On Thu, 2023-06-29 at 11:47 +0200, Thomas Witt wrote: > On 28/06/2023 12:59, Mika Westerberg wrote: > > I wonder if the patch actually helps here now because the reason we want > > to add it back is that it allows the CPU to enter lower power states and > > thus reducing the power consumption in S2idle too. Do you observe that > > when you have the patch applied? > > No joy. I did not check what its actually doing, but the computer takes > about 150mA over the charger at idle with screen off and 140mA in > suspend. With mem_sleep set to "deep", it takes about 20mA in suspend. > All with the battery at 100%, at 19.7V. > > So I guess I want to keep the "deep" setting in my cmdline. It's likely something is not entering the right state. You can try the following script to check what the cause may be. https://github.com/intel/S0ixSelftestTool David > > BR > Thomas >
