[RFC][PATCH 1/5] drm: Introduce scaling filter property

Laxminarayan Bharadiya, Pankaj pankaj.laxminarayan.bharadiya at intel.com
Tue Feb 25 10:09:42 UTC 2020



> -----Original Message-----
> From: Jani Nikula <jani.nikula at linux.intel.com>
> Sent: 25 February 2020 15:26
> To: Laxminarayan Bharadiya, Pankaj
> <pankaj.laxminarayan.bharadiya at intel.com>; daniel at ffwll.ch; intel-
> gfx at lists.freedesktop.org; dri-devel at lists.freedesktop.org;
> ville.syrjala at linux.intel.com; airlied at linux.ie;
> maarten.lankhorst at linux.intel.com; tzimmermann at suse.de;
> mripard at kernel.org; mihail.atanassov at arm.com
> Cc: Laxminarayan Bharadiya, Pankaj
> <pankaj.laxminarayan.bharadiya at intel.com>; linux-kernel at vger.kernel.org;
> Nautiyal, Ankit K <ankit.k.nautiyal at intel.com>
> Subject: Re: [RFC][PATCH 1/5] drm: Introduce scaling filter property
> 
> On Tue, 25 Feb 2020, Pankaj Bharadiya
> <pankaj.laxminarayan.bharadiya at intel.com> wrote:
> > Signed-off-by: Pankaj Bharadiya
> > <pankaj.laxminarayan.bharadiya at intel.com>
> > Signed-off-by: Shashank Sharma <shashank.sharma at intel.com>
> > Signed-off-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>
> 
> What did Shashank and Ankit do, should one or the other perhaps retain
> authorship?

I kind of refactored the code, added plane scaling support added new APIs & documentation
 and rewrote the scaling filter setting logic. Since I made significant changes (IMHO), I thought
of changing the authorship. I spoke with Ankit regarding this during my initial discussion with him.

Will you please review the present RFC and the older one and suggest.  I have
no issues with changing the authorship 😊.
 
> 
> In any case, when taking over code and submitting, you should add your sign-off
> *last*. Please see [1] for what Signed-off-by means.

Was not aware, will do the needful.

Thanks,
Pankaj

> 
> BR,
> Jani.
> 
> 
> [1] https://developercertificate.org/
> 
> --
> Jani Nikula, Intel Open Source Graphics Center


More information about the dri-devel mailing list