On 30/06/2023 12:41, Mika Westerberg wrote:
@Thomas, below is an updated patch. I wonder if you could try it out?
This one restores L1 substates first and then L0s/L1 as the spec
suggests. If this does not work, them I'm not sure what to do because
now we should be doing exactly what the spec is saying (unless I
misinterpret something):
- Write L1 enables on the upstream component first then downstream
(this is taken care by the parent child order of the Linux PM).
- Program L1 SS before L1 enables
- Program L1 SS enables after rest of the fields in the capability
Sadly, same as before. With s2idle, power consumption stays up, but
suspend/resume works, with deep it does not correctly suspend the PCI
devices.
