Re: [PATCH 1/3] auxdisplay: Make charlcd.[ch] more general

From: Andi Kleen
Date: Fri Oct 18 2019 - 17:55:54 EST


On Fri, Oct 18, 2019 at 08:33:26AM -0700, Joe Perches wrote:
> On Fri, 2019-10-18 at 17:08 +0200, Miguel Ojeda wrote:
> > On Thu, Oct 17, 2019 at 10:07 AM Lars Poeschel <poeschel@xxxxxxxxxxx> wrote:
> []
> > > Oh by the way: Do you know what I can do to make checkpatch happy with
> > > its describing of the config symbol ? I tried writing a help paragraph
> > > for the config symbols in Kconfig, but that did not help.
> >
> > CC'ing Joe.
>
> add
> --ignore=CONFIG_DESCRIPTION
> or
> --min-conf-desc-length=1 (default is 4)
>
> to the checkpatch command line, or just ignore it.
>
> AK: I guess there's still some debate as to the proper
> minimum length of a Kconfig help section paragraph.

I still think four lines is a good minimum

If it's not worth writing 4 lines for, perhaps the Kconfig
symbol is not needed at all?

-Andi