[U-Boot] [PATCH v2 05/10] powerpc/ppc4xx: Support gdsys multichannel iocon hardware

Joe Hershberger joe.hershberger at gmail.com
Mon May 6 21:31:33 CEST 2013


Hi Wolfgang,

On Mon, May 6, 2013 at 2:15 PM, Wolfgang Denk <wd at denx.de> wrote:
> Dear Dirk,
>
> In message <3628fded6de7e6b4286d3444a7e5e3ec at gdsys.cc> you wrote:
>>
>> > total: 3 errors, 0 warnings, 509 lines checked
>>
>> OK, thanks, that makes sense. Which version of checkpatch are you
>> using? Latest u-boot master checkpatch/patman does not show these (at
>> least not for me).
>
> It does for me, interestingly ONLY if the current directory is NOT in
> the U-Boot tree.
>
> Apparently both the COMPLEX_MACRO and
> MULTISTATEMENT_MACRO_USE_DO_WHILE option should better be enabled, or
> we will miss such issues.
>
>
>
> Added Joe to Cc: - Joe, what do you think?  I'd rather remove the
> silencing for these from .checkpatch.conf ?

Back in October 2011 checkpatch was generating false positives on
COMPLEX_MACRO.  If this has been fixed and copied from Linux, then we
can certainly stop ignoring it.  I reported the issue at that time.

As the .checkpatch.conf comments say, CONFIG_SYS_I2C_NOPROBES
definitions were causing false positives with
MULTISTATEMENT_MACRO_USE_DO_WHILE.  I agree that we can remove
ignoring MULTISTATEMENT_MACRO_USE_DO_WHILE as long as the false
positives are allowed when identified.

Cheers,
-Joe


More information about the U-Boot mailing list