On Wed, 2017-09-27 at 12:32 -0700, Rodrigo Vivi wrote:
> On Wed, Sep 27, 2017 at 12:03 AM, Jani Nikula
> <jani.nikula@xxxxxxxxxxxxxxx> wrote:
> > On Tue, 26 Sep 2017, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote:
> > > On Tue, Sep 26, 2017 at 09:21:43PM +0000, Paulo Zanoni wrote:
> > > > Em Ter, 2017-09-26 às 14:13 -0700, Rodrigo Vivi escreveu:
> > > > > Let's stop this usage before it spreads so much.
> > > > >
> > > > > 1. This check is not part of usual searches happening when adding
> > > > > new platform.
> > > > > 2. There is already a duplication here with INTEL_INFO(dev_priv)->gen
> > > > > and INTEL_GEN(dev_priv).
> > > > >
> > > > > So let's please avoid yet another way.
> > > > >
> > > > > Fixes: b22ca995ba1c ("drm/i915: prepare pipe for YCBCR420 output")
> > > > > Fixes: 27082493e9c6 ("drm/i915/skl: Update DDB values atomically with
> > > > > wms/plane attrs")
> > > >
> > > > Not sure if the Fixes tags are appropriate since this is not a bug fix.
> > >
> > > I wondered that... but since "dim fixes" provided me that tag along with the
> > > list of people I should cc I decided to include here. I thought it
> > > wouldn't hurt and also maybe good to propagate that to everywhere possible so
> > > we don't recieve more code based on that usage.
> > >
> > > But I won't merge today to give time to get Jani's view on that.
> >
> > Please only use Fixes: for functional fixes that need to be
> > backported. Like, nobody's going to be happier running a kernel they
> > know uses INTEL_GEN() consistently.
>
> Makes sense.
> Merged to dinq without the "Fixes:" tags.
> Thanks for all comments and reviews.
We discussed this with Chris too, I ended up suggesting there could be
something along; "Backport: none" or "Backport: v4.0+", instead of the
horrible #v4.0 that is causing pain every now and then, by getting
mixed to the To: fields in a wrong way.
Any thoughts on that?
Fixes: is an interesting metric in other sense too, or then just decide
to use X-Fixes: when we don't want it to end up backported.
Regards, Joonas
--
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
