[U-Boot] [PATCH] omap3: overo: Select fdtfile for expansion board

[Ash Charles]

On Sun, Jun 8, 2014 at 12:57 AM, Stefan Herbrechtsmeier
<stefan at herbrechtsmeier.net> wrote:
> At the moment I use this functionality to set the environment variable
> 'fdtfile'.
Cool :)
....
> Maybe you could set an environment variable 'boardname' to 'overo' or
> 'overo-storm' and use something like the following environment command:
>
> if test -n \"${boardname}\" && test -n \"${expansionname}\"; then
>     setenv fdtfile omap3-${boardname}-${expansionname}.dtb;
> fi;
>
> This allows the expansion board to set the expansionname via eeprom and
> works with both overo boards.
What is the value in testing for ${boardname} and ${expansionname}?
Shouldn't I just test for fdtfile and, if it is not set, proceed to
set it you propose? ${boardname} and ${expansionname} could be set on
boot. I'm sending an updated version of the patch for your comments
(sorry..compile-tested only as I'm not near hardware today).

--Ash