Re: [PATCH 2/3] kbuild: remove unnecessary variable initializaions
On Tue, Sep 9, 2014 at 12:26 PM, Masahiro Yamada wrote: > Clearing obj-y, obj-m, obj-n, obj- in each Makefile is > a useless habit. > > They are non-exported variables; therefore they are always empty > whenever descending into each subdirectory. > (Moreorver, obj-y and obj-m are also set to empty at the beginning > of scripts/Makefile.build) > > Signed-off-by: Masahiro Yamada Acked-by: Linus Walleij Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 2/3] kbuild: remove unnecessary variable initializaions
On Tue, Sep 9, 2014 at 6:26 AM, Masahiro Yamada wrote: > Clearing obj-y, obj-m, obj-n, obj- in each Makefile is > a useless habit. > > They are non-exported variables; therefore they are always empty > whenever descending into each subdirectory. > (Moreorver, obj-y and obj-m are also set to empty at the beginning > of scripts/Makefile.build) > > Signed-off-by: Masahiro Yamada Acked-by: Peter Foley -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 2/3] kbuild: remove unnecessary variable initializaions
On 09/09/2014 12:26, Masahiro Yamada : > Clearing obj-y, obj-m, obj-n, obj- in each Makefile is > a useless habit. > > They are non-exported variables; therefore they are always empty > whenever descending into each subdirectory. > (Moreorver, obj-y and obj-m are also set to empty at the beginning > of scripts/Makefile.build) > > Signed-off-by: Masahiro Yamada > --- > > arch/arm/mach-at91/Makefile | 3 --- > arch/arm/mach-ebsa110/Makefile| 3 --- > arch/arm/mach-ep93xx/Makefile | 3 --- > arch/arm/mach-exynos/Makefile | 5 - > arch/arm/mach-footbridge/Makefile | 3 --- > arch/arm/mach-iop13xx/Makefile| 5 - > arch/arm/mach-iop32x/Makefile | 3 --- > arch/arm/mach-iop33x/Makefile | 3 --- > arch/arm/mach-ks8695/Makefile | 3 --- > arch/arm/mach-rpc/Makefile| 4 > arch/arm/mach-s3c24xx/Makefile| 5 - > arch/arm/mach-s3c64xx/Makefile| 5 - > arch/arm/mach-s5pv210/Makefile| 5 - > arch/arm/mach-sa1100/Makefile | 3 --- > arch/arm/mach-u300/Makefile | 3 --- > arch/arm/plat-iop/Makefile| 6 -- > arch/arm/plat-omap/Makefile | 3 --- > arch/arm/plat-samsung/Makefile| 4 > 18 files changed, 69 deletions(-) > > diff --git a/arch/arm/mach-at91/Makefile b/arch/arm/mach-at91/Makefile > index 78e9cec..75033839 100644 > --- a/arch/arm/mach-at91/Makefile > +++ b/arch/arm/mach-at91/Makefile > @@ -3,9 +3,6 @@ > # > > obj-y:= irq.o gpio.o setup.o sysirq_mask.o > -obj-m:= > -obj-n:= > -obj- := I agree: Acked-by: Nicolas Ferre But I have patches that will mess with these changes already queued for 3.18. You may have to signal these conflicts when carrying the patch upstream. Thanks, best regards, > obj-$(CONFIG_OLD_CLK_AT91) += clock.o > obj-$(CONFIG_AT91_SAM9_ALT_RESET) += at91sam9_alt_reset.o > diff --git a/arch/arm/mach-ebsa110/Makefile b/arch/arm/mach-ebsa110/Makefile > index 935e4af..a7d68c1 100644 > --- a/arch/arm/mach-ebsa110/Makefile > +++ b/arch/arm/mach-ebsa110/Makefile > @@ -5,6 +5,3 @@ > # Object file lists. > > obj-y:= core.o io.o leds.o > -obj-m:= > -obj-n:= > -obj- := > diff --git a/arch/arm/mach-ep93xx/Makefile b/arch/arm/mach-ep93xx/Makefile > index 0dc51f9..78d427b 100644 > --- a/arch/arm/mach-ep93xx/Makefile > +++ b/arch/arm/mach-ep93xx/Makefile > @@ -2,9 +2,6 @@ > # Makefile for the linux kernel. > # > obj-y:= core.o clock.o > -obj-m:= > -obj-n:= > -obj- := > > obj-$(CONFIG_EP93XX_DMA) += dma.o > > diff --git a/arch/arm/mach-exynos/Makefile b/arch/arm/mach-exynos/Makefile > index 788f26d..27ae614 100644 > --- a/arch/arm/mach-exynos/Makefile > +++ b/arch/arm/mach-exynos/Makefile > @@ -7,11 +7,6 @@ > > ccflags-$(CONFIG_ARCH_MULTIPLATFORM) += -I$(srctree)/$(src)/include > -I$(srctree)/arch/arm/plat-samsung/include > > -obj-y:= > -obj-m:= > -obj-n:= > -obj- := > - > # Core > > obj-$(CONFIG_ARCH_EXYNOS)+= exynos.o pmu.o exynos-smc.o firmware.o > diff --git a/arch/arm/mach-footbridge/Makefile > b/arch/arm/mach-footbridge/Makefile > index c3faa3b..e83d5c8 100644 > --- a/arch/arm/mach-footbridge/Makefile > +++ b/arch/arm/mach-footbridge/Makefile > @@ -5,9 +5,6 @@ > # Object file lists. > > obj-y:= common.o dma.o isa-irq.o > -obj-m:= > -obj-n:= > -obj- := > > pci-y+= dc21285.o > pci-$(CONFIG_ARCH_CATS) += cats-pci.o > diff --git a/arch/arm/mach-iop13xx/Makefile b/arch/arm/mach-iop13xx/Makefile > index cad015f..a3d9260 100644 > --- a/arch/arm/mach-iop13xx/Makefile > +++ b/arch/arm/mach-iop13xx/Makefile > @@ -1,8 +1,3 @@ > -obj-y:= > -obj-m:= > -obj-n:= > -obj- := > - > obj-$(CONFIG_ARCH_IOP13XX) += setup.o > obj-$(CONFIG_ARCH_IOP13XX) += irq.o > obj-$(CONFIG_ARCH_IOP13XX) += pci.o > diff --git a/arch/arm/mach-iop32x/Makefile b/arch/arm/mach-iop32x/Makefile > index cfdf8a1..2d4010a 100644 > --- a/arch/arm/mach-iop32x/Makefile > +++ b/arch/arm/mach-iop32x/Makefile > @@ -3,9 +3,6 @@ > # > > obj-y:= irq.o > -obj-m:= > -obj-n:= > -obj- := > > obj-$(CONFIG_MACH_GLANTANK) += glantank.o > obj-$(CONFIG_ARCH_IQ80321) += iq80321.o > diff --git a/arch/arm/mach-iop33x/Makefile b/arch/arm/mach-iop33x/Makefile > index 90081d8..e95db30 100644 > --- a/arch/arm/mach-iop33x/Makefile > +++ b/arch/arm/mach-iop33x/Makefile > @@ -3,9 +3,6 @@ > # > > obj-y:= irq.o uart.o > -obj-m
[PATCH 2/3] kbuild: remove unnecessary variable initializaions
Clearing obj-y, obj-m, obj-n, obj- in each Makefile is a useless habit. They are non-exported variables; therefore they are always empty whenever descending into each subdirectory. (Moreorver, obj-y and obj-m are also set to empty at the beginning of scripts/Makefile.build) Signed-off-by: Masahiro Yamada --- arch/arm/mach-at91/Makefile | 3 --- arch/arm/mach-ebsa110/Makefile| 3 --- arch/arm/mach-ep93xx/Makefile | 3 --- arch/arm/mach-exynos/Makefile | 5 - arch/arm/mach-footbridge/Makefile | 3 --- arch/arm/mach-iop13xx/Makefile| 5 - arch/arm/mach-iop32x/Makefile | 3 --- arch/arm/mach-iop33x/Makefile | 3 --- arch/arm/mach-ks8695/Makefile | 3 --- arch/arm/mach-rpc/Makefile| 4 arch/arm/mach-s3c24xx/Makefile| 5 - arch/arm/mach-s3c64xx/Makefile| 5 - arch/arm/mach-s5pv210/Makefile| 5 - arch/arm/mach-sa1100/Makefile | 3 --- arch/arm/mach-u300/Makefile | 3 --- arch/arm/plat-iop/Makefile| 6 -- arch/arm/plat-omap/Makefile | 3 --- arch/arm/plat-samsung/Makefile| 4 18 files changed, 69 deletions(-) diff --git a/arch/arm/mach-at91/Makefile b/arch/arm/mach-at91/Makefile index 78e9cec..75033839 100644 --- a/arch/arm/mach-at91/Makefile +++ b/arch/arm/mach-at91/Makefile @@ -3,9 +3,6 @@ # obj-y := irq.o gpio.o setup.o sysirq_mask.o -obj-m := -obj-n := -obj- := obj-$(CONFIG_OLD_CLK_AT91) += clock.o obj-$(CONFIG_AT91_SAM9_ALT_RESET) += at91sam9_alt_reset.o diff --git a/arch/arm/mach-ebsa110/Makefile b/arch/arm/mach-ebsa110/Makefile index 935e4af..a7d68c1 100644 --- a/arch/arm/mach-ebsa110/Makefile +++ b/arch/arm/mach-ebsa110/Makefile @@ -5,6 +5,3 @@ # Object file lists. obj-y := core.o io.o leds.o -obj-m := -obj-n := -obj- := diff --git a/arch/arm/mach-ep93xx/Makefile b/arch/arm/mach-ep93xx/Makefile index 0dc51f9..78d427b 100644 --- a/arch/arm/mach-ep93xx/Makefile +++ b/arch/arm/mach-ep93xx/Makefile @@ -2,9 +2,6 @@ # Makefile for the linux kernel. # obj-y := core.o clock.o -obj-m := -obj-n := -obj- := obj-$(CONFIG_EP93XX_DMA) += dma.o diff --git a/arch/arm/mach-exynos/Makefile b/arch/arm/mach-exynos/Makefile index 788f26d..27ae614 100644 --- a/arch/arm/mach-exynos/Makefile +++ b/arch/arm/mach-exynos/Makefile @@ -7,11 +7,6 @@ ccflags-$(CONFIG_ARCH_MULTIPLATFORM) += -I$(srctree)/$(src)/include -I$(srctree)/arch/arm/plat-samsung/include -obj-y := -obj-m := -obj-n := -obj- := - # Core obj-$(CONFIG_ARCH_EXYNOS) += exynos.o pmu.o exynos-smc.o firmware.o diff --git a/arch/arm/mach-footbridge/Makefile b/arch/arm/mach-footbridge/Makefile index c3faa3b..e83d5c8 100644 --- a/arch/arm/mach-footbridge/Makefile +++ b/arch/arm/mach-footbridge/Makefile @@ -5,9 +5,6 @@ # Object file lists. obj-y := common.o dma.o isa-irq.o -obj-m := -obj-n := -obj- := pci-y += dc21285.o pci-$(CONFIG_ARCH_CATS) += cats-pci.o diff --git a/arch/arm/mach-iop13xx/Makefile b/arch/arm/mach-iop13xx/Makefile index cad015f..a3d9260 100644 --- a/arch/arm/mach-iop13xx/Makefile +++ b/arch/arm/mach-iop13xx/Makefile @@ -1,8 +1,3 @@ -obj-y := -obj-m := -obj-n := -obj- := - obj-$(CONFIG_ARCH_IOP13XX) += setup.o obj-$(CONFIG_ARCH_IOP13XX) += irq.o obj-$(CONFIG_ARCH_IOP13XX) += pci.o diff --git a/arch/arm/mach-iop32x/Makefile b/arch/arm/mach-iop32x/Makefile index cfdf8a1..2d4010a 100644 --- a/arch/arm/mach-iop32x/Makefile +++ b/arch/arm/mach-iop32x/Makefile @@ -3,9 +3,6 @@ # obj-y := irq.o -obj-m := -obj-n := -obj- := obj-$(CONFIG_MACH_GLANTANK) += glantank.o obj-$(CONFIG_ARCH_IQ80321) += iq80321.o diff --git a/arch/arm/mach-iop33x/Makefile b/arch/arm/mach-iop33x/Makefile index 90081d8..e95db30 100644 --- a/arch/arm/mach-iop33x/Makefile +++ b/arch/arm/mach-iop33x/Makefile @@ -3,9 +3,6 @@ # obj-y := irq.o uart.o -obj-m := -obj-n := -obj- := obj-$(CONFIG_ARCH_IQ80331) += iq80331.o obj-$(CONFIG_MACH_IQ80332) += iq80332.o diff --git a/arch/arm/mach-ks8695/Makefile b/arch/arm/mach-ks8695/Makefile index e370caf..8ecb797 100644 --- a/arch/arm/mach-ks8695/Makefile +++ b/arch/arm/mach-ks8695/Makefile @@ -4,9 +4,6 @@ # obj-y := cpu.o irq.o time.o devices.o -obj-m := -obj-n := -obj- := # PCI support is optional obj-$(CONFIG_PCI) += pci.o diff --git a/arch/arm/mach-rpc/Makefile b/arch/arm/mach-rpc/Makefile