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 devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
