Re: [PATCH 1/2] platform/x86: Add the VLV ISP PCI ID to atomisp2_pm

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 03-12-18 20:04, Andy Shevchenko wrote:
On Sun, Dec 02, 2018 at 02:36:24PM +0100, Hans de Goede wrote:
On 29-11-18 18:44, Ville Syrjala wrote:
From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>

If the ISP is exposed as a PCI device VLV machines need the
same treatment as CHV machines to power gate the ISP. Otherwise
s0ix will not work.

Thank you for the patches, both look good to me:

Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>

I've added both patches and the "drm/i915: Assert that VED
and ISP are power gated" patch to me personal tree, which I
regularly test on various BYT/CHT devices.  If I notice
anything wrt these patches I will let you know.

I have pushed them to my review and testing queue, thanks!

P.S. We have some time to hear from Hans if something wrong with them.

Sofar I've seen no adverse side-effects from these, but I've only
tested on 2 different BYT models sofar.

p.s.

Did I respond to the remark in the commit msg about there still
being an error with lspci ?  I think I may have forgotten to
respond to that bit.  So let me reply here, I think that living
with the error message showing up in dmesg on a lspci is fine,
much better then the mess which it will be to fix it.

Regards,

Hans




[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux