* [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 related [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 related [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 related [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 related [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 related [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).