Re: [PATCH v6] extcon: palmas: Added a new compatible type *ti,palmas-usb-vid*

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Chanwoo,

On Friday 11 October 2013 12:36 PM, Chanwoo Choi wrote:
> On 10/11/2013 03:15 PM, Kishon Vijay Abraham I wrote:
>> On Thursday 10 October 2013 06:41 PM, Laxman Dewangan wrote:
>>> On Thursday 10 October 2013 06:07 PM, Kishon Vijay Abraham I wrote:
>>>> The Palmas device contains only a USB VBUS-ID detector, so added a
>>>> compatible type *ti,palmas-usb-vid*. Didn't remove the existing compatible
>>>> types for backward compatibility.
>>>>
>>> I dont have much issue on this nomenclature but just second thought:
>>>
>>> The Palmas USB module is capable of detection VBUS, ID-GND, ID-RA, ID-RB, ID-RC
>>> and ID-FLOAT as per BC1.2.
>>> This sub-module only detect the cable, does nothing more than this.
>>> In this case, should we say "ti,palmas-usb-extcon"?
>>> Again extcon is linux specific terminology and this can be defer.
>>
>> I don't mind either way. Chanwoo?
> 
> The EXTCON is subsystem name. I don't prefer to use 'extcon' word on device name.
> I agree Kishon opinion.

Cool.. can you this patch as-is?

Thanks
Kishon
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux