Dear Albert ARIBAUD,
On 14/12/13 19:14, Albert ARIBAUD wrote:
Hi Minkyu,
On Sat, 14 Dec 2013 14:07:57 +0900, Minkyu Kang mk7.kang@samsung.com wrote:
Dear Albert,
On 13/12/13 18:00, Albert ARIBAUD wrote:
(adding Minkyu as the Samsung custodian)
On Mon, 9 Dec 2013 18:09:18 +0100, Albert ARIBAUD albert.u.boot@aribaud.net wrote:
mkexynos reads its input file which might be smaller than
( typo: s/mkexynos/mkexynosspl/ -- wil fix when applying if thee is no other change to be made )
its read buffer, but always writes the whole buffer out. This does not affect the functionalyty of the output file,
... and s/functionalyty/functionality/, too.
but it makes its content unpredictable as the end of the buffer is never initialized. Fix this by zeroing the buffer before reading the input file.
Minkyu: should the buffer be pre-filled with zeroes or should I use ones instead? Or maybe the output could be truncated to the size of the input?
Basically, the output file is fixed to 14K as same as buffer size. And the input file should be smaller than 14K. It's the rule of exynos spl. The role of mkexynosspl is just adding 4 byte checksum value to end of file. If the input file is bigger than 14K, output will be truncated.
Amicalement, Albert.
Signed-off-by: Albert ARIBAUD albert.u.boot@aribaud.net
btw, this change looks reasonable to me.
Acked-by: Minkyu Kang mk7.kang@samsung.com
Thanks! If that's ok with you, I'll apply 1/4 to ARM, considering it a kind of bugfix, and remove it from V4 of the patch series if there is any.
Or if you prefer to apply 1/4 to u-boot-samsung, that's fine with me.
please apply this patch to u-boot-arm.
Thanks, Minkyu Kang.
Amicalement,
Thanks, Minkyu Kang.