[U-Boot-Users] Big endian-ness - Config option?

[David Snowdon]

G'Day,

Feel free to harsh me if I've got this wrong. (I'm writing this so I 
don't get harshed when I submit my patch).

I've just borrowed the code from the sixnet board for talking to 
28F320C3 flash chips (advanced boot sector, yada, yada) which was 
written by Wolfgang at some point. Its dropped in very nicely and works 
very well (once I'd worked out what i was doing).

I've had two problems. The first has been that that the code uses the 
get_timer(x) function, which is not implemented properly on ARM. (It 
just returns the value of the OS counter, ignoring the value passed to 
it. I had to replace the get_timer stuff with get_timer_masked and 
reset_timer_masked. I could try fixing the arm implementation of 
get_timer, but figured you guys wouldn't want me touching that bit of 
code since other bits and pieces now depend on it. Is there some way of 
doing this nicely?

The second was that one part of the code munges some bytes into a word 
to get them ready for writing. This is broken with respect to 
endian-ness, and breaks on my XScale running in little-endian mode. Is 
it possible (or appropriate?) to add a CFG_LITTLE_ENDIAN (or something 
similar)? Is there a better way of switching between the two? It would 
be nice if this code could be re-used without these sorts of problems.

Cheers,

Dave.

-
David Snowdon
PhD Student
National ICT Australia (NICTA)
Phone: 9385 7355
Email: daves at cse.unsw.edu.au