linux-kernel - Re: [REGRESSION] `lis3lv02d_i2c_suspend()` causes `unbalanced disables for regulator-dummy` and `Failed to disable Vdd

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Thu, 8 Feb 2024 14:34:30 +0000
From: Mark Brown <broonie@...nel.org>
To: Paul Menzel <pmenzel@...gen.mpg.de>
Cc: Wolfram Sang <wsa+renesas@...g-engineering.com>,
	Pali Rohár <pali@...nel.org>,
	Andi Shyti <andi.shyti@...nel.org>, linux-i2c@...r.kernel.org,
	Liam Girdwood <lgirdwood@...il.com>,
	LKML <linux-kernel@...r.kernel.org>,
	Hans de Goede <hdegoede@...hat.com>, regressions@...ts.linux.dev
Subject: Re: [REGRESSION] `lis3lv02d_i2c_suspend()` causes `unbalanced
 disables for regulator-dummy` and `Failed to disable Vdd_IO: -EIO`

On Fri, Feb 02, 2024 at 12:38:04PM +0100, Paul Menzel wrote:

> Testing commit 2f189493ae32 (i2c: i801: Add lis3lv02d for Dell XPS 15 7590)
> [1], it’s very likely this commit, it turns out, that Linux logs the warning
> below during ACPI S3 suspend:

The driver is just buggy here AFAICT, it's powering off the device in
both runtime suspend and runtime resume so if the device is runtime
suspended when system suspend happens then it'll power off the device
again.  The runtime suspend and system suspend need to talk to each
other here.

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)