From patchwork Tue Sep 3 08:31:06 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vaishnav Achath X-Patchwork-Id: 13788267 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 6CC33CD13CF for ; Tue, 3 Sep 2024 08:32:38 +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-Type: Content-Transfer-Encoding:MIME-Version:Message-ID:Date:Subject:CC:To:From: Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender :Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=hqeyzmvGdqaZH7UmCcauPjdhGh8RVSEQWcfiZFqvHec=; b=D2lOkSpninVq6ViwROowacb4+J QDomnsIpb+/3rK0UsY6jnYl3a5cCLZ0HzXAnGBCCqIvNL2Y40LUGzlgVxsFCbtGs8au0GFmUyupBo 9MA8Ji8/B1TYUs/pX1API4rypLVaKkSBx7RNCG8va2WjC/3Jo3TSGoy3O7ZBfODwpqBEI6c7Hp7UT oXIXGhqb+1akXVVKLoUvxXltGc0K7WSbqrGQgMkp+/qOdJP908ruUGPWGnQpVTleBMNB/c0jVmOm5 n+gv4OiqLKajdi5bBS3GH9AgZXELUZUqGkNBw+Qm2Hh8PJTMgMqh5WmsyUfrPsBqrFi0FBYXDyWOD obAtmSzA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1slOxb-0000000GutX-3c8u; Tue, 03 Sep 2024 08:32:23 +0000 Received: from fllv0016.ext.ti.com ([198.47.19.142]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1slOwg-0000000Guiy-1v51 for linux-arm-kernel@lists.infradead.org; Tue, 03 Sep 2024 08:31:28 +0000 Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 4838VCSw012770; Tue, 3 Sep 2024 03:31:12 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1725352272; bh=hqeyzmvGdqaZH7UmCcauPjdhGh8RVSEQWcfiZFqvHec=; h=From:To:CC:Subject:Date; b=wyYEej1xCaI5MpeQQNwiZAQEoZWuo15tmtN2nVNydgU630xNy+rCOAkAA2ZmZNIVW HNkdYXUjshXE35Sai4jafegILJLCGQMeCrKXbeEfA6GT7zliLRqUJYDjefBIRoaxNF 2w6WZjAR9RFu5k9fkcJ00aTFm/DgFw8qOizfS110= Received: from DLEE104.ent.ti.com (dlee104.ent.ti.com [157.170.170.34]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTP id 4838VCrZ036671; Tue, 3 Sep 2024 03:31:12 -0500 Received: from DLEE109.ent.ti.com (157.170.170.41) by DLEE104.ent.ti.com (157.170.170.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Tue, 3 Sep 2024 03:31:11 -0500 Received: from lelvsmtp5.itg.ti.com (10.180.75.250) by DLEE109.ent.ti.com (157.170.170.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Tue, 3 Sep 2024 03:31:11 -0500 Received: from uda0490681.. ([10.24.69.142]) by lelvsmtp5.itg.ti.com (8.15.2/8.15.2) with ESMTP id 4838V7kU085647; Tue, 3 Sep 2024 03:31:08 -0500 From: Vaishnav Achath To: , , , , , , CC: , , , , , Subject: [RFC PATCH 0/1] Add interrupts property to interrupt aggregators/routers Date: Tue, 3 Sep 2024 14:01:06 +0530 Message-ID: <20240903083107.3562816-1-vaishnav.a@ti.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240903_013126_699628_13FBA6CC X-CRM114-Status: GOOD ( 12.81 ) 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 The interrupt aggregator in TI K3 devices currently uses a custom vendor property "ti,interrupt-ranges" [1] to specify the interrupt source to parent mapping. As per interrupt controller bindings [2], it is mandatory for Nodes that describe devices which generate interrupts to contain an "interrupts" property, an "interrupts-extended" property, or both. Without this, standard DT parsing code cannot identify the mapping, for example while booting these platforms with Xen hypervisor, we see that the interrupts are not injected to the guest Linux since the mapping is missing, and Xen expects standard properties to specify the mapping[3], while adding the interrupts fixes the issue, for some platforms the list of interrupts can be so large with more than 192 entries[4], moreover in some systems (like TI K3 platforms), system designers can change the mappings according to system needs and a long list of repeating entries is error prone and difficult to maintain. Are there any existing solution to solve this problem of maintaining long list of identical interrupt properties (mostly continuous) other than adding individual entries one by one? It looks like such general solutions will help multiple platforms, TI platforms use ti,interrupt-ranges[1], Socionext platforms make use of socionext,interrupt-ranges[5] and multiple other platforms which maintain long list of interrupts in DT[6] can benefit from such general implementation. If there are no existing solution is it okay to add new support for a general property, my proposal is for sometihing similar to interrupt-template and interrupt-ranges as described in [7]. 1- https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi#n111 2- https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/interrupt-controller/interrupts.txt 3- https://xenbits.xen.org/gitweb/?p=xen.git;a=blob;f=xen/common/device-tree/device-tree.c#l1135 4- https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi#n1346 5- https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/socionext/uniphier-ld11.dtsi#n228 6- https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/nvidia/tegra234.dtsi#n126 7- https://android.googlesource.com/kernel/common/+/android-trusty-4.4/Documentation/devicetree/bindings/trusty/trusty-irq.txt#10 Thanks and Regards, Vaishnav Vaishnav Achath (1): arm64: dts: ti: k3-am62p-main: Add interrupts property for DMSS INTA arch/arm64/boot/dts/ti/k3-am62p-main.dtsi | 35 +++++++++++++++++++++++ 1 file changed, 35 insertions(+)