Messages in this thread | | | From | Lh Kuo 郭力豪 <> | Subject | RE: [PATCH v9 1/2] thermal: Add thermal driver for Sunplus | Date | Wed, 29 Jun 2022 03:31:48 +0000 |
| |
I'm sorry for the wrong indentation problem, it was caused by a wrong setting in the mail server.I will fix it in the next patch.
> -----Original Message----- > From: Randy Dunlap <rdunlap@infradead.org> > Sent: Friday, June 10, 2022 11:20 AM > To: 郭力豪 <lhjeff911@gmail.com>; krzk@kernel.org; rafael@kernel.org; Daniel Lezcano > <daniel.lezcano@linaro.org>; amitk@kernel.org; rui.zhang@intel.com; robh+dt@kernel.org; > linux-pm@vger.kernel.org; devicetree@vger.kernel.org; linux-kernel@vger.kernel.org > Cc: Lh Kuo 郭力豪 <lh.Kuo@sunplus.com>; Wells Lu 呂芳騰 <wells.lu@sunplus.com> > Subject: Re: [PATCH v9 1/2] thermal: Add thermal driver for Sunplus > > Hi-- > > On 6/9/22 18:32, 郭力豪 wrote: > > diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index > > 0e5cc94..c85e668 100644 > > --- a/drivers/thermal/Kconfig > > +++ b/drivers/thermal/Kconfig > > @@ -504,4 +504,14 @@ config KHADAS_MCU_FAN_THERMAL > > If you say yes here you get support for the FAN controlled > > by the Microcontroller found on the Khadas VIM boards. > > > > +config SUNPLUS_THERMAL > > + tristate "Sunplus thermal drivers" > > + depends on SOC_SP7021 || COMPILE_TEST help > > + This enable the Sunplus SP7021 thermal driver, which supports the primitive > > + temperature sensor embedded in Sunplus SP7021 SoC. > > + > > + If you have a Sunplus SP7021 platform say Y here and enable this option > > + to have support for thermal management > > End the sentence above with a period ('.'). > > And the indentation here is all messed up. > See Documentation/process/coding-style.rst: > > 10) Kconfig configuration files > ------------------------------- > > For all of the Kconfig* configuration files throughout the source tree, the indentation is somewhat > different. Lines under a ``config`` definition are indented with one tab, while help text is indented an > additional two spaces. > > > > + > > endif > > -- > ~Randy
| |