diff mbox series

thermal: fix indentation in makefile

Message ID 20191221173737.30906-1-tiny.windzz@gmail.com (mailing list archive)
State Changes Requested
Delegated to: Zhang Rui
Headers show
Series thermal: fix indentation in makefile | expand

Commit Message

Yangtao Li Dec. 21, 2019, 5:37 p.m. UTC 
To unify code style.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/thermal/Makefile | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Daniel Lezcano Dec. 21, 2019, 6:57 p.m. UTC | #1 
On 21/12/2019 18:37, Yangtao Li wrote:
> To unify code style.
> 
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.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.

>  # 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
>
Yangtao Li Dec. 22, 2019, 5:38 p.m. UTC | #2 
：q

On Sun, Dec 22, 2019 at 2:57 AM Daniel Lezcano
<daniel.lezcano@linaro.org> wrote:
>
> On 21/12/2019 18:37, Yangtao Li wrote:
> > To unify code style.
> >
> > Signed-off-by: Yangtao Li <tiny.windzz@gmail.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?


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
>
Zhang Rui Feb. 17, 2020, 7:39 a.m. UTC | #3 
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@linaro.org> wrote:
> > 
> > On 21/12/2019 18:37, Yangtao Li wrote:
> > > To unify code style.
> > > 
> > > Signed-off-by: Yangtao Li <tiny.windzz@gmail.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
> >
diff mbox series

Patch

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
 
 # 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