On Thu, Jun 18, 2020 at 11:10:56AM +0200, Ahmad Fatoum wrote:
Prior to 8226f7f90973 ("of: base: parse all available memory nodes"),
barebox first looked at a /memory node, then at a node with a
device_type = "memory" property when probing available memory banks from
device tree. Linux instead only considered nodes with device_type =
"memory". The imx6ul-pico-hobbit predates the change in barebox behavior,
so it's unclear what is gained from dropping device_type = "memory" as
barebox used to parse the node by name anyway and Linux only saw the new
fixed up node.

Since then the Linux device tree has been split into a SoM and carrier
board device tree and the node name is now called memory@80000000 with
a size of 0 in the reg property. Because it's already used for boards
with differing memory size, we assume this will remain so and thus we
just drop the defunct barebox memory node override.

Cc: Michael Grzeschik <[email protected]>
Signed-off-by: Ahmad Fatoum <[email protected]>
---
arch/arm/dts/imx6ul-pico-hobbit.dts | 4 ----
1 file changed, 4 deletions(-)

diff --git a/arch/arm/dts/imx6ul-pico-hobbit.dts 
b/arch/arm/dts/imx6ul-pico-hobbit.dts
index 2f37b724b6e1..0c543de8c916 100644
--- a/arch/arm/dts/imx6ul-pico-hobbit.dts
+++ b/arch/arm/dts/imx6ul-pico-hobbit.dts
@@ -9,10 +9,6 @@
                        device-path = &environment_usdhc1;
                };
        };
-
-       memory {
-               /delete-property/ device_type;
-       };
};

&usdhc1 {
--
2.27.0

Ack

--
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

Attachment: signature.asc
Description: PGP signature

_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox

Reply via email to