In message <[EMAIL PROTECTED]> you wrote:
> The envcrc.c does sizeof(unsigned long) when calculating the crc, but this
> is done with the build toolchain instead of the target toolchain, so if
> the build is a 64bit system but the target is 32bits, the size will
> obviously be wrong.  This introduces a sizeof.sh script to calculate the
> required sizeof() value of any type with any compiler.
> 
> Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]>
> ---
> Changes:
>  - adds copyright/license
>  - increase limit to 512
>  - increment by 4 bytes to find "32bit naturals" faster:
> (old/new # = number of times gcc is called)
> type  | sizeof        | old   | new
> char  | 1     | 1     | 5
> short | 2     | 2     | 4
> int   | 4     | 4     | 2
> long  | 8     | 8     | 3
> double        | 8     | 8     | 3

Sorry, but I still think this is serious overkill which just slows down
the build process.

We need exactly one single case of  this,  i.  e.  sizeof(long).  And
instead  of  tricking  around all this here, I think we should rather
use somthing like "u32" in the code.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
Save yourself!  Reboot in 5 seconds!

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
U-Boot-Users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to