[U-Boot] [PATCH] OMAP3EVM: net_chip uses CS5 not CS6

[Scott Wood <scottwood@freescale.com>]

Wolfgang Denk wrote:
> Dear Scott Wood,
> 
> In message <4A034B09.7030105@freescale.com> you wrote:
>>> Or what replaced the "immr" structs?
>> The device tree, mainly...
> 
> Right, of course.
> 
>>                      ...  But #defines can work for u-boot.
> 
> Of course they _can_ work. But they can easily fail (as we just see
> in this patch), and we don't have typechecking. So until DT's are
> omnipresent, let's use structs in U-Boot, please.

You *do* have typechecking as long as the individual blocks are 
described with structs.

We could take immap to extremes by defining one big 4GiB struct that 
shows where memory, immr, flash, desired PCI bars, FPGAs, etc. are -- 
but that would be silly.  IMHO, so is doing it at the immr level. :-)

How would you deal with blocks being at different locations in different 
chips?  It's a lot easier to ifdef (or have the config file specify) a 
couple addresses than to ifdef the locations of fields in a struct, 
especially when you have more than a couple variations.

-Scott