linux-kernel - Re: [PATCH] extcon: Introduce EXTCON_PROP_DISP

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-id: <57D5FEC7.5020509@samsung.com>
Date:   Mon, 12 Sep 2016 10:03:03 +0900
From:   Chanwoo Choi <cw00.choi@...sung.com>
To:     Chris Zhong <zyw@...k-chips.com>, myungjoo.ham@...sung.com,
        linux@...ck-us.net
Cc:     linux-kernel@...r.kernel.org, cpgs@...sung.com, groeck@...omium.org
Subject: Re: [PATCH] extcon: Introduce EXTCON_PROP_DISP_HPD property

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@...k-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