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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <5348febc85ad3460f4e06ff11bf58ec70f6b3a48.camel@perches.com>
Date:   Fri, 18 Oct 2019 08:33:26 -0700
From:   Joe Perches <joe@...ches.com>
To:     Miguel Ojeda <miguel.ojeda.sandonis@...il.com>,
        Lars Poeschel <poeschel@...onage.de>,
        Vadim Bendebury <vbendeb@...omium.org>,
        Andi Kleen <ak@...ux.intel.com>
Cc:     Willy Tarreau <willy@...roxy.com>,
        Ksenija Stanojevic <ksenija.stanojevic@...il.com>,
        open list <linux-kernel@...r.kernel.org>,
        Geert Uytterhoeven <geert@...ux-m68k.org>,
        Geert Uytterhoeven <geert+renesas@...der.be>,
        Andy Shevchenko <andy.shevchenko@...il.com>
Subject: Re: [PATCH 1/3] auxdisplay: Make charlcd.[ch] more general

On Fri, 2019-10-18 at 17:08 +0200, Miguel Ojeda wrote:
> On Thu, Oct 17, 2019 at 10:07 AM Lars Poeschel <poeschel@...onage.de> 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.

commit 3354957a4f8b9bb4b43625232acdf0626851c82f
Author: Andi Kleen <ak@...ux.intel.com>
Date:   Mon May 24 14:33:29 2010 -0700

    checkpatch: add check for too short Kconfig descriptions
    
    I've seen various new Kconfigs with rather unhelpful one liner
    descriptions.  Add a Kconfig warning for a minimum length of the Kconfig
    help section.
    
    Right now I arbitarily chose 4. The exact value can be debated.