
Dear Michal Simek,
In message 1292933384-3032-3-git-send-email-monstr@monstr.eu you wrote:
Patch "Replace CONFIG_SYS_GBL_DATA_SIZE by auto-generated value" (sha1: 25ddd1fb0a2281b182529afbc8fda5de2dc16d96) introduce GENERATED_GBL_DATA_SIZE which is sizeof aligned gd_t (currently 0x40). Microblaze configs used 0x40(128) because this place also contained board info structure which lies on the top of ram.
This patch is fixing this scheme by extending CONFIG_SYS_GBL_DATA_OFFSET value. Doubled GENERATED_GBL_DATA_SIZE size is enough to store gd_t and bd_t structures.
No, I will not accept this.
Please do not make assumptions abouth the size of bd_t compared to GENERATED_GBL_DATA_SIZE. Use the correct size instead, as you can easily get using the sizeof() operator (plus some padding eventually to guarantee alignment, if needed).
Best regards,
Wolfgang Denk