[U-Boot] [PATCH 16/20] Roll crc32 into hash infrastructure
Wolfgang Denk
wd at denx.de
Mon Feb 18 12:35:58 CET 2013
Dear Tom,
In message <51216721.1010603 at ti.com> you wrote:
>
> There's another thread I don't have yet (and I don't have this one in
> gmail yet even). But, I am OK with custodians using their repos, but
> not the master branch, for unrelated but otherwise good patches. I'm
> also fine with patchwork bundles. I suppose we could use the staging
> repository for these changes instead.
What I mostly object about there is that these patches would go into
mainline basicly unreviewed, as patch submission and pull request is
all done from a single person, with no other feedback on the patches
at all. And this affects a lot of common code...
Actually, I see this change when pulling u-boot-x86.git/master:
-> bloat-o-meter u-boot-before u-boot
add/remove: 9/0 grow/shrink: 3/14 up/down: 1006/-560 (446)
function old new delta
hash_command - 424 +424
strncasecmp - 156 +156
simple_itoa - 104 +104
crc32_wd_buf - 76 +76
setenv_hex - 68 +68
setenv_ulong - 52 +52
strcasecmp - 36 +36
do_mem_loopw 304 328 +24
static.local - 22 +22
do_mem_loop 268 288 +20
hash_algo - 16 +16
do_mem_cmp 332 340 +8
do_mem_mw 224 220 -4
set_working_fdt_addr 72 52 -20
load_serial_ymodem 300 280 -20
load_serial 512 492 -20
index_partitions 200 180 -20
do_load_serial_bin 1844 1824 -20
do_load 468 448 -20
do_jffs2_fsload 320 300 -20
do_imgextract 636 592 -44
NetLoop 832 788 -44
do_mem_cp 312 252 -60
do_bootm 1244 1180 -64
do_mem_crc 188 88 -100
do_mem_mtest 1436 1332 -104
So there are changes all over the place, including a growth of the
memory footprint. I think this needs at least minimal review.
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: wd at denx.de
Time is an illusion perpetrated by the manufacturers of space.
More information about the U-Boot
mailing list