All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Bagas Sanjaya <bagasdotme@gmail.com>
Cc: Cixi Geng <cixi.geng@linux.dev>,
	apw@canonical.com, joe@perches.co, dwaipayanray1@gmail.com,
	lukas.bulwahn@gmail.com, akpm@linux-foundation.org,
	rdunlap@infradead.org, Jonathan Cameron <jic23@kernel.org>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Cosmin Tanislav <demonsingur@gmail.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Maksim Kiselev <bigunclemax@gmail.com>,
	Liam Beguin <liambeguin@gmail.com>,
	Marcus Folkesson <marcus.folkesson@gmail.com>,
	Ibrahim Tilki <Ibrahim.Tilki@analog.com>,
	Marius Cristea <marius.cristea@microchip.com>,
	ChiaEn Wu <chiaen_wu@richtek.com>,
	Niklas Schnelle <schnelle@linux.ibm.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linux Industrial I/O <linux-iio@vger.kernel.org>
Subject: Re: [PATCH] checkpatch: add judgment condition for Kconfig help test
Date: Fri, 17 Nov 2023 23:14:02 +0200	[thread overview]
Message-ID: <ZVfXmlBbB3Y12Zz3@smile.fi.intel.com> (raw)
In-Reply-To: <ZVarTLiczTZ8oCBg@archie.me>

On Fri, Nov 17, 2023 at 06:52:44AM +0700, Bagas Sanjaya wrote:
> On Thu, Nov 16, 2023 at 11:39:04PM +0800, Cixi Geng wrote:

...

> > the checkpatch result:
> >     WARNING: please write a help paragraph that fully describes the config symbol
> >     #23: FILE: drivers/iio/adc/Kconfig:1050:

> >      	help
> >     +	  Say yes here to build support for the integrated ADC inside of the
> >     +	  Say yes here to build support for the integrated ADC inside of the
> >     +	  Say yes here to build support for the integrated ADC inside of the
> >     +	  Spreadtrum SC27xx and UMPxx series PMICs.
> > 
> >      	  This driver can also be built as a module. If so, the module
> >      	  will be called sc27xx_adc.
> > 
> >     total: 0 errors, 1 warnings, 17 lines checked
> 
> This confuses me. What are you trying to achieve there?

Is the indentation correct in each line? No TABs/spaces mix?

-- 
With Best Regards,
Andy Shevchenko



  reply	other threads:[~2023-11-17 21:14 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-16 15:39 [PATCH] checkpatch: add judgment condition for Kconfig help test Cixi Geng
2023-11-16 23:52 ` Bagas Sanjaya
2023-11-17 21:14   ` Andy Shevchenko [this message]
2023-11-17 23:19     ` Bagas Sanjaya
2023-11-18  2:26 ` Randy Dunlap
2023-11-18 14:37   ` Cixi Geng

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZVfXmlBbB3Y12Zz3@smile.fi.intel.com \
    --to=andriy.shevchenko@linux.intel.com \
    --cc=Ibrahim.Tilki@analog.com \
    --cc=akpm@linux-foundation.org \
    --cc=apw@canonical.com \
    --cc=arnd@arndb.de \
    --cc=bagasdotme@gmail.com \
    --cc=bigunclemax@gmail.com \
    --cc=chiaen_wu@richtek.com \
    --cc=cixi.geng@linux.dev \
    --cc=demonsingur@gmail.com \
    --cc=dwaipayanray1@gmail.com \
    --cc=jic23@kernel.org \
    --cc=joe@perches.co \
    --cc=lars@metafoo.de \
    --cc=liambeguin@gmail.com \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lukas.bulwahn@gmail.com \
    --cc=marcus.folkesson@gmail.com \
    --cc=marius.cristea@microchip.com \
    --cc=rdunlap@infradead.org \
    --cc=schnelle@linux.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.