From: Alexandre TORGUE <alexandre.torgue@foss.st.com> To: Arnd Bergmann <arnd@arndb.de> Cc: Ahmad Fatoum <a.fatoum@pengutronix.de>, Rob Herring <robh+dt@kernel.org>, Marek Vasut <marex@denx.de>, Jagan Teki <jagan@amarulasolutions.com>, Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>, Marcin Sloniewski <marcin.sloniewski@gmail.com>, Linux ARM <linux-arm-kernel@lists.infradead.org>, DTML <devicetree@vger.kernel.org>, <linux-stm32@st-md-mailman.stormreply.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Lee Jones <lee.jones@linaro.org>, Jakub Kicinski <kuba@kernel.org> Subject: Re: [PATCH 12/13] ARM: dts: stm32: fix DSI port node on STM32MP15 Date: Mon, 19 Apr 2021 16:04:01 +0200 [thread overview] Message-ID: <f37abf6d-f82b-e253-d9f9-772df0b800d1@foss.st.com> (raw) In-Reply-To: <CAK8P3a1bGAUbqTGqyz+PB=7fuVLkJce0awtx1Z9PE3uiX6uysQ@mail.gmail.com> On 4/19/21 3:57 PM, Arnd Bergmann wrote: > On Thu, Apr 15, 2021 at 2:23 PM Alexandre TORGUE > <alexandre.torgue@foss.st.com> wrote: >> On 4/15/21 12:43 PM, Ahmad Fatoum wrote: >>> On 15.04.21 12:10, Alexandre Torgue wrote: >>>> Running "make dtbs_check W=1", some warnings are reported concerning >>>> DSI. This patch reorder DSI nodes to avoid: >>>> >>>> soc/dsi@5a000000: unnecessary #address-cells/#size-cells without >>>> "ranges" or child "reg" property >>> >>> This reverts parts of commit 9c32f980d9 ("ARM: dts: stm32: preset >>> stm32mp15x video #address- and #size-cells"): >>> >>> The cell count for address and size is defined by the binding and not >>> something a board would change. Avoid each board adding this >>> boilerplate by having the cell size specification in the SoC DTSI. >>> >>> >>> The DSI can have child nodes with a unit address (e.g. a panel) and ones >>> without (ports { } container). ports is described in the dtsi, panels are >>> described in the dts if available. >>> >>> Apparently, the checker is fine with >>> ports { >>> #address-cells = <1>; >>> #size-cells = <0>; >>> }; >>> >>> I think my rationale for the patch above was sound, so I think the checker >>> taking offense at the DSI cells here should be considered a false positive. >> >> If it's a "false positive" warning then we need to find a way to not >> print it out. Else, it'll be difficult to distinguish which warnings are >> "normal" and which are not. This question could also be applied to patch[3]. >> >> Arnd, Rob what is your feeling about this case ? > > I don't have a strong opinion on this either way, but I would just > not apply this one for 5.13 in this case. Rob, Alexandre, please > let me know if I should apply the other patches before the > merge window, I usually don't mind taking bugfixes late before the > merge window, but I still want some level of confidence that they > are actually correct. For me, we can keep this series for the v5.14 cycle. regards alex > > Ahmad, if you feel strongly about this particular issue, would you like > to suggest a patch for the checker? > > Arnd >
WARNING: multiple messages have this Message-ID (diff)
From: Alexandre TORGUE <alexandre.torgue@foss.st.com> To: Arnd Bergmann <arnd@arndb.de> Cc: Ahmad Fatoum <a.fatoum@pengutronix.de>, Rob Herring <robh+dt@kernel.org>, Marek Vasut <marex@denx.de>, Jagan Teki <jagan@amarulasolutions.com>, Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>, Marcin Sloniewski <marcin.sloniewski@gmail.com>, Linux ARM <linux-arm-kernel@lists.infradead.org>, DTML <devicetree@vger.kernel.org>, <linux-stm32@st-md-mailman.stormreply.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Lee Jones <lee.jones@linaro.org>, Jakub Kicinski <kuba@kernel.org> Subject: Re: [PATCH 12/13] ARM: dts: stm32: fix DSI port node on STM32MP15 Date: Mon, 19 Apr 2021 16:04:01 +0200 [thread overview] Message-ID: <f37abf6d-f82b-e253-d9f9-772df0b800d1@foss.st.com> (raw) In-Reply-To: <CAK8P3a1bGAUbqTGqyz+PB=7fuVLkJce0awtx1Z9PE3uiX6uysQ@mail.gmail.com> On 4/19/21 3:57 PM, Arnd Bergmann wrote: > On Thu, Apr 15, 2021 at 2:23 PM Alexandre TORGUE > <alexandre.torgue@foss.st.com> wrote: >> On 4/15/21 12:43 PM, Ahmad Fatoum wrote: >>> On 15.04.21 12:10, Alexandre Torgue wrote: >>>> Running "make dtbs_check W=1", some warnings are reported concerning >>>> DSI. This patch reorder DSI nodes to avoid: >>>> >>>> soc/dsi@5a000000: unnecessary #address-cells/#size-cells without >>>> "ranges" or child "reg" property >>> >>> This reverts parts of commit 9c32f980d9 ("ARM: dts: stm32: preset >>> stm32mp15x video #address- and #size-cells"): >>> >>> The cell count for address and size is defined by the binding and not >>> something a board would change. Avoid each board adding this >>> boilerplate by having the cell size specification in the SoC DTSI. >>> >>> >>> The DSI can have child nodes with a unit address (e.g. a panel) and ones >>> without (ports { } container). ports is described in the dtsi, panels are >>> described in the dts if available. >>> >>> Apparently, the checker is fine with >>> ports { >>> #address-cells = <1>; >>> #size-cells = <0>; >>> }; >>> >>> I think my rationale for the patch above was sound, so I think the checker >>> taking offense at the DSI cells here should be considered a false positive. >> >> If it's a "false positive" warning then we need to find a way to not >> print it out. Else, it'll be difficult to distinguish which warnings are >> "normal" and which are not. This question could also be applied to patch[3]. >> >> Arnd, Rob what is your feeling about this case ? > > I don't have a strong opinion on this either way, but I would just > not apply this one for 5.13 in this case. Rob, Alexandre, please > let me know if I should apply the other patches before the > merge window, I usually don't mind taking bugfixes late before the > merge window, but I still want some level of confidence that they > are actually correct. For me, we can keep this series for the v5.14 cycle. regards alex > > Ahmad, if you feel strongly about this particular issue, would you like > to suggest a patch for the checker? > > Arnd > _______________________________________________ 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:[~2021-04-19 14:04 UTC|newest] Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-04-15 10:10 [PATCH 00/13] ARM: dts: stm32: fix "make dtbs_check W=1" round1 Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-15 10:10 ` [PATCH 01/13] ARM: dts: stm32: fix gpio-keys node on STM32 MCU boards Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-15 10:10 ` [PATCH 02/13] ARM: dts: stm32: fix RCC node name on stm32f429 MCU Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-15 10:10 ` [PATCH 03/13] ARM: dts: stm32: fix timer nodes on STM32 MCU to prevent warnings Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-16 15:52 ` [Linux-stm32] " Fabrice Gasnier 2021-04-16 15:52 ` Fabrice Gasnier 2021-04-15 10:10 ` [PATCH 04/13] dt-bindings: mfd: stm32-timers: remove #address/size cells from required properties Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-16 15:52 ` [Linux-stm32] " Fabrice Gasnier 2021-04-16 15:52 ` Fabrice Gasnier 2021-04-20 20:04 ` Rob Herring 2021-04-20 20:04 ` Rob Herring 2021-06-01 16:26 ` Lee Jones 2021-06-01 16:26 ` Lee Jones 2021-04-15 10:10 ` [PATCH 05/13] ARM: dts: stm32: update pinctrl node name on STM32 MCU to prevent warnings Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-15 10:10 ` [PATCH 06/13] ARM: dts: stm32: fix i2c node name on stm32f746 " Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-15 10:10 ` [PATCH 07/13] ARM: dts: stm32: move stmmac axi config in ethernet node on stm32mp15 Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-15 10:10 ` [PATCH 08/13] dt-bindings: net: document ptp_ref clk in dwmac Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-20 20:05 ` Rob Herring 2021-04-20 20:05 ` Rob Herring 2021-04-15 10:10 ` [PATCH 09/13] ARM: dts: stm32: fix stpmic node for stm32mp1 boards Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-06-10 14:33 ` Alexandre TORGUE 2021-06-10 14:33 ` Alexandre TORGUE 2021-04-15 10:10 ` [PATCH 10/13] dt-bindings: mfd: add vref_ddr-supply to st,stpmic1 yaml Alexandre Torgue 2021-04-15 10:10 ` [PATCH 10/13] dt-bindings: mfd: add vref_ddr-supply to st, stpmic1 yaml Alexandre Torgue 2021-04-15 10:51 ` [PATCH 10/13] dt-bindings: mfd: add vref_ddr-supply to st,stpmic1 yaml Ahmad Fatoum 2021-04-15 10:51 ` [PATCH 10/13] dt-bindings: mfd: add vref_ddr-supply to st, stpmic1 yaml Ahmad Fatoum 2021-04-15 12:12 ` [PATCH 10/13] dt-bindings: mfd: add vref_ddr-supply to st,stpmic1 yaml Alexandre TORGUE 2021-04-15 12:12 ` [PATCH 10/13] dt-bindings: mfd: add vref_ddr-supply to st, stpmic1 yaml Alexandre TORGUE 2021-04-15 10:10 ` [PATCH 11/13] ARM: dts: stm32: fix LTDC port node on STM32 MCU ad MPU Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-15 13:21 ` Marek Vasut 2021-04-15 13:21 ` Marek Vasut 2021-04-15 13:34 ` Alexandre TORGUE 2021-04-15 13:34 ` Alexandre TORGUE 2021-04-15 14:30 ` Marek Vasut 2021-04-15 14:30 ` Marek Vasut 2021-04-15 14:35 ` Alexandre TORGUE 2021-04-15 14:35 ` Alexandre TORGUE 2021-04-15 14:59 ` Marek Vasut 2021-04-15 14:59 ` Marek Vasut 2021-04-15 10:10 ` [PATCH 12/13] ARM: dts: stm32: fix DSI port node on STM32MP15 Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-04-15 10:43 ` Ahmad Fatoum 2021-04-15 10:43 ` Ahmad Fatoum 2021-04-15 12:23 ` Alexandre TORGUE 2021-04-15 12:23 ` Alexandre TORGUE 2021-04-19 13:57 ` Arnd Bergmann 2021-04-19 13:57 ` Arnd Bergmann 2021-04-19 14:04 ` Alexandre TORGUE [this message] 2021-04-19 14:04 ` Alexandre TORGUE 2021-05-04 13:16 ` Ahmad Fatoum 2021-05-04 13:16 ` Ahmad Fatoum 2021-04-15 10:10 ` [PATCH 13/13] ARM: dts: stm32: fix ltdc pinctrl on microdev2.0-of7 Alexandre Torgue 2021-04-15 10:10 ` Alexandre Torgue 2021-06-01 10:37 ` [PATCH 00/13] ARM: dts: stm32: fix "make dtbs_check W=1" round1 Alexandre TORGUE 2021-06-01 10:37 ` Alexandre TORGUE
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=f37abf6d-f82b-e253-d9f9-772df0b800d1@foss.st.com \ --to=alexandre.torgue@foss.st.com \ --cc=a.fatoum@pengutronix.de \ --cc=arnd@arndb.de \ --cc=devicetree@vger.kernel.org \ --cc=jagan@amarulasolutions.com \ --cc=kuba@kernel.org \ --cc=lee.jones@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-stm32@st-md-mailman.stormreply.com \ --cc=manivannan.sadhasivam@linaro.org \ --cc=marcin.sloniewski@gmail.com \ --cc=marex@denx.de \ --cc=robh+dt@kernel.org \ /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.