Will do. I've been quite busy lately bringing up a 32-core chip and getting ready for a release but I should have some time now.
I've got some other patches I also need to submit. -Aaron On 04/30/2011 12:12 PM, Wolfgang Denk wrote: > Dear Aaron Williams, > > In message > <1302594796-26276-1-git-send-email-aaron.willi...@caviumnetworks.com> you > wrote: >> This patch corrects the addresses used when working with Spansion/AMD FLASH >> chips. Addressing for 8 and 16 bits is almost identical except in the 16 >> bit case the LSB of the address is always 0. The confusion arose because >> the addresses in the datasheet for the 16 bit mode are word addresses but >> the CFI driver assumed they were byte addresses. >> >> I have only been able to test this on our Octeon boards which use either >> an 8 or 16 bit bus. I have not tested 32-bits or when there are multiple >> parts in parallel on a wider bus. >> >> Several other people have also reported success with this patch. >> >> -Aaron Williams >> >> >> Signed-off-by: Aaron Williams <aaron.willi...@caviumnetworks.com> >> --- >> drivers/mtd/cfi_flash.c | 66 >> +++++++++++++++++++++++++++++++--------------- >> include/mtd/cfi_flash.h | 43 ++++++++++++++++-------------- >> 2 files changed, 67 insertions(+), 42 deletions(-) > This patch has a number of coding style isses (trailing whitespace, > space between function name and open parenthesis, line over 80 > characters, etc.) > > Please clean up and resubmit (after verifying the patch using > checkpatch). > > Best regards, > > Wolfgang Denk > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
