On Tue, 2017-08-15 at 08:42 +0200, Rafał Miłecki wrote: > 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 ;) thanks for explaining. Patch 1 and 2 queued for next merge window. -rui -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
