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]

 



On Friday 11 October 2013 01:53 PM, Chanwoo Choi wrote:
> On 10/11/2013 04:52 PM, Kishon Vijay Abraham I wrote:
>> 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?
>>
> 
> OK, applied it.

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