In message 1206719159-11200-1-git-send-email-Joakim.Tjernlund@transmode.se you wrote:
Global variables are not ideal before relocation to RAM.
Byt they don't cause any real problem either, or am I missing something?
I tend to reject the patch.
-init_fnc_t *init_sequence[] = {
The original idea of having such a list of funtion pointers which just get executed one after another was to be able to wrap this into some "#ifndef CONFIG_INIT_SEQUENCE" and use this to allow for board- specific init sequences by just adding a #define with the needed list of functions to the board config files.
Even though this feature never was used so far, I still hesitate to throw it away - at least as long as I don't see benefits for the new solution.
Best regards,
Wolfgang Denk