On Thu, Nov 16, 2023 at 11:39:04PM +0800, Cixi Geng wrote: > From: Cixi Geng <cixi.geng1@xxxxxxxxxx> > > The has_help only counted the situation which the patch file add > a help line, when the config was renamed and modify description, > the has_help is zero for the "help" line not added. > > here is one case: > the modify file: drivers/iio/adc/Kconfig line(1047) > > -config SC27XX_ADC > +config SPRD_ADC > tristate "Spreadtrum SC27xx series PMICs ADC" > depends on MFD_SC27XX_PMIC || COMPILE_TEST > + depends on ARCH_SPRD > help > - Say yes here to build support for the integrated ADC inside the > - Spreadtrum SC27xx series PMICs. > + 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. > > the checkpatch result: > WARNING: please write a help paragraph that fully describes the config symbol > #23: FILE: drivers/iio/adc/Kconfig:1050: > +config SPRD_ADC > + tristate "Spreadtrum's ADC PMICs driver" > depends on MFD_SC27XX_PMIC || COMPILE_TEST > + depends on ARCH_SPRD > 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? Confused... -- An old man doll... just what I always wanted! - Clara
Attachment:
signature.asc
Description: PGP signature
