All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: Vladimir Oltean <olteanv@gmail.com>
Cc: Linus Walleij <linus.walleij@linaro.org>,
	"David S. Miller" <davem@davemloft.net>,
	Jakub Kicinski <kuba@kernel.org>,
	Eric Dumazet <edumazet@google.com>,
	linux-arm-kernel@lists.infradead.org,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Andrew Lunn <andrew@lunn.ch>,
	Florian Fainelli <f.fainelli@gmail.com>,
	devicetree@vger.kernel.org,
	Christian Marangi <ansuelsmth@gmail.com>,
	netdev@vger.kernel.org, Paolo Abeni <pabeni@redhat.com>,
	Russell King <linux@armlinux.org.uk>,
	linux-kernel@vger.kernel.org,
	Gregory Clement <gregory.clement@bootlin.com>,
	Conor Dooley <conor+dt@kernel.org>
Subject: Re: [PATCH net-next v4 2/7] dt-bindings: net: mvusb: Fix up DSA example
Date: Thu, 19 Oct 2023 15:23:15 -0500	[thread overview]
Message-ID: <20231019202315.GA1031575-robh@kernel.org> (raw)
In-Reply-To: <20231019142646.iw6x72y6sqt4q2uw@skbuf>

On Thu, Oct 19, 2023 at 05:26:46PM +0300, Vladimir Oltean wrote:
> On Thu, Oct 19, 2023 at 08:45:14AM -0500, Rob Herring wrote:
> > On Wed, Oct 18, 2023 at 01:37:10PM +0200, Linus Walleij wrote:
> > > On Wed, Oct 18, 2023 at 12:32 PM Rob Herring <robh@kernel.org> wrote:
> > > 
> > > > dtschema/dtc warnings/errors:
> > > > Documentation/devicetree/bindings/net/marvell,mvusb.example.dtb: /example-0/usb/mdio@1/ethernet-switch@0: failed to match any schema with compatible: ['marvell,mv88e6190']
> > > 
> > > Isn't that just because the bindings now come last in the series.
> > > Which is in response to a review comment, hence this warning
> > > didn't appear before.
> > 
> > Yes. The only option that avoids this is squashing the 2 patches. I 
> > think it is fine to leave this as-is.
> > 
> > Rob
> 
> Anyway, I'm surprised that the bot would send this email, since the
> warning existed prior to this patch, and I would expect that the bot
> only notifies of newly introduced issues.

Oh right. Except that the node name changed in this patch and that 
changed the error message.

Rob

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

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Vladimir Oltean <olteanv@gmail.com>
Cc: Linus Walleij <linus.walleij@linaro.org>,
	"David S. Miller" <davem@davemloft.net>,
	Jakub Kicinski <kuba@kernel.org>,
	Eric Dumazet <edumazet@google.com>,
	linux-arm-kernel@lists.infradead.org,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Andrew Lunn <andrew@lunn.ch>,
	Florian Fainelli <f.fainelli@gmail.com>,
	devicetree@vger.kernel.org,
	Christian Marangi <ansuelsmth@gmail.com>,
	netdev@vger.kernel.org, Paolo Abeni <pabeni@redhat.com>,
	Russell King <linux@armlinux.org.uk>,
	linux-kernel@vger.kernel.org,
	Gregory Clement <gregory.clement@bootlin.com>,
	Conor Dooley <conor+dt@kernel.org>
Subject: Re: [PATCH net-next v4 2/7] dt-bindings: net: mvusb: Fix up DSA example
Date: Thu, 19 Oct 2023 15:23:15 -0500	[thread overview]
Message-ID: <20231019202315.GA1031575-robh@kernel.org> (raw)
In-Reply-To: <20231019142646.iw6x72y6sqt4q2uw@skbuf>

On Thu, Oct 19, 2023 at 05:26:46PM +0300, Vladimir Oltean wrote:
> On Thu, Oct 19, 2023 at 08:45:14AM -0500, Rob Herring wrote:
> > On Wed, Oct 18, 2023 at 01:37:10PM +0200, Linus Walleij wrote:
> > > On Wed, Oct 18, 2023 at 12:32 PM Rob Herring <robh@kernel.org> wrote:
> > > 
> > > > dtschema/dtc warnings/errors:
> > > > Documentation/devicetree/bindings/net/marvell,mvusb.example.dtb: /example-0/usb/mdio@1/ethernet-switch@0: failed to match any schema with compatible: ['marvell,mv88e6190']
> > > 
> > > Isn't that just because the bindings now come last in the series.
> > > Which is in response to a review comment, hence this warning
> > > didn't appear before.
> > 
> > Yes. The only option that avoids this is squashing the 2 patches. I 
> > think it is fine to leave this as-is.
> > 
> > Rob
> 
> Anyway, I'm surprised that the bot would send this email, since the
> warning existed prior to this patch, and I would expect that the bot
> only notifies of newly introduced issues.

Oh right. Except that the node name changed in this patch and that 
changed the error message.

Rob

  reply	other threads:[~2023-10-19 20:23 UTC|newest]

Thread overview: 110+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-18  9:03 [PATCH net-next v4 0/7] Create a binding for the Marvell MV88E6xxx DSA switches Linus Walleij
2023-10-18  9:03 ` Linus Walleij
2023-10-18  9:03 ` [PATCH net-next v4 1/7] dt-bindings: net: dsa: Require ports or ethernet-ports Linus Walleij
2023-10-18  9:03   ` Linus Walleij
2023-10-18 10:32   ` Rob Herring
2023-10-18 10:32     ` Rob Herring
2023-10-18 11:11     ` Linus Walleij
2023-10-18 11:11       ` Linus Walleij
2023-10-19 10:58       ` Russell King (Oracle)
2023-10-19 10:58         ` Russell King (Oracle)
2023-10-19 11:58         ` Conor Dooley
2023-10-19 11:58           ` Conor Dooley
2023-10-19 12:27           ` Russell King (Oracle)
2023-10-19 12:27             ` Russell King (Oracle)
2023-10-19 12:46             ` Conor Dooley
2023-10-19 12:46               ` Conor Dooley
2023-10-19 13:41               ` Rob Herring
2023-10-19 13:41                 ` Rob Herring
2023-10-20 11:10       ` Krzysztof Kozlowski
2023-10-20 11:10         ` Krzysztof Kozlowski
2023-10-20 11:41         ` Linus Walleij
2023-10-20 11:41           ` Linus Walleij
2023-10-20 12:16           ` Krzysztof Kozlowski
2023-10-20 12:16             ` Krzysztof Kozlowski
2023-10-20 12:27           ` Vladimir Oltean
2023-10-20 12:27             ` Vladimir Oltean
2023-10-20 12:56             ` Linus Walleij
2023-10-20 12:56               ` Linus Walleij
2023-10-19 12:03     ` Conor Dooley
2023-10-19 12:03       ` Conor Dooley
2023-10-19 12:28       ` Russell King (Oracle)
2023-10-19 12:28         ` Russell King (Oracle)
2023-10-19 12:33         ` Conor Dooley
2023-10-19 12:33           ` Conor Dooley
2023-10-18  9:03 ` [PATCH net-next v4 2/7] dt-bindings: net: mvusb: Fix up DSA example Linus Walleij
2023-10-18  9:03   ` Linus Walleij
2023-10-18 10:32   ` Rob Herring
2023-10-18 10:32     ` Rob Herring
2023-10-18 11:37     ` Linus Walleij
2023-10-18 11:37       ` Linus Walleij
2023-10-19 13:45       ` Rob Herring
2023-10-19 13:45         ` Rob Herring
2023-10-19 14:26         ` Vladimir Oltean
2023-10-19 14:26           ` Vladimir Oltean
2023-10-19 20:23           ` Rob Herring [this message]
2023-10-19 20:23             ` Rob Herring
2023-10-18 13:39   ` Andrew Lunn
2023-10-18 13:39     ` Andrew Lunn
2023-10-19 14:28   ` Vladimir Oltean
2023-10-19 14:28     ` Vladimir Oltean
2023-10-18  9:03 ` [PATCH net-next v4 3/7] ARM: dts: marvell: Fix some common switch mistakes Linus Walleij
2023-10-18  9:03   ` Linus Walleij
2023-10-18 13:41   ` Andrew Lunn
2023-10-18 13:41     ` Andrew Lunn
2023-10-19 11:04   ` Russell King (Oracle)
2023-10-19 11:04     ` Russell King (Oracle)
2023-10-19 14:29     ` Vladimir Oltean
2023-10-19 14:29       ` Vladimir Oltean
2023-10-18  9:03 ` [PATCH net-next v4 4/7] ARM: dts: nxp: " Linus Walleij
2023-10-18  9:03   ` Linus Walleij
2023-10-18 13:42   ` Andrew Lunn
2023-10-18 13:42     ` Andrew Lunn
2023-10-19 14:33   ` Vladimir Oltean
2023-10-19 14:33     ` Vladimir Oltean
2023-10-18  9:03 ` [PATCH net-next v4 5/7] ARM64: dts: marvell: " Linus Walleij
2023-10-18  9:03   ` Linus Walleij
2023-10-18 13:43   ` Andrew Lunn
2023-10-18 13:43     ` Andrew Lunn
2023-10-19 14:40   ` Vladimir Oltean
2023-10-19 14:40     ` Vladimir Oltean
2023-10-19 14:49     ` Vladimir Oltean
2023-10-19 14:49       ` Vladimir Oltean
2023-10-19 15:26       ` Marek Behún
2023-10-19 15:26         ` Marek Behún
2023-10-19 16:22         ` Vladimir Oltean
2023-10-19 16:22           ` Vladimir Oltean
2023-10-20 12:59           ` Linus Walleij
2023-10-20 12:59             ` Linus Walleij
2023-10-25  1:11             ` Vladimir Oltean
2023-10-25  1:11               ` Vladimir Oltean
2023-10-18  9:03 ` [PATCH net-next v4 6/7] dt-bindings: marvell: Rewrite MV88E6xxx in schema Linus Walleij
2023-10-18  9:03   ` Linus Walleij
2023-10-18 10:32   ` Rob Herring
2023-10-18 10:32     ` Rob Herring
2023-10-18 11:39     ` Linus Walleij
2023-10-18 11:39       ` Linus Walleij
2023-10-19 13:49       ` Rob Herring
2023-10-19 13:49         ` Rob Herring
2023-10-19 20:02         ` Rob Herring
2023-10-19 20:02           ` Rob Herring
2023-10-18 13:48   ` Andrew Lunn
2023-10-18 13:48     ` Andrew Lunn
2023-10-19 13:51   ` Rob Herring
2023-10-19 13:51     ` Rob Herring
2023-10-19 15:35   ` Vladimir Oltean
2023-10-19 15:35     ` Vladimir Oltean
2023-10-20  9:13     ` Arınç ÜNAL
2023-10-20  9:13       ` Arınç ÜNAL
2023-10-20 12:47     ` Linus Walleij
2023-10-20 12:47       ` Linus Walleij
2023-10-20 15:12       ` Vladimir Oltean
2023-10-20 15:12         ` Vladimir Oltean
2023-10-20 16:07       ` Andrew Lunn
2023-10-20 16:07         ` Andrew Lunn
2023-10-18  9:03 ` [PATCH net-next v4 7/7] dt-bindings: marvell: Add Marvell MV88E6060 DSA schema Linus Walleij
2023-10-18  9:03   ` Linus Walleij
2023-10-18 13:49   ` Andrew Lunn
2023-10-18 13:49     ` Andrew Lunn
2023-10-19 16:11   ` Vladimir Oltean
2023-10-19 16:11     ` Vladimir Oltean

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=20231019202315.GA1031575-robh@kernel.org \
    --to=robh@kernel.org \
    --cc=andrew@lunn.ch \
    --cc=ansuelsmth@gmail.com \
    --cc=conor+dt@kernel.org \
    --cc=davem@davemloft.net \
    --cc=devicetree@vger.kernel.org \
    --cc=edumazet@google.com \
    --cc=f.fainelli@gmail.com \
    --cc=gregory.clement@bootlin.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=kuba@kernel.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=netdev@vger.kernel.org \
    --cc=olteanv@gmail.com \
    --cc=pabeni@redhat.com \
    --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 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.