All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 0/7] Remove unneeded build directory traversals
@ 2017-03-15 16:37 ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

Hello all,

I was building a kernel for x86 and noticed Make still descended into
directories like drivers/gpu/drm/hisilicon, this seems kind of odd given
nothing will be built here. It looks to be due to some directories being
included in obj-y unconditionally instead of only when the relevant
CONFIG_ is set.

These patches are split by subsystem in-case, for some reason, a file in
a directory does need to be built, I believe I have checked for all
instances of this, but a quick review from some maintainers would be nice.

Thanks,
Andrew

Changes from v2:
 - Removed patches that would not work
 - Rebased on v4.11-rc1 (no changes needed)

Changes from v1:
 - Removed patches already taken by maintainers
 - Rebased on v4.10-rc1 (no changes needed)

Andrew F. Davis (7):
  pwm: Only descend into pwm directory when CONFIG_PWM is set
  amba: Only descend into amba directory when CONFIG_ARM_AMBA is set
  NFC: Only descend into nfc directory when CONFIG_NFC is set
  macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS
    is set
  auxdisplay: Only descend into directory when CONFIG_AUXDISPLAY is set
  lguest: Only descend into lguest directory when CONFIG_LGUEST is set
  mmc: Only descend into mmc directory when CONFIG_MMC is set

 drivers/Makefile | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

-- 
2.11.0

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [PATCH v3 0/7] Remove unneeded build directory traversals
@ 2017-03-15 16:37 ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linuxppc-dev, linux-media, linux-wireless, linux-pwm,
	linux-kernel, Andrew F . Davis

Hello all,

I was building a kernel for x86 and noticed Make still descended into
directories like drivers/gpu/drm/hisilicon, this seems kind of odd given
nothing will be built here. It looks to be due to some directories being
included in obj-y unconditionally instead of only when the relevant
CONFIG_ is set.

These patches are split by subsystem in-case, for some reason, a file in
a directory does need to be built, I believe I have checked for all
instances of this, but a quick review from some maintainers would be nice.

Thanks,
Andrew

Changes from v2:
 - Removed patches that would not work
 - Rebased on v4.11-rc1 (no changes needed)

Changes from v1:
 - Removed patches already taken by maintainers
 - Rebased on v4.10-rc1 (no changes needed)

Andrew F. Davis (7):
  pwm: Only descend into pwm directory when CONFIG_PWM is set
  amba: Only descend into amba directory when CONFIG_ARM_AMBA is set
  NFC: Only descend into nfc directory when CONFIG_NFC is set
  macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS
    is set
  auxdisplay: Only descend into directory when CONFIG_AUXDISPLAY is set
  lguest: Only descend into lguest directory when CONFIG_LGUEST is set
  mmc: Only descend into mmc directory when CONFIG_MMC is set

 drivers/Makefile | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

-- 
2.11.0

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [PATCH v3 1/7] pwm: Only descend into pwm directory when CONFIG_PWM is set
  2017-03-15 16:37 ` Andrew F. Davis
@ 2017-03-15 16:37   ` Andrew F. Davis
  -1 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

When CONFIG_PWM is not set make will still descend into the pwm
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 2eced9afba53..00d86749a843 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -13,7 +13,7 @@ obj-$(CONFIG_GENERIC_PHY)	+= phy/
 # GPIO must come after pinctrl as gpios may need to mux pins etc
 obj-$(CONFIG_PINCTRL)		+= pinctrl/
 obj-$(CONFIG_GPIOLIB)		+= gpio/
-obj-y				+= pwm/
+obj-$(CONFIG_PWM)		+= pwm/
 obj-$(CONFIG_PCI)		+= pci/
 # PCI dwc controller drivers
 obj-y				+= pci/dwc/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 1/7] pwm: Only descend into pwm directory when CONFIG_PWM is set
@ 2017-03-15 16:37   ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linuxppc-dev, linux-media, linux-wireless, linux-pwm,
	linux-kernel, Andrew F . Davis

When CONFIG_PWM is not set make will still descend into the pwm
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 2eced9afba53..00d86749a843 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -13,7 +13,7 @@ obj-$(CONFIG_GENERIC_PHY)	+= phy/
 # GPIO must come after pinctrl as gpios may need to mux pins etc
 obj-$(CONFIG_PINCTRL)		+= pinctrl/
 obj-$(CONFIG_GPIOLIB)		+= gpio/
-obj-y				+= pwm/
+obj-$(CONFIG_PWM)		+= pwm/
 obj-$(CONFIG_PCI)		+= pci/
 # PCI dwc controller drivers
 obj-y				+= pci/dwc/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 2/7] amba: Only descend into amba directory when CONFIG_ARM_AMBA is set
  2017-03-15 16:37 ` Andrew F. Davis
@ 2017-03-15 16:37   ` Andrew F. Davis
  -1 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

When CONFIG_ARM_AMBA is not set make will still descend into the amba
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 00d86749a843..23712a92a89a 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -31,7 +31,7 @@ obj-$(CONFIG_SFI)		+= sfi/
 # PnP must come after ACPI since it will eventually need to check if acpi
 # was used and do nothing if so
 obj-$(CONFIG_PNP)		+= pnp/
-obj-y				+= amba/
+obj-$(CONFIG_ARM_AMBA)		+= amba/
 
 obj-y				+= clk/
 # Many drivers will want to use DMA so this has to be made available
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 2/7] amba: Only descend into amba directory when CONFIG_ARM_AMBA is set
@ 2017-03-15 16:37   ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linuxppc-dev, linux-media, linux-wireless, linux-pwm,
	linux-kernel, Andrew F . Davis

When CONFIG_ARM_AMBA is not set make will still descend into the amba
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 00d86749a843..23712a92a89a 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -31,7 +31,7 @@ obj-$(CONFIG_SFI)		+= sfi/
 # PnP must come after ACPI since it will eventually need to check if acpi
 # was used and do nothing if so
 obj-$(CONFIG_PNP)		+= pnp/
-obj-y				+= amba/
+obj-$(CONFIG_ARM_AMBA)		+= amba/
 
 obj-y				+= clk/
 # Many drivers will want to use DMA so this has to be made available
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 3/7] NFC: Only descend into nfc directory when CONFIG_NFC is set
  2017-03-15 16:37 ` Andrew F. Davis
@ 2017-03-15 16:37   ` Andrew F. Davis
  -1 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

When CONFIG_NFC is not set make will still descend into the nfc
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 23712a92a89a..9cf52524ecab 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -69,7 +69,11 @@ obj-$(CONFIG_FB_INTEL)          += video/fbdev/intelfb/
 
 obj-$(CONFIG_PARPORT)		+= parport/
 obj-$(CONFIG_NVM)		+= lightnvm/
-obj-y				+= base/ block/ misc/ mfd/ nfc/
+obj-y				+= base/
+obj-y				+= block/
+obj-y				+= misc/
+obj-y				+= mfd/
+obj-$(CONFIG_NFC)		+= nfc/
 obj-$(CONFIG_LIBNVDIMM)		+= nvdimm/
 obj-$(CONFIG_DEV_DAX)		+= dax/
 obj-$(CONFIG_DMA_SHARED_BUFFER) += dma-buf/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 3/7] NFC: Only descend into nfc directory when CONFIG_NFC is set
@ 2017-03-15 16:37   ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linuxppc-dev, linux-media, linux-wireless, linux-pwm,
	linux-kernel, Andrew F . Davis

When CONFIG_NFC is not set make will still descend into the nfc
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 23712a92a89a..9cf52524ecab 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -69,7 +69,11 @@ obj-$(CONFIG_FB_INTEL)          += video/fbdev/intelfb/
 
 obj-$(CONFIG_PARPORT)		+= parport/
 obj-$(CONFIG_NVM)		+= lightnvm/
-obj-y				+= base/ block/ misc/ mfd/ nfc/
+obj-y				+= base/
+obj-y				+= block/
+obj-y				+= misc/
+obj-y				+= mfd/
+obj-$(CONFIG_NFC)		+= nfc/
 obj-$(CONFIG_LIBNVDIMM)		+= nvdimm/
 obj-$(CONFIG_DEV_DAX)		+= dax/
 obj-$(CONFIG_DMA_SHARED_BUFFER) += dma-buf/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 4/7] macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS is set
  2017-03-15 16:37 ` Andrew F. Davis
@ 2017-03-15 16:37   ` Andrew F. Davis
  -1 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

When CONFIG_MACINTOSH_DRIVERS is not set make will still descend into the
macintosh directory but nothing will be built. This produces unneeded
build artifacts and messages in addition to slowing the build.
Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 9cf52524ecab..cd92491bd76b 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -78,7 +78,7 @@ obj-$(CONFIG_LIBNVDIMM)		+= nvdimm/
 obj-$(CONFIG_DEV_DAX)		+= dax/
 obj-$(CONFIG_DMA_SHARED_BUFFER) += dma-buf/
 obj-$(CONFIG_NUBUS)		+= nubus/
-obj-y				+= macintosh/
+obj-$(CONFIG_MACINTOSH_DRIVERS)	+= macintosh/
 obj-$(CONFIG_IDE)		+= ide/
 obj-$(CONFIG_SCSI)		+= scsi/
 obj-y				+= nvme/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 4/7] macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS is set
@ 2017-03-15 16:37   ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linuxppc-dev, linux-media, linux-wireless, linux-pwm,
	linux-kernel, Andrew F . Davis

When CONFIG_MACINTOSH_DRIVERS is not set make will still descend into the
macintosh directory but nothing will be built. This produces unneeded
build artifacts and messages in addition to slowing the build.
Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 9cf52524ecab..cd92491bd76b 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -78,7 +78,7 @@ obj-$(CONFIG_LIBNVDIMM)		+= nvdimm/
 obj-$(CONFIG_DEV_DAX)		+= dax/
 obj-$(CONFIG_DMA_SHARED_BUFFER) += dma-buf/
 obj-$(CONFIG_NUBUS)		+= nubus/
-obj-y				+= macintosh/
+obj-$(CONFIG_MACINTOSH_DRIVERS)	+= macintosh/
 obj-$(CONFIG_IDE)		+= ide/
 obj-$(CONFIG_SCSI)		+= scsi/
 obj-y				+= nvme/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 5/7] auxdisplay: Only descend into directory when CONFIG_AUXDISPLAY is set
  2017-03-15 16:37 ` Andrew F. Davis
@ 2017-03-15 16:37   ` Andrew F. Davis
  -1 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

When CONFIG_AUXDISPLAY is not set make will still descend into the
auxdisplay directory but nothing will be built. This produces unneeded
build artifacts and messages in addition to slowing the build.
Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index cd92491bd76b..8a9ed5c59778 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -95,7 +95,7 @@ obj-y				+= firewire/
 obj-$(CONFIG_UIO)		+= uio/
 obj-$(CONFIG_VFIO)		+= vfio/
 obj-y				+= cdrom/
-obj-y				+= auxdisplay/
+obj-$(CONFIG_AUXDISPLAY)	+= auxdisplay/
 obj-$(CONFIG_PCCARD)		+= pcmcia/
 obj-$(CONFIG_DIO)		+= dio/
 obj-$(CONFIG_SBUS)		+= sbus/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 5/7] auxdisplay: Only descend into directory when CONFIG_AUXDISPLAY is set
@ 2017-03-15 16:37   ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linuxppc-dev, linux-media, linux-wireless, linux-pwm,
	linux-kernel, Andrew F . Davis

When CONFIG_AUXDISPLAY is not set make will still descend into the
auxdisplay directory but nothing will be built. This produces unneeded
build artifacts and messages in addition to slowing the build.
Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index cd92491bd76b..8a9ed5c59778 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -95,7 +95,7 @@ obj-y				+= firewire/
 obj-$(CONFIG_UIO)		+= uio/
 obj-$(CONFIG_VFIO)		+= vfio/
 obj-y				+= cdrom/
-obj-y				+= auxdisplay/
+obj-$(CONFIG_AUXDISPLAY)	+= auxdisplay/
 obj-$(CONFIG_PCCARD)		+= pcmcia/
 obj-$(CONFIG_DIO)		+= dio/
 obj-$(CONFIG_SBUS)		+= sbus/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 6/7] lguest: Only descend into lguest directory when CONFIG_LGUEST is set
  2017-03-15 16:37 ` Andrew F. Davis
@ 2017-03-15 16:37   ` Andrew F. Davis
  -1 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

When CONFIG_LGUEST is not set make will still descend into the lguest
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 8a9ed5c59778..3d758020d248 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -126,7 +126,7 @@ obj-$(CONFIG_ACCESSIBILITY)	+= accessibility/
 obj-$(CONFIG_ISDN)		+= isdn/
 obj-$(CONFIG_EDAC)		+= edac/
 obj-$(CONFIG_EISA)		+= eisa/
-obj-y				+= lguest/
+obj-$(CONFIG_LGUEST)		+= lguest/
 obj-$(CONFIG_CPU_FREQ)		+= cpufreq/
 obj-$(CONFIG_CPU_IDLE)		+= cpuidle/
 obj-y				+= mmc/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 6/7] lguest: Only descend into lguest directory when CONFIG_LGUEST is set
@ 2017-03-15 16:37   ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linuxppc-dev, linux-media, linux-wireless, linux-pwm,
	linux-kernel, Andrew F . Davis

When CONFIG_LGUEST is not set make will still descend into the lguest
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 8a9ed5c59778..3d758020d248 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -126,7 +126,7 @@ obj-$(CONFIG_ACCESSIBILITY)	+= accessibility/
 obj-$(CONFIG_ISDN)		+= isdn/
 obj-$(CONFIG_EDAC)		+= edac/
 obj-$(CONFIG_EISA)		+= eisa/
-obj-y				+= lguest/
+obj-$(CONFIG_LGUEST)		+= lguest/
 obj-$(CONFIG_CPU_FREQ)		+= cpufreq/
 obj-$(CONFIG_CPU_IDLE)		+= cpuidle/
 obj-y				+= mmc/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 7/7] mmc: Only descend into mmc directory when CONFIG_MMC is set
  2017-03-15 16:37 ` Andrew F. Davis
@ 2017-03-15 16:37   ` Andrew F. Davis
  -1 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

When CONFIG_MMC is not set make will still descend into the mmc
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 3d758020d248..03da7a38e989 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -129,7 +129,7 @@ obj-$(CONFIG_EISA)		+= eisa/
 obj-$(CONFIG_LGUEST)		+= lguest/
 obj-$(CONFIG_CPU_FREQ)		+= cpufreq/
 obj-$(CONFIG_CPU_IDLE)		+= cpuidle/
-obj-y				+= mmc/
+obj-$(CONFIG_MMC)		+= mmc/
 obj-$(CONFIG_MEMSTICK)		+= memstick/
 obj-$(CONFIG_NEW_LEDS)		+= leds/
 obj-$(CONFIG_INFINIBAND)	+= infiniband/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 7/7] mmc: Only descend into mmc directory when CONFIG_MMC is set
@ 2017-03-15 16:37   ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 16:37 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linuxppc-dev, linux-media, linux-wireless, linux-pwm,
	linux-kernel, Andrew F . Davis

When CONFIG_MMC is not set make will still descend into the mmc
directory but nothing will be built. This produces unneeded build
artifacts and messages in addition to slowing the build. Fix this here.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 drivers/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/Makefile b/drivers/Makefile
index 3d758020d248..03da7a38e989 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -129,7 +129,7 @@ obj-$(CONFIG_EISA)		+= eisa/
 obj-$(CONFIG_LGUEST)		+= lguest/
 obj-$(CONFIG_CPU_FREQ)		+= cpufreq/
 obj-$(CONFIG_CPU_IDLE)		+= cpuidle/
-obj-y				+= mmc/
+obj-$(CONFIG_MMC)		+= mmc/
 obj-$(CONFIG_MEMSTICK)		+= memstick/
 obj-$(CONFIG_NEW_LEDS)		+= leds/
 obj-$(CONFIG_INFINIBAND)	+= infiniband/
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 0/7] Remove unneeded build directory traversals
  2017-03-15 16:37 ` Andrew F. Davis
@ 2017-03-15 21:03   ` Arnd Bergmann
  -1 siblings, 0 replies; 32+ messages in thread
From: Arnd Bergmann @ 2017-03-15 21:03 UTC (permalink / raw)
  To: Andrew F. Davis
  Cc: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless,
	Linux Kernel Mailing List, linuxppc-dev, linux-media

On Wed, Mar 15, 2017 at 5:37 PM, Andrew F. Davis <afd@ti.com> wrote:
> Hello all,
>
> I was building a kernel for x86 and noticed Make still descended into
> directories like drivers/gpu/drm/hisilicon, this seems kind of odd given
> nothing will be built here. It looks to be due to some directories being
> included in obj-y unconditionally instead of only when the relevant
> CONFIG_ is set.
>
> These patches are split by subsystem in-case, for some reason, a file in
> a directory does need to be built, I believe I have checked for all
> instances of this, but a quick review from some maintainers would be nice.

I didn't see anything wrong with the patches, and made sure that there
are no tristate symbols controlling the subdirectory for anything that
requires a built-in driver (which would cause a link failure).

I'm not sure about drivers/lguest, which has some special magic
in its Makefile, it's possible that this now fails with CONFIG_LGUEST=m.

      Arnd

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 0/7] Remove unneeded build directory traversals
@ 2017-03-15 21:03   ` Arnd Bergmann
  0 siblings, 0 replies; 32+ messages in thread
From: Arnd Bergmann @ 2017-03-15 21:03 UTC (permalink / raw)
  To: Andrew F. Davis
  Cc: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless,
	Linux Kernel Mailing List, linuxppc-dev, linux-media

On Wed, Mar 15, 2017 at 5:37 PM, Andrew F. Davis <afd@ti.com> wrote:
> Hello all,
>
> I was building a kernel for x86 and noticed Make still descended into
> directories like drivers/gpu/drm/hisilicon, this seems kind of odd given
> nothing will be built here. It looks to be due to some directories being
> included in obj-y unconditionally instead of only when the relevant
> CONFIG_ is set.
>
> These patches are split by subsystem in-case, for some reason, a file in
> a directory does need to be built, I believe I have checked for all
> instances of this, but a quick review from some maintainers would be nice.

I didn't see anything wrong with the patches, and made sure that there
are no tristate symbols controlling the subdirectory for anything that
requires a built-in driver (which would cause a link failure).

I'm not sure about drivers/lguest, which has some special magic
in its Makefile, it's possible that this now fails with CONFIG_LGUEST=m.

      Arnd

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 0/7] Remove unneeded build directory traversals
  2017-03-15 21:03   ` Arnd Bergmann
  (?)
@ 2017-03-15 21:15     ` Andrew F. Davis
  -1 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 21:15 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless,
	Linux Kernel Mailing List, linuxppc-dev, linux-media

On 03/15/2017 04:03 PM, Arnd Bergmann wrote:
> On Wed, Mar 15, 2017 at 5:37 PM, Andrew F. Davis <afd@ti.com> wrote:
>> Hello all,
>>
>> I was building a kernel for x86 and noticed Make still descended into
>> directories like drivers/gpu/drm/hisilicon, this seems kind of odd given
>> nothing will be built here. It looks to be due to some directories being
>> included in obj-y unconditionally instead of only when the relevant
>> CONFIG_ is set.
>>
>> These patches are split by subsystem in-case, for some reason, a file in
>> a directory does need to be built, I believe I have checked for all
>> instances of this, but a quick review from some maintainers would be nice.
> 
> I didn't see anything wrong with the patches, and made sure that there
> are no tristate symbols controlling the subdirectory for anything that
> requires a built-in driver (which would cause a link failure).
> 
> I'm not sure about drivers/lguest, which has some special magic
> in its Makefile, it's possible that this now fails with CONFIG_LGUEST=m.
> 

lguest and mmc are the strange ones, so I put them last in the series in
case they did need to be dropped.

lguest was supposed to have been taken from v1:
https://lkml.org/lkml/2016/6/20/1086
but it looks like it didn't so I re-introduced it for v3.

mmc caught some 0-day build warnings but I never got to the bottom of them.

Anyway, I have no problem with these two being held back until the magic
in their Makefile is sorted out.

Thanks,
Andrew

>       Arnd
> 

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 0/7] Remove unneeded build directory traversals
@ 2017-03-15 21:15     ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 21:15 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless,
	Linux Kernel Mailing List, linuxppc-dev, linux-media

On 03/15/2017 04:03 PM, Arnd Bergmann wrote:
> On Wed, Mar 15, 2017 at 5:37 PM, Andrew F. Davis <afd@ti.com> wrote:
>> Hello all,
>>
>> I was building a kernel for x86 and noticed Make still descended into
>> directories like drivers/gpu/drm/hisilicon, this seems kind of odd given
>> nothing will be built here. It looks to be due to some directories being
>> included in obj-y unconditionally instead of only when the relevant
>> CONFIG_ is set.
>>
>> These patches are split by subsystem in-case, for some reason, a file in
>> a directory does need to be built, I believe I have checked for all
>> instances of this, but a quick review from some maintainers would be nice.
> 
> I didn't see anything wrong with the patches, and made sure that there
> are no tristate symbols controlling the subdirectory for anything that
> requires a built-in driver (which would cause a link failure).
> 
> I'm not sure about drivers/lguest, which has some special magic
> in its Makefile, it's possible that this now fails with CONFIG_LGUEST=m.
> 

lguest and mmc are the strange ones, so I put them last in the series in
case they did need to be dropped.

lguest was supposed to have been taken from v1:
https://lkml.org/lkml/2016/6/20/1086
but it looks like it didn't so I re-introduced it for v3.

mmc caught some 0-day build warnings but I never got to the bottom of them.

Anyway, I have no problem with these two being held back until the magic
in their Makefile is sorted out.

Thanks,
Andrew

>       Arnd
> 

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 0/7] Remove unneeded build directory traversals
@ 2017-03-15 21:15     ` Andrew F. Davis
  0 siblings, 0 replies; 32+ messages in thread
From: Andrew F. Davis @ 2017-03-15 21:15 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless,
	Linux Kernel Mailing List, linuxppc-dev, linux-media

On 03/15/2017 04:03 PM, Arnd Bergmann wrote:
> On Wed, Mar 15, 2017 at 5:37 PM, Andrew F. Davis <afd@ti.com> wrote:
>> Hello all,
>>
>> I was building a kernel for x86 and noticed Make still descended into
>> directories like drivers/gpu/drm/hisilicon, this seems kind of odd given
>> nothing will be built here. It looks to be due to some directories being
>> included in obj-y unconditionally instead of only when the relevant
>> CONFIG_ is set.
>>
>> These patches are split by subsystem in-case, for some reason, a file in
>> a directory does need to be built, I believe I have checked for all
>> instances of this, but a quick review from some maintainers would be nice.
> 
> I didn't see anything wrong with the patches, and made sure that there
> are no tristate symbols controlling the subdirectory for anything that
> requires a built-in driver (which would cause a link failure).
> 
> I'm not sure about drivers/lguest, which has some special magic
> in its Makefile, it's possible that this now fails with CONFIG_LGUEST=m.
> 

lguest and mmc are the strange ones, so I put them last in the series in
case they did need to be dropped.

lguest was supposed to have been taken from v1:
https://lkml.org/lkml/2016/6/20/1086
but it looks like it didn't so I re-introduced it for v3.

mmc caught some 0-day build warnings but I never got to the bottom of them.

Anyway, I have no problem with these two being held back until the magic
in their Makefile is sorted out.

Thanks,
Andrew

>       Arnd
> 

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 0/7] Remove unneeded build directory traversals
  2017-03-15 21:15     ` Andrew F. Davis
@ 2017-03-15 21:34       ` Arnd Bergmann
  -1 siblings, 0 replies; 32+ messages in thread
From: Arnd Bergmann @ 2017-03-15 21:34 UTC (permalink / raw)
  To: Andrew F. Davis
  Cc: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless,
	Linux Kernel Mailing List, linuxppc-dev, linux-media

On Wed, Mar 15, 2017 at 10:15 PM, Andrew F. Davis <afd@ti.com> wrote:
> On 03/15/2017 04:03 PM, Arnd Bergmann wrote:
>> On Wed, Mar 15, 2017 at 5:37 PM, Andrew F. Davis <afd@ti.com> wrote:
>>> Hello all,
>>>
>>> I was building a kernel for x86 and noticed Make still descended into
>>> directories like drivers/gpu/drm/hisilicon, this seems kind of odd given
>>> nothing will be built here. It looks to be due to some directories being
>>> included in obj-y unconditionally instead of only when the relevant
>>> CONFIG_ is set.
>>>
>>> These patches are split by subsystem in-case, for some reason, a file in
>>> a directory does need to be built, I believe I have checked for all
>>> instances of this, but a quick review from some maintainers would be nice.
>>
>> I didn't see anything wrong with the patches, and made sure that there
>> are no tristate symbols controlling the subdirectory for anything that
>> requires a built-in driver (which would cause a link failure).
>>
>> I'm not sure about drivers/lguest, which has some special magic
>> in its Makefile, it's possible that this now fails with CONFIG_LGUEST=m.
>>
>
> lguest and mmc are the strange ones, so I put them last in the series in
> case they did need to be dropped.
>
> lguest was supposed to have been taken from v1:
> https://lkml.org/lkml/2016/6/20/1086
> but it looks like it didn't so I re-introduced it for v3.
>
> mmc caught some 0-day build warnings but I never got to the bottom of them.

Ah, I see now what happened to mmc:

obj-$(subst m,y,$(CONFIG_MMC))  += host/
tmio_mmc_core-$(subst m,y,$(CONFIG_MMC_SDHI))   += tmio_mmc_dma.o
obj-$(subst m,y,$(CONFIG_MMC_SDHCI_PCI))        += sdhci-pci-data.o

with CONFIG_MMC=m, this will fail to build the built-in files in
drivers/mmc/host. I suppose this could be expressed in a different
way these days, but dropping the patch would be easier.

     Arnd

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 0/7] Remove unneeded build directory traversals
@ 2017-03-15 21:34       ` Arnd Bergmann
  0 siblings, 0 replies; 32+ messages in thread
From: Arnd Bergmann @ 2017-03-15 21:34 UTC (permalink / raw)
  To: Andrew F. Davis
  Cc: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless,
	Linux Kernel Mailing List, linuxppc-dev, linux-media

On Wed, Mar 15, 2017 at 10:15 PM, Andrew F. Davis <afd@ti.com> wrote:
> On 03/15/2017 04:03 PM, Arnd Bergmann wrote:
>> On Wed, Mar 15, 2017 at 5:37 PM, Andrew F. Davis <afd@ti.com> wrote:
>>> Hello all,
>>>
>>> I was building a kernel for x86 and noticed Make still descended into
>>> directories like drivers/gpu/drm/hisilicon, this seems kind of odd given
>>> nothing will be built here. It looks to be due to some directories being
>>> included in obj-y unconditionally instead of only when the relevant
>>> CONFIG_ is set.
>>>
>>> These patches are split by subsystem in-case, for some reason, a file in
>>> a directory does need to be built, I believe I have checked for all
>>> instances of this, but a quick review from some maintainers would be nice.
>>
>> I didn't see anything wrong with the patches, and made sure that there
>> are no tristate symbols controlling the subdirectory for anything that
>> requires a built-in driver (which would cause a link failure).
>>
>> I'm not sure about drivers/lguest, which has some special magic
>> in its Makefile, it's possible that this now fails with CONFIG_LGUEST=m.
>>
>
> lguest and mmc are the strange ones, so I put them last in the series in
> case they did need to be dropped.
>
> lguest was supposed to have been taken from v1:
> https://lkml.org/lkml/2016/6/20/1086
> but it looks like it didn't so I re-introduced it for v3.
>
> mmc caught some 0-day build warnings but I never got to the bottom of them.

Ah, I see now what happened to mmc:

obj-$(subst m,y,$(CONFIG_MMC))  += host/
tmio_mmc_core-$(subst m,y,$(CONFIG_MMC_SDHI))   += tmio_mmc_dma.o
obj-$(subst m,y,$(CONFIG_MMC_SDHCI_PCI))        += sdhci-pci-data.o

with CONFIG_MMC=m, this will fail to build the built-in files in
drivers/mmc/host. I suppose this could be expressed in a different
way these days, but dropping the patch would be easier.

     Arnd

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 4/7] macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS is set
  2017-03-15 16:37   ` Andrew F. Davis
  (?)
@ 2017-03-16  4:28     ` Michael Ellerman
  -1 siblings, 0 replies; 32+ messages in thread
From: Michael Ellerman @ 2017-03-16  4:28 UTC (permalink / raw)
  To: Andrew F. Davis, Miguel Ojeda Sandonis, Wolfram Sang,
	Richard Purdie, Benjamin Herrenschmidt, Mauro Carvalho Chehab,
	Ulf Hansson, Lauro Ramos Venancio, Aloisio Almeida Jr,
	Samuel Ortiz, kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

"Andrew F. Davis" <afd@ti.com> writes:

> When CONFIG_MACINTOSH_DRIVERS is not set make will still descend into the
> macintosh directory but nothing will be built. This produces unneeded
> build artifacts and messages in addition to slowing the build.
> Fix this here.
>
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> ---
>  drivers/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

LGTM.

Acked-by: Michael Ellerman <mpe@ellerman.id.au>

cheersj

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 4/7] macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS is set
@ 2017-03-16  4:28     ` Michael Ellerman
  0 siblings, 0 replies; 32+ messages in thread
From: Michael Ellerman @ 2017-03-16  4:28 UTC (permalink / raw)
  To: Andrew F. Davis, Miguel Ojeda Sandonis, Wolfram Sang,
	Richard Purdie, Benjamin Herrenschmidt, Mauro Carvalho Chehab,
	Ulf Hansson, Lauro Ramos Venancio, Aloisio Almeida Jr,
	Samuel Ortiz, kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, linuxppc-dev, linux-media

"Andrew F. Davis" <afd@ti.com> writes:

> When CONFIG_MACINTOSH_DRIVERS is not set make will still descend into the
> macintosh directory but nothing will be built. This produces unneeded
> build artifacts and messages in addition to slowing the build.
> Fix this here.
>
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> ---
>  drivers/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

LGTM.

Acked-by: Michael Ellerman <mpe@ellerman.id.au>

cheersj

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 4/7] macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS is set
@ 2017-03-16  4:28     ` Michael Ellerman
  0 siblings, 0 replies; 32+ messages in thread
From: Michael Ellerman @ 2017-03-16  4:28 UTC (permalink / raw)
  To: Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors
  Cc: linux-pwm, linux-wireless, linux-kernel, Andrew F . Davis,
	linuxppc-dev, linux-media

"Andrew F. Davis" <afd@ti.com> writes:

> When CONFIG_MACINTOSH_DRIVERS is not set make will still descend into the
> macintosh directory but nothing will be built. This produces unneeded
> build artifacts and messages in addition to slowing the build.
> Fix this here.
>
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> ---
>  drivers/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

LGTM.

Acked-by: Michael Ellerman <mpe@ellerman.id.au>

cheersj

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 7/7] mmc: Only descend into mmc directory when CONFIG_MMC is set
@ 2017-03-18  1:06     ` kbuild test robot
  0 siblings, 0 replies; 32+ messages in thread
From: kbuild test robot @ 2017-03-18  1:06 UTC (permalink / raw)
  To: Andrew F. Davis
  Cc: kbuild-all, Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless, linux-kernel,
	Andrew F . Davis, linuxppc-dev, linux-media

[-- Attachment #1: Type: text/plain, Size: 868 bytes --]

Hi Andrew,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.11-rc2 next-20170310]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Andrew-F-Davis/Remove-unneeded-build-directory-traversals/20170318-040719
config: x86_64-rhel (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

>> ERROR: "sdhci_pci_get_data" [drivers/mmc/host/sdhci-pci.ko] undefined!
>> ERROR: "sdhci_pci_spt_drive_strength" [drivers/mmc/host/sdhci-pci.ko] undefined!

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 38835 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 7/7] mmc: Only descend into mmc directory when CONFIG_MMC is set
@ 2017-03-18  1:06     ` kbuild test robot
  0 siblings, 0 replies; 32+ messages in thread
From: kbuild test robot @ 2017-03-18  1:06 UTC (permalink / raw)
  To: kernel-janitors

[-- Attachment #1: Type: text/plain, Size: 868 bytes --]

Hi Andrew,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.11-rc2 next-20170310]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Andrew-F-Davis/Remove-unneeded-build-directory-traversals/20170318-040719
config: x86_64-rhel (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

>> ERROR: "sdhci_pci_get_data" [drivers/mmc/host/sdhci-pci.ko] undefined!
>> ERROR: "sdhci_pci_spt_drive_strength" [drivers/mmc/host/sdhci-pci.ko] undefined!

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 38835 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 7/7] mmc: Only descend into mmc directory when CONFIG_MMC is set
@ 2017-03-18  1:06     ` kbuild test robot
  0 siblings, 0 replies; 32+ messages in thread
From: kbuild test robot @ 2017-03-18  1:06 UTC (permalink / raw)
  Cc: kbuild-all, Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless, linux-kernel,
	Andrew F . Davis, linuxppc-dev, linux-media

[-- Attachment #1: Type: text/plain, Size: 868 bytes --]

Hi Andrew,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.11-rc2 next-20170310]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Andrew-F-Davis/Remove-unneeded-build-directory-traversals/20170318-040719
config: x86_64-rhel (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

>> ERROR: "sdhci_pci_get_data" [drivers/mmc/host/sdhci-pci.ko] undefined!
>> ERROR: "sdhci_pci_spt_drive_strength" [drivers/mmc/host/sdhci-pci.ko] undefined!

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 38835 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 7/7] mmc: Only descend into mmc directory when CONFIG_MMC is set
  2017-03-18  1:06     ` kbuild test robot
  (?)
@ 2017-03-18  2:07     ` kbuild test robot
  -1 siblings, 0 replies; 32+ messages in thread
From: kbuild test robot @ 2017-03-18  2:07 UTC (permalink / raw)
  To: Andrew F. Davis
  Cc: kbuild-all, Miguel Ojeda Sandonis, Wolfram Sang, Richard Purdie,
	Benjamin Herrenschmidt, Mauro Carvalho Chehab, Ulf Hansson,
	Lauro Ramos Venancio, Aloisio Almeida Jr, Samuel Ortiz,
	kernel-janitors, linux-pwm, linux-wireless, linux-kernel,
	Andrew F . Davis, linuxppc-dev, linux-media

[-- Attachment #1: Type: text/plain, Size: 949 bytes --]

Hi Andrew,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.11-rc2 next-20170310]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Andrew-F-Davis/Remove-unneeded-build-directory-traversals/20170318-040719
config: x86_64-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   arch/x86/built-in.o: In function `mrfld_sd_init':
>> platform_mrfld_sd.c:(.init.text+0x397d5): undefined reference to `sdhci_pci_get_data'
   platform_mrfld_sd.c:(.init.text+0x397e3): undefined reference to `sdhci_pci_get_data'

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 58848 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 7/7] mmc: Only descend into mmc directory when CONFIG_MMC is set
@ 2017-03-18  2:07     ` kbuild test robot
  0 siblings, 0 replies; 32+ messages in thread
From: kbuild test robot @ 2017-03-18  2:07 UTC (permalink / raw)
  To: kernel-janitors

[-- Attachment #1: Type: text/plain, Size: 949 bytes --]

Hi Andrew,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.11-rc2 next-20170310]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Andrew-F-Davis/Remove-unneeded-build-directory-traversals/20170318-040719
config: x86_64-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   arch/x86/built-in.o: In function `mrfld_sd_init':
>> platform_mrfld_sd.c:(.init.text+0x397d5): undefined reference to `sdhci_pci_get_data'
   platform_mrfld_sd.c:(.init.text+0x397e3): undefined reference to `sdhci_pci_get_data'

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 58848 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 7/7] mmc: Only descend into mmc directory when CONFIG_MMC is set
@ 2017-03-18  2:07     ` kbuild test robot
  0 siblings, 0 replies; 32+ messages in thread
From: kbuild test robot @ 2017-03-18  2:07 UTC (permalink / raw)
  Cc: linux-pwm, Ulf Hansson, Samuel Ortiz, Mauro Carvalho Chehab,
	Wolfram Sang, kernel-janitors, linux-kernel,
	Lauro Ramos Venancio, Andrew F . Davis, Miguel Ojeda Sandonis,
	Richard Purdie, kbuild-all, linux-wireless, linuxppc-dev,
	Aloisio Almeida Jr, linux-media

[-- Attachment #1: Type: text/plain, Size: 949 bytes --]

Hi Andrew,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.11-rc2 next-20170310]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Andrew-F-Davis/Remove-unneeded-build-directory-traversals/20170318-040719
config: x86_64-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   arch/x86/built-in.o: In function `mrfld_sd_init':
>> platform_mrfld_sd.c:(.init.text+0x397d5): undefined reference to `sdhci_pci_get_data'
   platform_mrfld_sd.c:(.init.text+0x397e3): undefined reference to `sdhci_pci_get_data'

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 58848 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

end of thread, other threads:[~2017-03-18  2:07 UTC | newest]

Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-15 16:37 [PATCH v3 0/7] Remove unneeded build directory traversals Andrew F. Davis
2017-03-15 16:37 ` Andrew F. Davis
2017-03-15 16:37 ` [PATCH v3 1/7] pwm: Only descend into pwm directory when CONFIG_PWM is set Andrew F. Davis
2017-03-15 16:37   ` Andrew F. Davis
2017-03-15 16:37 ` [PATCH v3 2/7] amba: Only descend into amba directory when CONFIG_ARM_AMBA " Andrew F. Davis
2017-03-15 16:37   ` Andrew F. Davis
2017-03-15 16:37 ` [PATCH v3 3/7] NFC: Only descend into nfc directory when CONFIG_NFC " Andrew F. Davis
2017-03-15 16:37   ` Andrew F. Davis
2017-03-15 16:37 ` [PATCH v3 4/7] macintosh: Only descend into directory when CONFIG_MACINTOSH_DRIVERS " Andrew F. Davis
2017-03-15 16:37   ` Andrew F. Davis
2017-03-16  4:28   ` Michael Ellerman
2017-03-16  4:28     ` Michael Ellerman
2017-03-16  4:28     ` Michael Ellerman
2017-03-15 16:37 ` [PATCH v3 5/7] auxdisplay: Only descend into directory when CONFIG_AUXDISPLAY " Andrew F. Davis
2017-03-15 16:37   ` Andrew F. Davis
2017-03-15 16:37 ` [PATCH v3 6/7] lguest: Only descend into lguest directory when CONFIG_LGUEST " Andrew F. Davis
2017-03-15 16:37   ` Andrew F. Davis
2017-03-15 16:37 ` [PATCH v3 7/7] mmc: Only descend into mmc directory when CONFIG_MMC " Andrew F. Davis
2017-03-15 16:37   ` Andrew F. Davis
2017-03-18  1:06   ` kbuild test robot
2017-03-18  1:06     ` kbuild test robot
2017-03-18  1:06     ` kbuild test robot
2017-03-18  2:07   ` kbuild test robot
2017-03-18  2:07     ` kbuild test robot
2017-03-18  2:07     ` kbuild test robot
2017-03-15 21:03 ` [PATCH v3 0/7] Remove unneeded build directory traversals Arnd Bergmann
2017-03-15 21:03   ` Arnd Bergmann
2017-03-15 21:15   ` Andrew F. Davis
2017-03-15 21:15     ` Andrew F. Davis
2017-03-15 21:15     ` Andrew F. Davis
2017-03-15 21:34     ` Arnd Bergmann
2017-03-15 21:34       ` Arnd Bergmann

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.