The fixed-partitions compatible is meant to be used for the partitions container holding the individual partitions and not for every individual partition. This didn't lead to problems so far, because barebox ignored the compatible and parsed them as legacy partitions. Resolve the confusion by changing the description to use the new-style binding correctly. While at it, we also drop some leading zeroes for readability's sake. Cc: Michael Kopfensteiner <michael.kopfensteiner@xxxxxxxxx> Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> --- .../dts/imx8mp-var-dart-dt8mcustomboard.dts | 23 ++++++++++--------- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/arch/arm/dts/imx8mp-var-dart-dt8mcustomboard.dts b/arch/arm/dts/imx8mp-var-dart-dt8mcustomboard.dts index ab4c5790cf15..f11218c904f5 100644 --- a/arch/arm/dts/imx8mp-var-dart-dt8mcustomboard.dts +++ b/arch/arm/dts/imx8mp-var-dart-dt8mcustomboard.dts @@ -356,9 +356,6 @@ &usb_dwc3_1 { }; &usdhc2 { - #address-cells = <2>; - #size-cells = <1>; - pinctrl-names = "default", "state_100mhz", "state_200mhz"; pinctrl-0 = <&pinctrl_usdhc2>, <&pinctrl_usdhc2_gpio>; pinctrl-1 = <&pinctrl_usdhc2_100mhz>, <&pinctrl_usdhc2_gpio>; @@ -368,16 +365,20 @@ &usdhc2 { bus-width = <4>; status = "okay"; - partition@0 { + partitions { compatible = "fixed-partitions"; - label = "barebox"; - reg = <0x00000000 0x00000000 0x000e0000>; - }; + #size-cells = <2>; + #address-cells = <1>; - env_sd: partition@e0000 { - compatible = "fixed-partitions"; - label = "barebox-environment"; - reg = <0x00000000 0x000e0000 0x00020000>; + partition@0 { + label = "barebox"; + reg = <0x0 0x0 0xe0000>; + }; + + env_sd: partition@e0000 { + label = "barebox-environment"; + reg = <0x0 0xe0000 0x20000>; + }; }; }; -- 2.39.5