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 C9146C001B0 for ; Tue, 8 Aug 2023 09:45:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Y26G7+KOynAVgRIAWkKXWBcMzr5/LHbqAGB+yIdPwZ8=; b=2txhD2t5Hv/PuF udy2TKq8fcnmbX+O6hixyT4dMSkhz+QaqCVVfmRQVr2M/pgjgThLPvDp1YriqKTZrDU/KyXFJwjsB PyqSqNpphXDRJE42RR2d9eHUZp9/jH/+tqlnRtQXSNUbW3QckogDFJIetogsU+LybiAzROZlS+CMB vBGsfCrOd31vJv1vSJrG4ln4Xe2WaysX0C440a8h7ECQHmtZ1GC9Z8Lze6DtnTSQ3X1nLFQxmr6nF zLpIZXY724d45pOHd5U1Uz8J7Q/MmpDuEtqv9uPOLo5SHaGLo+V93UDrNP8AKw1iuy1xvKVzo971r JpXLOpTVnZe13ece0Cew==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qTJGf-002A6e-2Y; Tue, 08 Aug 2023 09:44:45 +0000 Received: from lelv0142.ext.ti.com ([198.47.23.249]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qTJGd-002A6F-0G for linux-arm-kernel@lists.infradead.org; Tue, 08 Aug 2023 09:44:44 +0000 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by lelv0142.ext.ti.com (8.15.2/8.15.2) with ESMTP id 3789ib6f033387; Tue, 8 Aug 2023 04:44:37 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1691487877; bh=x4KzLhx5BX2F6004xNI9OIlMNZqfZlvnhOagsXCzdtc=; h=Date:Subject:To:CC:References:From:In-Reply-To; b=DyIPmv9YiZPXPAz7XX1maF9evUxrDOSm5qnFSSLtrSTjKjqh8xPPoMJ0K4HFqdh7A 3dCpiq3h9ojI1G3y+s1+9Ium3bhM/a4eTDjRvu2ArylSDrqhS4QWeqcPP0vTHiw0oB WADKKshywpgMkgsrkftsRjWv9GLp8U57ZPeZUobY= Received: from DLEE115.ent.ti.com (dlee115.ent.ti.com [157.170.170.26]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 3789ibUJ001331 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Tue, 8 Aug 2023 04:44:37 -0500 Received: from DLEE112.ent.ti.com (157.170.170.23) by DLEE115.ent.ti.com (157.170.170.26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Tue, 8 Aug 2023 04:44:37 -0500 Received: from lelv0326.itg.ti.com (10.180.67.84) by DLEE112.ent.ti.com (157.170.170.23) 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, 8 Aug 2023 04:44:37 -0500 Received: from [172.24.227.217] (ileaxei01-snat.itg.ti.com [10.180.69.5]) by lelv0326.itg.ti.com (8.15.2/8.15.2) with ESMTP id 3789iWPm001940; Tue, 8 Aug 2023 04:44:33 -0500 Message-ID: <910a4a98-712a-5517-5a5b-ffb962f83463@ti.com> Date: Tue, 8 Aug 2023 15:14:31 +0530 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH] dt-bindings: remoteproc: pru: Add Interrupt property Content-Language: en-US To: Conor Dooley , MD Danish Anwar References: <20230807110836.2612730-1-danishanwar@ti.com> <20230807-euphemism-trailing-ef4130dc7437@spud> From: Md Danish Anwar Organization: Texas Instruments In-Reply-To: <20230807-euphemism-trailing-ef4130dc7437@spud> 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-20230808_024443_290504_F4ACA7E0 X-CRM114-Status: GOOD ( 16.46 ) 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: , Cc: srk@ti.com, devicetree@vger.kernel.org, Conor Dooley , linux-omap@vger.kernel.org, vigneshr@ti.com, Mathieu Poirier , Bjorn Andersson , linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org, nm@ti.com, Rob Herring , Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Conor, On 07/08/23 8:09 pm, Conor Dooley wrote: > On Mon, Aug 07, 2023 at 04:38:36PM +0530, MD Danish Anwar wrote: >> Add interrupts and interrupt-names protperties for PRU and RTU cores. >> >> Signed-off-by: MD Danish Anwar >> --- >> .../bindings/remoteproc/ti,pru-rproc.yaml | 22 +++++++++++++++++++ >> 1 file changed, 22 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,pru-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,pru-rproc.yaml >> index cd55d80137f7..6970316943bb 100644 >> --- a/Documentation/devicetree/bindings/remoteproc/ti,pru-rproc.yaml >> +++ b/Documentation/devicetree/bindings/remoteproc/ti,pru-rproc.yaml >> @@ -66,6 +66,16 @@ properties: >> Should contain the name of the default firmware image >> file located on the firmware search path. >> >> + interrupts: >> + maxItems: 1 >> + description: >> + Interrupt specifiers enable the virtio/rpmsg communication between MPU >> + and the PRU/RTU cores. >> + >> + interrupt-names: >> + items: >> + - const: vring >> + >> if: >> properties: >> compatible: >> @@ -171,6 +181,9 @@ examples: >> <0x22400 0x100>; >> reg-names = "iram", "control", "debug"; >> firmware-name = "am65x-pru0_0-fw"; >> + interrupt-parent = <&icssg0_intc>; >> + interrupts = <16 2 2>; >> + interrupt-names = "vring"; >> }; > > These examples would probably be more helpful if they used the > appropriate defines, no? > PRUSS Interrupt controller doesn't have any appropriate defines. This doesn't use GIC so defines from arm-gic.h can not be used here. These are specific to PRUSS INTC. I think these example are OK. Please let me know if this is OK to you. >> >> rtu0_0: rtu@4000 { >> @@ -180,6 +193,9 @@ examples: >> <0x23400 0x100>; >> reg-names = "iram", "control", "debug"; >> firmware-name = "am65x-rtu0_0-fw"; >> + interrupt-parent = <&icssg0_intc>; >> + interrupts = <20 4 4>; >> + interrupt-names = "vring"; >> }; >> >> tx_pru0_0: txpru@a000 { >> @@ -198,6 +214,9 @@ examples: >> <0x24400 0x100>; >> reg-names = "iram", "control", "debug"; >> firmware-name = "am65x-pru0_1-fw"; >> + interrupt-parent = <&icssg0_intc>; >> + interrupts = <18 3 3>; >> + interrupt-names = "vring"; >> }; >> >> rtu0_1: rtu@6000 { >> @@ -207,6 +226,9 @@ examples: >> <0x23c00 0x100>; >> reg-names = "iram", "control", "debug"; >> firmware-name = "am65x-rtu0_1-fw"; >> + interrupt-parent = <&icssg0_intc>; >> + interrupts = <22 5 5>; >> + interrupt-names = "vring"; >> }; >> >> tx_pru0_1: txpru@c000 { >> -- >> 2.34.1 >> -- Thanks and Regards, Danish. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel