From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] Patch sets coming
Date: Fri, 22 Mar 2013 21:45:02 +0100 [thread overview]
Message-ID: <20130322204502.39F332014CE@gemini.denx.de> (raw)
In-Reply-To: <1363970293-21228-1-git-send-email-yorksun@freescale.com>
Dear York Sun,
In message <1363970293-21228-1-git-send-email-yorksun@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@jpl-devvax.JPL.NASA.GOV>
next prev parent reply other threads:[~2013-03-22 20:45 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-22 16:38 [U-Boot] Patch sets coming York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 01/10] doc/ramboot.mpc85xx: Documented the RAMBOOT for MPC85xx York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 02/10] qoriq/p1_p2_rdb_pc: USB device-tree fixups for P1020 York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 03/10] powerpc/85xx: add SerDes bank 4 lanes only if we have a bank 4 York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 04/10] powerpc/p1010rdb: Change flexcan compatible string York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 05/10] powerpc/p1010rdb: add readme document for p1010rdb York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 06/10] powerpc/mpc85xx: Extend workaround for erratum DDR_A003 to other SoCs York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 07/10] Changed LIODN offset values York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 08/10] powerpc/p1_p2_rdb_pc: Add a pin to reset the DDR chip for P1021RDB-PC York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 09/10] powerpc/corenet_ds: Move SRIO enablement into specific boards York Sun
2013-03-22 16:38 ` [U-Boot] [PATCH 10/10] powerpc/85xx: set USB2 default mode to 'device' for (super)hydra boards York Sun
2013-03-22 20:45 ` Wolfgang Denk [this message]
2013-03-22 21:05 ` [U-Boot] Patch sets coming York Sun
2013-03-22 21:17 ` Wolfgang Denk
2013-03-22 21:25 ` Tom Rini
2013-03-22 21:33 ` York Sun
2013-03-22 21:43 ` York Sun
2013-03-22 22:45 ` Tom Rini
2013-03-22 22:55 ` York Sun
2013-03-23 1:09 ` Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130322204502.39F332014CE@gemini.denx.de \
--to=wd@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.