Hi Chris, On 2016년 09월 10일 09:33, Chris Zhong wrote: > EXTCON_PROP_DISP_HPD is need by display port, if the system has no hpd > interrupt, this property can be used.
What is meaning of HPD? So, you need to add the description and reference for HPD in commit message. For example, When adding EXTCON_PROP_USB_SS property[1], the commit message included the reference for USB SuperSpeed. [1] https://git.kernel.org/cgit/linux/kernel/git/chanwoo/extcon.git/commit/?h=extcon-next&id=8457a1b49a2af0a0e71f80afed9f7c80de361610 > > Change-Id: I8b3eb78429126eaa369b10711b7f857b0a3df8ed You have to remove the 'Change-Id'. > Signed-off-by: Chris Zhong <z...@rock-chips.com> > --- > include/linux/extcon.h | 14 +++++++++++++- > 1 file changed, 13 insertions(+), 1 deletion(-) > > diff --git a/include/linux/extcon.h b/include/linux/extcon.h > index 9147c42..4411893 100644 > --- a/include/linux/extcon.h > +++ b/include/linux/extcon.h > @@ -131,9 +131,21 @@ > #define EXTCON_PROP_JACK_MAX 100 > #define EXTCON_PROP_JACK_CNT (EXTCON_PROP_JACK_MAX - EXTCON_PROP_JACK_MIN + > 1) > > +/* > + * Properties of EXTCON_TYPE_DISP. > + * > + * - EXTCON_PROP_DISP_HPD You should add the full name of 'HPD'. > + * @type: integer (intval) > + * @value: 0 (no hpd) or 1 (hpd) > + * @default: 0 (no hpd) > + * > + */ > + > +#define EXTCON_PROP_DISP_HPD 150 > + > /* Properties of EXTCON_TYPE_DISP. */ > #define EXTCON_PROP_DISP_MIN 150 > -#define EXTCON_PROP_DISP_MAX 150 > +#define EXTCON_PROP_DISP_MAX 151 > #define EXTCON_PROP_DISP_CNT (EXTCON_PROP_DISP_MAX - EXTCON_PROP_DISP_MIN + > 1) > > /* > -- Best Regards, Chanwoo Choi