On 15/05/15 17:31, Chanwoo Choi wrote:
This patch uses the unique id to identify the type of external connector instead of string name. The string name have the many potential issues. So, this patch defines the 'extcon' enumeration which includes all supported external connector on EXTCON subsystem. If new external connector is necessary, the unique id of new connector have to be added in 'extcon' enumeration. There are current supported external connector in 'enum extcon' as following:enum extcon { EXTCON_NONE = 0x0, /* NONE */ /* USB external connector */ EXTCON_USB = 0x1, /* USB */ EXTCON_USB_HOST = 0x2, /* USB-HOST */ /* Charger external connector */ EXTCON_TA = 0x10, /* TA */ EXTCON_FAST_CHARGER = 0x11, /* FAST-CHARGER */ EXTCON_SLOW_CHARGER = 0x12, /* SLOW-CHARGER */ EXTCON_CHARGE_DOWNSTREAM= 0x13, /* CHARGE-DOWNSTREAM */ /* Audio and video external connector */ EXTCON_LINE_IN = 0x20, /* LINE-IN */ EXTCON_LINE_OUT = 0x21, /* LINE-OUT */ EXTCON_MICROPHONE = 0x22, /* MICROPHONE */ EXTCON_HEADPHONE = 0x23, /* HEADPHONE */ EXTCON_HDMI = 0x30, /* HDMI */ EXTCON_MHL = 0x31, /* MHL */ EXTCON_DVI = 0x32, /* DVI */ EXTCON_VGA = 0x33, /* VGA */ EXTCON_SPDIF_IN = 0x34, /* SPDIF-IN */ EXTCON_SPDIF_OUT = 0x35, /* SPDIF-OUT */ EXTCON_VIDEO_IN = 0x36, /* VIDEO-IN */ EXTCON_VIDEO_OUT = 0x37, /* VIDEO-OUT */ /* Miscellaneous external connector */ EXTCON_DOCK = 0x50, /* DOCK */ EXTCON_JIG = 0x51, /* JIG */ EXTCON_MECHANICAL = 0x52, /* MECHANICAL */ __EXTCON_END, }; For exmaple in extcon-arizoan.c: To use unique id removes the potential issue about handling the inconsistent name of external connector with string. - Previously, use the string to register the type of arizona jack connector static const char *arizona_cable[] = { "Mechanical", "Microphone", "Headphone", "Line-out", }; - Newly, use the unique id to register the type of arizona jack connector static const enum extcon arizona_cable[] = { EXTCON_MECHANICAL, EXTCON_MICROPHONE, EXTCON_HEADPHONE, EXTCON_LINE_OUT, EXTCON_NONE, }; And this patch modify the prototype of extcon_{get|set}_cable_state_() which uses the 'enum extcon id' instead of 'cable_index'. Because although one more extcon drivers support USB cable, each extcon driver might has the differnt 'cable_index' for USB cable. All extcon drivers can use the unique id number for same external connector with modified extcon_{get|set}_cable_state_(). - Previously, use 'cable_index' on these functions: extcon_get_cable_state_(struct extcon_dev*, int cable_index) extcon_set_cable_state_(struct extcon_dev*, int cable_index, bool state) -Newly, use 'enum extcon id' on these functions: extcon_get_cable_state_(struct extcon_dev*, enum extcon id) extcon_set_cable_state_(struct extcon_dev*, enum extcon id, bool state) Signed-off-by: Chanwoo Choi <cw00.c...@samsung.com> Cc: MyungJoo Ham <myungjoo....@samsung.com> Cc: Krzysztof Kozlowski <k.kozlow...@samsung.com> Cc: Charles Keepax <ckee...@opensource.wolfsonmicro.com> Cc: Graeme Gregory <g...@slimlogic.co.uk> Cc: Kishon Vijay Abraham I <kis...@ti.com> Cc: Jaewon Kim <jaewon02....@samsung.com> Cc: Roger Quadros <rog...@ti.com> Cc: Ramakrishna Pallala <ramakrishna.pall...@intel.com>
for drivers/extcon/extcon-usb-gpio.c Acked-by: Roger Quadros <rog...@ti.com> cheers, -roger
--- drivers/extcon/extcon-arizona.c | 38 +++----- drivers/extcon/extcon-axp288.c | 24 ++--- drivers/extcon/extcon-max14577.c | 45 ++++----- drivers/extcon/extcon-max77693.c | 95 +++++++++---------- drivers/extcon/extcon-max77843.c | 56 +++++------ drivers/extcon/extcon-max8997.c | 59 +++++------- drivers/extcon/extcon-palmas.c | 22 +++-- drivers/extcon/extcon-rt8973a.c | 40 +++----- drivers/extcon/extcon-sm5502.c | 32 ++----- drivers/extcon/extcon-usb-gpio.c | 32 ++----- drivers/extcon/extcon.c | 166 ++++++++++++++++++++------------- include/linux/extcon.h | 92 +++++++++--------- include/linux/extcon/extcon-adc-jack.h | 5 +- 13 files changed, 326 insertions(+), 380 deletions(-)
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
