* Re: linux-next: build warnings after merge of the ti tree [not found] <20231005141536.77538147@canb.auug.org.au> @ 2023-10-05 8:25 ` Raghavendra, Vignesh 2023-10-05 8:25 ` Raghavendra, Vignesh 2023-10-05 13:12 ` Roger Quadros 0 siblings, 2 replies; 7+ messages in thread From: Raghavendra, Vignesh @ 2023-10-05 8:25 UTC (permalink / raw) To: Stephen Rothwell, Nishanth Menon, Tero Kristo, Santosh Shilimkar, Rob Herring, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS Cc: Roger Quadros, Linux Kernel Mailing List, Linux Next Mailing List ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build warnings after merge of the ti tree 2023-10-05 8:25 ` linux-next: build warnings after merge of the ti tree Raghavendra, Vignesh @ 2023-10-05 8:25 ` Raghavendra, Vignesh 2023-10-05 13:12 ` Roger Quadros 1 sibling, 0 replies; 7+ messages in thread From: Raghavendra, Vignesh @ 2023-10-05 8:25 UTC (permalink / raw) To: Stephen Rothwell, Nishanth Menon, Tero Kristo, Santosh Shilimkar, Rob Herring, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS Cc: Roger Quadros, Linux Kernel Mailing List, Linux Next Mailing List + Rob and DT list Hi Stephen On 10/5/2023 8:45 AM, Stephen Rothwell wrote: > Hi all, > > [I may have missed this yesterday, sorry] > > After merging the ti tree, today's linux-next build (arm64 defconfig) > produced these warnings: > > arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #address-cells value > arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #size-cells value > > Introduced by commit > > 45a0c06571e1 ("arm64: dts: ti: am642-evm: Add overlay for NAND expansion card") > Thanks for the report. I will drop the offending comment. Roger, Sorry, this would need to be fixed in dtc or need exception from DT maintainers to ignore the warnings. Regards Vignesh ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build warnings after merge of the ti tree 2023-10-05 8:25 ` linux-next: build warnings after merge of the ti tree Raghavendra, Vignesh 2023-10-05 8:25 ` Raghavendra, Vignesh @ 2023-10-05 13:12 ` Roger Quadros 2023-10-05 13:29 ` Nishanth Menon 1 sibling, 1 reply; 7+ messages in thread From: Roger Quadros @ 2023-10-05 13:12 UTC (permalink / raw) To: Raghavendra, Vignesh, Stephen Rothwell, Nishanth Menon, Tero Kristo, Santosh Shilimkar, Rob Herring, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS Cc: Linux Kernel Mailing List, Linux Next Mailing List Hi, On 05/10/2023 11:25, Raghavendra, Vignesh wrote: > + Rob and DT list > > Hi Stephen > > On 10/5/2023 8:45 AM, Stephen Rothwell wrote: >> Hi all, >> >> [I may have missed this yesterday, sorry] >> >> After merging the ti tree, today's linux-next build (arm64 defconfig) >> produced these warnings: >> >> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #address-cells value >> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #size-cells value >> >> Introduced by commit >> >> 45a0c06571e1 ("arm64: dts: ti: am642-evm: Add overlay for NAND expansion card") >> > > Thanks for the report. I will drop the offending comment. > > Roger, > > Sorry, this would need to be fixed in dtc or need exception from DT > maintainers to ignore the warnings. Please don't drop this patch as the issue is not with the patch but with the dtc tool itself. As this is a DT overlay there is no way to specify address-cells/size-cells of parent here. This will be resolved only after merge with base tree. This will be fixed in next dtc sync. https://www.spinics.net/lists/devicetree-compiler/msg04036.html See further discussion here https://lore.kernel.org/all/CAL_JsqLmv904+_2EOmsQ__y1yLDvsT+_02i85phuh0cpe7X8NQ@mail.gmail.com/ -- cheers, -roger ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build warnings after merge of the ti tree 2023-10-05 13:12 ` Roger Quadros @ 2023-10-05 13:29 ` Nishanth Menon 2023-10-06 12:03 ` Roger Quadros 0 siblings, 1 reply; 7+ messages in thread From: Nishanth Menon @ 2023-10-05 13:29 UTC (permalink / raw) To: Roger Quadros Cc: Raghavendra, Vignesh, Stephen Rothwell, Tero Kristo, Santosh Shilimkar, Rob Herring, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux Kernel Mailing List, Linux Next Mailing List On 16:12-20231005, Roger Quadros wrote: > Hi, > > On 05/10/2023 11:25, Raghavendra, Vignesh wrote: > > + Rob and DT list > > > > Hi Stephen > > > > On 10/5/2023 8:45 AM, Stephen Rothwell wrote: > >> Hi all, > >> > >> [I may have missed this yesterday, sorry] > >> > >> After merging the ti tree, today's linux-next build (arm64 defconfig) > >> produced these warnings: > >> > >> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #address-cells value > >> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #size-cells value > >> > >> Introduced by commit > >> > >> 45a0c06571e1 ("arm64: dts: ti: am642-evm: Add overlay for NAND expansion card") > >> > > > > Thanks for the report. I will drop the offending comment. > > > > Roger, > > > > Sorry, this would need to be fixed in dtc or need exception from DT > > maintainers to ignore the warnings. > > Please don't drop this patch as the issue is not with the patch but with > the dtc tool itself. > > As this is a DT overlay there is no way to specify address-cells/size-cells > of parent here. This will be resolved only after merge with base tree. > > This will be fixed in next dtc sync. > https://www.spinics.net/lists/devicetree-compiler/msg04036.html > > See further discussion here > https://lore.kernel.org/all/CAL_JsqLmv904+_2EOmsQ__y1yLDvsT+_02i85phuh0cpe7X8NQ@mail.gmail.com/ > Roger, build warnings are a strict NO,NO for kernel. Lets bring in the series *after* the dtc sync is complete. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build warnings after merge of the ti tree 2023-10-05 13:29 ` Nishanth Menon @ 2023-10-06 12:03 ` Roger Quadros 2023-10-06 13:55 ` Rob Herring 0 siblings, 1 reply; 7+ messages in thread From: Roger Quadros @ 2023-10-06 12:03 UTC (permalink / raw) To: Nishanth Menon, Rob Herring Cc: Raghavendra, Vignesh, Stephen Rothwell, Tero Kristo, Santosh Shilimkar, Rob Herring, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux Kernel Mailing List, Linux Next Mailing List, Yadav, Nitin On 05/10/2023 16:29, Nishanth Menon wrote: > On 16:12-20231005, Roger Quadros wrote: >> Hi, >> >> On 05/10/2023 11:25, Raghavendra, Vignesh wrote: >>> + Rob and DT list >>> >>> Hi Stephen >>> >>> On 10/5/2023 8:45 AM, Stephen Rothwell wrote: >>>> Hi all, >>>> >>>> [I may have missed this yesterday, sorry] >>>> >>>> After merging the ti tree, today's linux-next build (arm64 defconfig) >>>> produced these warnings: >>>> >>>> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #address-cells value >>>> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #size-cells value >>>> >>>> Introduced by commit >>>> >>>> 45a0c06571e1 ("arm64: dts: ti: am642-evm: Add overlay for NAND expansion card") >>>> >>> >>> Thanks for the report. I will drop the offending comment. >>> >>> Roger, >>> >>> Sorry, this would need to be fixed in dtc or need exception from DT >>> maintainers to ignore the warnings. >> >> Please don't drop this patch as the issue is not with the patch but with >> the dtc tool itself. >> >> As this is a DT overlay there is no way to specify address-cells/size-cells >> of parent here. This will be resolved only after merge with base tree. >> >> This will be fixed in next dtc sync. >> https://www.spinics.net/lists/devicetree-compiler/msg04036.html >> >> See further discussion here >> https://lore.kernel.org/all/CAL_JsqLmv904+_2EOmsQ__y1yLDvsT+_02i85phuh0cpe7X8NQ@mail.gmail.com/ >> > > Roger, build warnings are a strict NO,NO for kernel. Lets bring in the > series *after* the dtc sync is complete. > Hi Rob, Following commit from dtc is required to resolve this issue. afbddcd418fb ("Suppress warnings on overlay fragments") FYI. Another patch of similar nature is on its way and will also require the above commit to DTC. https://lore.kernel.org/all/20231005093739.4071934-3-n-yadav@ti.com/ -- cheers, -roger ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build warnings after merge of the ti tree 2023-10-06 12:03 ` Roger Quadros @ 2023-10-06 13:55 ` Rob Herring 2023-10-06 15:41 ` Roger Quadros 0 siblings, 1 reply; 7+ messages in thread From: Rob Herring @ 2023-10-06 13:55 UTC (permalink / raw) To: Roger Quadros Cc: Nishanth Menon, Raghavendra, Vignesh, Stephen Rothwell, Tero Kristo, Santosh Shilimkar, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux Kernel Mailing List, Linux Next Mailing List, Yadav, Nitin On Fri, Oct 6, 2023 at 7:03 AM Roger Quadros <rogerq@kernel.org> wrote: > > > > On 05/10/2023 16:29, Nishanth Menon wrote: > > On 16:12-20231005, Roger Quadros wrote: > >> Hi, > >> > >> On 05/10/2023 11:25, Raghavendra, Vignesh wrote: > >>> + Rob and DT list > >>> > >>> Hi Stephen > >>> > >>> On 10/5/2023 8:45 AM, Stephen Rothwell wrote: > >>>> Hi all, > >>>> > >>>> [I may have missed this yesterday, sorry] > >>>> > >>>> After merging the ti tree, today's linux-next build (arm64 defconfig) > >>>> produced these warnings: > >>>> > >>>> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #address-cells value > >>>> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #size-cells value > >>>> > >>>> Introduced by commit > >>>> > >>>> 45a0c06571e1 ("arm64: dts: ti: am642-evm: Add overlay for NAND expansion card") > >>>> > >>> > >>> Thanks for the report. I will drop the offending comment. > >>> > >>> Roger, > >>> > >>> Sorry, this would need to be fixed in dtc or need exception from DT > >>> maintainers to ignore the warnings. > >> > >> Please don't drop this patch as the issue is not with the patch but with > >> the dtc tool itself. > >> > >> As this is a DT overlay there is no way to specify address-cells/size-cells > >> of parent here. This will be resolved only after merge with base tree. > >> > >> This will be fixed in next dtc sync. > >> https://www.spinics.net/lists/devicetree-compiler/msg04036.html > >> > >> See further discussion here > >> https://lore.kernel.org/all/CAL_JsqLmv904+_2EOmsQ__y1yLDvsT+_02i85phuh0cpe7X8NQ@mail.gmail.com/ > >> > > > > Roger, build warnings are a strict NO,NO for kernel. Lets bring in the > > series *after* the dtc sync is complete. > > > > Hi Rob, > > Following commit from dtc is required to resolve this issue. > > afbddcd418fb ("Suppress warnings on overlay fragments") That is from 2018, so it's been in the kernel's dtc for a long time. I believe what you need is the patch I referenced which hasn't been applied upstream. > FYI. Another patch of similar nature is on its way and will also require the > above commit to DTC. > https://lore.kernel.org/all/20231005093739.4071934-3-n-yadav@ti.com/ You'll need to fix dtc first. Rob ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build warnings after merge of the ti tree 2023-10-06 13:55 ` Rob Herring @ 2023-10-06 15:41 ` Roger Quadros 0 siblings, 0 replies; 7+ messages in thread From: Roger Quadros @ 2023-10-06 15:41 UTC (permalink / raw) To: Rob Herring Cc: Nishanth Menon, Raghavendra, Vignesh, Stephen Rothwell, Tero Kristo, Santosh Shilimkar, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux Kernel Mailing List, Linux Next Mailing List, Yadav, Nitin On 06/10/2023 16:55, Rob Herring wrote: > On Fri, Oct 6, 2023 at 7:03 AM Roger Quadros <rogerq@kernel.org> wrote: >> >> >> >> On 05/10/2023 16:29, Nishanth Menon wrote: >>> On 16:12-20231005, Roger Quadros wrote: >>>> Hi, >>>> >>>> On 05/10/2023 11:25, Raghavendra, Vignesh wrote: >>>>> + Rob and DT list >>>>> >>>>> Hi Stephen >>>>> >>>>> On 10/5/2023 8:45 AM, Stephen Rothwell wrote: >>>>>> Hi all, >>>>>> >>>>>> [I may have missed this yesterday, sorry] >>>>>> >>>>>> After merging the ti tree, today's linux-next build (arm64 defconfig) >>>>>> produced these warnings: >>>>>> >>>>>> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #address-cells value >>>>>> arch/arm64/boot/dts/ti/k3-am642-evm-nand.dtso:65.8-140.3: Warning (avoid_default_addr_size): /fragment@3/__overlay__: Relying on default #size-cells value >>>>>> >>>>>> Introduced by commit >>>>>> >>>>>> 45a0c06571e1 ("arm64: dts: ti: am642-evm: Add overlay for NAND expansion card") >>>>>> >>>>> >>>>> Thanks for the report. I will drop the offending comment. >>>>> >>>>> Roger, >>>>> >>>>> Sorry, this would need to be fixed in dtc or need exception from DT >>>>> maintainers to ignore the warnings. >>>> >>>> Please don't drop this patch as the issue is not with the patch but with >>>> the dtc tool itself. >>>> >>>> As this is a DT overlay there is no way to specify address-cells/size-cells >>>> of parent here. This will be resolved only after merge with base tree. >>>> >>>> This will be fixed in next dtc sync. >>>> https://www.spinics.net/lists/devicetree-compiler/msg04036.html >>>> >>>> See further discussion here >>>> https://lore.kernel.org/all/CAL_JsqLmv904+_2EOmsQ__y1yLDvsT+_02i85phuh0cpe7X8NQ@mail.gmail.com/ >>>> >>> >>> Roger, build warnings are a strict NO,NO for kernel. Lets bring in the >>> series *after* the dtc sync is complete. >>> >> >> Hi Rob, >> >> Following commit from dtc is required to resolve this issue. >> >> afbddcd418fb ("Suppress warnings on overlay fragments") > > That is from 2018, so it's been in the kernel's dtc for a long time. Oh, my bad. I saw your Reviewed-by tag and assumed the patch was already upstream. > > I believe what you need is the patch I referenced which hasn't been > applied upstream. > >> FYI. Another patch of similar nature is on its way and will also require the >> above commit to DTC. >> https://lore.kernel.org/all/20231005093739.4071934-3-n-yadav@ti.com/ > > You'll need to fix dtc first. -- cheers, -roger ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-10-06 15:42 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20231005141536.77538147@canb.auug.org.au>
2023-10-05 8:25 ` linux-next: build warnings after merge of the ti tree Raghavendra, Vignesh
2023-10-05 8:25 ` Raghavendra, Vignesh
2023-10-05 13:12 ` Roger Quadros
2023-10-05 13:29 ` Nishanth Menon
2023-10-06 12:03 ` Roger Quadros
2023-10-06 13:55 ` Rob Herring
2023-10-06 15:41 ` Roger Quadros
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).