* [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc @ 2018-01-11 5:58 Viresh Kumar [not found] ` <a062d9ed0d3365b578156a780202fa533e725374.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 0 siblings, 1 reply; 22+ messages in thread From: Viresh Kumar @ 2018-01-11 5:58 UTC (permalink / raw) To: arnd-r2nGTMty4D4, arm-DgEjT+Ai2ygdnm+yROfE0A Cc: Viresh Kumar, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, devicetree-u79uwXL29TY76Z2rM5mHXA The interrupt-parent of rtc was missing, add it. Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> --- arch/arm/boot/dts/spear600.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/spear600.dtsi b/arch/arm/boot/dts/spear600.dtsi index 6b32d20acc9f..00166eb9be86 100644 --- a/arch/arm/boot/dts/spear600.dtsi +++ b/arch/arm/boot/dts/spear600.dtsi @@ -194,6 +194,7 @@ rtc: rtc@fc900000 { compatible = "st,spear600-rtc"; reg = <0xfc900000 0x1000>; + interrupt-parent = <&vic0>; interrupts = <10>; status = "disabled"; }; -- 2.15.0.194.g9af6a3dea062 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 22+ messages in thread
[parent not found: <a062d9ed0d3365b578156a780202fa533e725374.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]
* [PATCH 2/3] arm: spear13xx: Fix dmas cells [not found] ` <a062d9ed0d3365b578156a780202fa533e725374.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> @ 2018-01-11 5:58 ` Viresh Kumar [not found] ` <6183c13fd3abcb2a6834a537a4809b6527204f10.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-11 5:58 ` [PATCH 3/3] arm: spear13xx: Fix spics gpio controller's warning Viresh Kumar ` (2 subsequent siblings) 3 siblings, 1 reply; 22+ messages in thread From: Viresh Kumar @ 2018-01-11 5:58 UTC (permalink / raw) To: arnd-r2nGTMty4D4, arm-DgEjT+Ai2ygdnm+yROfE0A Cc: Viresh Kumar, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, devicetree-u79uwXL29TY76Z2rM5mHXA The "dmas" cells for the designware DMA controller need to have only 3 properties apart from the phandle: request line, src master and destination master. But the commit 6e8887f60f60 updated it incorrectly while moving from platform code to DT. Fix it. Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.10+ Fixes: 6e8887f60f60 ("ARM: SPEAr13xx: Pass generic DW DMAC platform data from DT") Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> --- Documentation/devicetree/bindings/dma/snps-dma.txt | 2 +- arch/arm/boot/dts/spear1340.dtsi | 4 ++-- arch/arm/boot/dts/spear13xx.dtsi | 6 +++--- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/Documentation/devicetree/bindings/dma/snps-dma.txt b/Documentation/devicetree/bindings/dma/snps-dma.txt index a122723907ac..99acc712f83a 100644 --- a/Documentation/devicetree/bindings/dma/snps-dma.txt +++ b/Documentation/devicetree/bindings/dma/snps-dma.txt @@ -64,6 +64,6 @@ described in the dma.txt file, using a four-cell specifier for each channel. reg = <0xe0000000 0x1000>; interrupts = <0 35 0x4>; dmas = <&dmahost 12 0 1>, - <&dmahost 13 0 1 0>; + <&dmahost 13 1 0>; dma-names = "rx", "rx"; }; diff --git a/arch/arm/boot/dts/spear1340.dtsi b/arch/arm/boot/dts/spear1340.dtsi index 5f347054527d..d4dbc4098653 100644 --- a/arch/arm/boot/dts/spear1340.dtsi +++ b/arch/arm/boot/dts/spear1340.dtsi @@ -142,8 +142,8 @@ reg = <0xb4100000 0x1000>; interrupts = <0 105 0x4>; status = "disabled"; - dmas = <&dwdma0 0x600 0 0 1>, /* 0xC << 11 */ - <&dwdma0 0x680 0 1 0>; /* 0xD << 7 */ + dmas = <&dwdma0 12 0 1>, + <&dwdma0 13 1 0>; dma-names = "tx", "rx"; }; diff --git a/arch/arm/boot/dts/spear13xx.dtsi b/arch/arm/boot/dts/spear13xx.dtsi index 17ea0abcdbd7..086b4b333249 100644 --- a/arch/arm/boot/dts/spear13xx.dtsi +++ b/arch/arm/boot/dts/spear13xx.dtsi @@ -100,7 +100,7 @@ reg = <0xb2800000 0x1000>; interrupts = <0 29 0x4>; status = "disabled"; - dmas = <&dwdma0 0 0 0 0>; + dmas = <&dwdma0 0 0 0>; dma-names = "data"; }; @@ -290,8 +290,8 @@ #size-cells = <0>; interrupts = <0 31 0x4>; status = "disabled"; - dmas = <&dwdma0 0x2000 0 0 0>, /* 0x4 << 11 */ - <&dwdma0 0x0280 0 0 0>; /* 0x5 << 7 */ + dmas = <&dwdma0 4 0 0>, + <&dwdma0 5 0 0>; dma-names = "tx", "rx"; }; -- 2.15.0.194.g9af6a3dea062 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 22+ messages in thread
[parent not found: <6183c13fd3abcb2a6834a537a4809b6527204f10.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]
* Re: [PATCH 2/3] arm: spear13xx: Fix dmas cells [not found] ` <6183c13fd3abcb2a6834a537a4809b6527204f10.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> @ 2018-01-12 2:07 ` Olof Johansson 0 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2018-01-12 2:07 UTC (permalink / raw) To: Viresh Kumar Cc: arnd-r2nGTMty4D4, arm-DgEjT+Ai2ygdnm+yROfE0A, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, devicetree-u79uwXL29TY76Z2rM5mHXA On Thu, Jan 11, 2018 at 11:28:52AM +0530, Viresh Kumar wrote: > The "dmas" cells for the designware DMA controller need to have only 3 > properties apart from the phandle: request line, src master and > destination master. But the commit 6e8887f60f60 updated it incorrectly > while moving from platform code to DT. Fix it. > > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.10+ > Fixes: 6e8887f60f60 ("ARM: SPEAr13xx: Pass generic DW DMAC platform data from DT") > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Applied to next/dt. -Olof -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 3/3] arm: spear13xx: Fix spics gpio controller's warning [not found] ` <a062d9ed0d3365b578156a780202fa533e725374.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-11 5:58 ` [PATCH 2/3] arm: spear13xx: Fix dmas cells Viresh Kumar @ 2018-01-11 5:58 ` Viresh Kumar [not found] ` <9d8521e3990be6789f1ed3d5757b57c65483117b.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-11 13:47 ` [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc Rob Herring 2018-01-12 2:07 ` Olof Johansson 3 siblings, 1 reply; 22+ messages in thread From: Viresh Kumar @ 2018-01-11 5:58 UTC (permalink / raw) To: arnd-r2nGTMty4D4, arm-DgEjT+Ai2ygdnm+yROfE0A Cc: Viresh Kumar, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, devicetree-u79uwXL29TY76Z2rM5mHXA This fixes the following warning by also sending the flags argument for gpio controllers: Property 'cs-gpios', cell 6 is not a phandle reference in /ahb/apb/spi@e0100000 Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> --- arch/arm/boot/dts/spear1310-evb.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/spear1310-evb.dts b/arch/arm/boot/dts/spear1310-evb.dts index 84101e4eebbf..0f5f379323a8 100644 --- a/arch/arm/boot/dts/spear1310-evb.dts +++ b/arch/arm/boot/dts/spear1310-evb.dts @@ -349,7 +349,7 @@ spi0: spi@e0100000 { status = "okay"; num-cs = <3>; - cs-gpios = <&gpio1 7 0>, <&spics 0>, <&spics 1>; + cs-gpios = <&gpio1 7 0>, <&spics 0 0>, <&spics 1 0>; stmpe610@0 { compatible = "st,stmpe610"; -- 2.15.0.194.g9af6a3dea062 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 22+ messages in thread
[parent not found: <9d8521e3990be6789f1ed3d5757b57c65483117b.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]
* Re: [PATCH 3/3] arm: spear13xx: Fix spics gpio controller's warning [not found] ` <9d8521e3990be6789f1ed3d5757b57c65483117b.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> @ 2018-01-12 2:07 ` Olof Johansson 0 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2018-01-12 2:07 UTC (permalink / raw) To: Viresh Kumar Cc: arnd-r2nGTMty4D4, arm-DgEjT+Ai2ygdnm+yROfE0A, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, devicetree-u79uwXL29TY76Z2rM5mHXA On Thu, Jan 11, 2018 at 11:28:53AM +0530, Viresh Kumar wrote: > This fixes the following warning by also sending the flags argument for > gpio controllers: > > Property 'cs-gpios', cell 6 is not a phandle reference in > /ahb/apb/spi@e0100000 > > Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > --- Applied to next/dt. -Olof -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <a062d9ed0d3365b578156a780202fa533e725374.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-11 5:58 ` [PATCH 2/3] arm: spear13xx: Fix dmas cells Viresh Kumar 2018-01-11 5:58 ` [PATCH 3/3] arm: spear13xx: Fix spics gpio controller's warning Viresh Kumar @ 2018-01-11 13:47 ` Rob Herring 2018-01-12 2:07 ` Olof Johansson 3 siblings, 0 replies; 22+ messages in thread From: Rob Herring @ 2018-01-11 13:47 UTC (permalink / raw) To: Viresh Kumar Cc: Arnd Bergmann, arm-DgEjT+Ai2ygdnm+yROfE0A, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS On Wed, Jan 10, 2018 at 11:58 PM, Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: > The interrupt-parent of rtc was missing, add it. > > Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > --- > arch/arm/boot/dts/spear600.dtsi | 1 + > 1 file changed, 1 insertion(+) For all three patches: Reviewed-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <a062d9ed0d3365b578156a780202fa533e725374.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> ` (2 preceding siblings ...) 2018-01-11 13:47 ` [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc Rob Herring @ 2018-01-12 2:07 ` Olof Johansson 2018-01-12 3:22 ` Viresh Kumar 2018-01-14 11:17 ` Linus Walleij 3 siblings, 2 replies; 22+ messages in thread From: Olof Johansson @ 2018-01-12 2:07 UTC (permalink / raw) To: Viresh Kumar Cc: arnd-r2nGTMty4D4, arm-DgEjT+Ai2ygdnm+yROfE0A, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, devicetree-u79uwXL29TY76Z2rM5mHXA On Thu, Jan 11, 2018 at 11:28:51AM +0530, Viresh Kumar wrote: > The interrupt-parent of rtc was missing, add it. > > Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Applied to next/dt. Is stable really needed on this? It's been broken since pretty much forever, and nobody has complained... :) -Olof -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc 2018-01-12 2:07 ` Olof Johansson @ 2018-01-12 3:22 ` Viresh Kumar 2018-01-12 3:23 ` Olof Johansson 2018-01-14 11:17 ` Linus Walleij 1 sibling, 1 reply; 22+ messages in thread From: Viresh Kumar @ 2018-01-12 3:22 UTC (permalink / raw) To: Olof Johansson Cc: arnd-r2nGTMty4D4, arm-DgEjT+Ai2ygdnm+yROfE0A, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, devicetree-u79uwXL29TY76Z2rM5mHXA On 11-01-18, 18:07, Olof Johansson wrote: > On Thu, Jan 11, 2018 at 11:28:51AM +0530, Viresh Kumar wrote: > > The interrupt-parent of rtc was missing, add it. > > > > Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") > > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ > > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> > > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > > Applied to next/dt. Is stable really needed on this? It's been broken since > pretty much forever, and nobody has complained... :) Not sure. Just thought it may be useful for someone somewhere :) -- viresh -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc 2018-01-12 3:22 ` Viresh Kumar @ 2018-01-12 3:23 ` Olof Johansson [not found] ` <CAOesGMggU73asY1SEUDD_BsWcrGYR___o8e5omjziOPEwJgqBQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 22+ messages in thread From: Olof Johansson @ 2018-01-12 3:23 UTC (permalink / raw) To: Viresh Kumar Cc: Arnd Bergmann, ARM-SoC Maintainers, Linux ARM Mailing List, Rob Herring, devicetree-u79uwXL29TY76Z2rM5mHXA On Thu, Jan 11, 2018 at 7:22 PM, Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: > On 11-01-18, 18:07, Olof Johansson wrote: >> On Thu, Jan 11, 2018 at 11:28:51AM +0530, Viresh Kumar wrote: >> > The interrupt-parent of rtc was missing, add it. >> > >> > Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") >> > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ >> > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> >> > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >> >> Applied to next/dt. Is stable really needed on this? It's been broken since >> pretty much forever, and nobody has complained... :) > > Not sure. Just thought it may be useful for someone somewhere :) Ok. Left the tags there, but didn't merge into fixes since we're late in -rc and this didn't seem critical at this time. -Olof -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAOesGMggU73asY1SEUDD_BsWcrGYR___o8e5omjziOPEwJgqBQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CAOesGMggU73asY1SEUDD_BsWcrGYR___o8e5omjziOPEwJgqBQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-12 8:56 ` Arnd Bergmann [not found] ` <CAK8P3a2SO4=YKBE3L_AKsyfNe77YR7a8F=dvbS=ic03=6NK8MA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 22+ messages in thread From: Arnd Bergmann @ 2018-01-12 8:56 UTC (permalink / raw) To: Olof Johansson Cc: Viresh Kumar, ARM-SoC Maintainers, Linux ARM Mailing List, Rob Herring, DTML, gregkh On Fri, Jan 12, 2018 at 4:23 AM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: > On Thu, Jan 11, 2018 at 7:22 PM, Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: >> On 11-01-18, 18:07, Olof Johansson wrote: >>> On Thu, Jan 11, 2018 at 11:28:51AM +0530, Viresh Kumar wrote: >>> > The interrupt-parent of rtc was missing, add it. >>> > >>> > Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") >>> > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ >>> > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> >>> > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >>> >>> Applied to next/dt. Is stable really needed on this? It's been broken since >>> pretty much forever, and nobody has complained... :) >> >> Not sure. Just thought it may be useful for someone somewhere :) > > Ok. Left the tags there, but didn't merge into fixes since we're late > in -rc and this didn't seem critical at this time. My plan was to have these in the fixes branch in the hope of making it to a clean build for 4.15 after all, they fix warnings that got introduced by the updated dtc checks in 4.15-rc1. We are getting fairly close, but it seems we still miss a few, so we might as well give up at this point. The remaining fixes should be easy to backport into v4.15.y if we decide to do it, of further back even. For v4.14 and before, the in-kernel copy of dtc won't warn, but mainline dtc will. Greg, let me know your thoughts on this for the upcoming 4.15.y release. We had hundreds of dtc warnings in 4.15-rc1, many of them about important bugs, now we're down to a couple of warnings for platforms we don't care about much, and I expect the last of these fixes to land in 4.16-rc1 or maybe -rc2. Shall we backport them all to get a clean 4.15.y release? Note: there was at least one dtc warning fix that caused a serious regression in code that relied on a device probe to fail because of an invalid node (a fix is still in the works for 4.15), though generally the fixes are really harmless and can only make things better. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAK8P3a2SO4=YKBE3L_AKsyfNe77YR7a8F=dvbS=ic03=6NK8MA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CAK8P3a2SO4=YKBE3L_AKsyfNe77YR7a8F=dvbS=ic03=6NK8MA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-12 18:23 ` Olof Johansson [not found] ` <CAOesGMgcf9FovbzhPkKnrG4ETRFP3dsSJnTYNfp1wECMFeFOGA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 22+ messages in thread From: Olof Johansson @ 2018-01-12 18:23 UTC (permalink / raw) To: Arnd Bergmann Cc: Viresh Kumar, ARM-SoC Maintainers, Linux ARM Mailing List, Rob Herring, DTML, gregkh On Fri, Jan 12, 2018 at 12:56 AM, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> wrote: > On Fri, Jan 12, 2018 at 4:23 AM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: >> On Thu, Jan 11, 2018 at 7:22 PM, Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: >>> On 11-01-18, 18:07, Olof Johansson wrote: >>>> On Thu, Jan 11, 2018 at 11:28:51AM +0530, Viresh Kumar wrote: >>>> > The interrupt-parent of rtc was missing, add it. >>>> > >>>> > Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") >>>> > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ >>>> > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> >>>> > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >>>> >>>> Applied to next/dt. Is stable really needed on this? It's been broken since >>>> pretty much forever, and nobody has complained... :) >>> >>> Not sure. Just thought it may be useful for someone somewhere :) >> >> Ok. Left the tags there, but didn't merge into fixes since we're late >> in -rc and this didn't seem critical at this time. > > My plan was to have these in the fixes branch in the hope of making > it to a clean build for 4.15 after all, they fix warnings that got introduced > by the updated dtc checks in 4.15-rc1. > > We are getting fairly close, but it seems we still miss a few, so we > might as well give up at this point. The remaining fixes should be easy > to backport into v4.15.y if we decide to do it, of further back even. > For v4.14 and before, the in-kernel copy of dtc won't warn, but mainline > dtc will. > > Greg, let me know your thoughts on this for the upcoming 4.15.y > release. We had hundreds of dtc warnings in 4.15-rc1, many of them > about important bugs, now we're down to a couple of warnings > for platforms we don't care about much, and I expect the last of > these fixes to land in 4.16-rc1 or maybe -rc2. Shall we backport > them all to get a clean 4.15.y release? I think it makes more sense to disable the warnings than to backport a bunch of warning fixes this late. The code is working, has worked for a long time it's just that Rob enabled the warnings by default. We can keep them enabled for 4.16. Rob? > Note: there was at least one dtc warning fix that caused a serious > regression in code that relied on a device probe to fail because of > an invalid node (a fix is still in the works for 4.15), though generally > the fixes are really harmless and can only make things better. Exactly why picking up warning fixes this late is probably not a great idea. -Olof -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAOesGMgcf9FovbzhPkKnrG4ETRFP3dsSJnTYNfp1wECMFeFOGA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CAOesGMgcf9FovbzhPkKnrG4ETRFP3dsSJnTYNfp1wECMFeFOGA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-12 20:45 ` Rob Herring [not found] ` <CAL_JsqJVWLazDRB1me2OubqqMhdsqsR+cLSHR+K0OY3dZHcE-w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 22+ messages in thread From: Rob Herring @ 2018-01-12 20:45 UTC (permalink / raw) To: Olof Johansson, Arnd Bergmann Cc: Viresh Kumar, ARM-SoC Maintainers, Linux ARM Mailing List, DTML, gregkh On Fri, Jan 12, 2018 at 12:23 PM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: > On Fri, Jan 12, 2018 at 12:56 AM, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> wrote: >> On Fri, Jan 12, 2018 at 4:23 AM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: >>> On Thu, Jan 11, 2018 at 7:22 PM, Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: >>>> On 11-01-18, 18:07, Olof Johansson wrote: >>>>> On Thu, Jan 11, 2018 at 11:28:51AM +0530, Viresh Kumar wrote: >>>>> > The interrupt-parent of rtc was missing, add it. >>>>> > >>>>> > Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") >>>>> > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ >>>>> > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> >>>>> > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >>>>> >>>>> Applied to next/dt. Is stable really needed on this? It's been broken since >>>>> pretty much forever, and nobody has complained... :) >>>> >>>> Not sure. Just thought it may be useful for someone somewhere :) >>> >>> Ok. Left the tags there, but didn't merge into fixes since we're late >>> in -rc and this didn't seem critical at this time. >> >> My plan was to have these in the fixes branch in the hope of making >> it to a clean build for 4.15 after all, they fix warnings that got introduced >> by the updated dtc checks in 4.15-rc1. >> >> We are getting fairly close, but it seems we still miss a few, so we >> might as well give up at this point. The remaining fixes should be easy >> to backport into v4.15.y if we decide to do it, of further back even. >> For v4.14 and before, the in-kernel copy of dtc won't warn, but mainline >> dtc will. >> >> Greg, let me know your thoughts on this for the upcoming 4.15.y >> release. We had hundreds of dtc warnings in 4.15-rc1, many of them >> about important bugs, now we're down to a couple of warnings >> for platforms we don't care about much, and I expect the last of >> these fixes to land in 4.16-rc1 or maybe -rc2. Shall we backport >> them all to get a clean 4.15.y release? > > I think it makes more sense to disable the warnings than to backport a > bunch of warning fixes this late. The code is working, has worked for > a long time it's just that Rob enabled the warnings by default. We can > keep them enabled for 4.16. In some cases "working" was what's in the DT is unused by the kernel because the DT is broken. That's why this round was not off by default. It looks to me to be somewhere less than 5 fixes remaining (BTW, why is there no arm32 allmodconfig build on kernelci.org? That or allyesconfig are the only ways to build all dtbs). It would also be an exception to the the stable process because that patch would not be in Linus' tree. >> Note: there was at least one dtc warning fix that caused a serious >> regression in code that relied on a device probe to fail because of >> an invalid node (a fix is still in the works for 4.15), though generally >> the fixes are really harmless and can only make things better. > > Exactly why picking up warning fixes this late is probably not a great idea. Applying them now for 4.15 would be late, but tagging them as fixes in 4.16 would not be late (other than the normal problem that things get applied to stable before sufficient testing in master). I have more dtc checks in the works (nothing for 4.16 :) ). I'd like the process to work better. I'm not going to fix all the warnings. I don't think Arnd should either. Turning them off by default hasn't worked great either. For some, I'm not sure we can ever get to warning free, but I'd like new stuff to have warnings enabled and no one builds with W=1. We could put together tooling to just show new warnings, but someone has to run it and enforce it. I could stick dtc updates into linux-next for multiple cycles before sending to Linus. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAL_JsqJVWLazDRB1me2OubqqMhdsqsR+cLSHR+K0OY3dZHcE-w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CAL_JsqJVWLazDRB1me2OubqqMhdsqsR+cLSHR+K0OY3dZHcE-w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-12 21:50 ` Olof Johansson [not found] ` <CAOesGMiyfpRyoe4dxQeNMF5invysVBEy+J5OwUhuop=JyAvZoQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 22+ messages in thread From: Olof Johansson @ 2018-01-12 21:50 UTC (permalink / raw) To: Rob Herring Cc: Arnd Bergmann, Viresh Kumar, ARM-SoC Maintainers, Linux ARM Mailing List, DTML, gregkh On Fri, Jan 12, 2018 at 12:45 PM, Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote: > On Fri, Jan 12, 2018 at 12:23 PM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: >> On Fri, Jan 12, 2018 at 12:56 AM, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> wrote: >>> On Fri, Jan 12, 2018 at 4:23 AM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: >>>> On Thu, Jan 11, 2018 at 7:22 PM, Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: >>>>> On 11-01-18, 18:07, Olof Johansson wrote: >>>>>> On Thu, Jan 11, 2018 at 11:28:51AM +0530, Viresh Kumar wrote: >>>>>> > The interrupt-parent of rtc was missing, add it. >>>>>> > >>>>>> > Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") >>>>>> > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ >>>>>> > Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> >>>>>> > Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >>>>>> >>>>>> Applied to next/dt. Is stable really needed on this? It's been broken since >>>>>> pretty much forever, and nobody has complained... :) >>>>> >>>>> Not sure. Just thought it may be useful for someone somewhere :) >>>> >>>> Ok. Left the tags there, but didn't merge into fixes since we're late >>>> in -rc and this didn't seem critical at this time. >>> >>> My plan was to have these in the fixes branch in the hope of making >>> it to a clean build for 4.15 after all, they fix warnings that got introduced >>> by the updated dtc checks in 4.15-rc1. >>> >>> We are getting fairly close, but it seems we still miss a few, so we >>> might as well give up at this point. The remaining fixes should be easy >>> to backport into v4.15.y if we decide to do it, of further back even. >>> For v4.14 and before, the in-kernel copy of dtc won't warn, but mainline >>> dtc will. >>> >>> Greg, let me know your thoughts on this for the upcoming 4.15.y >>> release. We had hundreds of dtc warnings in 4.15-rc1, many of them >>> about important bugs, now we're down to a couple of warnings >>> for platforms we don't care about much, and I expect the last of >>> these fixes to land in 4.16-rc1 or maybe -rc2. Shall we backport >>> them all to get a clean 4.15.y release? >> >> I think it makes more sense to disable the warnings than to backport a >> bunch of warning fixes this late. The code is working, has worked for >> a long time it's just that Rob enabled the warnings by default. We can >> keep them enabled for 4.16. > > In some cases "working" was what's in the DT is unused by the kernel > because the DT is broken. That's why this round was not off by > default. > > It looks to me to be somewhere less than 5 fixes remaining (BTW, why > is there no arm32 allmodconfig build on kernelci.org? That or > allyesconfig are the only ways to build all dtbs). It would also be an > exception to the the stable process because that patch would not be in > Linus' tree. I build them but my script that analyses for warnings only looks for the gcc "warning:" output. Need to do grep -i to catch them. FWIW, logs are here: http://arm-soc.lixom.net/buildlogs/mainline/v4.15-rc7-200-gc92a9a4/buildall.arm.allmodconfig.log.passed >>> Note: there was at least one dtc warning fix that caused a serious >>> regression in code that relied on a device probe to fail because of >>> an invalid node (a fix is still in the works for 4.15), though generally >>> the fixes are really harmless and can only make things better. >> >> Exactly why picking up warning fixes this late is probably not a great idea. > > Applying them now for 4.15 would be late, but tagging them as fixes in > 4.16 would not be late (other than the normal problem that things get > applied to stable before sufficient testing in master). > > > I have more dtc checks in the works (nothing for 4.16 :) ). I'd like > the process to work better. I'm not going to fix all the warnings. I > don't think Arnd should either. Turning them off by default hasn't > worked great either. For some, I'm not sure we can ever get to warning > free, but I'd like new stuff to have warnings enabled and no one > builds with W=1. We could put together tooling to just show new > warnings, but someone has to run it and enforce it. I could stick dtc > updates into linux-next for multiple cycles before sending to Linus. I'll split up and report DT warnings separate from compiler, seems like a reasonable approach. -Olof -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAOesGMiyfpRyoe4dxQeNMF5invysVBEy+J5OwUhuop=JyAvZoQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CAOesGMiyfpRyoe4dxQeNMF5invysVBEy+J5OwUhuop=JyAvZoQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-12 21:53 ` Arnd Bergmann [not found] ` <CAK8P3a3KTz0sOSbO9+sAY1Sv95hZF4Yg1qMzOaXPAZ_1pOgGpg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 22+ messages in thread From: Arnd Bergmann @ 2018-01-12 21:53 UTC (permalink / raw) To: Olof Johansson Cc: Rob Herring, Viresh Kumar, ARM-SoC Maintainers, Linux ARM Mailing List, DTML, gregkh On Fri, Jan 12, 2018 at 10:50 PM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: > On Fri, Jan 12, 2018 at 12:45 PM, Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote: >> On Fri, Jan 12, 2018 at 12:23 PM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: >> I have more dtc checks in the works (nothing for 4.16 :) ). I'd like >> the process to work better. I'm not going to fix all the warnings. I >> don't think Arnd should either. Turning them off by default hasn't >> worked great either. For some, I'm not sure we can ever get to warning >> free, but I'd like new stuff to have warnings enabled and no one >> builds with W=1. We could put together tooling to just show new >> warnings, but someone has to run it and enforce it. I could stick dtc >> updates into linux-next for multiple cycles before sending to Linus. > > I'll split up and report DT warnings separate from compiler, seems > like a reasonable approach. Maybe also report any other output from the build process as another category. When things build fine, we should see no output at all. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAK8P3a3KTz0sOSbO9+sAY1Sv95hZF4Yg1qMzOaXPAZ_1pOgGpg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CAK8P3a3KTz0sOSbO9+sAY1Sv95hZF4Yg1qMzOaXPAZ_1pOgGpg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-12 21:57 ` Olof Johansson [not found] ` <CAOesGMiHeitS_j9r0DsPf3k1jVJT9_z-bHyrhbB7fQ__LjjaqA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 22+ messages in thread From: Olof Johansson @ 2018-01-12 21:57 UTC (permalink / raw) To: Arnd Bergmann Cc: Rob Herring, Viresh Kumar, ARM-SoC Maintainers, Linux ARM Mailing List, DTML, gregkh On Fri, Jan 12, 2018 at 1:53 PM, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> wrote: > On Fri, Jan 12, 2018 at 10:50 PM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: >> On Fri, Jan 12, 2018 at 12:45 PM, Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote: >>> On Fri, Jan 12, 2018 at 12:23 PM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: >>> I have more dtc checks in the works (nothing for 4.16 :) ). I'd like >>> the process to work better. I'm not going to fix all the warnings. I >>> don't think Arnd should either. Turning them off by default hasn't >>> worked great either. For some, I'm not sure we can ever get to warning >>> free, but I'd like new stuff to have warnings enabled and no one >>> builds with W=1. We could put together tooling to just show new >>> warnings, but someone has to run it and enforce it. I could stick dtc >>> updates into linux-next for multiple cycles before sending to Linus. >> >> I'll split up and report DT warnings separate from compiler, seems >> like a reasonable approach. > > Maybe also report any other output from the build process as another > category. When things build fine, we should see no output at all. Yep, true. Right now there's mostly DTC warnings and missing MODULE_LICENSE(). -Olof -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAOesGMiHeitS_j9r0DsPf3k1jVJT9_z-bHyrhbB7fQ__LjjaqA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CAOesGMiHeitS_j9r0DsPf3k1jVJT9_z-bHyrhbB7fQ__LjjaqA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-12 22:05 ` Arnd Bergmann [not found] ` <CAK8P3a3MZX-ZsxX9h1-8kVQhn9hqY=c8L1+=e3uTfjEjBCYuPg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 22+ messages in thread From: Arnd Bergmann @ 2018-01-12 22:05 UTC (permalink / raw) To: Olof Johansson Cc: Rob Herring, Viresh Kumar, ARM-SoC Maintainers, Linux ARM Mailing List, DTML, gregkh, Linus Walleij On Fri, Jan 12, 2018 at 10:57 PM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: > On Fri, Jan 12, 2018 at 1:53 PM, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> wrote: >> On Fri, Jan 12, 2018 at 10:50 PM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: >>> On Fri, Jan 12, 2018 at 12:45 PM, Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote: >>>> On Fri, Jan 12, 2018 at 12:23 PM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: >>>> I have more dtc checks in the works (nothing for 4.16 :) ). I'd like >>>> the process to work better. I'm not going to fix all the warnings. I >>>> don't think Arnd should either. Turning them off by default hasn't >>>> worked great either. For some, I'm not sure we can ever get to warning >>>> free, but I'd like new stuff to have warnings enabled and no one >>>> builds with W=1. We could put together tooling to just show new >>>> warnings, but someone has to run it and enforce it. I could stick dtc >>>> updates into linux-next for multiple cycles before sending to Linus. >>> >>> I'll split up and report DT warnings separate from compiler, seems >>> like a reasonable approach. >> >> Maybe also report any other output from the build process as another >> category. When things build fine, we should see no output at all. > > Yep, true. > > Right now there's mostly DTC warnings and missing MODULE_LICENSE(). I've sent patches for all the remaining MODULE_LICENSE warnings earlier this week, and we should now have patches for almost all the dtc warnings in flight. This is what I still see on my randconfig builder with linux-next + arm-soc/for-next + patches I sent: arch/arm/boot/dts/spear1310-evb.dtb: Warning (gpios_property): Missing property '#gpio-cells' in node /interrupt-controller@ec801000 or bad phandle (referred from /ahb/apb/spi@e0100000:cs-gpios[6]) arch/arm/boot/dts/spear1310-evb.dtb: Warning (gpios_property): Property 'cs-gpios', cell 6 is not a phandle reference in /ahb/apb/spi@e0100000 arch/arm/boot/dts/spear1340-evb.dtb: Warning (dmas_property): Missing property '#dma-cells' in node /interrupt-controller@ec801000 or bad phandle (referred from /ahb/apb/serial@b4100000:dmas[4]) arch/arm/boot/dts/spear1340-evb.dtb: Warning (dmas_property): Property 'dmas', cell 4 is not a phandle reference in /ahb/apb/serial@b4100000 arch/arm/boot/dts/spear600-evb.dtb: Warning (interrupts_property): Missing interrupt-parent for /ahb/apb/rtc@fc900000 arch/arm/boot/dts/ste-nomadik-nhk15.dtb: Warning (interrupts_property): Missing interrupt-parent for /amba/clcd@10120000 arch/arm/boot/dts/ste-nomadik-s8815.dtb: Warning (interrupts_property): Missing interrupt-parent for /amba/clcd@10120000 Not sure why the spear warnings are still there, maybe Viresh missed those or you missed one of his patches? I hope Linus Walleij can find out what the right interrupt-parent should be on the nomadik machine, that is otherwise the last remaining warning. Apparently the clcd driver doesn't actually need its interrupt, so we could decide to just comment out that line if we don't know which controller it's connected to. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAK8P3a3MZX-ZsxX9h1-8kVQhn9hqY=c8L1+=e3uTfjEjBCYuPg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CAK8P3a3MZX-ZsxX9h1-8kVQhn9hqY=c8L1+=e3uTfjEjBCYuPg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-14 11:20 ` Linus Walleij [not found] ` <CACRpkdag317LU2_3H-PPV_0xoX2RDLxBX5r4a3jpbn9LTvhkcA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-15 4:22 ` Viresh Kumar 1 sibling, 1 reply; 22+ messages in thread From: Linus Walleij @ 2018-01-14 11:20 UTC (permalink / raw) To: Arnd Bergmann Cc: Olof Johansson, Rob Herring, Viresh Kumar, ARM-SoC Maintainers, Linux ARM Mailing List, DTML, gregkh On Fri, Jan 12, 2018 at 11:05 PM, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> wrote: > arch/arm/boot/dts/ste-nomadik-nhk15.dtb: Warning > (interrupts_property): Missing interrupt-parent for > /amba/clcd@10120000 > arch/arm/boot/dts/ste-nomadik-s8815.dtb: Warning > (interrupts_property): Missing interrupt-parent for > /amba/clcd@10120000 (...) > I hope Linus Walleij can find out what the right interrupt-parent > should be on the > nomadik machine, that is otherwise the last remaining warning. OK I will look into it. > Apparently the > clcd driver doesn't actually need its interrupt, so we could decide to > just comment > out that line if we don't know which controller it's connected to. I am migrating it to the new DRM driver which uses the IRQ to sync screen output to avoid tearing so it would be unfortunate. I will look into it. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CACRpkdag317LU2_3H-PPV_0xoX2RDLxBX5r4a3jpbn9LTvhkcA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CACRpkdag317LU2_3H-PPV_0xoX2RDLxBX5r4a3jpbn9LTvhkcA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-15 16:34 ` Arnd Bergmann 0 siblings, 0 replies; 22+ messages in thread From: Arnd Bergmann @ 2018-01-15 16:34 UTC (permalink / raw) To: Linus Walleij Cc: Olof Johansson, Rob Herring, Viresh Kumar, ARM-SoC Maintainers, Linux ARM Mailing List, DTML, gregkh On Sun, Jan 14, 2018 at 12:20 PM, Linus Walleij <linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: > On Fri, Jan 12, 2018 at 11:05 PM, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> wrote: > >> arch/arm/boot/dts/ste-nomadik-nhk15.dtb: Warning >> (interrupts_property): Missing interrupt-parent for >> /amba/clcd@10120000 >> arch/arm/boot/dts/ste-nomadik-s8815.dtb: Warning >> (interrupts_property): Missing interrupt-parent for >> /amba/clcd@10120000 > (...) >> I hope Linus Walleij can find out what the right interrupt-parent >> should be on the >> nomadik machine, that is otherwise the last remaining warning. > > OK I will look into it. I took another look myself now since this was the last remaining dtc warning in linux-next. The answer was easy enough to find in the git history for the board files: all the on-chip devices are connected to the same VIC instance. I'll send the patch. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CAK8P3a3MZX-ZsxX9h1-8kVQhn9hqY=c8L1+=e3uTfjEjBCYuPg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-14 11:20 ` Linus Walleij @ 2018-01-15 4:22 ` Viresh Kumar 2018-01-15 12:49 ` Arnd Bergmann 1 sibling, 1 reply; 22+ messages in thread From: Viresh Kumar @ 2018-01-15 4:22 UTC (permalink / raw) To: Arnd Bergmann Cc: Olof Johansson, Rob Herring, ARM-SoC Maintainers, Linux ARM Mailing List, DTML, gregkh, Linus Walleij On 12-01-18, 23:05, Arnd Bergmann wrote: > I've sent patches for all the remaining MODULE_LICENSE warnings earlier > this week, and we should now have patches for almost all the dtc > warnings in flight. This is what I still see on my randconfig builder > with linux-next + arm-soc/for-next + patches I sent: > > arch/arm/boot/dts/spear1310-evb.dtb: Warning (gpios_property): > Missing property '#gpio-cells' in node /interrupt-controller@ec801000 > or bad phandle (referred from /ahb/apb/spi@e0100000:cs-gpios[6]) > arch/arm/boot/dts/spear1310-evb.dtb: Warning (gpios_property): > Property 'cs-gpios', cell 6 is not a phandle reference in > /ahb/apb/spi@e0100000 > arch/arm/boot/dts/spear1340-evb.dtb: Warning (dmas_property): > Missing property '#dma-cells' in node /interrupt-controller@ec801000 > or bad phandle (referred from /ahb/apb/serial@b4100000:dmas[4]) > arch/arm/boot/dts/spear1340-evb.dtb: Warning (dmas_property): > Property 'dmas', cell 4 is not a phandle reference in > /ahb/apb/serial@b4100000 > arch/arm/boot/dts/spear600-evb.dtb: Warning > (interrupts_property): Missing interrupt-parent for > /ahb/apb/rtc@fc900000 These are all the warnings we got earlier. So none of my patches are in the branch you tested I believe. Can you do a `git log` and see if they are present ? -- viresh -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc 2018-01-15 4:22 ` Viresh Kumar @ 2018-01-15 12:49 ` Arnd Bergmann 0 siblings, 0 replies; 22+ messages in thread From: Arnd Bergmann @ 2018-01-15 12:49 UTC (permalink / raw) To: Viresh Kumar Cc: DTML, ARM-SoC Maintainers, gregkh, Linus Walleij, Rob Herring, Olof Johansson, Linux ARM Mailing List On Mon, Jan 15, 2018 at 5:22 AM, Viresh Kumar <viresh.kumar@linaro.org> wrote: > On 12-01-18, 23:05, Arnd Bergmann wrote: >> I've sent patches for all the remaining MODULE_LICENSE warnings earlier >> this week, and we should now have patches for almost all the dtc >> warnings in flight. This is what I still see on my randconfig builder >> with linux-next + arm-soc/for-next + patches I sent: >> >> arch/arm/boot/dts/spear1310-evb.dtb: Warning (gpios_property): >> Missing property '#gpio-cells' in node /interrupt-controller@ec801000 >> or bad phandle (referred from /ahb/apb/spi@e0100000:cs-gpios[6]) >> arch/arm/boot/dts/spear1310-evb.dtb: Warning (gpios_property): >> Property 'cs-gpios', cell 6 is not a phandle reference in >> /ahb/apb/spi@e0100000 >> arch/arm/boot/dts/spear1340-evb.dtb: Warning (dmas_property): >> Missing property '#dma-cells' in node /interrupt-controller@ec801000 >> or bad phandle (referred from /ahb/apb/serial@b4100000:dmas[4]) >> arch/arm/boot/dts/spear1340-evb.dtb: Warning (dmas_property): >> Property 'dmas', cell 4 is not a phandle reference in >> /ahb/apb/serial@b4100000 >> arch/arm/boot/dts/spear600-evb.dtb: Warning >> (interrupts_property): Missing interrupt-parent for >> /ahb/apb/rtc@fc900000 > > These are all the warnings we got earlier. So none of my patches are in the > branch you tested I believe. Can you do a `git log` and see if they are present > ? I checked again on today's linux-next and all your patches are there, and the warning is gone. Whatever caused the warnings to still appear seems to be gone now. Sorry for the confusion. Arnd ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc 2018-01-12 2:07 ` Olof Johansson 2018-01-12 3:22 ` Viresh Kumar @ 2018-01-14 11:17 ` Linus Walleij [not found] ` <CACRpkdYtzWewTWrtWTaPWspHzRY6Y7MBNfm1wmy6yvLD_J8hFg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 1 sibling, 1 reply; 22+ messages in thread From: Linus Walleij @ 2018-01-14 11:17 UTC (permalink / raw) To: Olof Johansson Cc: Viresh Kumar, Rob Herring, arm-soc, Linux ARM, Arnd Bergmann, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS On Fri, Jan 12, 2018 at 3:07 AM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: > On Thu, Jan 11, 2018 at 11:28:51AM +0530, Viresh Kumar wrote: >> The interrupt-parent of rtc was missing, add it. >> >> Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") >> Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ >> Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> >> Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > > Applied to next/dt. Is stable really needed on this? It's been broken since > pretty much forever, and nobody has complained... :) The SPEAr architecture is widely used (in modified variants) by a large industrial automation company, 20+ years of support cycle. However I think they have a forked both kernel and hardware, so they will not notice any time soon, and when they eventually upgrade they can simply pick the latest I guess. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CACRpkdYtzWewTWrtWTaPWspHzRY6Y7MBNfm1wmy6yvLD_J8hFg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc [not found] ` <CACRpkdYtzWewTWrtWTaPWspHzRY6Y7MBNfm1wmy6yvLD_J8hFg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2018-01-14 13:17 ` Alexandre Belloni 0 siblings, 0 replies; 22+ messages in thread From: Alexandre Belloni @ 2018-01-14 13:17 UTC (permalink / raw) To: Linus Walleij Cc: Olof Johansson, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, arm-soc, Arnd Bergmann, Viresh Kumar, Rob Herring, Linux ARM Hi, On 14/01/2018 at 12:17:23 +0100, Linus Walleij wrote: > On Fri, Jan 12, 2018 at 3:07 AM, Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> wrote: > > On Thu, Jan 11, 2018 at 11:28:51AM +0530, Viresh Kumar wrote: > >> The interrupt-parent of rtc was missing, add it. > >> > >> Fixes: 8113ba917dfa ("ARM: SPEAr: DT: Update device nodes") > >> Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # v3.8+ > >> Reported-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> > >> Signed-off-by: Viresh Kumar <viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > > > > Applied to next/dt. Is stable really needed on this? It's been broken since > > pretty much forever, and nobody has complained... :) > > The SPEAr architecture is widely used (in modified variants) by > a large industrial automation company, 20+ years of support cycle. > > However I think they have a forked both kernel and hardware, so they > will not notice any time soon, and when they eventually upgrade they > can simply pick the latest I guess. > We have a customer using mainline v4.14 on their Spear600 based products. I guess the reason why nobody complained is simply because nobody is using the SoC RTC ;) -- Alexandre Belloni, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2018-01-15 16:34 UTC | newest] Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-01-11 5:58 [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc Viresh Kumar [not found] ` <a062d9ed0d3365b578156a780202fa533e725374.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-11 5:58 ` [PATCH 2/3] arm: spear13xx: Fix dmas cells Viresh Kumar [not found] ` <6183c13fd3abcb2a6834a537a4809b6527204f10.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-12 2:07 ` Olof Johansson 2018-01-11 5:58 ` [PATCH 3/3] arm: spear13xx: Fix spics gpio controller's warning Viresh Kumar [not found] ` <9d8521e3990be6789f1ed3d5757b57c65483117b.1515650104.git.viresh.kumar-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-12 2:07 ` Olof Johansson 2018-01-11 13:47 ` [PATCH 1/3] arm: spear600: Add missing interrupt-parent of rtc Rob Herring 2018-01-12 2:07 ` Olof Johansson 2018-01-12 3:22 ` Viresh Kumar 2018-01-12 3:23 ` Olof Johansson [not found] ` <CAOesGMggU73asY1SEUDD_BsWcrGYR___o8e5omjziOPEwJgqBQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-12 8:56 ` Arnd Bergmann [not found] ` <CAK8P3a2SO4=YKBE3L_AKsyfNe77YR7a8F=dvbS=ic03=6NK8MA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-12 18:23 ` Olof Johansson [not found] ` <CAOesGMgcf9FovbzhPkKnrG4ETRFP3dsSJnTYNfp1wECMFeFOGA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-12 20:45 ` Rob Herring [not found] ` <CAL_JsqJVWLazDRB1me2OubqqMhdsqsR+cLSHR+K0OY3dZHcE-w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-12 21:50 ` Olof Johansson [not found] ` <CAOesGMiyfpRyoe4dxQeNMF5invysVBEy+J5OwUhuop=JyAvZoQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-12 21:53 ` Arnd Bergmann [not found] ` <CAK8P3a3KTz0sOSbO9+sAY1Sv95hZF4Yg1qMzOaXPAZ_1pOgGpg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-12 21:57 ` Olof Johansson [not found] ` <CAOesGMiHeitS_j9r0DsPf3k1jVJT9_z-bHyrhbB7fQ__LjjaqA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-12 22:05 ` Arnd Bergmann [not found] ` <CAK8P3a3MZX-ZsxX9h1-8kVQhn9hqY=c8L1+=e3uTfjEjBCYuPg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-14 11:20 ` Linus Walleij [not found] ` <CACRpkdag317LU2_3H-PPV_0xoX2RDLxBX5r4a3jpbn9LTvhkcA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-15 16:34 ` Arnd Bergmann 2018-01-15 4:22 ` Viresh Kumar 2018-01-15 12:49 ` Arnd Bergmann 2018-01-14 11:17 ` Linus Walleij [not found] ` <CACRpkdYtzWewTWrtWTaPWspHzRY6Y7MBNfm1wmy6yvLD_J8hFg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-01-14 13:17 ` Alexandre Belloni
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).