linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Andrew Lunn <andrew@lunn.ch>, Jason Cooper <jason@lakedaemon.net>,
	arm-soc <arm@kernel.org>, Olof Johansson <olof@lixom.net>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Subject: Re: [GIT PULL] ARM: mvebu: dt for v5.1 (#1)
Date: Tue, 19 Feb 2019 17:21:02 +0100	[thread overview]
Message-ID: <87r2c321ht.fsf@FE-laptop> (raw)
In-Reply-To: <CAK8P3a1+anoAJvXmU8xCVnyuEAVC-Hs055ybZjmUmfJRdd0x=w@mail.gmail.com>

Hi Arnd,
 
 On ven., févr. 15 2019, Arnd Bergmann <arnd@arndb.de> wrote:

> On Fri, Feb 8, 2019 at 10:22 PM Gregory CLEMENT
> <gregory.clement@bootlin.com> wrote:
>
>>
>> ----------------------------------------------------------------
>> Andrew Lunn (1):
>>       arch: arm: dts: Remove disabled marvell,dsa properties
>>
>>  arch/arm/boot/dts/armada-370-rd.dts            | 42 -------------------
>>  arch/arm/boot/dts/armada-388-clearfog.dts      | 58 --------------------------
>>  arch/arm/boot/dts/armada-xp-linksys-mamba.dts  | 47 ---------------------
>>  arch/arm/boot/dts/kirkwood-dir665.dts          | 47 ---------------------
>>  arch/arm/boot/dts/kirkwood-linksys-viper.dts   | 47 ---------------------
>>  arch/arm/boot/dts/kirkwood-mv88f6281gtw-ge.dts | 47 ---------------------
>>  arch/arm/boot/dts/kirkwood-rd88f6281.dtsi      | 41 ------------------
>>  7 files changed, 329 deletions(-)
>
> Could you have a look at these warnings and send a follow-up pull request?
>
> ../arch/arm/boot/dts/kirkwood-rd88f6281-z0.dts:21.4-16: Warning
> (reg_format): /dsa/switch@0:reg: property has invalid length (8 bytes)
> (#address-cells == 2, #size-cells == 1)
> ../arch/arm/boot/dts/kirkwood-rd88f6281-z0.dts:23.5-15: Warning
> (reg_format): /dsa/switch@0/port@4:reg: property has invalid length (4
> bytes) (#address-cells == 2, #size-cells == 1)
> arch/arm/boot/dts/kirkwood-rd88f6281-z0.dtb: Warning
> (pci_device_bus_num): Failed prerequisite 'reg_format'
> arch/arm/boot/dts/kirkwood-rd88f6281-z0.dtb: Warning (i2c_bus_reg):
> Failed prerequisite 'reg_format'
> arch/arm/boot/dts/kirkwood-rd88f6281-z0.dtb: Warning (spi_bus_reg):
> Failed prerequisite 'reg_format'
> ../arch/arm/boot/dts/kirkwood-rd88f6281-z0.dts:20.12-26.5: Warning
> (avoid_default_addr_size): /dsa/switch@0: Relying on default
> #address-cells value
> ../arch/arm/boot/dts/kirkwood-rd88f6281-z0.dts:20.12-26.5: Warning
> (avoid_default_addr_size): /dsa/switch@0: Relying on default
> #size-cells value
> ../arch/arm/boot/dts/kirkwood-rd88f6281-z0.dts:22.11-25.6: Warning
> (avoid_default_addr_size): /dsa/switch@0/port@4: Relying on default
> #address-cells value
> ../arch/arm/boot/dts/kirkwood-rd88f6281-z0.dts:22.11-25.6: Warning
> (avoid_default_addr_size): /dsa/switch@0/port@4: Relying on default
> #size-cells value
>

Yes sure. I realize that I test the build with the wrong defconfig
(mvebu_v7_defconfig instead of mvebu_v5_defconfig) which made me miss
these warnings. I am going to send a patch to fix it and then I will do a
follow-up pull request.

Gregory


> Thanks,
>
>       Arnd

-- 
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

      reply	other threads:[~2019-02-19 16:21 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-08 21:22 [GIT PULL] ARM: mvebu: dt for v5.1 (#1) Gregory CLEMENT
2019-02-15 15:01 ` Arnd Bergmann
2019-02-15 15:44 ` Arnd Bergmann
2019-02-19 16:21   ` Gregory CLEMENT [this message]

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=87r2c321ht.fsf@FE-laptop \
    --to=gregory.clement@bootlin.com \
    --cc=andrew@lunn.ch \
    --cc=arm@kernel.org \
    --cc=arnd@arndb.de \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=olof@lixom.net \
    --cc=sebastian.hesselbarth@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: 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).