Hi Andreas,

On Thu, Dec 15, 2011 at 2:02 AM, Andreas Bießmann andreas.devel@googlemail.com wrote:

Hi all,

I encountered a compile warning in avr32 which lead me to the fact that every board.c utilizes these three lines:

---8<--- #if defined(CONFIG_CMD_NET)        s = getenv("bootfile");        if (s)                copy_filename(BootFile, s, sizeof(BootFile)); #endif --->8---

I know there is currently some effort to move the common stuff out of the respective board.c files and this part would be one of these. But I wonder if this specific part shouldn't move into the network framework. The BootFile is declared in net/ and is only required if some ethernet device is attached. The ethernet stuff will be initialized by eth_initialize(), this could be one place for setting BootFile initial but maybe there are better places.

Any comments?

Well arguably we could create a net function to do this and call it from each board. However I hope to reduce the number of board files to one soon so this might not be worthwhile (although of course I may fail in which case it might be :-)

I hope to complete an RFC series for ARM and x86 in the next few days.

Regards, Simon

best regards

Andreas Bießmann