[U-Boot] [PATCH] arm: socfpga: Fix mailbox command length checking

Marek Vasut marex at denx.de
Thu Apr 11 09:19:05 UTC 2019


On 4/11/19 11:16 AM, Ley Foon Tan wrote:
> On Thu, Apr 11, 2019 at 3:58 PM Marek Vasut <marex at denx.de> wrote:
>>
>> On 4/11/19 9:39 AM, Ley Foon Tan wrote:
>>> A mailbox command including header + arguments.
>>> But, "len" only including length of arguments, it missed the length
>>> of header in checking.
>>> Includes length of header (1) when checking the space size.
>>
>> I think the commit message could use some improvement, maybe reword it
>> so it's clear what you're fixing here. I guess it's some off-by-one error ?
> Okay, will resend again.

Thanks. The patch itself is fine.

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list