* [PATCH] drm/i915/psr: Adds psrwake options for all platforms
@ 2018-06-13 8:03 vathsala nagaraju
2018-06-13 8:18 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/psr: Adds psrwake options for all platforms (rev2) Patchwork
` (2 more replies)
0 siblings, 3 replies; 23+ messages in thread
From: vathsala nagaraju @ 2018-06-13 8:03 UTC (permalink / raw)
To: rodrigo.vivi, jani.nikula
Cc: Puthikorn Voravootivat, intel-gfx, Dhinakaran Pandiyan
From: Vathsala Nagaraju <vathsala.nagaraju@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.
v2: Jani
Keep the bdb version check.
Cc: Jani Nikula <jani.nikula@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Puthikorn Voravootivat <puthik@chromium.org>
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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..5517ca7 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 >= 209 && ((INTEL_GEN(dev_priv) >= 10) ||
+ (IS_GEN9_BC(dev_priv) && !IS_SKYLAKE(dev_priv)))) {
switch (psr_table->tp1_wakeup_time) {
case 0:
dev_priv->vbt.psr.tp1_wakeup_time_us = 500;
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 23+ messages in thread
* ✗ Fi.CI.CHECKPATCH: warning for drm/i915/psr: Adds psrwake options for all platforms (rev2)
2018-06-13 8:03 [PATCH] drm/i915/psr: Adds psrwake options for all platforms vathsala nagaraju
@ 2018-06-13 8:18 ` Patchwork
2018-06-13 8:33 ` ✓ Fi.CI.BAT: success " Patchwork
2018-06-13 9:22 ` ✓ Fi.CI.IGT: " Patchwork
2 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2018-06-13 8:18 UTC (permalink / raw)
To: vathsala nagaraju; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/psr: Adds psrwake options for all platforms (rev2)
URL : https://patchwork.freedesktop.org/series/44601/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
e6bea8c90022 drm/i915/psr: Adds psrwake options for all platforms
-:36: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#36: FILE: drivers/gpu/drm/i915/intel_bios.c:714:
+ if (bdb->version >= 209 && ((INTEL_GEN(dev_priv) >= 10) ||
+ (IS_GEN9_BC(dev_priv) && !IS_SKYLAKE(dev_priv)))) {
total: 0 errors, 0 warnings, 1 checks, 9 lines checked
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* ✓ Fi.CI.BAT: success for drm/i915/psr: Adds psrwake options for all platforms (rev2)
2018-06-13 8:03 [PATCH] drm/i915/psr: Adds psrwake options for all platforms vathsala nagaraju
2018-06-13 8:18 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/psr: Adds psrwake options for all platforms (rev2) Patchwork
@ 2018-06-13 8:33 ` Patchwork
2018-06-13 9:22 ` ✓ Fi.CI.IGT: " Patchwork
2 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2018-06-13 8:33 UTC (permalink / raw)
To: vathsala nagaraju; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/psr: Adds psrwake options for all platforms (rev2)
URL : https://patchwork.freedesktop.org/series/44601/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4309 -> Patchwork_9282 =
== Summary - SUCCESS ==
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/44601/revisions/2/mbox/
== Known issues ==
Here are the changes found in Patchwork_9282 that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@drv_module_reload@basic-reload-inject:
fi-glk-j4005: PASS -> DMESG-WARN (fdo#106725, fdo#106248)
igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
fi-glk-j4005: PASS -> FAIL (fdo#106765)
igt@kms_flip@basic-flip-vs-wf_vblank:
fi-cnl-psr: PASS -> FAIL (fdo#100368)
igt@kms_pipe_crc_basic@read-crc-pipe-b-frame-sequence:
fi-glk-j4005: PASS -> FAIL (fdo#103481)
==== Possible fixes ====
igt@gem_exec_suspend@basic-s4-devices:
fi-kbl-7500u: DMESG-WARN (fdo#105128) -> PASS
igt@kms_flip@basic-flip-vs-modeset:
fi-glk-j4005: DMESG-WARN (fdo#106000) -> PASS
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
fi-cnl-psr: DMESG-WARN (fdo#104951) -> PASS
fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
fdo#103481 https://bugs.freedesktop.org/show_bug.cgi?id=103481
fdo#104951 https://bugs.freedesktop.org/show_bug.cgi?id=104951
fdo#105128 https://bugs.freedesktop.org/show_bug.cgi?id=105128
fdo#106000 https://bugs.freedesktop.org/show_bug.cgi?id=106000
fdo#106248 https://bugs.freedesktop.org/show_bug.cgi?id=106248
fdo#106725 https://bugs.freedesktop.org/show_bug.cgi?id=106725
fdo#106765 https://bugs.freedesktop.org/show_bug.cgi?id=106765
== Participating hosts (43 -> 37) ==
Missing (6): fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-pnv-d510 fi-skl-6700hq
== Build changes ==
* Linux: CI_DRM_4309 -> Patchwork_9282
CI_DRM_4309: 2740c5b0d0f40092355b329a62ede8cced7f64b9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4517: e94ce40798e35d2e3c4494f50b617908066bbf8b @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_9282: e6bea8c90022b9d1b768a6ae2ba1ac3dd22502e5 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
e6bea8c90022 drm/i915/psr: Adds psrwake options for all platforms
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_9282/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* ✓ Fi.CI.IGT: success for drm/i915/psr: Adds psrwake options for all platforms (rev2)
2018-06-13 8:03 [PATCH] drm/i915/psr: Adds psrwake options for all platforms vathsala nagaraju
2018-06-13 8:18 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/psr: Adds psrwake options for all platforms (rev2) Patchwork
2018-06-13 8:33 ` ✓ Fi.CI.BAT: success " Patchwork
@ 2018-06-13 9:22 ` Patchwork
2 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2018-06-13 9:22 UTC (permalink / raw)
To: vathsala nagaraju; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/psr: Adds psrwake options for all platforms (rev2)
URL : https://patchwork.freedesktop.org/series/44601/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4309_full -> Patchwork_9282_full =
== Summary - WARNING ==
Minor unknown changes coming with Patchwork_9282_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_9282_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
== Possible new issues ==
Here are the unknown changes that may have been introduced in Patchwork_9282_full:
=== IGT changes ===
==== Warnings ====
igt@gem_exec_schedule@deep-bsd2:
shard-kbl: PASS -> SKIP +1
== Known issues ==
Here are the changes found in Patchwork_9282_full that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@drv_selftest@live_gtt:
shard-glk: NOTRUN -> INCOMPLETE (k.org#198133, fdo#103359)
igt@drv_selftest@live_hangcheck:
shard-kbl: PASS -> DMESG-FAIL (fdo#106560)
igt@drv_selftest@mock_scatterlist:
shard-glk: NOTRUN -> DMESG-WARN (fdo#103667)
igt@gem_ppgtt@blt-vs-render-ctxn:
shard-kbl: PASS -> INCOMPLETE (fdo#106023, fdo#103665)
igt@kms_flip@2x-flip-vs-dpms:
shard-hsw: PASS -> DMESG-WARN (fdo#102614)
igt@kms_flip@2x-plain-flip-ts-check:
shard-hsw: PASS -> FAIL (fdo#100368) +1
igt@kms_flip@plain-flip-fb-recreate-interruptible:
shard-glk: PASS -> FAIL (fdo#100368)
igt@kms_flip_tiling@flip-x-tiled:
shard-glk: NOTRUN -> FAIL (fdo#103822, fdo#104724)
igt@kms_sysfs_edid_timing:
shard-glk: NOTRUN -> WARN (fdo#100047)
igt@perf_pmu@busy-accuracy-50-vcs1:
shard-snb: SKIP -> INCOMPLETE (fdo#105411)
==== Possible fixes ====
igt@drv_selftest@live_gtt:
shard-kbl: FAIL (fdo#105347) -> PASS
igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic:
shard-glk: FAIL (fdo#105454, fdo#106509) -> PASS
igt@kms_flip@2x-flip-vs-expired-vblank:
shard-glk: FAIL (fdo#105363) -> PASS
igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
shard-hsw: FAIL (fdo#102887) -> PASS
igt@kms_flip@flip-vs-expired-vblank-interruptible:
shard-hsw: FAIL (fdo#102887, fdo#105363) -> PASS
igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-plflip-blt:
shard-glk: FAIL (fdo#104724, fdo#103167) -> PASS
fdo#100047 https://bugs.freedesktop.org/show_bug.cgi?id=100047
fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
fdo#102887 https://bugs.freedesktop.org/show_bug.cgi?id=102887
fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#103665 https://bugs.freedesktop.org/show_bug.cgi?id=103665
fdo#103667 https://bugs.freedesktop.org/show_bug.cgi?id=103667
fdo#103822 https://bugs.freedesktop.org/show_bug.cgi?id=103822
fdo#104724 https://bugs.freedesktop.org/show_bug.cgi?id=104724
fdo#105347 https://bugs.freedesktop.org/show_bug.cgi?id=105347
fdo#105363 https://bugs.freedesktop.org/show_bug.cgi?id=105363
fdo#105411 https://bugs.freedesktop.org/show_bug.cgi?id=105411
fdo#105454 https://bugs.freedesktop.org/show_bug.cgi?id=105454
fdo#106023 https://bugs.freedesktop.org/show_bug.cgi?id=106023
fdo#106509 https://bugs.freedesktop.org/show_bug.cgi?id=106509
fdo#106560 https://bugs.freedesktop.org/show_bug.cgi?id=106560
k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
== Participating hosts (5 -> 5) ==
No changes in participating hosts
== Build changes ==
* Linux: CI_DRM_4309 -> Patchwork_9282
CI_DRM_4309: 2740c5b0d0f40092355b329a62ede8cced7f64b9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4517: e94ce40798e35d2e3c4494f50b617908066bbf8b @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_9282: e6bea8c90022b9d1b768a6ae2ba1ac3dd22502e5 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_9282/shards.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-18 22:43 ` Dhinakaran Pandiyan
@ 2018-06-19 8:46 ` Jani Nikula
0 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2018-06-19 8:46 UTC (permalink / raw)
To: dhinakaran.pandiyan, vathsala nagaraju
Cc: Puthikorn Voravootivat, intel-gfx, Ashutosh D Shukla,
Maulik V Vaghela, Rodrigo Vivi
On Mon, 18 Jun 2018, Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> wrote:
> On Mon, 2018-06-18 at 11:42 +0530, vathsala nagaraju wrote:
>> From: Vathsala Nagaraju <vathsala.nagaraju@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.pandiyan@intel.com>
Thanks for the patch and review, pushed to dinq with the checkpatch
warnings fixed while at it.
BR,
Jani.
>
>>
>> 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.nikula@intel.com>
>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>> Cc: Puthikorn Voravootivat <puthik@chromium.org>
>> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>> Cc: Ashutosh D Shukla <ashutosh.d.shukla@intel.com>
>> Cc: Maulik V Vaghela <maulik.v.vaghela@intel.com>
>>
>> Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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;
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-18 6:12 [PATCH] drm/i915/psr: Adds psrwake options for all platforms vathsala nagaraju
@ 2018-06-18 22:43 ` Dhinakaran Pandiyan
2018-06-19 8:46 ` Jani Nikula
0 siblings, 1 reply; 23+ messages in thread
From: Dhinakaran Pandiyan @ 2018-06-18 22:43 UTC (permalink / raw)
To: vathsala nagaraju, jani.nikula
Cc: Ashutosh D Shukla, Jani Nikula, intel-gfx, Rodrigo Vivi,
Puthikorn Voravootivat, Maulik V Vaghela
On Mon, 2018-06-18 at 11:42 +0530, vathsala nagaraju wrote:
> From: Vathsala Nagaraju <vathsala.nagaraju@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.pandiyan@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.nikula@intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Cc: Puthikorn Voravootivat <puthik@chromium.org>
> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> Cc: Ashutosh D Shukla <ashutosh.d.shukla@intel.com>
> Cc: Maulik V Vaghela <maulik.v.vaghela@intel.com>
>
> Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH] drm/i915/psr: Adds psrwake options for all platforms
@ 2018-06-18 6:12 vathsala nagaraju
2018-06-18 22:43 ` Dhinakaran Pandiyan
0 siblings, 1 reply; 23+ messages in thread
From: vathsala nagaraju @ 2018-06-18 6:12 UTC (permalink / raw)
To: dhinakaran.pandiyan, jani.nikula
Cc: Ashutosh D Shukla, Jani Nikula, intel-gfx, Rodrigo Vivi,
Puthikorn Voravootivat, Maulik V Vaghela
From: Vathsala Nagaraju <vathsala.nagaraju@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.
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.nikula@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Puthikorn Voravootivat <puthik@chromium.org>
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Cc: Ashutosh D Shukla <ashutosh.d.shukla@intel.com>
Cc: Maulik V Vaghela <maulik.v.vaghela@intel.com>
Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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;
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-15 8:10 ` Jani Nikula
@ 2018-06-15 18:14 ` Dhinakaran Pandiyan
0 siblings, 0 replies; 23+ messages in thread
From: Dhinakaran Pandiyan @ 2018-06-15 18:14 UTC (permalink / raw)
To: Jani Nikula, Nagaraju, Vathsala, Vivi, Rodrigo, Shukla,
Ashutosh D, Vaghela, Maulik V
Cc: puthik, intel-gfx
On Fri, 2018-06-15 at 11:10 +0300, Jani Nikula wrote:
> On Thu, 14 Jun 2018, Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.c
> om> wrote:
> >
> > On Thu, 2018-06-14 at 16:56 +0000, Nagaraju, Vathsala wrote:
> > >
> > > + Ashutosh(VBT team) + maulik
> > >
> > > 209 is confirmed version on kbl both by vbt team (Maulik) and
> > > google,
> > > so we had used it.
> > >
> > > DK's suggestion is
> > > if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
> > > IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
> > > /* new mapping */
> > >
> > > As per Ashutosh, 203 Is not the right version, 205 is fine , but
> > > user
> > > can still provide decimal value for SKL.
> >
> > I am confused, why does the commit message mention 203 then?
> Whatever the version, I want that to be required always. i.e.
>
> if (version >= N && (bunch of other conditions))
Using the version number as a guard makes sense but this means we need
a version number that's applicable to all platforms.
Vathsala,
Can you please confirm if all platforms that you want to use the new
mapping for have version numbers >= 205?
If that's the case, it should be easy to change the check to
if (version >= 205 && (IS_GEN9_BC() || IS_GEMINILAKE() || INTEL_GEN()
>= 10))
>
> *not*
>
> if (version >= N || (bunch of other conditions))
>
> BR,
> Jani.
>
> >
> >
> >
> > >
> > > Jani/Rodrigo, should we use 205 for SKL or drop SKL from the new
> > > mapping?
> > >
> > > -----Original Message-----
> > > From: Pandiyan, Dhinakaran
> > > Sent: Thursday, June 14, 2018 9:30 PM
> > > To: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Nikula, Jani
> > > <jani.nikula
> > > @intel.com>; Nagaraju, Vathsala <vathsala.nagaraju@intel.com>
> > > Cc: puthik@chromium.org; intel-gfx@lists.freedesktop.org
> > > Subject: Re: [PATCH] drm/i915/psr: Adds psrwake options for all
> > > platforms
> > >
> > > On Thu, 2018-06-14 at 11:59 +0530, Nagaraju, Vathsala wrote:
> > > >
> > > >
> > > >
> > > > On 6/13/2018 11:10 PM, Dhinakaran Pandiyan wrote:
> > > > >
> > > > >
> > > > >
> > > > > 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@
> > > > > > > in
> > > > > > > tel.
> > > > > > > co
> > > > > > > m> wrote:
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > On 6/12/2018 2:30 PM, Jani Nikula wrote:
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > On Tue, 12 Jun 2018, vathsala nagaraju
> > > > > > > > > <vathsala.nagaraju
> > > > > > > > > @i
> > > > > > > > > ntel .c
> > > > > > > > > om> wrote:
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > From: Vathsala Nagaraju <vathsala.nagaraju@intel.co
> > > > > > > > > > m>
> > > > > > > > > >
> > > > > > > > > > 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@intel.com>
> > > > > > > > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > > > > > > > Cc: Puthikorn Voravootivat <puthik@chromium.org>
> > > > > > > > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.
> > > > > > > > > > com>
> > > > > > > > > >
> > > > > > > > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju
> > > > > > > > > > @int
> > > > > > > > > > el
> > > > > > > > > > .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.
> > > > > Said laptop is a SKL with bdb version 205.
> > > > + ashutosh(VBT team)
> > > > Since VBT team cannot confirm version for SKL ,so skipped for
> > > > skylake.
> > > > I did a copy paste of the table provided by vbt team, will edit
> > > > for
> > > > skylake.
> > > >
> > > We are not going to get this right for all combinations, the best
> > > we
> > > can do is make sure things work in most cases. I prefer to err on
> > > the
> > > side of using the new mapping because when translated
> > > incorrectly, 3
> > > out of 4 values lead to >= intended training time. Given the fact
> > > that
> > > SKL PV releases also used the new mapping, I suggest you do this
> > >
> > > if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
> > > IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
> > > /* new mapping */
> > >
> > > }
> > >
> > > I don't know what the basis for the current check for version 209
> > > is,
> > > other than it was the version on the KBL you tested.
> > >
> > > -DK
> > >
> > >
> > >
> > >
> > >
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-14 17:28 ` Dhinakaran Pandiyan
@ 2018-06-15 8:10 ` Jani Nikula
2018-06-15 18:14 ` Dhinakaran Pandiyan
0 siblings, 1 reply; 23+ messages in thread
From: Jani Nikula @ 2018-06-15 8:10 UTC (permalink / raw)
To: dhinakaran.pandiyan, Nagaraju, Vathsala, Vivi, Rodrigo, Shukla,
Ashutosh D, Vaghela, Maulik V
Cc: puthik, intel-gfx
On Thu, 14 Jun 2018, Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> wrote:
> On Thu, 2018-06-14 at 16:56 +0000, Nagaraju, Vathsala wrote:
>> + Ashutosh(VBT team) + maulik
>>
>> 209 is confirmed version on kbl both by vbt team (Maulik) and google,
>> so we had used it.
>>
>> DK's suggestion is
>> if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
>> IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
>> /* new mapping */
>>
>> As per Ashutosh, 203 Is not the right version, 205 is fine , but user
>> can still provide decimal value for SKL.
>
> I am confused, why does the commit message mention 203 then?
Whatever the version, I want that to be required always. i.e.
if (version >= N && (bunch of other conditions))
*not*
if (version >= N || (bunch of other conditions))
BR,
Jani.
>
>
>> Jani/Rodrigo, should we use 205 for SKL or drop SKL from the new
>> mapping?
>>
>> -----Original Message-----
>> From: Pandiyan, Dhinakaran
>> Sent: Thursday, June 14, 2018 9:30 PM
>> To: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Nikula, Jani <jani.nikula
>> @intel.com>; Nagaraju, Vathsala <vathsala.nagaraju@intel.com>
>> Cc: puthik@chromium.org; intel-gfx@lists.freedesktop.org
>> Subject: Re: [PATCH] drm/i915/psr: Adds psrwake options for all
>> platforms
>>
>> On Thu, 2018-06-14 at 11:59 +0530, Nagaraju, Vathsala wrote:
>> >
>> >
>> > On 6/13/2018 11:10 PM, Dhinakaran Pandiyan wrote:
>> > >
>> > >
>> > > 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@
>> > > > > in
>> > > > > tel.
>> > > > > co
>> > > > > m> wrote:
>> > > > > >
>> > > > > >
>> > > > > >
>> > > > > > On 6/12/2018 2:30 PM, Jani Nikula wrote:
>> > > > > > >
>> > > > > > >
>> > > > > > >
>> > > > > > > On Tue, 12 Jun 2018, vathsala nagaraju <vathsala.nagaraju
>> > > > > > > @i
>> > > > > > > ntel .c
>> > > > > > > om> wrote:
>> > > > > > > >
>> > > > > > > >
>> > > > > > > >
>> > > > > > > > From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
>> > > > > > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>> > > > > > > > Cc: Puthikorn Voravootivat <puthik@chromium.org>
>> > > > > > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>> > > > > > > >
>> > > > > > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@int
>> > > > > > > > el
>> > > > > > > > .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.
>> > > Said laptop is a SKL with bdb version 205.
>> > + ashutosh(VBT team)
>> > Since VBT team cannot confirm version for SKL ,so skipped for
>> > skylake.
>> > I did a copy paste of the table provided by vbt team, will edit for
>> > skylake.
>> >
>> We are not going to get this right for all combinations, the best we
>> can do is make sure things work in most cases. I prefer to err on the
>> side of using the new mapping because when translated incorrectly, 3
>> out of 4 values lead to >= intended training time. Given the fact
>> that
>> SKL PV releases also used the new mapping, I suggest you do this
>>
>> if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
>> IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
>> /* new mapping */
>>
>> }
>>
>> I don't know what the basis for the current check for version 209 is,
>> other than it was the version on the KBL you tested.
>>
>> -DK
>>
>>
>>
>>
>>
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-14 16:56 ` Nagaraju, Vathsala
@ 2018-06-14 17:28 ` Dhinakaran Pandiyan
2018-06-15 8:10 ` Jani Nikula
0 siblings, 1 reply; 23+ messages in thread
From: Dhinakaran Pandiyan @ 2018-06-14 17:28 UTC (permalink / raw)
To: Nagaraju, Vathsala, Vivi, Rodrigo, Nikula, Jani, Shukla,
Ashutosh D, Vaghela, Maulik V
Cc: puthik, intel-gfx
On Thu, 2018-06-14 at 16:56 +0000, Nagaraju, Vathsala wrote:
> + Ashutosh(VBT team) + maulik
>
> 209 is confirmed version on kbl both by vbt team (Maulik) and google,
> so we had used it.
>
> DK's suggestion is
> if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
> IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
> /* new mapping */
>
> As per Ashutosh, 203 Is not the right version, 205 is fine , but user
> can still provide decimal value for SKL.
I am confused, why does the commit message mention 203 then?
> Jani/Rodrigo, should we use 205 for SKL or drop SKL from the new
> mapping?
>
> -----Original Message-----
> From: Pandiyan, Dhinakaran
> Sent: Thursday, June 14, 2018 9:30 PM
> To: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Nikula, Jani <jani.nikula
> @intel.com>; Nagaraju, Vathsala <vathsala.nagaraju@intel.com>
> Cc: puthik@chromium.org; intel-gfx@lists.freedesktop.org
> Subject: Re: [PATCH] drm/i915/psr: Adds psrwake options for all
> platforms
>
> On Thu, 2018-06-14 at 11:59 +0530, Nagaraju, Vathsala wrote:
> >
> >
> > On 6/13/2018 11:10 PM, Dhinakaran Pandiyan wrote:
> > >
> > >
> > > 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@
> > > > > in
> > > > > tel.
> > > > > co
> > > > > m> wrote:
> > > > > >
> > > > > >
> > > > > >
> > > > > > On 6/12/2018 2:30 PM, Jani Nikula wrote:
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > On Tue, 12 Jun 2018, vathsala nagaraju <vathsala.nagaraju
> > > > > > > @i
> > > > > > > ntel .c
> > > > > > > om> wrote:
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
> > > > > > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > > > > > Cc: Puthikorn Voravootivat <puthik@chromium.org>
> > > > > > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> > > > > > > >
> > > > > > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@int
> > > > > > > > el
> > > > > > > > .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.
> > > Said laptop is a SKL with bdb version 205.
> > + ashutosh(VBT team)
> > Since VBT team cannot confirm version for SKL ,so skipped for
> > skylake.
> > I did a copy paste of the table provided by vbt team, will edit for
> > skylake.
> >
> We are not going to get this right for all combinations, the best we
> can do is make sure things work in most cases. I prefer to err on the
> side of using the new mapping because when translated incorrectly, 3
> out of 4 values lead to >= intended training time. Given the fact
> that
> SKL PV releases also used the new mapping, I suggest you do this
>
> if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
> IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
> /* new mapping */
>
> }
>
> I don't know what the basis for the current check for version 209 is,
> other than it was the version on the KBL you tested.
>
> -DK
>
>
>
>
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-14 16:00 ` Pandiyan, Dhinakaran
2018-06-14 16:48 ` Rodrigo Vivi
@ 2018-06-14 16:56 ` Nagaraju, Vathsala
2018-06-14 17:28 ` Dhinakaran Pandiyan
1 sibling, 1 reply; 23+ messages in thread
From: Nagaraju, Vathsala @ 2018-06-14 16:56 UTC (permalink / raw)
To: Pandiyan, Dhinakaran, Vivi, Rodrigo, Nikula, Jani, Shukla,
Ashutosh D, Vaghela, Maulik V
Cc: puthik, intel-gfx
+ Ashutosh(VBT team) + maulik
209 is confirmed version on kbl both by vbt team (Maulik) and google, so we had used it.
DK's suggestion is
if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
/* new mapping */
As per Ashutosh, 203 Is not the right version, 205 is fine , but user can still provide decimal value for SKL.
Jani/Rodrigo, should we use 205 for SKL or drop SKL from the new mapping?
-----Original Message-----
From: Pandiyan, Dhinakaran
Sent: Thursday, June 14, 2018 9:30 PM
To: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Nikula, Jani <jani.nikula@intel.com>; Nagaraju, Vathsala <vathsala.nagaraju@intel.com>
Cc: puthik@chromium.org; intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
On Thu, 2018-06-14 at 11:59 +0530, Nagaraju, Vathsala wrote:
>
> On 6/13/2018 11:10 PM, Dhinakaran Pandiyan wrote:
> >
> > 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@in
> > > > tel.
> > > > co
> > > > m> wrote:
> > > > >
> > > > >
> > > > > On 6/12/2018 2:30 PM, Jani Nikula wrote:
> > > > > >
> > > > > >
> > > > > > On Tue, 12 Jun 2018, vathsala nagaraju <vathsala.nagaraju@i
> > > > > > ntel .c
> > > > > > om> wrote:
> > > > > > >
> > > > > > >
> > > > > > > From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
> > > > > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > > > > Cc: Puthikorn Voravootivat <puthik@chromium.org>
> > > > > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> > > > > > >
> > > > > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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.
> > Said laptop is a SKL with bdb version 205.
> + ashutosh(VBT team)
> Since VBT team cannot confirm version for SKL ,so skipped for
> skylake.
> I did a copy paste of the table provided by vbt team, will edit for
> skylake.
>
We are not going to get this right for all combinations, the best we
can do is make sure things work in most cases. I prefer to err on the
side of using the new mapping because when translated incorrectly, 3
out of 4 values lead to >= intended training time. Given the fact that
SKL PV releases also used the new mapping, I suggest you do this
if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
/* new mapping */
}
I don't know what the basis for the current check for version 209 is,
other than it was the version on the KBL you tested.
-DK
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-14 16:00 ` Pandiyan, Dhinakaran
@ 2018-06-14 16:48 ` Rodrigo Vivi
2018-06-14 16:56 ` Nagaraju, Vathsala
1 sibling, 0 replies; 23+ messages in thread
From: Rodrigo Vivi @ 2018-06-14 16:48 UTC (permalink / raw)
To: Pandiyan, Dhinakaran; +Cc: Nikula, Jani, intel-gfx, puthik
On Thu, Jun 14, 2018 at 09:00:15AM -0700, Pandiyan, Dhinakaran wrote:
> On Thu, 2018-06-14 at 11:59 +0530, Nagaraju, Vathsala wrote:
> >
> > On 6/13/2018 11:10 PM, Dhinakaran Pandiyan wrote:
> > >
> > > 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@in
> > > > > tel.
> > > > > co
> > > > > m> wrote:
> > > > > >
> > > > > >
> > > > > > On 6/12/2018 2:30 PM, Jani Nikula wrote:
> > > > > > >
> > > > > > >
> > > > > > > On Tue, 12 Jun 2018, vathsala nagaraju <vathsala.nagaraju@i
> > > > > > > ntel
> > > > > > > .c
> > > > > > > om> wrote:
> > > > > > > >
> > > > > > > >
> > > > > > > > From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
> > > > > > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > > > > > Cc: Puthikorn Voravootivat <puthik@chromium.org>
> > > > > > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> > > > > > > >
> > > > > > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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.
> > > Said laptop is a SKL with bdb version 205.
> > + ashutosh(VBT team)
> > Since VBT team cannot confirm version for SKL ,so skipped for
> > skylake.
> > I did a copy paste of the table provided by vbt team, will edit for
> > skylake.
> >
> We are not going to get this right for all combinations,
:(
it seems we have to live with it
> the best we
> can do is make sure things work in most cases.
I agree.
> I prefer to err on the
> side of using the new mapping because when translated incorrectly, 3
> out of 4 values lead to >= intended training time. Given the fact that
> SKL PV releases also used the new mapping, I suggest you do this
>
> if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
> IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
+1
> /* new mapping */
>
> }
>
> I don't know what the basis for the current check for version 209 is,
> other than it was the version on the KBL you tested.
>
> -DK
>
>
>
>
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-14 6:29 ` Nagaraju, Vathsala
@ 2018-06-14 16:00 ` Pandiyan, Dhinakaran
2018-06-14 16:48 ` Rodrigo Vivi
2018-06-14 16:56 ` Nagaraju, Vathsala
0 siblings, 2 replies; 23+ messages in thread
From: Pandiyan, Dhinakaran @ 2018-06-14 16:00 UTC (permalink / raw)
To: Vivi, Rodrigo, Nikula, Jani, Nagaraju, Vathsala; +Cc: puthik, intel-gfx
On Thu, 2018-06-14 at 11:59 +0530, Nagaraju, Vathsala wrote:
>
> On 6/13/2018 11:10 PM, Dhinakaran Pandiyan wrote:
> >
> > 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@in
> > > > tel.
> > > > co
> > > > m> wrote:
> > > > >
> > > > >
> > > > > On 6/12/2018 2:30 PM, Jani Nikula wrote:
> > > > > >
> > > > > >
> > > > > > On Tue, 12 Jun 2018, vathsala nagaraju <vathsala.nagaraju@i
> > > > > > ntel
> > > > > > .c
> > > > > > om> wrote:
> > > > > > >
> > > > > > >
> > > > > > > From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
> > > > > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > > > > Cc: Puthikorn Voravootivat <puthik@chromium.org>
> > > > > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> > > > > > >
> > > > > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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.
> > Said laptop is a SKL with bdb version 205.
> + ashutosh(VBT team)
> Since VBT team cannot confirm version for SKL ,so skipped for
> skylake.
> I did a copy paste of the table provided by vbt team, will edit for
> skylake.
>
We are not going to get this right for all combinations, the best we
can do is make sure things work in most cases. I prefer to err on the
side of using the new mapping because when translated incorrectly, 3
out of 4 values lead to >= intended training time. Given the fact that
SKL PV releases also used the new mapping, I suggest you do this
if ((bdb->version >= 203 && IS_GEN9_BC(dev_priv)) ||
IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10) {
/* new mapping */
}
I don't know what the basis for the current check for version 209 is,
other than it was the version on the KBL you tested.
-DK
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-13 17:40 ` Dhinakaran Pandiyan
2018-06-14 6:29 ` Nagaraju, Vathsala
2018-06-14 7:38 ` Nagaraju, Vathsala
@ 2018-06-14 7:46 ` Nagaraju, Vathsala
2 siblings, 0 replies; 23+ messages in thread
From: Nagaraju, Vathsala @ 2018-06-14 7:46 UTC (permalink / raw)
To: dhinakaran.pandiyan, Jani Nikula, rodrigo.vivi
Cc: Puthikorn Voravootivat, intel-gfx
On 6/13/2018 11:10 PM, Dhinakaran Pandiyan wrote:
> 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@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.nikula@intel.com>
>>>>>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>>>>>> Cc: Puthikorn Voravootivat <puthik@chromium.org>
>>>>>> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>>>>>>
>>>>>> Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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.
> Said laptop is a SKL with bdb version 205.
I did a copy paste of the table provided by vbt team, will edit for skylake
Since VBT team cannot confirm version for SKL ,so skipping for skylake.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-13 17:40 ` Dhinakaran Pandiyan
2018-06-14 6:29 ` Nagaraju, Vathsala
@ 2018-06-14 7:38 ` Nagaraju, Vathsala
2018-06-14 7:46 ` Nagaraju, Vathsala
2 siblings, 0 replies; 23+ messages in thread
From: Nagaraju, Vathsala @ 2018-06-14 7:38 UTC (permalink / raw)
To: dhinakaran.pandiyan, ashutosh.d.shukla, Jani Nikula, rodrigo.vivi
Cc: Puthikorn Voravootivat, intel-gfx
On 6/13/2018 11:10 PM, Dhinakaran Pandiyan wrote:
> 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@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.nikula@intel.com>
>>>>>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>>>>>> Cc: Puthikorn Voravootivat <puthik@chromium.org>
>>>>>> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>>>>>>
>>>>>> Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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.
> Said laptop is a SKL with bdb version 205.
+ ashutosh(VBT team)
Since VBT team cannot confirm version for SKL ,so skipped for skylake.
I did a copy paste of the table provided by vbt team, will edit for skylake
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-13 17:40 ` Dhinakaran Pandiyan
@ 2018-06-14 6:29 ` Nagaraju, Vathsala
2018-06-14 16:00 ` Pandiyan, Dhinakaran
2018-06-14 7:38 ` Nagaraju, Vathsala
2018-06-14 7:46 ` Nagaraju, Vathsala
2 siblings, 1 reply; 23+ messages in thread
From: Nagaraju, Vathsala @ 2018-06-14 6:29 UTC (permalink / raw)
To: dhinakaran.pandiyan, Jani Nikula, rodrigo.vivi
Cc: Puthikorn Voravootivat, intel-gfx
On 6/13/2018 11:10 PM, Dhinakaran Pandiyan wrote:
> 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@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.nikula@intel.com>
>>>>>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>>>>>> Cc: Puthikorn Voravootivat <puthik@chromium.org>
>>>>>> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>>>>>>
>>>>>> Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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.
> Said laptop is a SKL with bdb version 205.
+ ashutosh(VBT team)
Since VBT team cannot confirm version for SKL ,so skipped for skylake.
I did a copy paste of the table provided by vbt team, will edit for skylake.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-13 17:32 ` Dhinakaran Pandiyan
@ 2018-06-13 17:40 ` Dhinakaran Pandiyan
2018-06-14 6:29 ` Nagaraju, Vathsala
` (2 more replies)
0 siblings, 3 replies; 23+ messages in thread
From: Dhinakaran Pandiyan @ 2018-06-13 17:40 UTC (permalink / raw)
To: Jani Nikula, Nagaraju, Vathsala, rodrigo.vivi
Cc: Puthikorn Voravootivat, intel-gfx
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@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.nikula@intel.com>
> > > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > > Cc: Puthikorn Voravootivat <puthik@chromium.org>
> > > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> > > > >
> > > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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.
Said laptop is a SKL with bdb version 205.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-13 6:41 ` Jani Nikula
@ 2018-06-13 17:32 ` Dhinakaran Pandiyan
2018-06-13 17:40 ` Dhinakaran Pandiyan
0 siblings, 1 reply; 23+ messages in thread
From: Dhinakaran Pandiyan @ 2018-06-13 17:32 UTC (permalink / raw)
To: Jani Nikula, Nagaraju, Vathsala, rodrigo.vivi
Cc: Puthikorn Voravootivat, intel-gfx
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@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.nikula@intel.com>
> > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > Cc: Puthikorn Voravootivat <puthik@chromium.org>
> > > > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> > > >
> > > > Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-12 5:19 vathsala nagaraju
2018-06-12 9:00 ` Jani Nikula
@ 2018-06-13 10:42 ` Ville Syrjälä
1 sibling, 0 replies; 23+ messages in thread
From: Ville Syrjälä @ 2018-06-13 10:42 UTC (permalink / raw)
To: vathsala nagaraju
Cc: jani.nikula, Puthikorn Voravootivat, intel-gfx,
Dhinakaran Pandiyan, rodrigo.vivi
On Tue, Jun 12, 2018 at 10:49:09AM +0530, vathsala nagaraju wrote:
> From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Cc: Puthikorn Voravootivat <puthik@chromium.org>
> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>
> Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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))) {
That doesn't match your commit message.
> switch (psr_table->tp1_wakeup_time) {
> case 0:
> dev_priv->vbt.psr.tp1_wakeup_time_us = 500;
> --
> 1.9.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-13 6:30 ` Nagaraju, Vathsala
@ 2018-06-13 6:41 ` Jani Nikula
2018-06-13 17:32 ` Dhinakaran Pandiyan
0 siblings, 1 reply; 23+ messages in thread
From: Jani Nikula @ 2018-06-13 6:41 UTC (permalink / raw)
To: Nagaraju, Vathsala, rodrigo.vivi
Cc: Puthikorn Voravootivat, intel-gfx, Dhinakaran Pandiyan
On Wed, 13 Jun 2018, "Nagaraju, Vathsala" <vathsala.nagaraju@intel.com> wrote:
> On 6/12/2018 2:30 PM, Jani Nikula wrote:
>> On Tue, 12 Jun 2018, vathsala nagaraju <vathsala.nagaraju@intel.com> wrote:
>>> From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
>>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>>> Cc: Puthikorn Voravootivat <puthik@chromium.org>
>>> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>>>
>>> Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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.
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;
>
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-12 9:00 ` Jani Nikula
@ 2018-06-13 6:30 ` Nagaraju, Vathsala
2018-06-13 6:41 ` Jani Nikula
0 siblings, 1 reply; 23+ messages in thread
From: Nagaraju, Vathsala @ 2018-06-13 6:30 UTC (permalink / raw)
To: Jani Nikula, rodrigo.vivi
Cc: Puthikorn Voravootivat, intel-gfx, Dhinakaran Pandiyan
On 6/12/2018 2:30 PM, Jani Nikula wrote:
> On Tue, 12 Jun 2018, vathsala nagaraju <vathsala.nagaraju@intel.com> wrote:
>> From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
>> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
>> Cc: Puthikorn Voravootivat <puthik@chromium.org>
>> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>>
>> Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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?
> 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
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH] drm/i915/psr: Adds psrwake options for all platforms
2018-06-12 5:19 vathsala nagaraju
@ 2018-06-12 9:00 ` Jani Nikula
2018-06-13 6:30 ` Nagaraju, Vathsala
2018-06-13 10:42 ` Ville Syrjälä
1 sibling, 1 reply; 23+ messages in thread
From: Jani Nikula @ 2018-06-12 9:00 UTC (permalink / raw)
To: vathsala nagaraju, rodrigo.vivi
Cc: Puthikorn Voravootivat, intel-gfx, Dhinakaran Pandiyan
On Tue, 12 Jun 2018, vathsala nagaraju <vathsala.nagaraju@intel.com> wrote:
> From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Cc: Puthikorn Voravootivat <puthik@chromium.org>
> Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
>
> Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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.
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;
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH] drm/i915/psr: Adds psrwake options for all platforms
@ 2018-06-12 5:19 vathsala nagaraju
2018-06-12 9:00 ` Jani Nikula
2018-06-13 10:42 ` Ville Syrjälä
0 siblings, 2 replies; 23+ messages in thread
From: vathsala nagaraju @ 2018-06-12 5:19 UTC (permalink / raw)
To: jani.nikula, rodrigo.vivi
Cc: Puthikorn Voravootivat, intel-gfx, Dhinakaran Pandiyan
From: Vathsala Nagaraju <vathsala.nagaraju@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.nikula@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Puthikorn Voravootivat <puthik@chromium.org>
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Signed-off-by: Vathsala Nagaraju <vathsala.nagaraju@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))) {
switch (psr_table->tp1_wakeup_time) {
case 0:
dev_priv->vbt.psr.tp1_wakeup_time_us = 500;
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 23+ messages in thread
end of thread, other threads:[~2018-06-19 8:46 UTC | newest]
Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-13 8:03 [PATCH] drm/i915/psr: Adds psrwake options for all platforms vathsala nagaraju
2018-06-13 8:18 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/psr: Adds psrwake options for all platforms (rev2) Patchwork
2018-06-13 8:33 ` ✓ Fi.CI.BAT: success " Patchwork
2018-06-13 9:22 ` ✓ Fi.CI.IGT: " Patchwork
-- strict thread matches above, loose matches on Subject: below --
2018-06-18 6:12 [PATCH] drm/i915/psr: Adds psrwake options for all platforms vathsala nagaraju
2018-06-18 22:43 ` Dhinakaran Pandiyan
2018-06-19 8:46 ` Jani Nikula
2018-06-12 5:19 vathsala nagaraju
2018-06-12 9:00 ` Jani Nikula
2018-06-13 6:30 ` Nagaraju, Vathsala
2018-06-13 6:41 ` Jani Nikula
2018-06-13 17:32 ` Dhinakaran Pandiyan
2018-06-13 17:40 ` Dhinakaran Pandiyan
2018-06-14 6:29 ` Nagaraju, Vathsala
2018-06-14 16:00 ` Pandiyan, Dhinakaran
2018-06-14 16:48 ` Rodrigo Vivi
2018-06-14 16:56 ` Nagaraju, Vathsala
2018-06-14 17:28 ` Dhinakaran Pandiyan
2018-06-15 8:10 ` Jani Nikula
2018-06-15 18:14 ` Dhinakaran Pandiyan
2018-06-14 7:38 ` Nagaraju, Vathsala
2018-06-14 7:46 ` Nagaraju, Vathsala
2018-06-13 10:42 ` Ville Syrjälä
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.