[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <520CFD52.4060103@baylibre.com>
Date: Thu, 15 Aug 2013 18:09:54 +0200
From: Benoit Cousson <bcousson@...libre.com>
To: Kishon Vijay Abraham I <kishon@...com>
CC: Stephen Warren <swarren@...dotorg.org>, mark.rutland@....com,
devicetree@...r.kernel.org, george.cherian@...com,
linux@....linux.org.uk, ian.campbell@...rix.com,
pawel.moll@....com, linux-doc@...r.kernel.org, tony@...mide.com,
gregkh@...uxfoundation.org, linux-kernel@...r.kernel.org,
rob.herring@...xeda.com, cw00.choi@...sung.com,
myungjoo.ham@...sung.com, balbi@...com, rob@...dley.net,
benoit.cousson@...aro.org, linux-omap@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 1/2] extcon: palmas: Added a new compatible type *ti,
palmas-usb-vid*
Hi Kishon,
On 14/08/2013 07:24, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Wednesday 14 August 2013 12:43 AM, Stephen Warren wrote:
>> On 08/12/2013 11:37 PM, Kishon Vijay Abraham I wrote:
>>> The Palmas device contains only a USB VID detector, so added a
>>> compatible type *ti,palmas-usb-vid*. Dint remove the existing compatible
>>
>> s/Dint/Didn't/
>>
>>> diff --git a/Documentation/devicetree/bindings/extcon/extcon-twl.txt b/Documentation/devicetree/bindings/extcon/extcon-twl.txt
>>
>>> PALMAS USB COMPARATOR
>>> Required Properties:
>>> - - compatible : Should be "ti,palmas-usb" or "ti,twl6035-usb"
>>> + - compatible : Should be "ti,palmas-usb" or "ti,twl6035-usb" or
>>> + "ti,palmas-usb-vid".
>>
>> So are ti,palmas-usb and ti,twl6035-usb full EHCI controllers then?
>
> No. I thought I shouldn't remove those if someone is already using those
> compatible value.
Well, I think we still have a short period of time where we can clean
some badly defined bindings before it is really too late.
Both kernel and DTS are still in sync for the moment, so changing both
at the same time should be safe.
Regards,
Benoit
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists