Dear Jean-Christophe PLAGNIOL-VILLARD, In message <20090502194009.gk25...@game.jcrosoft.org> you wrote: > > > Please just remove this. I mean, without attempting to clean it up. > > > > [And not only on ARM, but everywhere.] > Actually For ARM we ask us tp use it so to simplify the import from linux > this "helper" are also imported
I don't know who "we" is, but you don;t have my ACK for this, and as long as it's not even correct youwill not get it. > > > #define SZ_1K 0x00000400 > > > #define SZ_4K 0x00001000 > > > #define SZ_8K 0x00002000 > > > > Please keep in mind that "K", "M", and "G" are non-standard prefixes, > > and it is not clear if they refer to decimal or binary multiples. > > This makes their use in the code a PITA. > I do not choose it We can simply chose NOT to use this. > > Prefixes and symbols for binary multiples are standardized in IEC > > 60027-2. [They are not part of the SI metric system of units, > > though.] Correct names for 1024 = (1 << 10), 1024*1024 = ( 1 << 20) > > and 1024*1024*1024 = (1 << 30) are "kibi" (kilobinary, abbreviated as > > "Ki"), "mebi" (megabinary, "Mi"), and "gibi" (gigabinary, "Gi"), > > respective. Note that this is "Ki", not "K". > I known but it's the linux implementation choice not mine There is zero need to copy bugs or incorrect code from Linux. Instead, someone should send patches to fix this in Linux, too (or better to kill it alltogether). > > So if you think youmust define something that means 8192, then please > > name it "8 Ki", as "Ki" is the correct, standardized symbol for the > > name "kibi", meaning "kilobinary". > Honnestly I think I'll let you try to deal it with Russel Russel? In case you are referring to RMK you better let not see him that you misspell his name. He is pretty allergic on this. Anyway: RMK has zero votes on U-Boot code. > > With "8 K", nobody ever knows for sure what is meant - is it 8*10**3 > > or is it 8*2**10. > > > > Thus my urgent recommendation: let's get rid of all this ambiguous > > SZ_* definitions and use numbers. "8 * 1024" is at least as readable > > as "SZ_8K", and it leaves no doubt what might be meant. > To simplify linux import I'll not prefer it But I insist. My preference is to get rid of this, but at least fix the bugs before we use it. 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: w...@denx.de I have the simplest tastes. I am always satisfied with the best. -- Oscar Wilde _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot