Re: [PATCH v4 2/4] thermal: add brcmstb AVS TMON driver

[RafaÅ MiÅecki]

On 2017-08-15 08:08, Zhang Rui wrote:
> > diff --git a/drivers/thermal/broadcom/Kconfig
> > b/drivers/thermal/broadcom/Kconfig
> > index 42c098e..c106a15 100644
> > --- a/drivers/thermal/broadcom/Kconfig
> > +++ b/drivers/thermal/broadcom/Kconfig
> > @@ -6,6 +6,13 @@ config BCM2835_THERMAL
> > Â	help
> > Â	ÂÂSupport for thermal sensors on Broadcom bcm2835 SoCs.
> > Â
> > +config BRCMSTB_THERMAL
> > +	tristate "Broadcom STB AVS TMON thermal driver"
> > +	depends on ARCH_BRCMSTB || COMPILE_TEST
> > +	help
> > +	ÂÂEnable this driver if you have a Broadcom STB SoC and
> > would like
> > +	ÂÂthermal framework support.
> > +
>
> I don't understand why I got the following checkpatch warning
>
> WARNING: please write a paragraph that describes the config symbol
> fully
> #73: FILE: drivers/thermal/broadcom/Kconfig:9:
> +config BRCMSTB_THERMAL
>
> I didn't see this for other Kconfig changes.

It's because your help message is only 2 lines long (instead of 3).

Some (many?) maintainers aren't pedantic about that, a common sense 
should
be applied ;)