Following Jean-Christophe's recommendation the patch has been splitted in two, supressed extra spaces and substituted a ==NULL for a !.
Regarding to the questions: I'm not checking the return value of fdt_initrd because in case of error 1) fdt_initrd will print a message to inform the user. 2) it's not a blocking error (from the bootloader point-of-view), so we can still try to boot. fdt_setprop gets a buffer of data that must be in BigEndian order to be standard compliant. It's defined to be BigEndian probably because it comes from PowerPC. I've also tested it before submitting and it works fine. Thank you for your comments. Regards, Vicente. Vicente Bergas (2): DeviceTree: add support for initrd in the DT ArchosG9: boot linux kernel with DT arch/arm/boards/archosg9/env/config | 1 + arch/arm/configs/archosg9_defconfig | 4 ++ arch/arm/configs/archosg9_xload_defconfig | 1 + arch/arm/lib/armlinux.c | 2 + common/oftree.c | 66 +++++++++++++++++++++++++++++++ include/of.h | 1 + 6 files changed, 75 insertions(+) -- 1.8.1.3 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
