mvebu_armada-37xx: Wrong usage of of_machine_is_compatible in u-boot master
Pali Rohár
pali at kernel.org
Tue Sep 29 11:52:16 CEST 2020
Hello!
In current U-Boot master branch is board_late_init() function for
mvebu_armada-37xx/board.c slightly broken.
There are two of_machine_is_compatible() checks for non-existent
compatible string "globalscale,espressobin"
https://gitlab.denx.de/u-boot/u-boot/-/blob/master/board/Marvell/mvebu_armada-37xx/board.c#L91
On all other places is correct compatible "marvell,armada-3720-db" or
"marvell,armada-3720-espressobin":
https://gitlab.denx.de/u-boot/u-boot/-/blob/master/board/Marvell/mvebu_armada-37xx/board.c#L121
https://gitlab.denx.de/u-boot/u-boot/-/blob/master/board/Marvell/mvebu_armada-37xx/board.c#L159
https://gitlab.denx.de/u-boot/u-boot/-/blob/master/board/Marvell/mvebu_armada-37xx/board.c#L303
In DTS files in U-Boot master branch is really "marvell,armada-3720-db"
and "marvell,armada-3720-espressobin":
https://gitlab.denx.de/u-boot/u-boot/-/blob/master/arch/arm/dts/armada-3720-db.dts#L53
https://gitlab.denx.de/u-boot/u-boot/-/blob/master/arch/arm/dts/armada-3720-espressobin.dts#L54
So "globalscale,espressobin" would not work.
I think that "globalscale,espressobin" in board.c#L91 should be fixed,
changed to "marvell,armada-3720-espressobin" to not have broken
mvebu_armada-37xx code in 2020.10 U-Boot release.
Stefan, can you do this fix for 2020.10 release, so
mvebu_armada-37xx/board.c file would not be broken?
More information about the U-Boot
mailing list