[U-Boot] Patch sets coming
Wolfgang Denk
wd at denx.de
Fri Mar 22 21:45:02 CET 2013
Dear York Sun,
In message <1363970293-21228-1-git-send-email-yorksun at freescale.com> you wrote:
>
> Andy,
>
> I am sending out several sets of backlog patches. Some of them may have been
> sent before. To make them easier to apply, here they come in sets.
>
> In case my git send-email fails to add the author(s) to the CC list, please add
> them when you reply (it happened before).
It would be _really_ nice if you could get used to running your
patches through checkpatch _before_ posting these.
I see again a number of checkpatch issues, like:
WARNING: quoted string split across lines
#149: FILE: board/freescale/p1_p2_rdb_pc/p1_p2_rdb_pc.c:456:
+ printf("WARNING: could not find compatible node"
+ " %s: %s.\n", soc_elbc_compat,
CHECK: Alignment should match open parenthesis
#156: FILE: board/freescale/p1_p2_rdb_pc/p1_p2_rdb_pc.c:463:
+ printf("WARNING: could not remove %s: %s.\n",
+ soc_elbc_compat, fdt_strerror(err));
WARNING: quoted string split across lines
#167: FILE: board/freescale/p1_p2_rdb_pc/p1_p2_rdb_pc.c:474:
+ printf("WARNING: could not find compatible node"
+ "%s: %s.\n", soc_usb_compat,
WARNING: quoted string split across lines
#175: FILE: board/freescale/p1_p2_rdb_pc/p1_p2_rdb_pc.c:482:
+ printf("WARNING: could not find compatible node"
+ " %s: %s.\n", soc_usb_compat,
CHECK: Alignment should match open parenthesis
#182: FILE: board/freescale/p1_p2_rdb_pc/p1_p2_rdb_pc.c:489:
+ printf("WARNING: could not remove %s: %s.\n",
+ soc_usb_compat, fdt_strerror(err));
ERROR: Unrecognized email address: 'Timur Tabi'
#116:
Signed-off-by: Timur Tabi
CHECK: No space is necessary after a cast
#178: FILE: board/freescale/p1_p2_rdb_pc/spl_minimal.c:85:
+#elif defined(CONFIG_P1021RDB)
+ par_io_t *par_io = (par_io_t *) &(gur->qe_par_io);
ERROR: Unrecognized email address: 'Timur Tabi'
#122:
Signed-off-by: Timur Tabi
Can you please fix these?
Thanks.
NAK for the series unless these issues are fixed.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I don't know if it's what you want, but it's what you get. :-)
- Larry Wall in <10502 at jpl-devvax.JPL.NASA.GOV>
More information about the U-Boot
mailing list