* [PATCH] ARM: imx: removed unused MACH_MXLADS Kconfig parameter @ 2013-11-04 5:19 ` Michael Opdenacker 0 siblings, 0 replies; 6+ messages in thread From: Michael Opdenacker @ 2013-11-04 5:19 UTC (permalink / raw) To: linux-arm-kernel This removes the MACH_MXLADS Kconfig parameter, which was no longer used anywhere in the source code and Makefiles. Signed-off-by: Michael Opdenacker <michael.opdenacker@free-electrons.com> --- arch/arm/mach-imx/Kconfig | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig index 29a8af6922a8..d84a772bbc68 100644 --- a/arch/arm/mach-imx/Kconfig +++ b/arch/arm/mach-imx/Kconfig @@ -158,14 +158,11 @@ config SOC_IMX51 if ARCH_MULTI_V4T comment "MX1 platforms:" -config MACH_MXLADS - bool config ARCH_MX1ADS bool "MX1ADS platform" select IMX_HAVE_PLATFORM_IMX_I2C select IMX_HAVE_PLATFORM_IMX_UART - select MACH_MXLADS select SOC_IMX1 help Say Y here if you are using Motorola MX1ADS/MXLADS boards -- 1.8.1.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] ARM: imx: removed unused MACH_MXLADS Kconfig parameter @ 2013-11-04 5:19 ` Michael Opdenacker 0 siblings, 0 replies; 6+ messages in thread From: Michael Opdenacker @ 2013-11-04 5:19 UTC (permalink / raw) To: kernel, linux; +Cc: linux-arm-kernel, linux-kernel, Michael Opdenacker This removes the MACH_MXLADS Kconfig parameter, which was no longer used anywhere in the source code and Makefiles. Signed-off-by: Michael Opdenacker <michael.opdenacker@free-electrons.com> --- arch/arm/mach-imx/Kconfig | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig index 29a8af6922a8..d84a772bbc68 100644 --- a/arch/arm/mach-imx/Kconfig +++ b/arch/arm/mach-imx/Kconfig @@ -158,14 +158,11 @@ config SOC_IMX51 if ARCH_MULTI_V4T comment "MX1 platforms:" -config MACH_MXLADS - bool config ARCH_MX1ADS bool "MX1ADS platform" select IMX_HAVE_PLATFORM_IMX_I2C select IMX_HAVE_PLATFORM_IMX_UART - select MACH_MXLADS select SOC_IMX1 help Say Y here if you are using Motorola MX1ADS/MXLADS boards -- 1.8.1.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] ARM: imx: removed unused MACH_MXLADS Kconfig parameter 2013-11-04 5:19 ` Michael Opdenacker @ 2013-11-04 8:51 ` Sascha Hauer -1 siblings, 0 replies; 6+ messages in thread From: Sascha Hauer @ 2013-11-04 8:51 UTC (permalink / raw) To: linux-arm-kernel On Mon, Nov 04, 2013 at 06:19:26AM +0100, Michael Opdenacker wrote: > This removes the MACH_MXLADS Kconfig parameter, > which was no longer used anywhere in the source code > and Makefiles. Hehe, this one is a bit tricky. Indeed MACH_MXLADS isn't used directly anywhere. Nevertheless having CONFIG_MACH_MXLADS defined makes machine_is_mxlads() become true when the kernel is started with a machine id of 1851. machine_is_mxlads() is unused in the kernel, so this patch indeed doesn't hurt. Anyway, the code is correct currently, so I suggest keeping it until the mx[1l]ads is either removed or converted to devicetree. Sascha > > Signed-off-by: Michael Opdenacker <michael.opdenacker@free-electrons.com> > --- > arch/arm/mach-imx/Kconfig | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig > index 29a8af6922a8..d84a772bbc68 100644 > --- a/arch/arm/mach-imx/Kconfig > +++ b/arch/arm/mach-imx/Kconfig > @@ -158,14 +158,11 @@ config SOC_IMX51 > if ARCH_MULTI_V4T > > comment "MX1 platforms:" > -config MACH_MXLADS > - bool > > config ARCH_MX1ADS > bool "MX1ADS platform" > select IMX_HAVE_PLATFORM_IMX_I2C > select IMX_HAVE_PLATFORM_IMX_UART > - select MACH_MXLADS > select SOC_IMX1 > help > Say Y here if you are using Motorola MX1ADS/MXLADS boards > -- > 1.8.1.2 > > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: imx: removed unused MACH_MXLADS Kconfig parameter @ 2013-11-04 8:51 ` Sascha Hauer 0 siblings, 0 replies; 6+ messages in thread From: Sascha Hauer @ 2013-11-04 8:51 UTC (permalink / raw) To: Michael Opdenacker; +Cc: kernel, linux, linux-arm-kernel, linux-kernel On Mon, Nov 04, 2013 at 06:19:26AM +0100, Michael Opdenacker wrote: > This removes the MACH_MXLADS Kconfig parameter, > which was no longer used anywhere in the source code > and Makefiles. Hehe, this one is a bit tricky. Indeed MACH_MXLADS isn't used directly anywhere. Nevertheless having CONFIG_MACH_MXLADS defined makes machine_is_mxlads() become true when the kernel is started with a machine id of 1851. machine_is_mxlads() is unused in the kernel, so this patch indeed doesn't hurt. Anyway, the code is correct currently, so I suggest keeping it until the mx[1l]ads is either removed or converted to devicetree. Sascha > > Signed-off-by: Michael Opdenacker <michael.opdenacker@free-electrons.com> > --- > arch/arm/mach-imx/Kconfig | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig > index 29a8af6922a8..d84a772bbc68 100644 > --- a/arch/arm/mach-imx/Kconfig > +++ b/arch/arm/mach-imx/Kconfig > @@ -158,14 +158,11 @@ config SOC_IMX51 > if ARCH_MULTI_V4T > > comment "MX1 platforms:" > -config MACH_MXLADS > - bool > > config ARCH_MX1ADS > bool "MX1ADS platform" > select IMX_HAVE_PLATFORM_IMX_I2C > select IMX_HAVE_PLATFORM_IMX_UART > - select MACH_MXLADS > select SOC_IMX1 > help > Say Y here if you are using Motorola MX1ADS/MXLADS boards > -- > 1.8.1.2 > > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: imx: removed unused MACH_MXLADS Kconfig parameter 2013-11-04 8:51 ` Sascha Hauer @ 2013-11-04 9:00 ` Michael Opdenacker -1 siblings, 0 replies; 6+ messages in thread From: Michael Opdenacker @ 2013-11-04 9:00 UTC (permalink / raw) To: linux-arm-kernel Hi Sascha, On 11/04/2013 09:51 AM, Sascha Hauer wrote: > On Mon, Nov 04, 2013 at 06:19:26AM +0100, Michael Opdenacker wrote: >> This removes the MACH_MXLADS Kconfig parameter, >> which was no longer used anywhere in the source code >> and Makefiles. > Hehe, this one is a bit tricky. Indeed MACH_MXLADS isn't used directly > anywhere. Nevertheless having CONFIG_MACH_MXLADS defined makes > machine_is_mxlads() become true when the kernel is started with a > machine id of 1851. Oops, that's right, got it! Thanks! > > machine_is_mxlads() is unused in the kernel, so this patch indeed > doesn't hurt. Anyway, the code is correct currently, so I suggest > keeping it until the mx[1l]ads is either removed or converted to > devicetree. Sounds good to me. Thanks for your review! Cheers, Michael. -- Michael Opdenacker, CEO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com +33 484 258 098 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: imx: removed unused MACH_MXLADS Kconfig parameter @ 2013-11-04 9:00 ` Michael Opdenacker 0 siblings, 0 replies; 6+ messages in thread From: Michael Opdenacker @ 2013-11-04 9:00 UTC (permalink / raw) To: kernel, linux, linux-arm-kernel, linux-kernel Hi Sascha, On 11/04/2013 09:51 AM, Sascha Hauer wrote: > On Mon, Nov 04, 2013 at 06:19:26AM +0100, Michael Opdenacker wrote: >> This removes the MACH_MXLADS Kconfig parameter, >> which was no longer used anywhere in the source code >> and Makefiles. > Hehe, this one is a bit tricky. Indeed MACH_MXLADS isn't used directly > anywhere. Nevertheless having CONFIG_MACH_MXLADS defined makes > machine_is_mxlads() become true when the kernel is started with a > machine id of 1851. Oops, that's right, got it! Thanks! > > machine_is_mxlads() is unused in the kernel, so this patch indeed > doesn't hurt. Anyway, the code is correct currently, so I suggest > keeping it until the mx[1l]ads is either removed or converted to > devicetree. Sounds good to me. Thanks for your review! Cheers, Michael. -- Michael Opdenacker, CEO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com +33 484 258 098 ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-11-04 9:00 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-11-04 5:19 [PATCH] ARM: imx: removed unused MACH_MXLADS Kconfig parameter Michael Opdenacker 2013-11-04 5:19 ` Michael Opdenacker 2013-11-04 8:51 ` Sascha Hauer 2013-11-04 8:51 ` Sascha Hauer 2013-11-04 9:00 ` Michael Opdenacker 2013-11-04 9:00 ` Michael Opdenacker
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.