[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAO-hwJJ926=ivk0avup_H3nAOYCbjkWT1M12wjyizJA2uojqtQ@mail.gmail.com>
Date: Fri, 29 Mar 2019 08:51:37 +0100
From: Benjamin Tissoires <benjamin.tissoires@...hat.com>
To: "Junge, Terry" <terry.junge@...y.com>
Cc: Nicolas Saenz Julienne <nsaenzjulienne@...e.de>,
Jiri Kosina <jikos@...nel.org>,
"oneukum@...e.de" <oneukum@...e.de>,
"linux-input@...r.kernel.org" <linux-input@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v4] HID: core: move Usage Page concatenation to Main item
On Fri, Mar 29, 2019 at 1:48 AM Junge, Terry <terry.junge@...y.com> wrote:
>
> On Thursday, March 28, 2019 6:49 AM Benjamin Tissoires <benjamin.tissoires@...hat.com> wrote:
> >>
> >> Hi Nicolas,
> >>
> >> V4 looks good to me.
> >
> >Looks good to me too.
> >
> >Terry, can I consider this as a formal Rev-by you?
>
> Benjamin,
>
> Yes, thank you. What is the normal way of formally indicating Rev-by?
>
Thanks.
So giving a rev-by is described in
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst
section 13.
Basically, you just answer a patch with: "Reviewed-by: Your Name
<your@...ress.sth>" (without quotes).
This has 2 benefits: it's formal, and everybody understands it, and
patchwork (at https://patchwork.kernel.org/patch/10873179/ for this
patch) will catch it. Then with the patchwork client, I can apply the
patch without having to manually add your tag :) (no need to resend
one here, unless you want to give a try, I'll apply it manually).
Cheers,
Benjamin
Powered by blists - more mailing lists