Re: [PATCH 1/12] mips: PMC MSP71xx core platform

[Ralf Baechle <ralf@linux-mips.org>]

On Thu, Jun 14, 2007 at 03:54:47PM -0600, Marc St-Jean wrote:

> Re-posting patch as requested by Ralf. Changes since last post:
> -Minor cleanups as recommended by checkpatch.pl.

Be careful with that script.  Some of it's recommendations are harmful,
for example when it sees an inclusion of <asm/time.h> it will suggest
to use <linux/time.h>.  That's all fine because generally the file under
linux/ will include the asm/ version but there are exception.

But it's great that you discovered it already, it saves me the for both
sides annoying part of dealing out tons of trivialities, first such as
formatting trivialities.  One thing checkpatch.pl doesn't yet complain
about is trailing whitespace such as in the first of the patches you
sent:

Warning: trailing whitespace in line 106 of arch/mips/pmc-sierra/msp71xx/msp_usb.c
Warning: trailing whitespace in line 12 of arch/mips/pmc-sierra/msp71xx/msp_irq_slp.c
Warning: trailing whitespace in lines 43,53,63,79 of arch/mips/pmc-sierra/msp71xx/msp_time.c
Warning: trailing whitespace in lines 49,59,62,68,78,84 of arch/mips/pmc-sierra/msp71xx/msp_irq.c
Warning: trailing whitespace in line 84 of arch/mips/pmc-sierra/msp71xx/msp_irq_cic.c
Warning: trailing whitespace in line 157 of arch/mips/pmc-sierra/msp71xx/msp_hwbutton.c
Warning: trailing whitespace in lines 154,307,537,548,558 of arch/mips/pmc-sierra/msp71xx/msp_prom.c
Warning: trailing whitespace in lines 66,229,238,252 of arch/mips/pmc-sierra/msp71xx/msp_setup.c

No need to resend the patch, I can strip that of trivially

  Ralf