linux-kernel - RE: [PATCH] cros_ec_keyb: Add 3 buttons for monitor function

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: <dad487c20e134f5a96909315c0e7911d@msi.com>
Date:   Fri, 16 Dec 2022 01:17:40 +0000
From:   joewu(吳仲振) <joewu@....com>
To:     Prashant Malani <pmalani@...omium.org>,
        Joe Wu <joewu@....corp-partner.google.com>
CC:     LKML <linux-kernel@...r.kernel.org>,
        Stephen Boyd <swboyd@...omium.org>,
        Xiang wangx <wangxiang@...rlc.com>,
        "linux-input@...r.kernel.org" <linux-input@...r.kernel.org>,
        Derek Huang <derekhuang@...gle.com>,
        "Greg Kroah-Hartman" <gregkh@...uxfoundation.org>,
        Benson Leung <bleung@...omium.org>,
        Daisuke Nojiri <dnojiri@...omium.org>,
        Furquan Shaikh <furquan@...omium.org>,
        Tzung-Bi Shih <tzungbi@...nel.org>,
        "chrome-platform@...ts.linux.dev" <chrome-platform@...ts.linux.dev>,
        "Douglas Anderson" <dianders@...omium.org>,
        "Dustin L . Howett" <dustin@...ett.net>,
        Dmitry Torokhov <dmitry.torokhov@...il.com>,
        Guenter Roeck <groeck@...omium.org>,
        "Gustavo A . R . Silva" <gustavoars@...nel.org>,
        Sebastian Reichel <sebastian.reichel@...labora.com>
Subject: RE: [PATCH] cros_ec_keyb: Add 3 buttons for monitor function

Hi Prashant,

Thank you for comments. I will correct the description to be imperative mood.

BRs,
Joe


-----Original Message-----
From: Prashant Malani <pmalani@...omium.org> 
Sent: Friday, December 16, 2022 6:59 AM
To: Joe Wu <joewu@....corp-partner.google.com>
Cc: LKML <linux-kernel@...r.kernel.org>; Stephen Boyd <swboyd@...omium.org>; Xiang wangx <wangxiang@...rlc.com>; linux-input@...r.kernel.org; Derek Huang <derekhuang@...gle.com>; Greg Kroah-Hartman <gregkh@...uxfoundation.org>; Benson Leung <bleung@...omium.org>; Daisuke Nojiri <dnojiri@...omium.org>; Furquan Shaikh <furquan@...omium.org>; Tzung-Bi Shih <tzungbi@...nel.org>; chrome-platform@...ts.linux.dev; Douglas Anderson <dianders@...omium.org>; Dustin L . Howett <dustin@...ett.net>; Dmitry Torokhov <dmitry.torokhov@...il.com>; Guenter Roeck <groeck@...omium.org>; Gustavo A . R . Silva <gustavoars@...nel.org>; Sebastian Reichel <sebastian.reichel@...labora.com>
Subject: Re: [PATCH] cros_ec_keyb: Add 3 buttons for monitor function

Hi Joe,

On Wed, Dec 14, 2022 at 10:15 PM Joe Wu
<joewu@....corp-partner.google.com> wrote:
>
> This patch is to add extra 3 buttons: 'brightness up', 'brightness 
> down' and 'leave PC(cros) mode' to support monitor navigation 
> function.

nit: Please use the imperative form [1] while describing patches.
In this case, that would translate to:
"Add extra 3 buttons..."

Thanks,

-Prashant

[1] https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes

*****CONFIDENTIAL INFORMATION*****

This email is intended only for the use of the person or entity to whom it is
addressed and contains information that may be subject to and/or may be
restricted from disclosure by contract or applicable law. If you are not the 
intended recipient of this email, be advised that any disclosure, copy, 
distribution or use of the contents of this message is strictly prohibited. 
If you are not the intended recipient of this email, please notify the sender 
that you have received this in error by replying to this message. Then, 
please delete it from your system. Our Privacy Policy is available here 
https://www.msi.com/page/privacy-policy. Thank you.