Re: [PATCH] checkpatch: add judgment condition for Kconfig help test

From: Bagas Sanjaya
Date: Fri Nov 17 2023 - 18:36:55 EST


On 11/18/23 04:14, Andy Shevchenko wrote:
> 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?
>

IDK as I'm not the patch author...

--
An old man doll... just what I always wanted! - Clara