From: Sekhar Nori <nsekhar@ti.com> To: Nishanth Menon <nm@ti.com>, Lokesh Vutla <lokeshvutla@ti.com> Cc: Faiz Abbas <faiz_abbas@ti.com>, <linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <t-kristo@ti.com>, <robh+dt@kernel.org>, Grygorii Strashko <grygorii.strashko@ti.com>, "Vutla, Lokesh" <lokeshvutla@ti.com> Subject: Re: [PATCH 1/3] arm64: dts: ti: k3-j7200-main: Add gpio nodes in main domain Date: Sat, 14 Nov 2020 00:39:18 +0530 [thread overview] Message-ID: <c6267288-581e-3b34-cec5-d5879f3d1283@ti.com> (raw) In-Reply-To: <20201113184020.drntugqsnj7dzsnh@ultimatum> On 14/11/20 12:10 AM, Nishanth Menon wrote: > On 23:59-20201113, Sekhar Nori wrote: > [..] >>> dtbs_check: we added: >>> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/gpio@600000: Missing #address-cells in interrupt provider >>> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/gpio@610000: Missing #address-cells in interrupt provider >>> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/gpio@620000: Missing #address-cells in interrupt provider >>> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/gpio@630000: Missing #address-cells in interrupt provider >> >> Hmm, running dtbs_check, I did not really see this. These are all the >> warnings I see for TI platforms: https://pastebin.ubuntu.com/p/m2my62mjQq/ > > Here is the full list of checks I ran through with kernel_patch_verify > (docker) > https://pastebin.ubuntu.com/p/tcnWw89CMD/ > > See lines 128 onwards for this series. kernel_patch_verify does'nt > complain on existing warnings, but just prints when there are additional > ones added in. Also make sure we have the right dtc as well > dtc 1.6.0 and dt_schema 2020.8.1 was used. I was using the latest schema from master. But I changed to 2020.08.1 also, and still don't see the warning. $ dt-doc-validate --version 2020.12.dev1+gab5a73fcef26 I dont have a system-wide dtc installed. One in kernel tree is updated. $ scripts/dtc/dtc --version Version: DTC 1.6.0-gcbca977e Looking at your logs, it looks like you have more patches than just this applied. I wonder if thats making a difference. Can you check with just these patches applied to linux-next or share your tree which includes other patches? In your logs, you have such error for other interrupt controller nodes as well. For example: arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/bus@30000000/interrupt-controller1: Missing #address-cells in interrupt provider Which I don't see in my logs. My guess is some other patch(es) in your patch stack either uncovers this warning or causes it. > >> >> The tree I am testing is linux-next of 12th Nov + these three patches >> applied. >> >> Also, #address-cells for interrupt provider being compulsory does not >> make full sense to me. Nothing in >> Documentation/devicetree/bindings/interrupt-controller/interrupts.txt or >> Documentation/devicetree/bindings/gpio/gpio-davinci.txt suggests that as >> well. >> >> Existing GPIO nodes for AM654 or J721E does not have #address-cells as well. >> >> Adding Grygorii as well, in case he knows more about this. > > > Yes - we need to have this conversation in the community :) I had > tagged this internally already during the 5.10 merge cycle that we > need to clean up the #address-cells warning and in some cases, maybe > the bindings are probably not accurate to attempt an enforcement. > I'd really like a conclusion on the topic as I recollect Lokesh and > Grygorii had a debate internally, but reached no conclusion, lets get > the wisdom of the community to help us here. Adding Lokesh to cc as well. Thanks, Sekhar
WARNING: multiple messages have this Message-ID (diff)
From: Sekhar Nori <nsekhar@ti.com> To: Nishanth Menon <nm@ti.com>, Lokesh Vutla <lokeshvutla@ti.com> Cc: devicetree@vger.kernel.org, Grygorii Strashko <grygorii.strashko@ti.com>, "Vutla, Lokesh" <lokeshvutla@ti.com>, linux-kernel@vger.kernel.org, robh+dt@kernel.org, t-kristo@ti.com, Faiz Abbas <faiz_abbas@ti.com>, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 1/3] arm64: dts: ti: k3-j7200-main: Add gpio nodes in main domain Date: Sat, 14 Nov 2020 00:39:18 +0530 [thread overview] Message-ID: <c6267288-581e-3b34-cec5-d5879f3d1283@ti.com> (raw) In-Reply-To: <20201113184020.drntugqsnj7dzsnh@ultimatum> On 14/11/20 12:10 AM, Nishanth Menon wrote: > On 23:59-20201113, Sekhar Nori wrote: > [..] >>> dtbs_check: we added: >>> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/gpio@600000: Missing #address-cells in interrupt provider >>> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/gpio@610000: Missing #address-cells in interrupt provider >>> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/gpio@620000: Missing #address-cells in interrupt provider >>> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/gpio@630000: Missing #address-cells in interrupt provider >> >> Hmm, running dtbs_check, I did not really see this. These are all the >> warnings I see for TI platforms: https://pastebin.ubuntu.com/p/m2my62mjQq/ > > Here is the full list of checks I ran through with kernel_patch_verify > (docker) > https://pastebin.ubuntu.com/p/tcnWw89CMD/ > > See lines 128 onwards for this series. kernel_patch_verify does'nt > complain on existing warnings, but just prints when there are additional > ones added in. Also make sure we have the right dtc as well > dtc 1.6.0 and dt_schema 2020.8.1 was used. I was using the latest schema from master. But I changed to 2020.08.1 also, and still don't see the warning. $ dt-doc-validate --version 2020.12.dev1+gab5a73fcef26 I dont have a system-wide dtc installed. One in kernel tree is updated. $ scripts/dtc/dtc --version Version: DTC 1.6.0-gcbca977e Looking at your logs, it looks like you have more patches than just this applied. I wonder if thats making a difference. Can you check with just these patches applied to linux-next or share your tree which includes other patches? In your logs, you have such error for other interrupt controller nodes as well. For example: arch/arm64/boot/dts/ti/k3-j7200-main.dtsi: /bus@100000/bus@30000000/interrupt-controller1: Missing #address-cells in interrupt provider Which I don't see in my logs. My guess is some other patch(es) in your patch stack either uncovers this warning or causes it. > >> >> The tree I am testing is linux-next of 12th Nov + these three patches >> applied. >> >> Also, #address-cells for interrupt provider being compulsory does not >> make full sense to me. Nothing in >> Documentation/devicetree/bindings/interrupt-controller/interrupts.txt or >> Documentation/devicetree/bindings/gpio/gpio-davinci.txt suggests that as >> well. >> >> Existing GPIO nodes for AM654 or J721E does not have #address-cells as well. >> >> Adding Grygorii as well, in case he knows more about this. > > > Yes - we need to have this conversation in the community :) I had > tagged this internally already during the 5.10 merge cycle that we > need to clean up the #address-cells warning and in some cases, maybe > the bindings are probably not accurate to attempt an enforcement. > I'd really like a conclusion on the topic as I recollect Lokesh and > Grygorii had a debate internally, but reached no conclusion, lets get > the wisdom of the community to help us here. Adding Lokesh to cc as well. Thanks, Sekhar _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-11-13 19:09 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-11-02 19:11 [PATCH 0/3] Add gpio support for TI's J7200 platform Faiz Abbas 2020-11-02 19:11 ` Faiz Abbas 2020-11-02 19:11 ` [PATCH 1/3] arm64: dts: ti: k3-j7200-main: Add gpio nodes in main domain Faiz Abbas 2020-11-02 19:11 ` Faiz Abbas 2020-11-12 16:39 ` Nishanth Menon 2020-11-12 16:39 ` Nishanth Menon 2020-11-13 18:29 ` Sekhar Nori 2020-11-13 18:29 ` Sekhar Nori 2020-11-13 18:40 ` Nishanth Menon 2020-11-13 18:40 ` Nishanth Menon 2020-11-13 19:09 ` Sekhar Nori [this message] 2020-11-13 19:09 ` Sekhar Nori 2020-11-13 20:55 ` Nishanth Menon 2020-11-13 20:55 ` Nishanth Menon 2020-11-14 4:15 ` Grygorii Strashko 2020-11-14 4:15 ` Grygorii Strashko 2020-11-14 5:56 ` Sekhar Nori 2020-11-14 5:56 ` Sekhar Nori 2020-11-02 19:11 ` [PATCH 2/3] arm64: dts: ti: k3-j7200: Add gpio nodes in wakeup domain Faiz Abbas 2020-11-02 19:11 ` Faiz Abbas 2020-11-02 19:11 ` [PATCH 3/3] arm64: dts: ti: k3-j7200-common-proc-board: Disable unused gpio modules Faiz Abbas 2020-11-02 19:11 ` Faiz Abbas 2020-11-04 11:40 ` [PATCH 0/3] Add gpio support for TI's J7200 platform Lokesh Vutla 2020-11-04 11:40 ` Lokesh Vutla 2020-11-12 16:42 ` Nishanth Menon 2020-11-12 16:42 ` Nishanth Menon
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=c6267288-581e-3b34-cec5-d5879f3d1283@ti.com \ --to=nsekhar@ti.com \ --cc=devicetree@vger.kernel.org \ --cc=faiz_abbas@ti.com \ --cc=grygorii.strashko@ti.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=lokeshvutla@ti.com \ --cc=nm@ti.com \ --cc=robh+dt@kernel.org \ --cc=t-kristo@ti.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.