linux-kernel - Re: [PATCH] thermal: fix indentation in makefile

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <6abe2d62a08d38d53c85b3892ed77dbcc543c8fc.camel@intel.com>
Date:   Mon, 17 Feb 2020 15:39:48 +0800
From:   Zhang Rui <rui.zhang@...el.com>
To:     Frank Lee <tiny.windzz@...il.com>,
        Daniel Lezcano <daniel.lezcano@...aro.org>
Cc:     Amit Kucheria <amit.kucheria@...durent.com>,
        Linux PM <linux-pm@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] thermal: fix indentation in makefile

On Mon, 2019-12-23 at 01:38 +0800, Frank Lee wrote:
> ：q
> 
> On Sun, Dec 22, 2019 at 2:57 AM Daniel Lezcano
> <daniel.lezcano@...aro.org> wrote:
> > 
> > On 21/12/2019 18:37, Yangtao Li wrote:
> > > To unify code style.
> > > 
> > > Signed-off-by: Yangtao Li <tiny.windzz@...il.com>
> > > ---
> > >  drivers/thermal/Makefile | 10 +++++-----
> > >  1 file changed, 5 insertions(+), 5 deletions(-)
> > > 
> > > diff --git a/drivers/thermal/Makefile b/drivers/thermal/Makefile
> > > index baeb70bf0568..ab9389bc42b2 100644
> > > --- a/drivers/thermal/Makefile
> > > +++ b/drivers/thermal/Makefile
> > > @@ -5,7 +5,7 @@
> > > 
> > >  obj-$(CONFIG_THERMAL)                += thermal_sys.o
> > >  thermal_sys-y                        += thermal_core.o
> > > thermal_sysfs.o \
> > > -                                     thermal_helpers.o
> > > +                                thermal_helpers.o
> > 
> > Here a tabulation is replaced with spaces.
> 
> It won't align after using tab here, shouldn't modify it?

I'd prefer using tab here. Other changes seem okay to me.

thanks,
rui
> 
> 
> Yours,
> Yangtao
> 
> > 
> > >  # interface to/from other layers providing sensors
> > >  thermal_sys-$(CONFIG_THERMAL_HWMON)          += thermal_hwmon.o
> > > @@ -25,11 +25,11 @@ thermal_sys-$(CONFIG_CPU_THERMAL) +=
> > > cpu_cooling.o
> > >  thermal_sys-$(CONFIG_CLOCK_THERMAL)  += clock_cooling.o
> > > 
> > >  # devfreq cooling
> > > -thermal_sys-$(CONFIG_DEVFREQ_THERMAL) += devfreq_cooling.o
> > > +thermal_sys-$(CONFIG_DEVFREQ_THERMAL)        +=
> > > devfreq_cooling.o
> > > 
> > >  # platform thermal drivers
> > >  obj-y                                += broadcom/
> > > -obj-$(CONFIG_THERMAL_MMIO)           += thermal_mmio.o
> > > +obj-$(CONFIG_THERMAL_MMIO)   += thermal_mmio.o
> > >  obj-$(CONFIG_SPEAR_THERMAL)  += spear_thermal.o
> > >  obj-$(CONFIG_ROCKCHIP_THERMAL)       += rockchip_thermal.o
> > >  obj-$(CONFIG_RCAR_THERMAL)   += rcar_thermal.o
> > > @@ -49,9 +49,9 @@ obj-$(CONFIG_TI_SOC_THERMAL)        += ti-soc-
> > > thermal/
> > >  obj-y                                += st/
> > >  obj-$(CONFIG_QCOM_TSENS)     += qcom/
> > >  obj-y                                += tegra/
> > > -obj-$(CONFIG_HISI_THERMAL)     += hisi_thermal.o
> > > +obj-$(CONFIG_HISI_THERMAL)   += hisi_thermal.o
> > >  obj-$(CONFIG_MTK_THERMAL)    += mtk_thermal.o
> > >  obj-$(CONFIG_GENERIC_ADC_THERMAL)    += thermal-generic-adc.o
> > >  obj-$(CONFIG_ZX2967_THERMAL) += zx2967_thermal.o
> > >  obj-$(CONFIG_UNIPHIER_THERMAL)       += uniphier_thermal.o
> > > -obj-$(CONFIG_AMLOGIC_THERMAL)     += amlogic_thermal.o
> > > +obj-$(CONFIG_AMLOGIC_THERMAL)        += amlogic_thermal.o
> > > 
> > 
> > 
> > --
> >  <http://www.linaro.org/> Linaro.org │ Open source software for ARM
> > SoCs
> > 
> > Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
> > <http://twitter.com/#!/linaroorg> Twitter |
> > <http://www.linaro.org/linaro-blog/> Blog
> >