[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <da5658565bc28f1d9d7cc49bf31dce7e77b7982d.camel@redhat.com>
Date: Thu, 23 May 2024 13:39:58 +0200
From: Paolo Abeni <pabeni@...hat.com>
To: Heiko Stübner <heiko@...ech.de>, Rob Herring
<robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, "David S. Miller" <davem@...emloft.net>, Eric
Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Alexandre
Torgue <alexandre.torgue@...s.st.com>, Giuseppe Cavallaro
<peppe.cavallaro@...com>, Jose Abreu <joabreu@...opsys.com>, Tobias
Schramm <t.schramm@...jaro.org>, Jonas Karlman <jonas@...boo.se>
Cc: devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-rockchip@...ts.infradead.org, linux-kernel@...r.kernel.org,
netdev@...r.kernel.org
Subject: Re: [PATCH 01/13] dt-bindings: net: rockchip-dwmac: Fix
rockchip,rk3308-gmac compatible
On Tue, 2024-05-21 at 23:34 +0200, Heiko Stübner wrote:
> Am Dienstag, 21. Mai 2024, 23:10:04 CEST schrieb Jonas Karlman:
> > Schema validation using rockchip,rk3308-gmac compatible fails with:
> >
> > ethernet@...e0000: compatible: ['rockchip,rk3308-gmac'] does not contain items matching the given schema
> > from schema $id: http://devicetree.org/schemas/net/rockchip-dwmac.yaml#
> > ethernet@...e0000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts', 'phy-mode',
> > 'reg', 'reset-names', 'resets', 'snps,reset-active-low', 'snps,reset-delays-us',
> > 'snps,reset-gpio' were unexpected)
> > from schema $id: http://devicetree.org/schemas/net/rockchip-dwmac.yaml#
> >
> > Add rockchip,rk3308-gmac to snps,dwmac.yaml to fix DT schema validation.
> >
> > Fixes: 2cc8c910f515 ("dt-bindings: net: rockchip-dwmac: add rk3308 gmac compatible")
> > Signed-off-by: Jonas Karlman <jonas@...boo.se>
>
> Reviewed-by: Heiko Stuebner <heiko@...ech.de>
>
> I'm not sure how the network tree works these days, but this patch is the
> only one that should go through the network tree.
It looks like the rest of the series has no dependency and will be
merged
via other trees.
If so, I think it would be cleaner if you could submit this patch
individually to the net tree, including the target tree 'net' into the
subj.
That will trigger successfully our CI - that currently does little for
DT changes but still more then nothing and things could improve in the
future.
Thanks,
Paolo
Powered by blists - more mailing lists