Scott, > If readw() is returning the bytes in the correct endianness, then that > means the register is little-endian. > > It's not clear to me what the default assumption in nand_base.c is, > though. read_byte16() suggests the default is native endian, but > read_buf() and read_word() suggest it's little endian. :-(
I am not sure either; the nand_read_byte16() function confuses me quite a bit. I really have no idea whether what I have done is wrong and an evil hack or if I found a typo or something. > Is there any currently working host-big-endian platform with 16-bit NAND > that doesn't override these functions? I really can't say. I haven't looked through the source of any of the other boards. As Stefan said, all previous 4xx based boards use 8 bit NAND so the possibility that there might be a problem with the 16 bit code on 4xx based boards doesn't surprise me. > BTW, as for read_word(), looking at its only user, I think nand_block_bad() > should be checking a 16-bit bad block marker on 16-bit NAND, rather than the > low byte. Why is there a separate mechanism for checking bad block markers > than the one in nand_bbt.c (not the bbt itself, but the code used to read > the markers to create the bbt)? As long as a BBT is used, I don't think > read_word() will ever matter. I think it may have something to do with the NAND SPL loader. Since the NDFC code is being used by the SPL, it has to be small. Pulling in the all of the nand_bbt.c code for just the code that checks if the block is actually bad may (probably) have been to much for the SPL. Thats my guess at least. If the 16 bit chips used 0xffff as a bad block marker then yeah, I suppose the endianness wouldn't matter. Is that something that we can do? Just choose to use 0xffff? Will that not potentially mess up JFFS support or what not from the Linux kernel that gets booted? Regards, Alex -- Alex Waterman Computer Engineer Phone: 215-896-4920 Email: awater...@dawning.com _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
