LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile.
@ 2011-04-05 16:58 Justin P. Mattock
  2011-04-05 16:58 ` [RFC 2/5]arch:powerpc:sysdev:Makefile " Justin P. Mattock
                   ` (4 more replies)
  0 siblings, 5 replies; 16+ messages in thread
From: Justin P. Mattock @ 2011-04-05 16:58 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock, Russell King, linux-arm-kernel

The patch below removes an unused config variable found by using a kernel
cleanup script.
Note: I did try to cross compile these but hit erros while doing so..
(gcc is not setup to cross compile) and am unsure if anymore needs to be done.
Please have a look if/when anybody has free time.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
CC: Russell King <linux@arm.linux.org.uk>
CC: linux-arm-kernel@lists.infradead.org

---
 arch/arm/common/Makefile |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile
index e7521bca..6ea9b6f 100644
--- a/arch/arm/common/Makefile
+++ b/arch/arm/common/Makefile
@@ -16,5 +16,4 @@ obj-$(CONFIG_SHARP_SCOOP)	+= scoop.o
 obj-$(CONFIG_ARCH_IXP2000)	+= uengine.o
 obj-$(CONFIG_ARCH_IXP23XX)	+= uengine.o
 obj-$(CONFIG_PCI_HOST_ITE8152)  += it8152.o
-obj-$(CONFIG_COMMON_CLKDEV)	+= clkdev.o
 obj-$(CONFIG_ARM_TIMER_SP804)	+= timer-sp.o
-- 
1.7.4.2


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

* [RFC 2/5]arch:powerpc:sysdev:Makefile Remove unused config in the Makefile.
  2011-04-05 16:58 [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile Justin P. Mattock
@ 2011-04-05 16:58 ` Justin P. Mattock
  2011-04-05 18:15   ` Scott Wood
  2011-04-05 16:58 ` [RFC 3/5]arch:mips:pmc-sierra:msp71xx:Makefile " Justin P. Mattock
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2011-04-05 16:58 UTC (permalink / raw)
  To: trivial
  Cc: linux-kernel, Justin P. Mattock, Benjamin Herrenschmidt, linuxppc-dev

The patch below removes an unused config variable found by using a kernel
cleanup script.
Note: I did try to cross compile these but hit erros while doing so..
(gcc is not setup to cross compile) and am unsure if anymore needs to be done.
Please have a look if/when anybody has free time.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
CC: Benjamin Herrenschmidt <benh@kernel.crashing.org>
CC: linuxppc-dev@lists.ozlabs.org

---
 arch/powerpc/sysdev/Makefile |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/powerpc/sysdev/Makefile b/arch/powerpc/sysdev/Makefile
index 1e0c933..243b6ad 100644
--- a/arch/powerpc/sysdev/Makefile
+++ b/arch/powerpc/sysdev/Makefile
@@ -18,7 +18,6 @@ obj-$(CONFIG_FSL_PMC)		+= fsl_pmc.o
 obj-$(CONFIG_FSL_LBC)		+= fsl_lbc.o
 obj-$(CONFIG_FSL_GTM)		+= fsl_gtm.o
 obj-$(CONFIG_MPC8xxx_GPIO)	+= mpc8xxx_gpio.o
-obj-$(CONFIG_FSL_85XX_CACHE_SRAM)	+= fsl_85xx_l2ctlr.o fsl_85xx_cache_sram.o
 obj-$(CONFIG_SIMPLE_GPIO)	+= simple_gpio.o
 obj-$(CONFIG_FSL_RIO)		+= fsl_rio.o
 obj-$(CONFIG_TSI108_BRIDGE)	+= tsi108_pci.o tsi108_dev.o
-- 
1.7.4.2


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

* [RFC 3/5]arch:mips:pmc-sierra:msp71xx:Makefile Remove unused config in the Makefile.
  2011-04-05 16:58 [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile Justin P. Mattock
  2011-04-05 16:58 ` [RFC 2/5]arch:powerpc:sysdev:Makefile " Justin P. Mattock
@ 2011-04-05 16:58 ` Justin P. Mattock
  2011-04-08 13:08   ` Ralf Baechle
  2011-04-05 16:58 ` [RFC 4/5]arch:mn10300:kernel:Makefile " Justin P. Mattock
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2011-04-05 16:58 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock, Ralf Baechle, linux-mips

The patch below removes an unused config variable found by using a kernel
cleanup script.
Note: I did try to cross compile these but hit erros while doing so..
(gcc is not setup to cross compile) and am unsure if anymore needs to be done.
Please have a look if/when anybody has free time.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
CC: Ralf Baechle <ralf@linux-mips.org>
CC: linux-mips@linux-mips.org

---
 arch/mips/pmc-sierra/msp71xx/Makefile |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/mips/pmc-sierra/msp71xx/Makefile b/arch/mips/pmc-sierra/msp71xx/Makefile
index cefba77..9201c8b 100644
--- a/arch/mips/pmc-sierra/msp71xx/Makefile
+++ b/arch/mips/pmc-sierra/msp71xx/Makefile
@@ -3,7 +3,6 @@
 #
 obj-y += msp_prom.o msp_setup.o msp_irq.o \
 	 msp_time.o msp_serial.o msp_elb.o
-obj-$(CONFIG_HAVE_GPIO_LIB) += gpio.o gpio_extended.o
 obj-$(CONFIG_PMC_MSP7120_GW) += msp_hwbutton.o
 obj-$(CONFIG_IRQ_MSP_SLP) += msp_irq_slp.o
 obj-$(CONFIG_IRQ_MSP_CIC) += msp_irq_cic.o msp_irq_per.o
-- 
1.7.4.2


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

* [RFC 4/5]arch:mn10300:kernel:Makefile Remove unused config in the Makefile.
  2011-04-05 16:58 [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile Justin P. Mattock
  2011-04-05 16:58 ` [RFC 2/5]arch:powerpc:sysdev:Makefile " Justin P. Mattock
  2011-04-05 16:58 ` [RFC 3/5]arch:mips:pmc-sierra:msp71xx:Makefile " Justin P. Mattock
@ 2011-04-05 16:58 ` Justin P. Mattock
  2011-04-05 16:58 ` [RFC 5/5]arch:powerpc:kernel:Makefile " Justin P. Mattock
  2011-04-06  8:05 ` [RFC 1/5]arch/arm/common/Makefile " Uwe Kleine-König
  4 siblings, 0 replies; 16+ messages in thread
From: Justin P. Mattock @ 2011-04-05 16:58 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, Justin P. Mattock, David Howells, linux-am33-list

The patch below removes an unused config variable found by using a kernel
cleanup script.
Note: I did try to cross compile these but hit erros while doing so..
(gcc is not setup to cross compile) and am unsure if anymore needs to be done.
Please have a look if/when anybody has free time.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
CC: David Howells <dhowells@redhat.com> 
CC: linux-am33-list@redhat.com

---
 arch/mn10300/kernel/Makefile |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/mn10300/kernel/Makefile b/arch/mn10300/kernel/Makefile
index 47ed30f..74b20ad 100644
--- a/arch/mn10300/kernel/Makefile
+++ b/arch/mn10300/kernel/Makefile
@@ -22,7 +22,6 @@ obj-$(CONFIG_GDBSTUB_ON_TTYSx) += gdb-io-serial.o gdb-io-serial-low.o
 obj-$(CONFIG_GDBSTUB_ON_TTYSMx) += gdb-io-ttysm.o gdb-io-ttysm-low.o
 
 obj-$(CONFIG_MN10300_RTC) += rtc.o
-obj-$(CONFIG_PROFILE) += profile.o profile-low.o
 obj-$(CONFIG_MODULES) += module.o
 obj-$(CONFIG_KPROBES) += kprobes.o
 obj-$(CONFIG_KGDB) += kgdb.o
-- 
1.7.4.2


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

* [RFC 5/5]arch:powerpc:kernel:Makefile Remove unused config in the Makefile.
  2011-04-05 16:58 [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile Justin P. Mattock
                   ` (2 preceding siblings ...)
  2011-04-05 16:58 ` [RFC 4/5]arch:mn10300:kernel:Makefile " Justin P. Mattock
@ 2011-04-05 16:58 ` Justin P. Mattock
  2011-04-06  8:05 ` [RFC 1/5]arch/arm/common/Makefile " Uwe Kleine-König
  4 siblings, 0 replies; 16+ messages in thread
From: Justin P. Mattock @ 2011-04-05 16:58 UTC (permalink / raw)
  To: trivial
  Cc: linux-kernel, Justin P. Mattock, Benjamin Herrenschmidt, linuxppc-dev

The patch below removes an unused config variable found by using a kernel
cleanup script.
Note: I did try to cross compile these but hit erros while doing so..
(gcc is not setup to cross compile) and am unsure if anymore needs to be done.
Please have a look if/when anybody has free time.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
CC: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: linuxppc-dev@lists.ozlabs.org

---
 arch/powerpc/kernel/Makefile |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/powerpc/kernel/Makefile b/arch/powerpc/kernel/Makefile
index 3bb2a3e..4fa0d52 100644
--- a/arch/powerpc/kernel/Makefile
+++ b/arch/powerpc/kernel/Makefile
@@ -75,7 +75,6 @@ obj-$(CONFIG_PPC_FSL_BOOK3E)	+= cpu_setup_fsl_booke.o dbell.o
 obj-$(CONFIG_PPC_BOOK3E_64)	+= dbell.o
 
 extra-y				:= head_$(CONFIG_WORD_SIZE).o
-extra-$(CONFIG_PPC_BOOK3E_32)	:= head_new_booke.o
 extra-$(CONFIG_40x)		:= head_40x.o
 extra-$(CONFIG_44x)		:= head_44x.o
 extra-$(CONFIG_FSL_BOOKE)	:= head_fsl_booke.o
-- 
1.7.4.2


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

* Re: [RFC 2/5]arch:powerpc:sysdev:Makefile Remove unused config in the Makefile.
  2011-04-05 16:58 ` [RFC 2/5]arch:powerpc:sysdev:Makefile " Justin P. Mattock
@ 2011-04-05 18:15   ` Scott Wood
  2011-04-06 15:07     ` Justin Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Scott Wood @ 2011-04-05 18:15 UTC (permalink / raw)
  To: Justin P. Mattock; +Cc: trivial, linuxppc-dev, linux-kernel, Harninder Rai

On Tue, 5 Apr 2011 09:58:19 -0700
"Justin P. Mattock" <justinmattock@gmail.com> wrote:

> The patch below removes an unused config variable found by using a kernel
> cleanup script.
> Note: I did try to cross compile these but hit erros while doing so..
> (gcc is not setup to cross compile) and am unsure if anymore needs to be done.
> Please have a look if/when anybody has free time.
> 
> Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
> CC: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> CC: linuxppc-dev@lists.ozlabs.org
> 
> ---
>  arch/powerpc/sysdev/Makefile |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/powerpc/sysdev/Makefile b/arch/powerpc/sysdev/Makefile
> index 1e0c933..243b6ad 100644
> --- a/arch/powerpc/sysdev/Makefile
> +++ b/arch/powerpc/sysdev/Makefile
> @@ -18,7 +18,6 @@ obj-$(CONFIG_FSL_PMC)		+= fsl_pmc.o
>  obj-$(CONFIG_FSL_LBC)		+= fsl_lbc.o
>  obj-$(CONFIG_FSL_GTM)		+= fsl_gtm.o
>  obj-$(CONFIG_MPC8xxx_GPIO)	+= mpc8xxx_gpio.o
> -obj-$(CONFIG_FSL_85XX_CACHE_SRAM)	+= fsl_85xx_l2ctlr.o fsl_85xx_cache_sram.o
>  obj-$(CONFIG_SIMPLE_GPIO)	+= simple_gpio.o
>  obj-$(CONFIG_FSL_RIO)		+= fsl_rio.o
>  obj-$(CONFIG_TSI108_BRIDGE)	+= tsi108_pci.o tsi108_dev.o

Those files do exist, and aren't pulled in by any other means I can see.
It was introduced by commit 6db92cc9d07db9f713da8554b4bcdfc8e54ad386, whose
changelog says:
	Drivers can do the following in Kconfig to use these APIs "select
FSL_85XX_CACHE_SRAM if MPC85xx"

Now, the absence of such a kconfig option[1] is a problem, but I don't think
outright removal (labelled "trivial cleanup") is appropriate, unless nobody
fixes it after the problem is pointed out.  And if it is removed, the files
should go with it.

-Scott

[1] and of any drivers that select it, though this was added fairly
recently -- perhaps such a driver change is on its way?


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

* Re: [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile.
  2011-04-05 16:58 [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile Justin P. Mattock
                   ` (3 preceding siblings ...)
  2011-04-05 16:58 ` [RFC 5/5]arch:powerpc:kernel:Makefile " Justin P. Mattock
@ 2011-04-06  8:05 ` Uwe Kleine-König
  2011-04-06 15:10   ` Justin P. Mattock
  2011-04-07  3:59   ` Jean-Christophe PLAGNIOL-VILLARD
  4 siblings, 2 replies; 16+ messages in thread
From: Uwe Kleine-König @ 2011-04-06  8:05 UTC (permalink / raw)
  To: Justin P. Mattock
  Cc: trivial, linux-arm-kernel, Russell King, linux-kernel,
	Paul Mundt, Jean-Christophe PLAGNIOL-VILLARD

Hello,

On Tue, Apr 05, 2011 at 09:58:18AM -0700, Justin P. Mattock wrote:
> The patch below removes an unused config variable found by using a kernel
> cleanup script.
> Note: I did try to cross compile these but hit erros while doing so..
> (gcc is not setup to cross compile) and am unsure if anymore needs to be done.
> Please have a look if/when anybody has free time.
> 
> Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
> CC: Russell King <linux@arm.linux.org.uk>
> CC: linux-arm-kernel@lists.infradead.org
The file was removed by commit

	6d803ba (ARM: 6483/1: arm & sh: factorised duplicated clkdev.c)

(added the persons involved to Cc).

Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

Thanks
Uwe

> ---
>  arch/arm/common/Makefile |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile
> index e7521bca..6ea9b6f 100644
> --- a/arch/arm/common/Makefile
> +++ b/arch/arm/common/Makefile
> @@ -16,5 +16,4 @@ obj-$(CONFIG_SHARP_SCOOP)	+= scoop.o
>  obj-$(CONFIG_ARCH_IXP2000)	+= uengine.o
>  obj-$(CONFIG_ARCH_IXP23XX)	+= uengine.o
>  obj-$(CONFIG_PCI_HOST_ITE8152)  += it8152.o
> -obj-$(CONFIG_COMMON_CLKDEV)	+= clkdev.o
>  obj-$(CONFIG_ARM_TIMER_SP804)	+= timer-sp.o
> -- 
> 1.7.4.2
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

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

* Re: [RFC 2/5]arch:powerpc:sysdev:Makefile Remove unused config in the Makefile.
  2011-04-05 18:15   ` Scott Wood
@ 2011-04-06 15:07     ` Justin Mattock
  2011-04-06 16:03       ` Scott Wood
  0 siblings, 1 reply; 16+ messages in thread
From: Justin Mattock @ 2011-04-06 15:07 UTC (permalink / raw)
  To: Scott Wood; +Cc: trivial, linuxppc-dev, linux-kernel, Harninder Rai

On Tue, Apr 5, 2011 at 11:15 AM, Scott Wood <scottwood@freescale.com> wrote:
> On Tue, 5 Apr 2011 09:58:19 -0700
> "Justin P. Mattock" <justinmattock@gmail.com> wrote:
>
>> The patch below removes an unused config variable found by using a kernel
>> cleanup script.
>> Note: I did try to cross compile these but hit erros while doing so..
>> (gcc is not setup to cross compile) and am unsure if anymore needs to be done.
>> Please have a look if/when anybody has free time.
>>
>> Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
>> CC: Benjamin Herrenschmidt <benh@kernel.crashing.org>
>> CC: linuxppc-dev@lists.ozlabs.org
>>
>> ---
>>  arch/powerpc/sysdev/Makefile |    1 -
>>  1 files changed, 0 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/powerpc/sysdev/Makefile b/arch/powerpc/sysdev/Makefile
>> index 1e0c933..243b6ad 100644
>> --- a/arch/powerpc/sysdev/Makefile
>> +++ b/arch/powerpc/sysdev/Makefile
>> @@ -18,7 +18,6 @@ obj-$(CONFIG_FSL_PMC)               += fsl_pmc.o
>>  obj-$(CONFIG_FSL_LBC)                += fsl_lbc.o
>>  obj-$(CONFIG_FSL_GTM)                += fsl_gtm.o
>>  obj-$(CONFIG_MPC8xxx_GPIO)   += mpc8xxx_gpio.o
>> -obj-$(CONFIG_FSL_85XX_CACHE_SRAM)    += fsl_85xx_l2ctlr.o fsl_85xx_cache_sram.o
>>  obj-$(CONFIG_SIMPLE_GPIO)    += simple_gpio.o
>>  obj-$(CONFIG_FSL_RIO)                += fsl_rio.o
>>  obj-$(CONFIG_TSI108_BRIDGE)  += tsi108_pci.o tsi108_dev.o
>
> Those files do exist, and aren't pulled in by any other means I can see.
> It was introduced by commit 6db92cc9d07db9f713da8554b4bcdfc8e54ad386, whose
> changelog says:
>        Drivers can do the following in Kconfig to use these APIs "select
> FSL_85XX_CACHE_SRAM if MPC85xx"
>
> Now, the absence of such a kconfig option[1] is a problem, but I don't think
> outright removal (labelled "trivial cleanup") is appropriate, unless nobody
> fixes it after the problem is pointed out.  And if it is removed, the files
> should go with it.
>
> -Scott
>
> [1] and of any drivers that select it, though this was added fairly
> recently -- perhaps such a driver change is on its way?
>
>

ahh.. so the:  fsl_85xx_l2ctlr.o fsl_85xx_cache_sram.o is still in use
even though FSL_85XX_CACHE_SRAM is not really used, but really is used!!

but might be wrong with this.

-- 
Justin P. Mattock

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

* Re: [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile.
  2011-04-06  8:05 ` [RFC 1/5]arch/arm/common/Makefile " Uwe Kleine-König
@ 2011-04-06 15:10   ` Justin P. Mattock
  2011-04-07 22:23     ` Russell King - ARM Linux
  2011-04-07  3:59   ` Jean-Christophe PLAGNIOL-VILLARD
  1 sibling, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2011-04-06 15:10 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: trivial, linux-arm-kernel, Russell King, linux-kernel,
	Paul Mundt, Jean-Christophe PLAGNIOL-VILLARD

On 04/06/2011 01:05 AM, Uwe Kleine-König wrote:
> Hello,
>
> On Tue, Apr 05, 2011 at 09:58:18AM -0700, Justin P. Mattock wrote:
>> The patch below removes an unused config variable found by using a kernel
>> cleanup script.
>> Note: I did try to cross compile these but hit erros while doing so..
>> (gcc is not setup to cross compile) and am unsure if anymore needs to be done.
>> Please have a look if/when anybody has free time.
>>
>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com>
>> CC: Russell King<linux@arm.linux.org.uk>
>> CC: linux-arm-kernel@lists.infradead.org
> The file was removed by commit
>
> 	6d803ba (ARM: 6483/1: arm&  sh: factorised duplicated clkdev.c)
>
> (added the persons involved to Cc).
>
> Acked-by: Uwe Kleine-König<u.kleine-koenig@pengutronix.de>
>
> Thanks
> Uwe

o.k.!! Thanks for having a look at this..

>
>> ---
>>   arch/arm/common/Makefile |    1 -
>>   1 files changed, 0 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile
>> index e7521bca..6ea9b6f 100644
>> --- a/arch/arm/common/Makefile
>> +++ b/arch/arm/common/Makefile
>> @@ -16,5 +16,4 @@ obj-$(CONFIG_SHARP_SCOOP)	+= scoop.o
>>   obj-$(CONFIG_ARCH_IXP2000)	+= uengine.o
>>   obj-$(CONFIG_ARCH_IXP23XX)	+= uengine.o
>>   obj-$(CONFIG_PCI_HOST_ITE8152)  += it8152.o
>> -obj-$(CONFIG_COMMON_CLKDEV)	+= clkdev.o
>>   obj-$(CONFIG_ARM_TIMER_SP804)	+= timer-sp.o
>> --
>> 1.7.4.2
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>>
>

Justin P. Mattock

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

* Re: [RFC 2/5]arch:powerpc:sysdev:Makefile Remove unused config in the Makefile.
  2011-04-06 15:07     ` Justin Mattock
@ 2011-04-06 16:03       ` Scott Wood
  2011-04-06 16:09         ` Justin Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Scott Wood @ 2011-04-06 16:03 UTC (permalink / raw)
  To: Justin Mattock; +Cc: trivial, linuxppc-dev, linux-kernel, Harninder Rai

On Wed, 6 Apr 2011 08:07:58 -0700
Justin Mattock <justinmattock@gmail.com> wrote:

> ahh.. so the:  fsl_85xx_l2ctlr.o fsl_85xx_cache_sram.o is still in use
> even though FSL_85XX_CACHE_SRAM is not really used, but really is used!!
> 
> but might be wrong with this.

More like there are plans to use it, or possibly out-of-tree users.  We
should prod people a bit to submit the driver patches that use this before
we just yank it out.

-Scott


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

* Re: [RFC 2/5]arch:powerpc:sysdev:Makefile Remove unused config in the Makefile.
  2011-04-06 16:03       ` Scott Wood
@ 2011-04-06 16:09         ` Justin Mattock
  0 siblings, 0 replies; 16+ messages in thread
From: Justin Mattock @ 2011-04-06 16:09 UTC (permalink / raw)
  To: Scott Wood; +Cc: trivial, linuxppc-dev, linux-kernel, Harninder Rai

On Wed, Apr 6, 2011 at 9:03 AM, Scott Wood <scottwood@freescale.com> wrote:
> On Wed, 6 Apr 2011 08:07:58 -0700
> Justin Mattock <justinmattock@gmail.com> wrote:
>
>> ahh.. so the:  fsl_85xx_l2ctlr.o fsl_85xx_cache_sram.o is still in use
>> even though FSL_85XX_CACHE_SRAM is not really used, but really is used!!
>>
>> but might be wrong with this.
>
> More like there are plans to use it, or possibly out-of-tree users.  We
> should prod people a bit to submit the driver patches that use this before
> we just yank it out.
>
> -Scott
>
>

well if this is going to be used for something down the line, then
best leave it in..

-- 
Justin P. Mattock

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

* Re: [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile.
  2011-04-06  8:05 ` [RFC 1/5]arch/arm/common/Makefile " Uwe Kleine-König
  2011-04-06 15:10   ` Justin P. Mattock
@ 2011-04-07  3:59   ` Jean-Christophe PLAGNIOL-VILLARD
  1 sibling, 0 replies; 16+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2011-04-07  3:59 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Justin P. Mattock, trivial, linux-arm-kernel, Russell King,
	linux-kernel, Paul Mundt

On 10:05 Wed 06 Apr     , Uwe Kleine-König wrote:
> Hello,
> 
> On Tue, Apr 05, 2011 at 09:58:18AM -0700, Justin P. Mattock wrote:
> > The patch below removes an unused config variable found by using a kernel
> > cleanup script.
> > Note: I did try to cross compile these but hit erros while doing so..
> > (gcc is not setup to cross compile) and am unsure if anymore needs to be done.
> > Please have a look if/when anybody has free time.
> > 
> > Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>
> > CC: Russell King <linux@arm.linux.org.uk>
> > CC: linux-arm-kernel@lists.infradead.org
> The file was removed by commit
> 
> 	6d803ba (ARM: 6483/1: arm & sh: factorised duplicated clkdev.c)
> 
> (added the persons involved to Cc).
> 
> Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

Best Regards,
J.

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

* Re: [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile.
  2011-04-06 15:10   ` Justin P. Mattock
@ 2011-04-07 22:23     ` Russell King - ARM Linux
  2011-04-07 23:40       ` Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Russell King - ARM Linux @ 2011-04-07 22:23 UTC (permalink / raw)
  To: Justin P. Mattock
  Cc: Uwe Kleine-König, trivial, linux-arm-kernel, linux-kernel,
	Paul Mundt, Jean-Christophe PLAGNIOL-VILLARD

On Wed, Apr 06, 2011 at 08:10:36AM -0700, Justin P. Mattock wrote:
> o.k.!! Thanks for having a look at this..

Could you queue this up in the patch system with everyone's acks please?
You don't need to include a Cc line for any of my addresses (and I'd
prefer it not to be in submissions.)

Thanks.

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

* Re: [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile.
  2011-04-07 22:23     ` Russell King - ARM Linux
@ 2011-04-07 23:40       ` Justin P. Mattock
  0 siblings, 0 replies; 16+ messages in thread
From: Justin P. Mattock @ 2011-04-07 23:40 UTC (permalink / raw)
  To: Russell King - ARM Linux
  Cc: Uwe Kleine-König, trivial, linux-arm-kernel, linux-kernel,
	Paul Mundt, Jean-Christophe PLAGNIOL-VILLARD

On 04/07/2011 03:23 PM, Russell King - ARM Linux wrote:
> On Wed, Apr 06, 2011 at 08:10:36AM -0700, Justin P. Mattock wrote:
>> o.k.!! Thanks for having a look at this..
>
> Could you queue this up in the patch system with everyone's acks please?
> You don't need to include a Cc line for any of my addresses (and I'd
> prefer it not to be in submissions.)
>
> Thanks.
>


o.k. I resent.. hopefully I got the addresses proper.

Justin P. Mattock

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

* Re: [RFC 3/5]arch:mips:pmc-sierra:msp71xx:Makefile Remove unused config in the Makefile.
  2011-04-05 16:58 ` [RFC 3/5]arch:mips:pmc-sierra:msp71xx:Makefile " Justin P. Mattock
@ 2011-04-08 13:08   ` Ralf Baechle
  2011-04-08 15:10     ` Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Ralf Baechle @ 2011-04-08 13:08 UTC (permalink / raw)
  To: Justin P. Mattock; +Cc: trivial, linux-kernel, linux-mips

On Tue, Apr 05, 2011 at 09:58:20AM -0700, Justin P. Mattock wrote:

> The patch below removes an unused config variable found by using a kernel
> cleanup script.
> Note: I did try to cross compile these but hit erros while doing so..
> (gcc is not setup to cross compile) and am unsure if anymore needs to be done.
> Please have a look if/when anybody has free time.

NACK.

This need a rewrite, not just chainsawing.  Your code was only touching
the makefile anyway, not the two referenced C files.

 Ralf

  Ralf

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

* Re: [RFC 3/5]arch:mips:pmc-sierra:msp71xx:Makefile Remove unused config in the Makefile.
  2011-04-08 13:08   ` Ralf Baechle
@ 2011-04-08 15:10     ` Justin P. Mattock
  0 siblings, 0 replies; 16+ messages in thread
From: Justin P. Mattock @ 2011-04-08 15:10 UTC (permalink / raw)
  To: Ralf Baechle; +Cc: trivial, linux-kernel, linux-mips

On 04/08/2011 06:08 AM, Ralf Baechle wrote:
> On Tue, Apr 05, 2011 at 09:58:20AM -0700, Justin P. Mattock wrote:
>
>> The patch below removes an unused config variable found by using a kernel
>> cleanup script.
>> Note: I did try to cross compile these but hit erros while doing so..
>> (gcc is not setup to cross compile) and am unsure if anymore needs to be done.
>> Please have a look if/when anybody has free time.
>
> NACK.
>
> This need a rewrite, not just chainsawing.  Your code was only touching
> the makefile anyway, not the two referenced C files.
>
>   Ralf
>
>    Ralf
>

alright!!

Justin P. Mattock

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

end of thread, other threads:[~2011-04-08 15:10 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-04-05 16:58 [RFC 1/5]arch/arm/common/Makefile Remove unused config in the Makefile Justin P. Mattock
2011-04-05 16:58 ` [RFC 2/5]arch:powerpc:sysdev:Makefile " Justin P. Mattock
2011-04-05 18:15   ` Scott Wood
2011-04-06 15:07     ` Justin Mattock
2011-04-06 16:03       ` Scott Wood
2011-04-06 16:09         ` Justin Mattock
2011-04-05 16:58 ` [RFC 3/5]arch:mips:pmc-sierra:msp71xx:Makefile " Justin P. Mattock
2011-04-08 13:08   ` Ralf Baechle
2011-04-08 15:10     ` Justin P. Mattock
2011-04-05 16:58 ` [RFC 4/5]arch:mn10300:kernel:Makefile " Justin P. Mattock
2011-04-05 16:58 ` [RFC 5/5]arch:powerpc:kernel:Makefile " Justin P. Mattock
2011-04-06  8:05 ` [RFC 1/5]arch/arm/common/Makefile " Uwe Kleine-König
2011-04-06 15:10   ` Justin P. Mattock
2011-04-07 22:23     ` Russell King - ARM Linux
2011-04-07 23:40       ` Justin P. Mattock
2011-04-07  3:59   ` Jean-Christophe PLAGNIOL-VILLARD

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).