linux-kernel - Re: [PATCH] i2c: tegra: Don't mark devices with pins as IRQ safe

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] [day] [month] [year] [list]

Message-ID: <13293829.O9o76ZdvQC@senjougahara>
Date: Fri, 06 Feb 2026 12:38:08 +0900
From: Mikko Perttunen <mperttunen@...dia.com>
To: Andi Shyti <andi.shyti@...nel.org>
Cc: Dmitry Osipenko <digetx@...il.com>,
 Thierry Reding <thierry.reding@...il.com>,
 Jonathan Hunter <jonathanh@...dia.com>, linux-i2c@...r.kernel.org,
 linux-tegra@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] i2c: tegra: Don't mark devices with pins as IRQ safe

On Thursday, February 5, 2026 7:47 PM Andi Shyti wrote:
> Hi Mikko,
> 
> On Mon, Feb 02, 2026 at 01:42:34PM +0900, Mikko Perttunen wrote:
> > I2C devices with associated pinctrl states (DPAUX I2C controllers)
> > will change pinctrl state during runtime PM. This requires taking
> > a mutex, so these devices cannot be marked as IRQ safe.
> > 
> > Discovered through lockdep.
> 
> I removed this sentence...

Sorry about that.

> 
> > Signed-off-by: Mikko Perttunen <mperttunen@...dia.com>
> 
> ... and merged the patch in i2c/i2c-host-2.
> 
> Thanks,
> Andi
> 

Thank you!

Looks like there's a COMPILE_TEST failure so we'll need to do something like ..

diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index e11d50750e63..09152088b79f 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -1211,7 +1211,7 @@ config I2C_SYNQUACER
 
 config I2C_TEGRA
 	tristate "NVIDIA Tegra internal I2C controller"
-	depends on ARCH_TEGRA || (COMPILE_TEST && (ARC || ARM || ARM64 || M68K || RISCV || SUPERH || SPARC))
+	depends on ARCH_TEGRA || (COMPILE_TEST && (ARC || ARM || ARM64 || M68K || RISCV || SUPERH || SPARC) && PINCTRL)
 	# COMPILE_TEST needs architectures with readsX()/writesX() primitives
 	help
 	  If you say yes to this option, support will be included for the
-- 
2.52.0

Please let me know how you'd like to go about it.

Mikko