> >> diff --git a/include/linux/extcon.h b/include/linux/extcon.h index > >> c0f8c4fc5d45..c8dd881e2b8d 100644 > >> --- a/include/linux/extcon.h > >> +++ b/include/linux/extcon.h > >> @@ -31,32 +31,42 @@ > >> /* > >> * Define the unique id of supported external connectors > >> */ > >> -#define EXTCON_NONE 0 > >> - > >> -#define EXTCON_USB 1 /* USB connector */ > >> -#define EXTCON_USB_HOST 2 > >> - > >> -#define EXTCON_TA 3 /* Charger connector */ > >> -#define EXTCON_FAST_CHARGER 4 > >> -#define EXTCON_SLOW_CHARGER 5 > >> -#define EXTCON_CHARGE_DOWNSTREAM 6 > >> - > >> -#define EXTCON_LINE_IN 7 /* Audio/Video > >> connector */ > >> -#define EXTCON_LINE_OUT 8 > >> -#define EXTCON_MICROPHONE 9 > >> -#define EXTCON_HEADPHONE 10 > >> -#define EXTCON_HDMI 11 > >> -#define EXTCON_MHL 12 > >> -#define EXTCON_DVI 13 > >> -#define EXTCON_VGA 14 > >> -#define EXTCON_SPDIF_IN 15 > >> -#define EXTCON_SPDIF_OUT 16 > >> -#define EXTCON_VIDEO_IN 17 > >> -#define EXTCON_VIDEO_OUT 18 > >> - > >> -#define EXTCON_DOCK 19 /* Misc connector */ > >> -#define EXTCON_JIG 20 > >> -#define EXTCON_MECHANICAL 21 > >> +#define EXTCON_NONE 0 > >> + > >> +/* USB external connector */ > >> +#define EXTCON_USB 1 /* Universal Serial Bus */ > >> +#define EXTCON_USB_HOST 2 > >> + > >> +/* Charging external connector */ > >> +#define EXTCON_CHG_USB 5 /* Standard Downstream Port > >> */ > >> +#define EXTCON_CHG_USB_CDP 6 /* Charging Downstream Port > >> */ > >> +#define EXTCON_CHG_USB_DCP 7 /* Dedicated Charging Port */ > >> +#define EXTCON_CHG_USB_DCP_FAST 8 > >> +#define EXTCON_CHG_USB_DCP_SLOW 9 > >> +#define EXTCON_CHG_USB_ACA 10 /* Accessory Charger Adapter > >> */ > >> + > >> +/* Jack external connector */ > >> +#define EXTCON_JACK_MICROPHONE 20 > >> +#define EXTCON_JACK_HEADPHONE 21 > >> +#define EXTCON_JACK_LINE_IN 22 > >> +#define EXTCON_JACK_LINE_OUT 23 > >> +#define EXTCON_JACK_VIDEO_IN 24 > >> +#define EXTCON_JACK_VIDEO_OUT 25 > >> +#define EXTCON_JACK_SPDIF_IN 26 /* Sony Philips Digital > >> InterFace */ > >> +#define EXTCON_JACK_SPDIF_OUT 27 > >> + > >> +/* Display external connector */ > >> +#define EXTCON_DISP_HDMI 40 /* High-Definition Multimedia > >> Interface */ > >> +#define EXTCON_DISP_MHL 41 /* Mobile High-Definition Link > >> */ > >> +#define EXTCON_DISP_DVI 42 /* Digital Visual Inteface */ > >> +#define EXTCON_DISP_VGA 43 /* Video Graphics Array */ > >> + > >> +/* Miscellaneous external connector */ > >> +#define EXTCON_DOCK 60 > >> +#define EXTCON_JIG 61 > >> +#define EXTCON_MECHANICAL 62 > >> + > >> +#define EXTCON_NUM 63 > > > > Can we change the #define macro's to enum's? is there problem with that? > > It is possible. But, the unique id of external connector will be used on > device tree > file as following patch[1]. Following patch[1] defines the include/dt- > bindings/extcon/extcon.h > which is include in *.dts file. In *.dts file, we can not use the 'enum'. > [1] https://lkml.org/lkml/2015/10/5/38 > So, I use the "#define" keyword instead of enum. > > I used the 'enum' on first patch to define the unique id but I altered it by > using > '#define' instead of 'enum' on following patch[2]. > [2] commit 73b6ecdb93e8e ("extcon: Redefine the unique id of supported > external connectors without 'enum extcon' type")
Ok. Thanks. -Ram