Re: [U-Boot] [PATCH 1/8] imx: i2c: Zap unnecessary malloc() calls

30 Dec 2014


      On 16/12/2014 14:09, Marek Vasut wrote:
...
The malloc() calls are unnecessary, just allocate the stuff on stack.
While at it, reorder the code a little, so that only one variable is
used for the text, use snprintf() instead of sprintf() and use %01d
as a formatting string to avoid any possible overflows.
Signed-off-by: Marek Vasut marex@denx.de
Cc: Igor Grinberg grinberg@compulab.co.il
Cc: Nikita Kiryanov nikita@compulab.co.il
Cc: Sean Cross xobs@kosagi.com
Cc: Simon Glass sjg@chromium.org
Cc: Stefano Babic sbabic@denx.de
Cc: Tim Harvey tharvey@gateworks.com

Applied to u-boot-imx, thanks !
Best regards,
Stefano Babic
-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================