linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Arınç ÜNAL" <arinc.unal@arinc9.com>
To: Arnd Bergmann <arnd@arndb.de>, Stephen Rothwell <sfr@canb.auug.org.au>
Cc: "Olof Johansson" <olof@lixom.net>,
	ARM <linux-arm-kernel@lists.infradead.org>,
	"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
	"Linux Next Mailing List" <linux-next@vger.kernel.org>,
	"Rafał Miłecki" <rafal@milecki.pl>,
	"Florian Fainelli" <f.fainelli@gmail.com>,
	bcm-kernel-feedback-list <bcm-kernel-feedback-list@broadcom.com>
Subject: Re: linux-next: build warnings after merge of the arm-soc tree
Date: Fri, 13 May 2022 13:22:31 +0300	[thread overview]
Message-ID: <8e5d2f7f-0b48-6317-d1d9-dee6a7c06d3c@arinc9.com> (raw)
In-Reply-To: <CAK8P3a27bPttwrOBKPwNZ=B1zGZqKV=Js-zWcO7ofnqc=xyHQA@mail.gmail.com>

On 13/05/2022 12:18, Arnd Bergmann wrote:
> On Fri, May 13, 2022 at 9:29 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>> On Fri, 6 May 2022 08:58:15 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>>>
>>> After merging the arm-soc tree, today's linux-next build (arm
>>> multi_v7_defconfig) produced these warnings:
>>>
>>> arch/arm/boot/dts/bcm953012hr.dts:57.3-33: Warning (reg_format): /nand-controller@18028000/nand@0/partition@0:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 1)
>>> arch/arm/boot/dts/bcm953012hr.dts:62.3-33: Warning (reg_format): /nand-controller@18028000/nand@0/partition@200000:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 1)
>>> arch/arm/boot/dts/bcm953012hr.dts:66.3-33: Warning (reg_format): /nand-controller@18028000/nand@0/partition@600000:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 1)
>>> arch/arm/boot/dts/bcm953012hr.dts:70.3-33: Warning (reg_format): /nand-controller@18028000/nand@0/partition@1000000:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 1)
>>> arch/arm/boot/dts/bcm953012hr.dtb: Warning (pci_device_reg): Failed prerequisite 'reg_format'
>>> arch/arm/boot/dts/bcm953012hr.dtb: Warning (pci_device_bus_num): Failed prerequisite 'reg_format'
>>> arch/arm/boot/dts/bcm953012hr.dtb: Warning (i2c_bus_reg): Failed prerequisite 'reg_format'
>>> arch/arm/boot/dts/bcm953012hr.dtb: Warning (spi_bus_reg): Failed prerequisite 'reg_format'
>>> arch/arm/boot/dts/bcm953012hr.dts:55.14-59.4: Warning (avoid_default_addr_size): /nand-controller@18028000/nand@0/partition@0: Relying on default #address-cells value
>>> arch/arm/boot/dts/bcm953012hr.dts:55.14-59.4: Warning (avoid_default_addr_size): /nand-controller@18028000/nand@0/partition@0: Relying on default #size-cells value
>>> arch/arm/boot/dts/bcm953012hr.dts:60.19-63.4: Warning (avoid_default_addr_size): /nand-controller@18028000/nand@0/partition@200000: Relying on default #address-cells value
>>> arch/arm/boot/dts/bcm953012hr.dts:60.19-63.4: Warning (avoid_default_addr_size): /nand-controller@18028000/nand@0/partition@200000: Relying on default #size-cells value
>>> arch/arm/boot/dts/bcm953012hr.dts:64.19-67.4: Warning (avoid_default_addr_size): /nand-controller@18028000/nand@0/partition@600000: Relying on default #address-cells value
>>> arch/arm/boot/dts/bcm953012hr.dts:64.19-67.4: Warning (avoid_default_addr_size): /nand-controller@18028000/nand@0/partition@600000: Relying on default #size-cells value
>>> arch/arm/boot/dts/bcm953012hr.dts:68.20-71.4: Warning (avoid_default_addr_size): /nand-controller@18028000/nand@0/partition@1000000: Relying on default #address-cells value
>>> arch/arm/boot/dts/bcm953012hr.dts:68.20-71.4: Warning (avoid_default_addr_size): /nand-controller@18028000/nand@0/partition@1000000: Relying on default #size-cells value
>>>
>>> I don't know what caused this now.
>>
>> I am still getting these warnings.
> 
> Thank you for the reminder, I have now reverted commit 90103611d573 ("ARM: dts:
> BCM5301X: Fix DTC warning for NAND node"), which caused the problem.
> 
> The patch was meant to fix a different warning
> 
> Warning (avoid_unnecessary_addr_size):
> /nand-controller@18028000/nand@0: unnecessary
> #address-cells/#size-cells without "ranges" or child "reg" property
> 
> but I don't see that here, so I don't know if it was actually required.
> 
> Arınç, Rafał, Florian: if the original problem comes back now, please
> send a follow-up patch
> that addresses all the warnings.

I was using an unusual method to compile DTs which produced the warning:

$ cpp -nostdinc -I include -I arch -undef -x assembler-with-cpp 
arch/arm/boot/dts/bcm47094-asus-rt-ac88u.dts processed.dts && 
scripts/dtc/dtc -I dts -O dtb processed.dts -o processed.dtb
arch/arm/boot/dts/bcm5301x-nand-cs0.dtsi:10.18-19.5: Warning 
(avoid_unnecessary_addr_size): /nand-controller@18028000/nand@0: 
unnecessary #address-cells/#size-cells without "ranges" or child "reg" 
property
   also defined at arch/arm/boot/dts/bcm5301x-nand-cs0-bch8.dtsi:13.9-17.3
   also defined at arch/arm/boot/dts/bcm47094-asus-rt-ac88u.dts:249.9-272.3
arch/arm/boot/dts/bcm5301x.dtsi:22.28-45.4: Warning 
(unique_unit_address): /chipcommon-a-bus@18000000: duplicate 
unit-address (also used in node /axi@18000000)
arch/arm/boot/dts/bcm5301x.dtsi:358.22-363.4: Warning 
(unique_unit_address): /mdio@18003000: duplicate unit-address (also used 
in node /mdio-mux@18003000)

Now that I know how to properly compile the dt files, I don't see any 
issues before the fix DTC commit or after the revert:

$ ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- make clean dtbs -j$(nproc)
   SYNC    include/config/auto.conf.cmd
   HOSTCC  scripts/basic/fixdep
   HOSTCC  scripts/kconfig/conf.o
   HOSTCC  scripts/kconfig/confdata.o
   HOSTCC  scripts/kconfig/expr.o
   LEX     scripts/kconfig/lexer.lex.c
   YACC    scripts/kconfig/parser.tab.[ch]
   HOSTCC  scripts/kconfig/menu.o
   HOSTCC  scripts/kconfig/preprocess.o
   HOSTCC  scripts/kconfig/symbol.o
   HOSTCC  scripts/kconfig/util.o
   HOSTCC  scripts/kconfig/lexer.lex.o
   HOSTCC  scripts/kconfig/parser.tab.o
   HOSTLD  scripts/kconfig/conf
   HOSTCC  scripts/dtc/dtc.o
   HOSTCC  scripts/dtc/flattree.o
   HOSTCC  scripts/dtc/fstree.o
   HOSTCC  scripts/dtc/data.o
   HOSTCC  scripts/dtc/livetree.o
   HOSTCC  scripts/dtc/treesource.o
   HOSTCC  scripts/dtc/srcpos.o
   HOSTCC  scripts/dtc/checks.o
   HOSTCC  scripts/dtc/util.o
   LEX     scripts/dtc/dtc-lexer.lex.c
   YACC    scripts/dtc/dtc-parser.tab.[ch]
   HOSTCC  scripts/dtc/libfdt/fdt.o
   HOSTCC  scripts/dtc/libfdt/fdt_ro.o
   HOSTCC  scripts/dtc/libfdt/fdt_wip.o
   HOSTCC  scripts/dtc/libfdt/fdt_sw.o
   HOSTCC  scripts/dtc/libfdt/fdt_rw.o
   HOSTCC  scripts/dtc/libfdt/fdt_strerror.o
   HOSTCC  scripts/dtc/libfdt/fdt_empty_tree.o
   HOSTCC  scripts/dtc/libfdt/fdt_addresses.o
   HOSTCC  scripts/dtc/libfdt/fdt_overlay.o
   HOSTCC  scripts/dtc/fdtoverlay.o
   HOSTCC  scripts/dtc/dtc-lexer.lex.o
   HOSTCC  scripts/dtc/dtc-parser.tab.o
   UPD     include/config/kernel.release
   HOSTLD  scripts/dtc/fdtoverlay
   HOSTLD  scripts/dtc/dtc
   DTC     arch/arm/boot/dts/bcm4708-asus-rt-ac56u.dtb
   DTC     arch/arm/boot/dts/bcm4708-asus-rt-ac68u.dtb
   DTC     arch/arm/boot/dts/bcm4708-buffalo-wzr-1750dhp.dtb
   DTC     arch/arm/boot/dts/bcm4708-linksys-ea6300-v1.dtb
   DTC     arch/arm/boot/dts/bcm4708-linksys-ea6500-v2.dtb
   DTC     arch/arm/boot/dts/bcm4708-luxul-xap-1510.dtb
   DTC     arch/arm/boot/dts/bcm4708-luxul-xwc-1000.dtb
   DTC     arch/arm/boot/dts/bcm4708-netgear-r6250.dtb
   DTC     arch/arm/boot/dts/bcm4708-netgear-r6300-v2.dtb
   DTC     arch/arm/boot/dts/bcm4708-smartrg-sr400ac.dtb
   DTC     arch/arm/boot/dts/bcm47081-asus-rt-n18u.dtb
   DTC     arch/arm/boot/dts/bcm47081-buffalo-wzr-600dhp2.dtb
   DTC     arch/arm/boot/dts/bcm47081-buffalo-wzr-900dhp.dtb
   DTC     arch/arm/boot/dts/bcm47081-luxul-xap-1410.dtb
   DTC     arch/arm/boot/dts/bcm47081-luxul-xwr-1200.dtb
   DTC     arch/arm/boot/dts/bcm47081-tplink-archer-c5-v2.dtb
   DTC     arch/arm/boot/dts/bcm4709-asus-rt-ac87u.dtb
   DTC     arch/arm/boot/dts/bcm4709-buffalo-wxr-1900dhp.dtb
   DTC     arch/arm/boot/dts/bcm4709-linksys-ea9200.dtb
   DTC     arch/arm/boot/dts/bcm4709-netgear-r7000.dtb
   DTC     arch/arm/boot/dts/bcm4709-netgear-r8000.dtb
   DTC     arch/arm/boot/dts/bcm4709-tplink-archer-c9-v1.dtb
   DTC     arch/arm/boot/dts/bcm47094-asus-rt-ac88u.dtb
   DTC     arch/arm/boot/dts/bcm47094-dlink-dir-885l.dtb
   DTC     arch/arm/boot/dts/bcm47094-linksys-panamera.dtb
   DTC     arch/arm/boot/dts/bcm47094-luxul-abr-4500.dtb
   DTC     arch/arm/boot/dts/bcm47094-luxul-xap-1610.dtb
   DTC     arch/arm/boot/dts/bcm47094-luxul-xbr-4500.dtb
   DTC     arch/arm/boot/dts/bcm47094-luxul-xwc-2000.dtb
   DTC     arch/arm/boot/dts/bcm47094-luxul-xwr-3100.dtb
   DTC     arch/arm/boot/dts/bcm47094-luxul-xwr-3150-v1.dtb
   DTC     arch/arm/boot/dts/bcm47094-netgear-r8500.dtb
   DTC     arch/arm/boot/dts/bcm47094-phicomm-k3.dtb
   DTC     arch/arm/boot/dts/bcm53016-meraki-mr32.dtb
   DTC     arch/arm/boot/dts/bcm94708.dtb
   DTC     arch/arm/boot/dts/bcm94709.dtb
   DTC     arch/arm/boot/dts/bcm953012er.dtb
   DTC     arch/arm/boot/dts/bcm953012hr.dtb
   DTC     arch/arm/boot/dts/bcm953012k.dtb

Sorry for the noise, I'll be more careful in the future.

Arınç

  reply	other threads:[~2022-05-13 10:23 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-05 22:58 linux-next: build warnings after merge of the arm-soc tree Stephen Rothwell
2022-05-13  7:29 ` Stephen Rothwell
2022-05-13  9:18   ` Arnd Bergmann
2022-05-13 10:22     ` Arınç ÜNAL [this message]
  -- strict thread matches above, loose matches on Subject: below --
2020-07-22 23:27 Stephen Rothwell
2020-08-09 22:21 ` Stephen Rothwell

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=8e5d2f7f-0b48-6317-d1d9-dee6a7c06d3c@arinc9.com \
    --to=arinc.unal@arinc9.com \
    --cc=arnd@arndb.de \
    --cc=bcm-kernel-feedback-list@broadcom.com \
    --cc=f.fainelli@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-next@vger.kernel.org \
    --cc=olof@lixom.net \
    --cc=rafal@milecki.pl \
    --cc=sfr@canb.auug.org.au \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
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).