[U-Boot] [PATCH resend] Added UBL_MAGIC_PLL number for ublimage + added automatic page size calculation

Tom Rini trini at ti.com
Mon Jun 4 23:29:23 CEST 2012


On 06/02/2012 01:38 PM, Stijn Souffriau wrote:
> ---
>  tools/mkimage.h  |    6 ++++++
>  tools/ublimage.c |   29 +++++++++++++++++++++++++++--
>  tools/ublimage.h |    2 ++
>  3 files changed, 35 insertions(+), 2 deletions(-)

In addition to what I just said about v2, v3 and so forth (and needing a
body of the commit message, in this case explain what PLL mode is and so
forth), you have a few cases of '//' or:
/* multi line
 * comments.
 */
that don't match the required style.  Using checkpatch.pl should catch
one or both of these cases.

[snip]
> +	{UBL_MAGIC_SAFE,"safe",	"Safe boot mode",	},
> +	{UBL_MAGIC_PLL,	"pll", "With PLL enabled to have higher ARM/DMA clocks",	},

Missing space.  Thanks!

-- 
Tom


More information about the U-Boot mailing list