Re: [PATCH v2 2/7] usb: extcon: Fix USB-Host cable name

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

 



Hi Roger,

On 02/02/2015 06:09 PM, Roger Quadros wrote:
> Chanwoo,
> 
> On 02/02/15 07:04, Chanwoo Choi wrote:
>> Hi Roger,
>>
>> On 01/30/2015 11:05 PM, Roger Quadros wrote:
>>> Hi,
>>>
>>> On 30/01/15 13:04, Roger Quadros wrote:
>>>> Felipe & Chanwoo,
>>>>
>>>> On 26/01/15 14:15, Roger Quadros wrote:
>>>>> The recommended name for USB-Host cable state is "USB-Host" and not
>>>>> "USB-HOST" as per drivers/extcon/extcon-class.c extcon_cable_name.
>>>>>
>>>>> Change all instances of "USB-HOST" to "USB-Host".
>>>>>
>>>>> Signed-off-by: Roger Quadros <rogerq@xxxxxx>
>>>>> Reviewed-by: Felipe Balbi <balbi@xxxxxx>
>>>>> Acked-by: Felipe Balbi <balbi@xxxxxx>
>>>>
>>>> This patch has no dependency to the rest so can be picked up as soon as possible.
>>>>
>>>> Do you think it is better to go via the USB tree?
>>>> If yes then Chanwoo, can you please Ack this one? Thanks.
>>>>
>>>> This would mean that only the first patch needs to go through extcon tree as Tony
>>>> will pick the rest.
>>>
>>> Hold on. Let's first decide what we really want to go ahead with
>>> "USB-Host" or "USB-HOST".
>>
>> Currently, extcon driver have used the specific cable name(USB-Host or USB-HOST)
>> without any standard way. So, I have plan to define common cable name in extcon
>> header file by using capital letter.
> 
> OK. In that case, this patch is not required.
> I will resend patch 1 with cable name corrected to "USB-HOST".

If you possbile, I want to use 'USB-HOST' cable name in drivers related to extcon.
If we use different cable name, this cause the confusion to control cable.

Thanks,
Chanwoo


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




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux