Re: ixp4xx stuff

[David Vrabel <dvrabel@arcom.com>]

John Bowler wrote:
> From: David Vrabel
> 
>>>along with the other NSLU2 patches.  We found that it is now necessary to
>>>set CONFIG_MTD_LE_BYTE_SWAP - with the NSLU2 patches CONFIG_MTD_BE_BYTE_SWAP
>>>must be set.  If this is not done (i.e. if BE is set) the flash probe fails.
>>
>>Neither of these options should be enabled.  The flash data is returned
>>in the same order regardless of the endianness of the CPU.
> 
> 
> What is the difference between CONFIG_MTD_BE_BYTE_SWAP and ..._NO_SWAP on a
> BE machine?

They're both no-ops on BE CPUs.

To conclude: With CONFIG_MTD_NOSWAP set the current patches:

a) Work on BE systems as they have always done (tested by myself and
it's fairly obvious it's right since it doesn't do anything different).
b) Work on LE systems (as you have verified -- CONFIG_MTD_LE_BYTE_SWAP
is a no-op on LE CPUs).

So it looks like everything is good to go, yes?

David Vrabel
-- 
David Vrabel, Design Engineer

Arcom, Clifton Road           Tel: +44 (0)1223 411200 ext. 3233
Cambridge CB1 7EA, UK         Web: http://www.arcom.com/