From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752098AbbJOIkG (ORCPT ); Thu, 15 Oct 2015 04:40:06 -0400 Received: from mailout3.samsung.com ([203.254.224.33]:58034 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750880AbbJOIkB (ORCPT ); Thu, 15 Oct 2015 04:40:01 -0400 MIME-version: 1.0 Content-type: text/plain; charset=utf-8 X-AuditID: cbfee68e-f791c6d000001498-df-561f665fc9a2 Content-transfer-encoding: 8BIT Message-id: <561F665F.2010505@samsung.com> Date: Thu, 15 Oct 2015 17:39:59 +0900 From: Chanwoo Choi User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 To: Roger Quadros , linux-kernel@vger.kernel.org Cc: k.kozlowski@samsung.com, ckeepax@opensource.wolfsonmicro.com, gregkh@linuxfoundation.org, ramakrishna.pallala@intel.com, patches@opensource.wolfsonmicro.com, myungjoo.ham@samsung.com Subject: Re: [PATCH v4] extcon: Modify the id and name of external connector References: <1444296281-5284-1-git-send-email-cw00.choi@samsung.com> <561E00AD.50500@ti.com> <561F0DE0.1070405@samsung.com> <561F52B6.6020800@ti.com> In-reply-to: <561F52B6.6020800@ti.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrOIsWRmVeSWpSXmKPExsWyRsSkRDc+TT7MoP+4ksW/KTfYLZoXr2ez eP3C0OLyrjlsFrcbV7BZLH/7n81i4ZubTBY9j7QcODwW73nJ5LF/7hp2j5cTf7N59G1Zxehx /MZ2Jo/Pm+QC2KK4bFJSczLLUov07RK4Ms61XWIvaHOp+DnjPHsD427zLkZODgkBE4mj1/uZ IWwxiQv31rOB2EICKxglNt8MgKk5uXk9axcjF1B8FqPE7dk3GUESvAKCEj8m32PpYuTgYBaQ lzhyKRvCVJeYMiUXovwBo8SjT/+YIMq1JF7u/c4KYrMIqEqs6u0D28sGFN//4gYbSK+oQIRE 94lKkLCIgL1E7+xZYOXMAvsYJdbPKAexhQV8JA7MOgB1zkxGifYPB8HO4RRQkzh9ZT87SEJC 4B67xIcLzWwQywQkvk0+BHanhICsxKYDUP9KShxccYNlAqPYLCTfzEL4ZhbCNwsYmVcxiqYW JBcUJ6UXGekVJ+YWl+al6yXn525iBMbf6X/P+nYw3jxgfYhRgINRiYf3xAO5MCHWxLLiytxD jKZAN0xklhJNzgdGeV5JvKGxmZGFqYmpsZG5pZmSOG+C1M9gIYH0xJLU7NTUgtSi+KLSnNTi Q4xMHJxSDYwsOY2WJs1H1I+yvHywX+a9zoFf987dc81+YpOu9Mr5i9q+mLRmDd/qwMehr7QP ZFmf285g+fC5lewcJ3fmIINFnKKvhRwzP+qmvYrSyWRyTNC9p8TF7hNyUnyb0ZzTKl6PbRtW 5FwU1qq8N+v+hAnTgw+uFNu68V+ZZ5FZ1Wwu70LPJQ+jI5RYijMSDbWYi4oTAfqPmzO6AgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrKIsWRmVeSWpSXmKPExsVy+t9jQd34NPkwgw/ThCz+TbnBbtG8eD2b xesXhhaXd81hs7jduILNYvnb/2wWC9/cZLLoeaTlwOGxeM9LJo/9c9ewe7yc+JvNo2/LKkaP 4ze2M3l83iQXwBbVwGiTkZqYklqkkJqXnJ+SmZduq+QdHO8cb2pmYKhraGlhrqSQl5ibaqvk 4hOg65aZA3SPkkJZYk4pUCggsbhYSd8O04TQEDddC5jGCF3fkCC4HiMDNJCwhjHjXNsl9oI2 l4qfM86zNzDuNu9i5OSQEDCROLl5PSuELSZx4d56ti5GLg4hgVmMErdn32QESfAKCEr8mHyP pYuRg4NZQF7iyKVsCFNdYsqUXIjyB4wSjz79Y4Io15J4ufc72EwWAVWJVb19zCA2G1B8/4sb bCC9ogIREt0nKkHCIgL2Er2zZ4GVMwvsY5RYP6McxBYW8JE4MOsAK8T8mYwS7R8Ogp3DKaAm cfrKfvYJjEBHIlw3C+G6WQjXLWBkXsUokVqQXFCclJ5rmJdarlecmFtcmpeul5yfu4kRHOXP pHYwHtzlfohRgINRiYf3xAO5MCHWxLLiytxDjBIczEoivB5x8mFCvCmJlVWpRfnxRaU5qcWH GE2B3pvILCWanA9MQHkl8YbGJmZGlkbmhhZGxuZK4rw3DjGECQmkJ5akZqemFqQWwfQxcXBK NTCyvch1eP7n44bOZ4xmD7ON/3hMEWRujUku572cHfmjevO+5gmn1m2rqfq5+hmzl3odn/CM Qxt6apPKQjzMRIsebzQQX3Fjfluh4SHmbXHpp6yPpSyVdXnI3K2Rs/P9doHOt3tNXTfsY/ze GP9S7L1+YWj+gfSp6tMMl8w6z3ExQPJgEMfvfWuVWIozEg21mIuKEwE9LPXtCAMAAA== DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Roger, On 2015년 10월 15일 16:16, Roger Quadros wrote: > Chanwoo, > > On 15/10/15 05:22, Chanwoo Choi wrote: >> Hi Roger, >> >> On 2015년 10월 14일 16:13, Roger Quadros wrote: >>> Chanwoo, >>> >>> On 08/10/15 12:24, Chanwoo Choi wrote: >>>> This patch modifies the id and name of external connector with the >>>> additional prefix to clarify both attribute and meaning of external >>>> connector as following: >>>> - EXTCON_CHG_* mean the charger connector. >>>> - EXTCON_JACK_* mean the jack connector. >>>> - EXTCON_DISP_* mean the display port connector. >>>> >>>> Following table show the new name of external connector with old name: >>>> -------------------------------------------------- >>>> Old extcon name | New extcon name | >>>> -------------------------------------------------- >>>> EXTCON_TA | EXTCON_CHG_USB_DCP | >>>> EXTCON_CHARGE_DOWNSTREAM| EXTCON_CHG_USB_CDP | >>>> EXTCON_FAST_CHARGER | EXTCON_CHG_USB_FAST | >>>> EXTCON_SLOW_CHARGER | EXTCON_CHG_USB_SLOW | >>>> -------------------------------------------------- >>>> EXTCON_MICROPHONE | EXTCON_JACK_MICROPHONE | >>>> EXTCON_HEADPHONE | EXTCON_JACK_HEADPHONE | >>>> EXTCON_LINE_IN | EXTCON_JACK_LINE_IN | >>>> EXTCON_LINE_OUT | EXTCON_JACK_LINE_OUT | >>>> EXTCON_VIDEO_IN | EXTCON_JACK_VIDEO_IN | >>>> EXTCON_VIDEO_OUT | EXTCON_JACK_VIDEO_OUT | >>>> EXTCON_SPDIF_IN | EXTCON_JACK_SPDIF_IN | >>>> EXTCON_SPDIF_OUT | EXTCON_JACK_SPDIF_OUT | >>>> -------------------------------------------------- >>>> EXTCON_HMDI | EXTCON_DISP_HDMI | >>>> EXTCON_MHL | EXTCON_DISP_MHL | >>>> EXTCON_DVI | EXTCON_DISP_DVI | >>>> EXTCON_VGA | EXTCON_DISP_VGA | >>>> -------------------------------------------------- >>>> >>>> And, when altering the name of USB charger connector, EXTCON refers to the >>>> "Battery Charging v1.2 Spec and Adopters Agreement"[1] to use the standard >>>> name of USB charging port as following. Following name of USB charging port >>>> are already used in power_supply subsystem. We chan check it on patch[2]. >>>> - EXTCON_CHG_USB_SDP /* Standard Downstream Port */ >>>> - EXTCON_CHG_USB_DCP /* Dedicated Charging Port */ >>>> - EXTCON_CHG_USB_CDP /* Charging Downstream Port */ >>>> - EXTCON_CHG_USB_ACA /* Accessory Charger Adapter */ >>>> >>>> [1] www.usb.org/developers/docs/devclass_docs/BCv1.2_070312.zip >>>> [2] commit 85efc8a18ced ("power_supply: Add types for USB chargers") >>>> >>>> Signed-off-by: Chanwoo Choi >>>> [ckeepax: For the Arizona changes] >>>> Acked-by: Charles Keepax >>>> --- >>>> Changes from v3: >>>> (https://lkml.org/lkml/2015/10/6/984) >>>> - Modify the name of fast/slow charger connector as following: >>>> : EXTCON_CHG_USB_DCP_FAST -> EXTCON_CHG_USB_FAST >>>> : EXTCON_CHG_USB_DCP_SLOW -> EXTCON_CHG_USB_SLOW >>>> - Add EXTCON_CHG_USB_SDP to mean the charging connector of SDP (Standard >>>> Downstream Port) >>>> >>>> Changes from v2: >>>> (https://lkml.org/lkml/2015/10/6/239) >>>> - Remove the EXTCON_CHG_USB type to remove the possible confusion according to >>>> Roger's comment and drop patch2 about EXTCON_CHG_USB. >>>> - Fix the warning issue provided by scripts/checkpatch.pl >>>> >>>> Changes from v1: >>>> (https://lkml.org/lkml/2015/10/3/304) >>>> - Add acked tag by Charles Keepax for arizona changes >>>> - Modify the name of USB charger connector as following: >>>> : EXTCON_CHG_USB_FAST -> EXTCON_CHG_USB_DCP_FAST >>>> : EXTCON_CHG_USB_SLOW -> EXTCON_CHG_USB_DCP_SLOW >>>> - Add the missing EXTCON_CHG_USB_ACA charger connector >>>> - Add one more patch to support the EXTCON_CHG_USB when SDP port is >>>> connected or not >>>> >>>> drivers/extcon/extcon-arizona.c | 18 ++++++------ >>>> drivers/extcon/extcon-axp288.c | 12 ++++---- >>>> drivers/extcon/extcon-max14577.c | 17 +++++------ >>>> drivers/extcon/extcon-max77693.c | 32 +++++++++++---------- >>>> drivers/extcon/extcon-max77843.c | 27 +++++++++-------- >>>> drivers/extcon/extcon-max8997.c | 21 +++++++------- >>>> drivers/extcon/extcon-rt8973a.c | 4 +-- >>>> drivers/extcon/extcon-sm5502.c | 4 +-- >>>> drivers/extcon/extcon.c | 61 ++++++++++++++++++++------------------- >>>> include/linux/extcon.h | 62 +++++++++++++++++++++++----------------- >>>> 10 files changed, 139 insertions(+), 119 deletions(-) >>>> > > > >>>> diff --git a/drivers/extcon/extcon-rt8973a.c b/drivers/extcon/extcon-rt8973a.c >>>> index 1bc3737ea01c..36bf1d63791c 100644 >>>> --- a/drivers/extcon/extcon-rt8973a.c >>>> +++ b/drivers/extcon/extcon-rt8973a.c >>>> @@ -93,7 +93,7 @@ static struct reg_data rt8973a_reg_data[] = { >>>> static const unsigned int rt8973a_extcon_cable[] = { >>>> EXTCON_USB, >>>> EXTCON_USB_HOST, >>>> - EXTCON_TA, >>>> + EXTCON_CHG_USB_DCP, >>>> EXTCON_JIG, >>>> EXTCON_NONE, >>>> }; >>>> @@ -333,7 +333,7 @@ static int rt8973a_muic_cable_handler(struct rt8973a_muic_info *info, >>>> con_sw = DM_DP_SWITCH_USB; >>>> break; >>>> case RT8973A_MUIC_ADC_TA: >>>> - id = EXTCON_TA; >>>> + id = EXTCON_CHG_USB_DCP; >>>> con_sw = DM_DP_SWITCH_OPEN; >>>> break; >>>> case RT8973A_MUIC_ADC_FACTORY_MODE_BOOT_OFF_USB: >>>> diff --git a/drivers/extcon/extcon-sm5502.c b/drivers/extcon/extcon-sm5502.c >>>> index 2945091bfd0e..7aac3cc7efd7 100644 >>>> --- a/drivers/extcon/extcon-sm5502.c >>>> +++ b/drivers/extcon/extcon-sm5502.c >>>> @@ -95,7 +95,7 @@ static struct reg_data sm5502_reg_data[] = { >>>> static const unsigned int sm5502_extcon_cable[] = { >>>> EXTCON_USB, >>>> EXTCON_USB_HOST, >>>> - EXTCON_TA, >>>> + EXTCON_CHG_USB_DCP, >>>> EXTCON_NONE, >>>> }; >>>> >>>> @@ -389,7 +389,7 @@ static int sm5502_muic_cable_handler(struct sm5502_muic_info *info, >>>> vbus_sw = VBUSIN_SWITCH_VBUSOUT_WITH_USB; >>>> break; >>>> case SM5502_MUIC_ADC_OPEN_TA: >>>> - id = EXTCON_TA; >>>> + id = EXTCON_CHG_USB_DCP; >>>> con_sw = DM_DP_SWITCH_OPEN; >>>> vbus_sw = VBUSIN_SWITCH_VBUSOUT; >>>> break; >>>> diff --git a/drivers/extcon/extcon.c b/drivers/extcon/extcon.c >>>> index 8dd0af1d50bc..f345d492d4a1 100644 >>>> --- a/drivers/extcon/extcon.c >>>> +++ b/drivers/extcon/extcon.c >>>> @@ -39,37 +39,40 @@ >>>> #define CABLE_NAME_MAX 30 >>>> >>>> static const char *extcon_name[] = { >>>> - [EXTCON_NONE] = "NONE", >>>> + [EXTCON_NONE] = "EXTCON_NONE", >>>> >>>> /* USB external connector */ >>>> - [EXTCON_USB] = "USB", >>>> - [EXTCON_USB_HOST] = "USB-HOST", >>>> - >>>> - /* Charger external connector */ >>>> - [EXTCON_TA] = "TA", >>>> - [EXTCON_FAST_CHARGER] = "FAST-CHARGER", >>>> - [EXTCON_SLOW_CHARGER] = "SLOW-CHARGER", >>>> - [EXTCON_CHARGE_DOWNSTREAM] = "CHARGE-DOWNSTREAM", >>>> - >>>> - /* Audio/Video external connector */ >>>> - [EXTCON_LINE_IN] = "LINE-IN", >>>> - [EXTCON_LINE_OUT] = "LINE-OUT", >>>> - [EXTCON_MICROPHONE] = "MICROPHONE", >>>> - [EXTCON_HEADPHONE] = "HEADPHONE", >>>> - >>>> - [EXTCON_HDMI] = "HDMI", >>>> - [EXTCON_MHL] = "MHL", >>>> - [EXTCON_DVI] = "DVI", >>>> - [EXTCON_VGA] = "VGA", >>>> - [EXTCON_SPDIF_IN] = "SPDIF-IN", >>>> - [EXTCON_SPDIF_OUT] = "SPDIF-OUT", >>>> - [EXTCON_VIDEO_IN] = "VIDEO-IN", >>>> - [EXTCON_VIDEO_OUT] = "VIDEO-OUT", >>>> - >>>> - /* Etc external connector */ >>>> - [EXTCON_DOCK] = "DOCK", >>>> - [EXTCON_JIG] = "JIG", >>>> - [EXTCON_MECHANICAL] = "MECHANICAL", >>>> + [EXTCON_USB] = "EXTCON_USB", >>> >>> Should the name string be "USB-PERIPHERAL"? >> >> I think 'PERIPHERAL' is not necessary. The extcon name is >> used for only end user using the platform developer. >> 'PERIPHERAL' might cause the confusion to end user. >> > OK. >>> >>>> + [EXTCON_USB_HOST] = "EXTCON_USB_HOST", >>> >>> Why prefix EXTCON and change hyphen to underscore? >>> Wasn't the original version i.e. "USB-HOST" better? >> >> Agreee. >> >>> >>> Why the change in the name strings? Who is the end user of the name string? >>> If the end use is just for information to a human user then the human readable >>> format makes more sense. i.e. "MHL" or "MICROPHONE" makes more sense than >>> "EXTCON_DISP_MHL" or "EXTCON_JACK_MICROPHONE" >> >> Your comment make sense. I'll not modify the name of external connector. >> I'll use the existing name. >> > > OK. Sorry for not pointing this out earlier. No problem. Thanks for your review. Regards, Chanwoo Choi