On 10 December 2013 14:47, Alexey Brodkin Alexey.Brodkin@synopsys.com wrote:
On Thu, 2013-11-28 at 13:55 +0400, Alexey Brodkin wrote:
On Wed, 2013-11-27 at 18:43 -0700, Simon Glass wrote:
I have a similar patch locally, but it actually does memset() on the whole structure. Some archs handle this setup differently. For example both ARM and x86 now allocate it in low level code so there is no need for the board_f code to allocate a global data structure.
Another approach would be to move "zero_global_data" from "init_sequence_f" to the very beginning of "board_init_f" right after global data allocation.
But IMHO it is a bit more significant modification so I decided to start from the simplest fix that resolves a particular issue I see.
Any chance for this patch to be applied or described problem has to be resolved/worked-around in any other way?
I say apply it. Tom would you like me to do a pull request, or do you want to pick it up?
Regards, Simon