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 X-Spam-Level: X-Spam-Status: No, score=-13.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,UNWANTED_LANGUAGE_BODY,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 96A19C433B4 for ; Tue, 11 May 2021 10:50:37 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1859461932 for ; Tue, 11 May 2021 10:50:37 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1859461932 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=1lKgVe86I1hPQ9dT63Aexf9MnUgRM10ozvGtX5pLATs=; b=IEINwI1K3qvtsktvdQPYs1OOG vndXdGV2+RVsINOCe2lLphAllJRUWRLquBEpbfnvkpTHJYXf0vXHs9/8ZSPef08gjHnhqqp5LTWGx DWwerT5uYKd2Dobb2GvtJjnygJ1dquuwgL+HVRbyHefOTAWv9y9CiFlrq5OV7pXDkegv3hQBSAhrA eQ2bOlIM4phCZSTyMwkWWXyXqJKk3LDmNvzF7v1UmqBeTQohIl2ugH+jcHY3nHX/yxytNgeMr+4/8 7V5/l24Y/AI1JtNhzyywaABorqBFh9pHNJKixZHdgyBbGwpRuK1tuJXj1kA0AiAnTr8Bz5abH7IRn iR5GP6Lqg==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lgPws-00H0ke-CN; Tue, 11 May 2021 10:49:10 +0000 Received: from [2607:7c80:54:e::133] (helo=bombadil.infradead.org) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lgMaf-00GU7l-Ph for linux-arm-kernel@desiato.infradead.org; Tue, 11 May 2021 07:14:04 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Transfer-Encoding: Content-Type:In-Reply-To:MIME-Version:Date:Message-ID:From:References:Cc:To: Subject:Sender:Reply-To:Content-ID:Content-Description; bh=bI+Y29NolRf9RqH8suQr680G4AEyP4IGk4hVvctTTnI=; b=Yvbj5nI69DvD5qZYpgEGTEl8jT LNNMQFcommc4c4ZD2KPDTFylizbAwhwvr44P0+OxMY6++AnWOnnppxjvFNynL+idH7vV26neyeEjC tcGIpjBATau7mdb8TQpqCpDAefnHjJ3ANqkUKB3Px+cLTpyiey3+S68P05MOGD4uRtSmkhwAFqkE7 bN2upv2v9Qkm/3aXO2X1uX8ENxAodgVpjcP4tjVjcH6Ejlfp5hhZfYhZ43mj08ugCI7eXL/rcawhN 0j2rLCfGx3Lllx6Rmo5YbKc8W0Exh8DZLCwHRvAnZdoIwr9QOEtMcQhpb++Mov8hUJcaTRPpd4eX7 iNiFytVA==; Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lgMaa-009M61-SH for linux-arm-kernel@lists.infradead.org; Tue, 11 May 2021 07:13:58 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 71B2861920; Tue, 11 May 2021 07:13:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1620717235; bh=8Ng+KnzWu9dMfLHfX55NiwAREP/QuApMDcjkSM9xUN0=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=UC2nTetGxX97hx4A4rBxNGIsduWo+WK38h26kaqQUBp0FzO4vWVhFEuw/IwwTi56+ ULhzDH0MSRpCapYoqxShi5MUTpUv7HAd4DbB6wOCC9Ry6muwpGP5Ll1ToPixjmEhsr /pJG1YoCSb9cCRDkg1fufAUtLHveYouv0WVNxK2frYm9473wdG6uQv52EPRO4ULKM6 lha5TuhW/JobVZQen+HCMBX5WdsvrbGQnQlL9ToRh6D55AFurFLuw+5my14ITvjxCv QxBGcc1SlLOLhLWjFbzKy2szYw/5pvyZyTb0I6/DKIVJhVC2Fvrbs3aUReX2LvFVd6 ev3k2oS26qglA== Subject: Re: [PATCH] arm64: dts: ti: k3*: Introduce reg definition for interrupt routers To: Nishanth Menon , grygorii.strashko@ti.com Cc: lokeshvutla@ti.com, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org References: <20210510145508.8994-1-nm@ti.com> From: Tero Kristo Message-ID: Date: Tue, 11 May 2021 10:13:50 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20210510145508.8994-1-nm@ti.com> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210511_001357_021472_960718A6 X-CRM114-Status: GOOD ( 21.52 ) /bin/ln: failed to access 'reaver_cache/texts/20210511_001357_021472_960718A6': No such file or directory X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210511_001357_021472_960718A6 X-CRM114-Status: GOOD ( 18.22 ) 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: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 10/05/2021 17:55, Nishanth Menon wrote: > Interrupt routers are memory mapped peripherals, that are organized > in our dts bus hierarchy to closely represents the actual hardware > behavior. > > However, without explicitly calling out the reg property, using > 2021.03+ dt-schema package, this exposes the following problem with > dtbs_check: > > /arch/arm64/boot/dts/ti/k3-am654-base-board.dt.yaml: bus@100000: > interrupt-controller0: {'type': 'object'} is not allowed for > {'compatible': ['ti,sci-intr'], ..... > > Even though we don't use interrupt router directly via memory mapped > registers and have to use it via the system controller, the hardware > block is memory mapped, so describe the base address in device tree. > > This is a valid, comprehensive description of hardware and permitted > by the existing ti,sci-intr schema. > > Signed-off-by: Nishanth Menon Reviewed-by: Tero Kristo > --- > > if possible, I'd like to pick this fixup for 5.13 window.. > > arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 3 ++- > arch/arm64/boot/dts/ti/k3-am64-mcu.dtsi | 3 ++- > arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 6 ++++-- > arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 3 ++- > arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 6 ++++-- > arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 3 ++- > arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 6 ++++-- > arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 3 ++- > 8 files changed, 22 insertions(+), 11 deletions(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi > index b2bcbf23eefd..a49e41021573 100644 > --- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi > @@ -373,8 +373,9 @@ main_spi4: spi@20140000 { > clocks = <&k3_clks 145 0>; > }; > > - main_gpio_intr: interrupt-controller0 { > + main_gpio_intr: interrupt-controller@a00000 { > compatible = "ti,sci-intr"; > + reg = <0x00 0xa00000 0x00 0x800>; > ti,intr-trigger-type = <1>; > interrupt-controller; > interrupt-parent = <&gic500>; > diff --git a/arch/arm64/boot/dts/ti/k3-am64-mcu.dtsi b/arch/arm64/boot/dts/ti/k3-am64-mcu.dtsi > index 99e94dee1bd4..deb19ae5e168 100644 > --- a/arch/arm64/boot/dts/ti/k3-am64-mcu.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am64-mcu.dtsi > @@ -74,8 +74,9 @@ mcu_spi1: spi@4b10000 { > clocks = <&k3_clks 148 0>; > }; > > - mcu_gpio_intr: interrupt-controller1 { > + mcu_gpio_intr: interrupt-controller@4210000 { > compatible = "ti,sci-intr"; > + reg = <0x00 0x04210000 0x00 0x200>; > ti,intr-trigger-type = <1>; > interrupt-controller; > interrupt-parent = <&gic500>; > diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi > index cb340d1b401f..037f9776c4c8 100644 > --- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi > @@ -433,8 +433,9 @@ usb1_phy: phy@4110000 { > #phy-cells = <0>; > }; > > - intr_main_gpio: interrupt-controller0 { > + intr_main_gpio: interrupt-controller@a00000 { > compatible = "ti,sci-intr"; > + reg = <0x0 0xa00000 0x0 0x400>; > ti,intr-trigger-type = <1>; > interrupt-controller; > interrupt-parent = <&gic500>; > @@ -454,8 +455,9 @@ main-navss { > > ti,sci-dev-id = <118>; > > - intr_main_navss: interrupt-controller1 { > + intr_main_navss: interrupt-controller@310e0000 { > compatible = "ti,sci-intr"; > + reg = <0x0 0x310e0000 0x0 0x2000>; > ti,intr-trigger-type = <4>; > interrupt-controller; > interrupt-parent = <&gic500>; > diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > index ed42f13e7663..62a18b110c52 100644 > --- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > @@ -69,8 +69,9 @@ wkup_i2c0: i2c@42120000 { > power-domains = <&k3_pds 115 TI_SCI_PD_EXCLUSIVE>; > }; > > - intr_wkup_gpio: interrupt-controller2 { > + intr_wkup_gpio: interrupt-controller@42200000 { > compatible = "ti,sci-intr"; > + reg = <0x42200000 0x200>; > ti,intr-trigger-type = <1>; > interrupt-controller; > interrupt-parent = <&gic500>; > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > index f86c493a44f1..4403ac0fc8a7 100644 > --- a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > @@ -68,8 +68,9 @@ gic_its: msi-controller@1820000 { > }; > }; > > - main_gpio_intr: interrupt-controller0 { > + main_gpio_intr: interrupt-controller@a00000 { > compatible = "ti,sci-intr"; > + reg = <0x00 0xa00000 0x00 0x800>; > ti,intr-trigger-type = <1>; > interrupt-controller; > interrupt-parent = <&gic500>; > @@ -86,8 +87,9 @@ main_navss: bus@30000000 { > ranges = <0x00 0x30000000 0x00 0x30000000 0x00 0x0c400000>; > ti,sci-dev-id = <199>; > > - main_navss_intr: interrupt-controller1 { > + main_navss_intr: interrupt-controller@310e0000 { > compatible = "ti,sci-intr"; > + reg = <0x00 0x310e0000 0x00 0x4000>; > ti,intr-trigger-type = <4>; > interrupt-controller; > interrupt-parent = <&gic500>; > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi > index 5e74e43822c3..825b9f1931b7 100644 > --- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi > @@ -96,8 +96,9 @@ mcu_uart0: serial@40a00000 { > clock-names = "fclk"; > }; > > - wkup_gpio_intr: interrupt-controller2 { > + wkup_gpio_intr: interrupt-controller@42200000 { > compatible = "ti,sci-intr"; > + reg = <0x00 0x42200000 0x00 0x400>; > ti,intr-trigger-type = <1>; > interrupt-controller; > interrupt-parent = <&gic500>; > diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > index c2aa45a3ac79..512371e36a30 100644 > --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > @@ -76,8 +76,9 @@ gic_its: msi-controller@1820000 { > }; > }; > > - main_gpio_intr: interrupt-controller0 { > + main_gpio_intr: interrupt-controller@a00000 { > compatible = "ti,sci-intr"; > + reg = <0x00 0xa00000 0x00 0x800>; > ti,intr-trigger-type = <1>; > interrupt-controller; > interrupt-parent = <&gic500>; > @@ -97,8 +98,9 @@ main-navss { > > ti,sci-dev-id = <199>; > > - main_navss_intr: interrupt-controller1 { > + main_navss_intr: interrupt-controller@310e0000 { > compatible = "ti,sci-intr"; > + reg = <0x0 0x310e0000 0x0 0x4000>; > ti,intr-trigger-type = <4>; > interrupt-controller; > interrupt-parent = <&gic500>; > diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi > index d56e3475aee7..ad12a5c9f209 100644 > --- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi > @@ -96,8 +96,9 @@ mcu_uart0: serial@40a00000 { > clock-names = "fclk"; > }; > > - wkup_gpio_intr: interrupt-controller2 { > + wkup_gpio_intr: interrupt-controller@42200000 { > compatible = "ti,sci-intr"; > + reg = <0x00 0x42200000 0x00 0x400>; > ti,intr-trigger-type = <1>; > interrupt-controller; > interrupt-parent = <&gic500>; > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel