Dear Anton Staaf, In message <1315007128-10710-1-git-send-email-robot...@chromium.org> you wrote: > Currently when you call ROUND with a value that is already a > multiple of the second parameter it will return a value that is > one multiple larger, instead of returning the value passed in. > > There are only two types of usage of ROUND currently, one in > various config files to round CONFIG_SYS_MALLOC_LEN to a multiple > of 4096 bytes. The other in cmd_sf.c where the incorrect behavior > of ROUND is worked around be subtracting one from the length argument > before passing it to ROUND. > > This patch fixes ROUND and removes the workaround from cmd_sf. It > also results in all of the malloc pools that use ROUND to compute > their size shrinking by 4KB. > > Cc: Lukasz Majewski <l.majew...@samsung.com> > Cc: Mike Frysinger <vap...@gentoo.org> > --- > > I'm not sure if it makes sense to go and add 4KB to each malloc pool > effected. What do other people think? I'd be happy to make a v2 that does > that. This just seemed like a cleaner v1 to start with. > > common/cmd_sf.c | 2 +- > include/common.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-)
Applied, thanks. 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 "There are three principal ways to lose money: wine, women, and engi- neers. While the first two are more pleasant, the third is by far the more certain." - Baron Rothschild, ca. 1800 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
