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

[Chanwoo Choi]

On 02/02/2015 07:01 PM, Roger Quadros wrote:
> On 02/02/15 11:55, Chanwoo Choi wrote:
>> 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.
>>
> 
> Kernel tree shows following users of "USB-Host" that will have to be changed to
> "USB-HOST".

You're right. I'll modify all cable name of 'USB-HOST'.
Also, I have plan to use only capital letter for cable name.

> 
> extcon-class.c:	[EXTCON_USB_HOST]	= "USB-Host",
> extcon-max77693.c:	[EXTCON_CABLE_USB_HOST]			= "USB-Host",
> extcon-max77693.c:		extcon_set_cable_state(info->edev, "USB-Host", attached);
> extcon-max8997.c:	[EXTCON_CABLE_USB_HOST]			= "USB-Host",
> extcon-max8997.c:		extcon_set_cable_state(info->edev, "USB-Host", attached);
> extcon-rt8973a.c:	[EXTCON_CABLE_USB_HOST]		= "USB-Host",
> extcon-sm5502.c:	[EXTCON_CABLE_USB_HOST]	= "USB-Host",
> 
> I'm not aware if any user space programs depend on this name. Do you know of any?

As I knew, released samsung smart-phone used the cable name to detect the cable state
becaues extcon send the uevent with both cable name and cable state.

Thanks,
Chanwoo Choi

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