[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