On Tue, Apr 07, 2020 at 09:00:49PM -0600, Simon Glass wrote:
> Add the C version of this header. It includes a few Chrome OS bits which
> are disabled for a normal build.

> +     u8      unused[0x100 - 0x3d];           /* Pad out to 0x100 */
> +     u8      unused2[0x1000 - 0x100];        /* Pad out to 4KB */

Inconsistent size representation in the comment: 256 bytes, 4096 bytes?
Also I would rather name unused1 and unused2.

-- 
With Best Regards,
Andy Shevchenko


Reply via email to