a long time ago I had agreed with Daniel that we would only add new
platforms after it was enabled by default on previous platforms.
a big reason for that is that we was willing to reduce the platforms
to validate and do better validation one by one before enabling.

However now I believe it would be beneficial to have that supported
added so we can get more brave people using in different platforms so
we could capture more corner cases before we enable it by default.
Also we can still enable by default one platform at time if needed.

So:

Acked-by: Rodrigo Vivi <rodrigo.v...@intel.com>

I also checked the spec to see if there was anything else new or
different for these platforms and didn't find anything so:

Reviewed-by: Rodrigo Vivi <rodrigo.v...@intel.com>

But let's wait a bit to merge to give Daniel or others a time to nack ;)

Cheers,
Rodrigo.


On Tue, Aug 8, 2017 at 3:09 AM, David Weinehall
<david.weineh...@linux.intel.com> wrote:
> While testing Jim Bride's latest batch of PSR patches I noticed
> that gen9lp doesn't include the has_psr flag, and that our GLK
> system thus reported PSR as unsupported.
>
> This patch simply adds has_psr.
>
> Signed-off-by: David Weinehall <david.weineh...@linux.intel.com>
> ---
>  drivers/gpu/drm/i915/i915_pci.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> index 09d97e0990b7..11f0e8aa1fe4 100644
> --- a/drivers/gpu/drm/i915/i915_pci.c
> +++ b/drivers/gpu/drm/i915/i915_pci.c
> @@ -377,6 +377,7 @@ static const struct intel_device_info 
> intel_skylake_gt3_info = {
>         .has_ddi = 1, \
>         .has_fpga_dbg = 1, \
>         .has_fbc = 1, \
> +       .has_psr = 1, \
>         .has_runtime_pm = 1, \
>         .has_pooled_eu = 0, \
>         .has_csr = 1, \
> --
> 2.14.0
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to