Hi, Florian, This patch set was dropped in the last minute because of this discussion https://patchwork.kernel.org/patch/9936325/ as I don’t want to rebase the patch before sending the pull request.
I don’t think we can make it for 4.14. Eduardo will be back and pick the soc patches for 4.15. Thanks, rui -----Original Message----- From: Florian Fainelli [mailto:[email protected]] Sent: Monday, September 25, 2017 5:11 AM To: Zhang, Rui <[email protected]>; Rafał Miłecki <[email protected]> Cc: Markus Mayer <[email protected]>; Eduardo Valentin <[email protected]>; Rob Herring <[email protected]>; Mark Rutland <[email protected]>; Doug Berger <[email protected]>; Brian Norris <[email protected]>; Gregory Fong <[email protected]>; Russell King <[email protected]>; Catalin Marinas <[email protected]>; Will Deacon <[email protected]>; Arnd Bergmann <[email protected]>; Olof Johansson <[email protected]>; Broadcom Kernel List <[email protected]>; Power Management List <[email protected]>; Device Tree List <[email protected]>; ARM Kernel List <[email protected]>; Linux Kernel Mailing List <[email protected]>; Markus Mayer <[email protected]> Subject: Re: [PATCH v4 2/4] thermal: add brcmstb AVS TMON driver Importance: High Le 08/14/17 à 23:48, Zhang Rui a écrit : > 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. Humm, I don't see this driver in your latest 4.14 pull request to Linus, so what happened here exactly? Can we expect this driver to be submitted for 4.14 or we just happened to have missed this window now? -- Florian
