| Message ID | 20201110033113.31090-4-f.fainelli@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | Broadcom b53 YAML bindings | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On Mon, Nov 09, 2020 at 07:31:06PM -0800, Florian Fainelli wrote: > Update the switch unit name from srab to ethernet-switch, allowing us to > fix warnings such as: > > CHECK arch/arm/boot/dts/bcm4708-buffalo-wzr-1750dhp.dt.yaml > arch/arm/boot/dts/bcm4708-buffalo-wzr-1750dhp.dt.yaml: > srab@18007000: $nodename:0: 'srab@18007000' does not match > '^(ethernet-)?switch(@.*)?$' > From schema: > Documentation/devicetree/bindings/net/dsa/b53.yaml > > Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> > --- You're lucky that you don't have any firmware that fixes up DT properties by path. Reviewed-by: Vladimir Oltean <olteanv@gmail.com>
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi index ac3a99cf2079..ee23c0841699 100644 --- a/arch/arm/boot/dts/bcm5301x.dtsi +++ b/arch/arm/boot/dts/bcm5301x.dtsi @@ -482,7 +482,7 @@ thermal: thermal@1800c2c0 { #thermal-sensor-cells = <0>; }; - srab: srab@18007000 { + srab: ethernet-switch@18007000 { compatible = "brcm,bcm5301x-srab"; reg = <0x18007000 0x1000>;
Update the switch unit name from srab to ethernet-switch, allowing us to fix warnings such as: CHECK arch/arm/boot/dts/bcm4708-buffalo-wzr-1750dhp.dt.yaml arch/arm/boot/dts/bcm4708-buffalo-wzr-1750dhp.dt.yaml: srab@18007000: $nodename:0: 'srab@18007000' does not match '^(ethernet-)?switch(@.*)?$' From schema: Documentation/devicetree/bindings/net/dsa/b53.yaml Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> --- arch/arm/boot/dts/bcm5301x.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)