[BUG] patman: refused to send patches because of warnings

[Tony Dinh]

Hi Sean,

Just tried again with -i and it works fine now!

Thanks,
Tony

On Fri, Dec 17, 2021 at 8:02 PM Sean Anderson <seanga2 at gmail.com> wrote:
>
> On 12/17/21 10:59 PM, Tony Dinh wrote:
> > Hi Simon,
> >
> > I tried to send in patches to add support for a new board. patman
> > seems to hit a problem with "Bad divisor in main" error and refuses to
> > send the patches. The warnings are not really important, are they? In
> > some instances, we need camel cases because that is what the
> > convention in the specs for MPPs. And the #ifdef or #if is necessary
> > because the "if (IS_ENABLED(CONFIG...))" cannot be used to compile-out
> > a function. And the warnings about MAINTAINERS seem unnecessary.
> >
> > Is there an option to tell patman to ignore the warnings or checks?
> >
> > Here is the patman log.
> >
> >
> > ./tools/patman/patman -c8 send
>
> you can use -i to ignore errors
>
> --Sean
>
> >
> > Cleaned 8 patches
> > 0 errors, 1 warnings, 0 checks for
> > 0001-arm-kirkwood-Pogoplug-V4-Add-DTS-files.patch:
> > <patch>:29: warning: added, moved or deleted file(s), does MAINTAINERS
> > need updating?
> > 0 errors, 1 warnings, 0 checks for
> > 0002-arm-kirkwood-Pogoplug-V4-Add-board-defconfig-file.patch:
> > <patch>:16: warning: added, moved or deleted file(s), does MAINTAINERS
> > need updating?
> > 0 errors, 2 warnings, 0 checks for
> > 0003-arm-kirkwood-Pogoplug-V4-Add-Kconfig-files.patch:
> > arch/arm/mach-kirkwood/Kconfig:28: warning: please write a paragraph
> > that describes the config symbol fully
> > <patch>:39: warning: added, moved or deleted file(s), does MAINTAINERS
> > need updating?
> > 0 errors, 1 warnings, 0 checks for
> > 0004-arm-kirkwood-Pogoplug-V4-Add-board-include-configs-f.patch:
> > <patch>:17: warning: added, moved or deleted file(s), does MAINTAINERS
> > need updating?
> > 0 errors, 1 warnings, 0 checks for
> > 0005-arm-kirkwood-Pogoplug-V4-Add-board-kwbimage-file.patch:
> > <patch>:16: warning: added, moved or deleted file(s), does MAINTAINERS
> > need updating?
> > 0 errors, 1 warnings, 0 checks for
> > 0006-arm-kirkwood-Pogoplug-V4-Add-board-implementation-he.patch:
> > <patch>:19: warning: added, moved or deleted file(s), does MAINTAINERS
> > need updating?
> > 0 errors, 3 warnings, 3 checks for
> > 0007-arm-kirkwood-Pogoplug-V4-Add-board-implementation.patch:
> > <patch>:23: warning: added, moved or deleted file(s), does MAINTAINERS
> > need updating?
> > board/cloudengines/pogo_v4/pogo_v4.c:49: check: Avoid CamelCase:
> > <MPP6_SYSRST_OUTn>
> > board/cloudengines/pogo_v4/pogo_v4.c:63: check: Avoid CamelCase:
> > <MPP20_SATA1_ACTn>
> > board/cloudengines/pogo_v4/pogo_v4.c:64: check: Avoid CamelCase:
> > <MPP21_SATA0_ACTn>
> > board/cloudengines/pogo_v4/pogo_v4.c:137: warning: Use 'if
> > (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where possible
> > board/cloudengines/pogo_v4/pogo_v4.c:170: warning: Use 'if
> > (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where possible
> > 0 errors, 1 warnings, 0 checks for
> > 0008-arm-kirkwood-Pogoplug-V4-Add-board-maintainer.patch:
> > <patch>:16: warning: added, moved or deleted file(s), does MAINTAINERS
> > need updating?
> > checkpatch.pl found 0 error(s), 11 warning(s), 3 checks(s)
> > Bad divisor in main::vcs_assign: 0
> > Not sending emails due to errors/warnings
> >
> > Thanks,
> > Tony
> >
>