| Message ID | 20240405064624.18997-1-krzysztof.kozlowski@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | [v2,1/5] ARM: dts: aspeed: greatlakes: correct Mellanox multi-host property | expand |
On Fri, 2024-04-05 at 08:46 +0200, Krzysztof Kozlowski wrote: > "mlx,multi-host" is using incorrect vendor prefix and is not documented. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Thanks, I've applied these to my tree to be picked up by Joel. Andrew
On Fri, 05 Apr 2024 08:46:20 +0200, Krzysztof Kozlowski wrote: > "mlx,multi-host" is using incorrect vendor prefix and is not documented. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > Changes in v2: > 1. None > --- > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade New warnings running 'make CHECK_DTBS=y aspeed/aspeed-bmc-ampere-mtmitchell.dtb aspeed/aspeed-bmc-facebook-greatlakes.dtb aspeed/aspeed-bmc-facebook-harma.dtb aspeed/aspeed-bmc-facebook-yosemite4.dtb aspeed/aspeed-bmc-facebook-yosemitev2.dtb' for 20240405064624.18997-1-krzysztof.kozlowski@linaro.org: arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dtb: ftgmac@1e690000: Unevaluated properties are not allowed ('#address-cells', '#size-cells', 'mellanox,multi-host' were unexpected) from schema $id: http://devicetree.org/schemas/net/faraday,ftgmac100.yaml#
On 09/04/2024 15:38, Rob Herring wrote: > > On Fri, 05 Apr 2024 08:46:20 +0200, Krzysztof Kozlowski wrote: >> "mlx,multi-host" is using incorrect vendor prefix and is not documented. >> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> >> --- >> >> Changes in v2: >> 1. None >> --- >> arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> > > > My bot found new DTB warnings on the .dts files added or changed in this > series. > > Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings > are fixed by another series. Ultimately, it is up to the platform > maintainer whether these warnings are acceptable or not. No need to reply > unless the platform maintainer has comments. > > If you already ran DT checks and didn't see these error(s), then > make sure dt-schema is up to date: > > pip3 install dtschema --upgrade > > > New warnings running 'make CHECK_DTBS=y aspeed/aspeed-bmc-ampere-mtmitchell.dtb aspeed/aspeed-bmc-facebook-greatlakes.dtb aspeed/aspeed-bmc-facebook-harma.dtb aspeed/aspeed-bmc-facebook-yosemite4.dtb aspeed/aspeed-bmc-facebook-yosemitev2.dtb' for 20240405064624.18997-1-krzysztof.kozlowski@linaro.org: > > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dtb: ftgmac@1e690000: Unevaluated properties are not allowed ('#address-cells', '#size-cells', 'mellanox,multi-host' were unexpected) > from schema $id: http://devicetree.org/schemas/net/faraday,ftgmac100.yaml# > Just FYI These warnings are expected and not a result of my patches. Best regards, Krzysztof
diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts index 7a53f54833a0..9a6757dd203f 100644 --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts @@ -66,7 +66,7 @@ &mac3 { pinctrl-0 = <&pinctrl_rmii4_default>; no-hw-checksum; use-ncsi; - mlx,multi-host; + mellanox,multi-host; ncsi-ctrl,start-redo-probe; ncsi-ctrl,no-channel-monitor; ncsi-package = <1>;
"mlx,multi-host" is using incorrect vendor prefix and is not documented. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Changes in v2: 1. None --- arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)