> -----Original Message-----
> From: Jani Nikula <jani.nik...@linux.intel.com>
> Sent: Wednesday, September 3, 2025 1:11 PM
> To: Cavitt, Jonathan <jonathan.cav...@intel.com>; Shankar, Uma
> <uma.shan...@intel.com>; intel-gfx@lists.freedesktop.org; intel-
> x...@lists.freedesktop.org
> Cc: ville.syrj...@linux.intel.com; Govindapillai, Vinod
> <vinod.govindapil...@intel.com>; Shankar, Uma <uma.shan...@intel.com>;
> Cavitt, Jonathan <jonathan.cav...@intel.com>
> Subject: RE: [PATCH 1/1] drm/i915/display: Remove FBC modulo 4 restriction for
> ADL+
>
> On Tue, 02 Sep 2025, "Cavitt, Jonathan" <jonathan.cav...@intel.com> wrote:
> > Reviewed-bys should go below Signed-off-bys.
>
> Mmh, I don't think there's a hard and fast rule about that one.
>
> When applying patches, I usually go by what b4 does.
>
> But when sending patches that have previously been reviewed, I usually stick
> the
> Signed-off-by at the bottom instead. And I think that's quite common practise.
Yeah, the usage seems mixed in the tree. Have updated it though, also dropped
the
Credits-to tag.
Thanks Jani and Jonathan.
Regards,
Uma Shankar
>
> BR,
> Jani.
>
>
> --
> Jani Nikula, Intel
