[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CACeCKae01WNKbP8wWONdpbNJQjLWFF67AFay0kwA_zmoaUNVUQ@mail.gmail.com>
Date: Thu, 15 Dec 2022 14:58:41 -0800
From: Prashant Malani <pmalani@...omium.org>
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
Powered by blists - more mailing lists