Re: [PATCH] drm/i915: Fix iboost setting for SKL Y/U DP DDI buffer translation entry 2

Daniel Vetter
2016-08-05T09:26:54Z

On Fri, Aug 05, 2016 at 09:43:59AM +0300, Ville Syrjälä wrote:
> On Tue, Aug 02, 2016 at 03:37:27PM +0300, Jani Nikula wrote:
> > On Tue, 02 Aug 2016, ville.syrjala@linux.intel.com wrote:
> > > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > >
> > > The spec was recently fixed to have the correct iboost setting for the
> > > SKL Y/U DP DDI buffer translation table entry 2. Update our tables
> > > to match.
> > >
> > > Cc: David Weinehall <david.weinehall@linux.intel.com>
> > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >
> > Cc: stable@vger.kernel.org
>
> Added that and pushed to dinq.
>
> It looks like've had quite a few updates to these tables slip by w/o
> adding the cc:stable :( Hopefully stable maintainers will notice
> and perhaps pick them all up when they grab this one.

For next time around, you can give stable maintainers explicit hints about
additional patches they need to cherry-pick:

Cc: <stable@vger.kernel.org> # 3.3.x: a1f84a3: sched: Check for idle
Cc: <stable@vger.kernel.org> # 3.3.x: 1b9508f: sched: Rate-limit newidle
Cc: <stable@vger.kernel.org> # 3.3.x: fd21073: sched: Fix affinity logic
Cc: <stable@vger.kernel.org> # 3.3.x
Signed-off-by: Ingo Molnar <mingo@elte.hu>

Copied directly fromt he docs even ;-)
-Daniel

>
> >
> > > ---
> > > drivers/gpu/drm/i915/intel_ddi.c | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
> > > index fc2ef2d76091..c55a2ad50c0d 100644
> > > --- a/drivers/gpu/drm/i915/intel_ddi.c
> > > +++ b/drivers/gpu/drm/i915/intel_ddi.c
> > > @@ -145,7 +145,7 @@ static const struct ddi_buf_trans skl_ddi_translations_dp[] = {
> > > static const struct ddi_buf_trans skl_u_ddi_translations_dp[] = {
> > > { 0x0000201B, 0x000000A2, 0x0 },
> > > { 0x00005012, 0x00000088, 0x0 },
> > > - { 0x80007011, 0x000000CD, 0x0 },
> > > + { 0x80007011, 0x000000CD, 0x1 },
> > > { 0x80009010, 0x000000C0, 0x1 },
> > > { 0x0000201B, 0x0000009D, 0x0 },
> > > { 0x80005012, 0x000000C0, 0x1 },
> > > @@ -158,7 +158,7 @@ static const struct ddi_buf_trans skl_u_ddi_translations_dp[] = {
> > > static const struct ddi_buf_trans skl_y_ddi_translations_dp[] = {
> > > { 0x00000018, 0x000000A2, 0x0 },
> > > { 0x00005012, 0x00000088, 0x0 },
> > > - { 0x80007011, 0x000000CD, 0x0 },
> > > + { 0x80007011, 0x000000CD, 0x3 },
> > > { 0x80009010, 0x000000C0, 0x3 },
> > > { 0x00000018, 0x0000009D, 0x0 },
> > > { 0x80005012, 0x000000C0, 0x3 },
> >
> > --
> > Jani Nikula, Intel Open Source Technology Center
>
> --
> Ville Syrjälä
> Intel OTC
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

--
Daniel Vetter
Software Engineer, Intel Corporation