[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <037f9a23-6a90-bacb-5f9c-05f9a8be3be7@linux.intel.com>
Date: Fri, 14 Jan 2022 15:40:40 +0000
From: Tvrtko Ursulin <tvrtko.ursulin@...ux.intel.com>
To: Jani Nikula <jani.nikula@...ux.intel.com>,
Colin Ian King <colin.i.king@...il.com>,
Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>,
Rodrigo Vivi <rodrigo.vivi@...el.com>,
David Airlie <airlied@...ux.ie>,
Daniel Vetter <daniel@...ll.ch>,
intel-gfx@...ts.freedesktop.org, dri-devel@...ts.freedesktop.org
Cc: kernel-janitors@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH][RESEND] i915: make array flex_regs static const
On 13/01/2022 09:21, Jani Nikula wrote:
> On Wed, 12 Jan 2022, Colin Ian King <colin.i.king@...il.com> wrote:
>> Don't populate the read-only array flex_regs on the stack but
>> instead it static const. Also makes the object code a little smaller.
>>
>> Signed-off-by: Colin Ian King <colin.i.king@...il.com>
>>
>> ---
>>
>> RESEND: Use correct e-mail address for sign-off and From: in e-mail.
>
> There are a number of things at play here:
>
> - Our tooling checks for certain things, such as author/signed-off-by
> match, after applying the patch.
>
> - You have an entry in the kernel git .mailmap.
>
> - The git log.mailmap config default has changed from false to true.
>
> We apply the patch, and, with the default options, git show will display
> a different author than the signed-off-by.
>
> We should use --no-use-mailmap when examining a commit to ensure the
> commit is internally consistent, regardless of .mailmap.
Yes that sounds like the right thing to do.
In the meantime I have pushed this patch.
Regards,
Tvrtko
Powered by blists - more mailing lists