Re: [PATCH] extcon: Introduce EXTCON_PROP_DISP_HPD property
From: Chanwoo Choi
Date: Sun Sep 11 2016 - 21:03:10 EST
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 <zyw@xxxxxxxxxxxxxx>
> ---
> 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