From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f49.google.com (mail-wr1-f49.google.com [209.85.221.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 08CE52868AB for ; Sun, 23 Aug 2026 18:08:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787508521; cv=none; b=thZ/D9Y8loQf6Pk2RtjRNAfQDBouZ3Rc98+x1y73BbmZTNj6sPg16JFcBXM8AdxoJxsRcn/6VEuPBinEWXT0dG3fhbB9tUfEuPWHpyhaC2owymigTS5ueCSC1GX4m5dzQWwPRDbuKKgt6TxZYfQqfdU/Ip+M6t30NXPBfho4z+M= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787508521; c=relaxed/simple; bh=d4+SpXzXQg1I0AEgLHL/3ZdYMeyuisXWriRMDooKRJY=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=SDLBKCJBT1s3WeVY/ExA0WsBZ6iQe9fAQem4o95q5URQAxaZEXrE4HSY6555xeTvOfedilz2E3bDvPIZ44i+ivvDUNGRo6KQVJw45H3c1qD0DB4yGbQnJPRfi81FQr2cTI0v3C3oq+im3XNADHYkJQfqV2aa5lztEds2xuNHWCU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=P5rx7kcI; arc=none smtp.client-ip=209.85.221.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="P5rx7kcI" Received: by mail-wr1-f49.google.com with SMTP id ffacd0b85a97d-47f703a9d05so1311016f8f.0 for ; Sun, 23 Aug 2026 11:08:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1787508518; x=1788113318; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:sender:from:to:cc:subject:date:message-id:reply-to :content-type; bh=3gq37rj4bzCyBTGwM7oPPXQqhYUmg5gV71MEIz1jHkg=; b=P5rx7kcIwNuHZI6SyFCZXqs+zxJIguVERNSeE8X/tZT44ZhdN+06Xh63Y4poRxK7ya L3y8QPC3knC29GrkYdnQLYBGf/7JyfoU7U0i40giLLeMOrbkui9hFcZUI1gTmbTp2pga uUaEmXt+Yqfc2gTo/iacYVCA9LzLk7KHmZNPjoooSoJI9/z4l1v9y5xMndGl/0TXvxmP EKc11aBdYf9IMzMBOFKXIB0SWKLybSTXXXfZHkr5zh7Brc7+UBPqBteCrLkY6ZACK0sW ahAfeaL9EPwgH5y8MxKFuN0m8W/2dOJ5w+R0va2CwQIxX8XjOwqumcPDFNs9olqLRX3O vOLQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787508518; x=1788113318; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:sender:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=3gq37rj4bzCyBTGwM7oPPXQqhYUmg5gV71MEIz1jHkg=; b=awaa/o30VnQO7aoUEfTb05McD0l+HybkKIytSxh8a3cwz7YNUQ7dSXHT8ATErC9sKu o5eaWpQCQBlXc63MXJi/eyqA6MJ2xuha+y/AmNMurrFCkzdWpSfgJ9H4ERkOr+ZXvdYx 7/IdWg/OeKEhpx1Xw7uaebjgz1tD3+qFEgdstjGFQR+htPHXYvFLzKiYk5/FVG/n+rAB 1vS9GiR0xyiqrBkOyuTMISLoIltnRVAUkeP4z7rqoUzEm1YX/j6umMqG0Fq/eJ79B25M DdP3dXr7kyrZ5S1SAoczzMSZIC7sz3QvTBuw1gfnvXH3HdRgC7Ev7JGH+yD8EjYUKW8M EjLA== X-Forwarded-Encrypted: i=1; AHgh+Rrk7+p0PIw2/fo4iPGFwEtB7Mp6Vln1HcyB/uuRtyB+jEH0CnO5xRGbTjkyr8lD83otAFyW0Wt+gTbTrJs=@vger.kernel.org X-Gm-Message-State: AFuF++mYD+5QWkgswSqa6dDZG5/YvZmCo5DK1tUAFw4M7ckFiQae+4Ra WZp/WDdJ1Ayv+/M4o79w2IMVmXsiL4WjBG2Y/IkZiru+UyyhKSgD1fLq X-Gm-Gg: AR+sD11WngagWqfRW91EtUw0BK7bIxA7KptMh9+9udj0kNF8q70ZRdBC9NRUOl1EoVE hWEUFaWxyVDj/4Jz/4A2o0whRcu1LXRwtom3k4NKkhtEParnqrLwW9Zk+vxaFrU+seSoG8TR5ea oBgXCKmvmzfsGdHsMZMrtWWzJ3RXAeE4CfKdThULAPfh00+Bjg0KV0kUg+uCeeKj5wcdHAFSZ55 L4/XmrYzHijpSwVDKaxPIHySU0MNbqQvN+GCFgugcfHXg97WPHZ1couemViqckjr8/mrqOD+AGi vqaFVEfsQLtEp7LSV4dkHQZa3e+FnBSoX+MeWUTW77iPC4PZEPlVuXmsKIzhn35gZD9kHB9EWjQ 8qXH+HcrjhwOsS5ngVAU1oDLi4u/p0ty/vjfDYwxoPgZ1fgifGAmb5PIfRi6ZGgkKcdqgL7AzOg 5UepRIsrqNdMajBR7UI7CJN1iQRFXvxQE4Xn6vjziyafF6Jusgg9aMHUor66tbYYrDfOv2CHZFp /cmv+hbZnh2FPG4fqIN9pp2au1bckKuzajT4zG0dY3tXF4ocWz4 X-Received: by 2002:a05:6000:310f:b0:47f:f4b9:88b7 with SMTP id ffacd0b85a97d-482c0b58818mr25065061f8f.1.1787508518013; Sun, 23 Aug 2026 11:08:38 -0700 (PDT) Received: from nixos-office (195-23-151-163.net.novis.pt. [195.23.151.163]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-482d4710ac5sm219571f8f.32.2026.08.23.11.08.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 23 Aug 2026 11:08:37 -0700 (PDT) Sender: Julian Braha From: Julian Braha To: florian.fainelli@broadcom.com, rjui@broadcom.com, sbranden@broadcom.com, linux@armlinux.org.uk, tglx@kernel.org, Frank.Li@nxp.com, s.hauer@pengutronix.de Cc: bcm-kernel-feedback-list@broadcom.com, radu@rendec.net, kernel@pengutronix.de, festevam@gmail.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, imx@lists.linux.dev, arnd@arndb.de, maz@kernel.org, sudeep.holla@kernel.org, mark.rutland@arm.com, lpieralisi@kernel.org, Julian Braha Subject: [PATCH v2] irqchip: gic-v3: fix unmet dependency on ARM_PSCI_FW Date: Sun, 23 Aug 2026 19:08:32 +0100 Message-ID: <20260823180832.1246363-1-julianbraha@gmail.com> X-Mailer: git-send-email 2.55.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit ARM_GIC_V3 currently selects HAVE_ARM_SMCCC_DISCOVERY without ensuring its dependency, ARM_PSCI_FW, is met. However, ARM_PSCI_FW is always enabled on for arm64, and the GICv3 driver only uses HAVE_ARM_SMCCC_DISCOVERY to identify the NVIDIA T241, which is arm64 anyway. Also adds fallbacks for callers to avoid needing #ifdef guards when HAVE_ARM_SMCCC_DISCOVERY is disabled (32bit arm). This unmet dependency was found by kconfirm, a static analysis tool for Kconfig. Fixes: 35727af2b15d ("irqchip/gicv3: Workaround for NVIDIA erratum T241-FABRIC-4") Suggested-by: Marc Zyngier Signed-off-by: Julian Braha --- Changes since v1: - remove select and dependency on ARM_PSCI_FW, - make the HAVE_ARM_SMCCC_DISCOVERY 'select' conditional on arm64 Link to v1: https://lore.kernel.org/all/20260801212235.2692694-1-julianbraha@gmail.com/ --- drivers/irqchip/Kconfig | 2 +- include/linux/arm-smccc.h | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/irqchip/Kconfig b/drivers/irqchip/Kconfig index 20b77fbc51ee..dae023fed7d6 100644 --- a/drivers/irqchip/Kconfig +++ b/drivers/irqchip/Kconfig @@ -37,7 +37,7 @@ config ARM_GIC_V3 bool select IRQ_DOMAIN_HIERARCHY select GENERIC_IRQ_EFFECTIVE_AFF_MASK if SMP - select HAVE_ARM_SMCCC_DISCOVERY + select HAVE_ARM_SMCCC_DISCOVERY if ARM64 select IRQ_MSI_IOMMU config ARM_GIC_ITS_PARENT diff --git a/include/linux/arm-smccc.h b/include/linux/arm-smccc.h index 4de81848fe2e..248624c1af7c 100644 --- a/include/linux/arm-smccc.h +++ b/include/linux/arm-smccc.h @@ -324,6 +324,7 @@ enum arm_smccc_conduit { SMCCC_CONDUIT_HVC, }; +#ifdef CONFIG_HAVE_ARM_SMCCC_DISCOVERY /** * arm_smccc_1_1_get_conduit() * @@ -363,6 +364,12 @@ s32 arm_smccc_get_soc_id_version(void); * When ARM_SMCCC_ARCH_SOC_ID is not present, returns SMCCC_RET_NOT_SUPPORTED. */ s32 arm_smccc_get_soc_id_revision(void); +#else /* CONFIG_HAVE_ARM_SMCCC_DISCOVERY */ +#define arm_smccc_1_1_get_conduit() SMCCC_CONDUIT_NONE +#define arm_smccc_get_version() 0 +#define arm_smccc_get_soc_id_version() SMCCC_RET_NOT_SUPPORTED +#define arm_smccc_get_soc_id_revision() SMCCC_RET_NOT_SUPPORTED +#endif /* CONFIG_HAVE_ARM_SMCCC_DISCOVERY */ #ifndef __ASSEMBLY__ -- 2.55.0