Messages in this thread | | | Date | Tue, 15 Aug 2017 08:42:11 +0200 | From | Rafał Miłecki <> | Subject | Re: [PATCH v4 2/4] thermal: add brcmstb AVS TMON driver |
| |
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 ;)
| |