On Mon, 2018-06-18 at 11:42 +0530, vathsala nagaraju 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(205+)
> BXT           Uses old interpretation.
> CNL/ICL+      All(205+)
> GLK           All(205+)
> SKL           All PV releases (Check for 205+ might help but
> cannot be foolproof)
> 
> We will continue with newer interpretation for SKL from 205.
Let's hope this works for most machines out there.

It'd be easier to distinguish newer updated patches if you use version
numbers.

Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandi...@intel.com>

> 
> v2: Jani
>     Keep the bdb version check.
> v3:
>     Apply newer version for skl from 205+(DK).
>     Add (version check && platform list) (Jani).
>     Add bdb version for each platform in commit message(DK).
> 
> 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>
> Cc: Ashutosh D Shukla <ashutosh.d.shu...@intel.com>
> Cc: Maulik V Vaghela <maulik.v.vagh...@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..9ef0464 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 (bdb->version >= 205 && (IS_GEN9_BC(dev_priv) ||
> +         IS_GEMINILAKE(dev_priv) || (INTEL_GEN(dev_priv) >= 10)))
> {
>               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

Reply via email to