Please ignore my last question. Lately, I find out in your .c file that there are still some places using __LITTLE_ENDIAN.
However, I still have a question that in which condition CFG_FLASH_CFI_SWAP is required to set?
Best Regards, Leo
-----Original Message----- From: Li Yang-r58472 Sent: Tuesday, August 08, 2006 7:11 PM To: 'Yuli Barcohen'; u-boot-users@lists.sourceforge.net Cc: Sam Song; Wolfgang Denk Subject: RE: [U-Boot-Users] [RFC] CFI Driver Little-Endian write Issue
+#if defined(__LITTLE_ENDIAN) && !defined(CFG_FLASH_CFI_SWAP) +#define CFG_FLASH_CFI_SWAP +#endif
[snip] -#if defined(__LITTLE_ENDIAN) +#if defined(CFG_FLASH_CFI_SWAP)
Why use CFG_FLASH_CFI_SWAP instead of __LITTLE_ENDIAN. It seems that
they are
equivalent.
Best Regards, Leo
-----Original Message----- From: u-boot-users-bounces@lists.sourceforge.net [mailto:u-boot-users-bounces@lists.sourceforge.net] On Behalf Of
Yuli Barcohen
Sent: Tuesday, August 08, 2006 7:00 PM To: u-boot-users@lists.sourceforge.net Cc: Sam Song; Wolfgang Denk Subject: Re: [U-Boot-Users] [RFC] CFI Driver Little-Endian write
Issue
> Sam Song writes:
Zang> Where is the patch? I do not object. Our final goal is
to
Zang> enable the general flash driver work on the board. Sam> Sorry, I misused the word patch:-). The "patch" is Yuli's Sam> cfi_flash.c in his attached file. See your mail archive. It Sam> would be there. I tested that it was OK but still need your Sam> confirmation, I am afraid.OK, the patch is attached. As we know, it works on big-endian
systems.
On little-endian ones, it should change nothing.
--
========================================================================
Yuli Barcohen | Phone +972-9-765-1788 | Software Project
Leader
yuli@arabellasw.com | Fax +972-9-765-7494 | Arabella Software,
Israel
========================================================================