[U-Boot] [PATCH v3] crc32: Impl. linux optimized crc32()

Mike Frysinger vapier at gentoo.org
Thu Nov 19 19:48:50 CET 2009


On Thursday 19 November 2009 13:22:43 Thiago A. Corrêa wrote:
> 2009/11/19 Joakim Tjernlund wrote:
> > The whole point of the tole define is to make the table more
> > readable. Try and replace tole in the table and then
> > look at it.
> 
> Ah, ok. Then I think this one at make_crc_table should use the
> __cpu_to_le32() as tole only porpuse is a more readable table, then
> you could lose the first tole define as __cpu_to_le32 and keep only
> the one defined as __constant_cpu_to_le32. Also, I don't know how you
> are not getting a warning from the preprocessor about the redefined
> tole macro:

except then you revert the fix i posted and restore the bug i referred to 
earlier about cross-compiling.

since the define needs to exist to keep the table understandable, keeping the 
two table code paths similar makes sense too.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091119/a99d2192/attachment.pgp 


More information about the U-Boot mailing list