Hi Jean-Christophe, >> Hi Wolfgang, >> >>> Dear Michal, >>> >>>> >>>> -#if defined(CONFIG_PPC) || defined(CONFIG_M68K) || defined(CONFIG_SPARC) >>>> +#if defined(CONFIG_M68K) || defined(CONFIG_MICROBLAZE) || >>>> defined(CONFIG_PPC) \ >>>> + || defined(CONFIG_SPARC) >>>> /** >>>> * boot_ramdisk_high - relocate init ramdisk >>>> * @lmb: pointer to lmb handle, will be used for memory mgmt >>> I reject this patch. >>> >>> I know this is not your fault - you just add to existing code. But the >>> existing code is wrong, and we should not add even more bad code (and >>> rather fix the current, incorrect one). >>> >>> The thing is that the code we are #ifdef'ing here depends on a >>> *feature* (FDT boot support), and thus can be selected on a board >>> level. It is *not* an architecture thing, so it is incorrect to >>> enable or disable it depending on the current arch settings. >> I understand. >> When will be remove this part of code? >> Because without any fix all Microblaze boards can't be successfully compiled. >> > try my patch > > [PATCH V3] Fix FIT and fdt blob support to have CONFIG_OF_LIBFDT and > CONFIG_FIT independant
I'll disable FDT support in all MB boards. This is straight forward to me. Microblaze FDT support for bootm command is not there yet. I have it in my internal repo. Thanks, Michal > Best Regards, > J. > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
