From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B3FBBC44501 for ; Fri, 10 Jul 2026 08:28:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Subject:Cc:To:From: Message-ID:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=F6KudrFDGEDQqOhegar75aieWQMc2k5wBts8J2fU69w=; b=m7PqdOwJBF+W9j4w2VnLrsdrPG ddpeD8YixYKuEOh2aOFFk+RqDnpUCcHvg7oymXaQ+wUqx0flcxGr2m4cLXcazIf/pbrnSpSnnRRKg 1bzGQklk6dsphj2/Gnho9sM08YZep4GrGso0cZymox9Byj1tjNtdpuSP5HraQ+JJSmymgENZ9Qigx 41DWr5wYyB7YQ4dfPFfrZTvp7Wi+nF7e8GZEqVQRMjgptCHliyh6RV8KsRQ655PiGDva8OSoDCDIZ iZ/8JvUgOJeeG7kbfezSlxQRWYCuwb/evEbkzADTStij3ZOScItZXtyceNDF1IltLwFj/13bVEXT7 Bo8soe8g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wi6an-00000004Sgy-2qn2; Fri, 10 Jul 2026 08:28:17 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wi6am-00000004Sgo-1l2l for linux-arm-kernel@lists.infradead.org; Fri, 10 Jul 2026 08:28:16 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id A4BEA601D9; Fri, 10 Jul 2026 08:28:15 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 587891F000E9; Fri, 10 Jul 2026 08:28:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1783672095; bh=F6KudrFDGEDQqOhegar75aieWQMc2k5wBts8J2fU69w=; h=Date:From:To:Cc:Subject:In-Reply-To:References; b=N+SX3jrlYmsL5bMWiQ1GMDeRKxW9iAigjmPFC06Wz319fosORES2zhht2QaH/6qFq DwmmN17OAX/kzxh+a2JI+o4ifiTmNWEPHKxuiUuX0B0MULGUhJi0t3Fevbi0OiDEXz A7laFPH8WbFKGhOC36Yygn6qAz+sNpKGn9pdookoPdWdYqt+2MlTojVKF+9R54TIcR /FYacKL0pAatQHo3G/oFCy5hp54dc69KxWmNbAxfdmpv20n+KQyxYbVLNn/aCzECBJ 3dUBpVjy1o7y2Roi5/yQdpsZn102AV8KToXzOnO+/W/q0dLmu64YnwWJiju6W4aqJU lmzAH9Fcl7WeQ== Received: from sofa.misterjones.org ([185.219.108.64] helo=lobster-girl.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.98.2) (envelope-from ) id 1wi6aj-00000003b2r-1kdZ; Fri, 10 Jul 2026 08:28:13 +0000 Date: Fri, 10 Jul 2026 09:30:02 +0100 Message-ID: <87qzlb5jsl.wl-maz@kernel.org> From: Marc Zyngier To: Marek Vasut Cc: linux-pci@vger.kernel.org, kernel test robot , Krzysztof =?UTF-8?B?V2lsY3p5xYRza2k=?= , Bjorn Helgaas , Catalin Marinas , Conor Dooley , Geert Uytterhoeven , Krzysztof Kozlowski , Lorenzo Pieralisi , Manivannan Sadhasivam , Rob Herring , Yoshihiro Shimoda , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org Subject: Re: [PATCH] PCI: rcar-gen4: Inline GIC_TRANSLATER offset macro In-Reply-To: <20260709201103.90162-1-marek.vasut+renesas@mailbox.org> References: <20260709201103.90162-1-marek.vasut+renesas@mailbox.org> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/30.1 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: marek.vasut+renesas@mailbox.org, linux-pci@vger.kernel.org, lkp@intel.com, kwilczynski@kernel.org, bhelgaas@google.com, catalin.marinas@arm.com, conor+dt@kernel.org, geert+renesas@glider.be, krzk+dt@kernel.org, lpieralisi@kernel.org, mani@kernel.org, robh@kernel.org, yoshihiro.shimoda.uh@renesas.com, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, 09 Jul 2026 21:10:03 +0100, Marek Vasut wrote: >=20 > Instead of pulling in the whole linux/irqchip/arm-gic-v3.h , copy the > one GITS_TRANSLATER register offset macro directly into the driver. > This repairs the ability to build the driver on non-ARM non-GIC targets > the way it was possible until now, which retains good build test coverage. >=20 > Reported-by: kernel test robot > Closes: https://lore.kernel.org/oe-kbuild-all/202607100310.iQw5m9Uo-lkp@i= ntel.com/ > Signed-off-by: Marek Vasut > --- > Cc: "Krzysztof Wilczy=C5=84ski" > Cc: Bjorn Helgaas > Cc: Catalin Marinas > Cc: Conor Dooley > Cc: Geert Uytterhoeven > Cc: Krzysztof Kozlowski > Cc: Lorenzo Pieralisi > Cc: Manivannan Sadhasivam > Cc: Marc Zyngier > Cc: Rob Herring > Cc: Yoshihiro Shimoda > Cc: devicetree@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-doc@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Cc: linux-pci@vger.kernel.org > Cc: linux-renesas-soc@vger.kernel.org > --- > Note: The alternative I could think of would be ifdeffery which > is not nice and thwarts the build coverage, or limit the > driver to ARM/ARM64 in Kconfig which also thwarts the build > coverage. I could also split off the register macros in > linux/irqchip/arm-gic-v3.h into some separate header > linux/irqchip/arm-gic-v3-regs.h and include that which > might be OKish and avoids duplication. Thoughts ? No, I'm not hacking something that is purely architecture specific for the purpose of a bizarre integration quirk that should be handled by the boot firmware, and not Linux. Add whatever you want to the PCI glue code, limit this to arm64 (which is the only architecture this can ever be used on, and the build coverage argument really doesn't hold), but please leave the GIC code alone. M. --=20 Jazz isn't dead. It just smells funny.