On Wed, 2018-06-13 at 09:41 +0300, Jani Nikula wrote:
> On Wed, 13 Jun 2018, "Nagaraju, Vathsala" <vathsala.nagar...@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.nagar...@intel.com>
> > > >
> > > > 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.nik...@intel.com>
> > > > Cc: Rodrigo Vivi <rodrigo.v...@intel.com>
> > > > Cc: Puthikorn Voravootivat <put...@chromium.org>
> > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandi...@intel.com>
> > > >
> > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagar...@intel.com>
> > > > ---
> > > > 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.
> BR,
> Jani.
>
>
> >
> >
> > >
> > > Please tell anyone who asks, and also those who don't, that *all*
> > > of the
> > > VBT changes should be based on the *version*, and *none* of them
> > > should
> > > be based on the *platform*.
> > >
> > > BR,
> > > Jani.
> > >
> > > >
> > > > switch (psr_table->tp1_wakeup_time) {
> > > > case 0:
> > > > dev_priv->vbt.psr.tp1_wakeup_time_us
> > > > = 500;
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
