On Wed, 2018-06-13 at 10:32 -0700, Dhinakaran Pandiyan wrote:
> On Wed, 2018-06-13 at 09:41 +0300, Jani Nikula wrote:
> >
> > On Wed, 13 Jun 2018, "Nagaraju, Vathsala" <vathsala.nagaraju@intel.
> > co
> > m> wrote:
> > >
> > >
> > > On 6/12/2018 2:30 PM, Jani Nikula wrote:
> > > >
> > > >
> > > > On Tue, 12 Jun 2018, vathsala nagaraju <vathsala.nagaraju@intel
> > > > .c
> > > > om> wrote:
> > > > >
> > > > >
> > > > > From: Vathsala Nagaraju <vathsala.nagaraju@xxxxxxxxx>
> > > > >
> > > > > Adds new psrwake options defined in the below table.
> > > > > Platform PSR wake options vbt version
> > > > > KBL/CFL/WHL All
> > > > > SKL All PV releases (Check for 203+ might help
> > > > > but cannot be foolproof)
> > > > > BXT Uses old interpretation.
> > > > > CNL/ICL+ All
> > > > > GLK All
> > > > >
> > > > > For SKL, we will continue to use older interpretation for the
> > > > > above reason.
> > > > >
> > > > > Cc: Jani Nikula <jani.nikula@xxxxxxxxx>
> > > > > Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> > > > > Cc: Puthikorn Voravootivat <puthik@xxxxxxxxxxxx>
> > > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@xxxxxxxxx>
> > > > >
> > > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@xxxxxxxxx
> > > > > >
> > > > > ---
> > > > > drivers/gpu/drm/i915/intel_bios.c | 3 ++-
> > > > > 1 file changed, 2 insertions(+), 1 deletion(-)
> > > > >
> > > > > diff --git a/drivers/gpu/drm/i915/intel_bios.c
> > > > > b/drivers/gpu/drm/i915/intel_bios.c
> > > > > index 465dff4..010ff68 100644
> > > > > --- a/drivers/gpu/drm/i915/intel_bios.c
> > > > > +++ b/drivers/gpu/drm/i915/intel_bios.c
> > > > > @@ -710,7 +710,8 @@ static int
> > > > > intel_bios_ssc_frequency(struct
> > > > > drm_i915_private *dev_priv,
> > > > > * New psr options 0=500us, 1=100us, 2=2500us,
> > > > > 3=0us
> > > > > * Old decimal value is wake up time in multiples
> > > > > of
> > > > > 100 us.
> > > > > */
> > > > > - if (bdb->version >= 209 && IS_GEN9_BC(dev_priv)) {
> > > > > + if ((INTEL_GEN(dev_priv) >= 10) ||
> > > > > + (IS_GEN9_BC(dev_priv) && !IS_SKYLAKE(dev_priv)))
> > > > > {
> > > > Please keep the version check.
> > > Sure. For SKL , shall we use older interpretation for all bdb
> > > version as
> > > vbt team cannot confirm bdb version for SKL?
> > I guess.
> >
> Why not change the version check to >= 203, if that's what PV
> releases
> had as per your commit message? With the current code, Linux and
> Windows set 500 us and 2.5 ms respectively on my laptop.
Said laptop is a SKL with bdb version 205.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
