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.
Thanks, Michal
Best regards,
Wolfgang Denk