diff mbox

ARM: mx25: Remove mach-mx25_3ds board file

Message ID 1425138035-19715-1-git-send-email-festevam@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Fabio Estevam Feb. 28, 2015, 3:40 p.m. UTC
From: Fabio Estevam <fabio.estevam@freescale.com>

imx25-pdk.dts provides a more complete support than the board file version, so
let's get rid of the board file.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 arch/arm/mach-imx/Kconfig         |  15 ---
 arch/arm/mach-imx/Makefile        |   1 -
 arch/arm/mach-imx/mach-mx25_3ds.c | 270 --------------------------------------
 3 files changed, 286 deletions(-)
 delete mode 100644 arch/arm/mach-imx/mach-mx25_3ds.c

diff --git a/arch/arm/mach-imx/mach-mx25_3ds.c b/arch/arm/mach-imx/mach-mx25_3ds.c
deleted file mode 100644
index 0d01e36..0000000

Comments

Fabio Estevam Feb. 28, 2015, 3:42 p.m. UTC | #1
Hi Eric,

On Sat, Feb 28, 2015 at 12:40 PM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> imx25-pdk.dts provides a more complete support than the board file version, so
> let's get rid of the board file.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

Do you mind if I submit a patch removing the eukrea mx25 board files?

If this is possible, then we can convert mx25 to a dt-only platform.

Regards,

Fabio Estevam
Eric Bénard Feb. 28, 2015, 8:24 p.m. UTC | #2
Hi Fabio,

Le Sat, 28 Feb 2015 12:42:45 -0300,
Fabio Estevam <festevam@gmail.com> a écrit :
> On Sat, Feb 28, 2015 at 12:40 PM, Fabio Estevam <festevam@gmail.com> wrote:
> > From: Fabio Estevam <fabio.estevam@freescale.com>
> >
> > imx25-pdk.dts provides a more complete support than the board file version, so
> > let's get rid of the board file.
> >
> > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> 
> Do you mind if I submit a patch removing the eukrea mx25 board files?
> 
> If this is possible, then we can convert mx25 to a dt-only platform.
> 
no problem on our side, it was supposed to be removed after DT support
for this board was accepted but it seems the patch was never sent ;-)

Best regards,
Eric
Shawn Guo March 3, 2015, 3:54 a.m. UTC | #3
On Sat, Feb 28, 2015 at 12:40:35PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> imx25-pdk.dts provides a more complete support than the board file version, so
> let's get rid of the board file.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

It applies neither my branch nor v4.0-rc1.  Can you please generate the
patch against imx/soc branch?

Shawn

> ---
>  arch/arm/mach-imx/Kconfig         |  15 ---
>  arch/arm/mach-imx/Makefile        |   1 -
>  arch/arm/mach-imx/mach-mx25_3ds.c | 270 --------------------------------------
>  3 files changed, 286 deletions(-)
>  delete mode 100644 arch/arm/mach-imx/mach-mx25_3ds.c
> 
> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> index e8627e0..aaea738 100644
> --- a/arch/arm/mach-imx/Kconfig
> +++ b/arch/arm/mach-imx/Kconfig
> @@ -151,21 +151,6 @@ config MACH_MX21ADS
>  
>  comment "MX25 platforms:"
>  
> -config MACH_MX25_3DS
> -	bool "Support MX25PDK (3DS) Platform"
> -	select IMX_HAVE_PLATFORM_FLEXCAN
> -	select IMX_HAVE_PLATFORM_FSL_USB2_UDC
> -	select IMX_HAVE_PLATFORM_IMX2_WDT
> -	select IMX_HAVE_PLATFORM_IMXDI_RTC
> -	select IMX_HAVE_PLATFORM_IMX_FB
> -	select IMX_HAVE_PLATFORM_IMX_I2C
> -	select IMX_HAVE_PLATFORM_IMX_KEYPAD
> -	select IMX_HAVE_PLATFORM_IMX_UART
> -	select IMX_HAVE_PLATFORM_MXC_EHCI
> -	select IMX_HAVE_PLATFORM_MXC_NAND
> -	select IMX_HAVE_PLATFORM_SDHCI_ESDHC_IMX
> -	select SOC_IMX25
> -
>  config MACH_EUKREA_CPUIMX25SD
>  	bool "Support Eukrea CPUIMX25 Platform"
>  	select IMX_HAVE_PLATFORM_FLEXCAN
> diff --git a/arch/arm/mach-imx/Makefile b/arch/arm/mach-imx/Makefile
> index 8d1b101..e92ede9 100644
> --- a/arch/arm/mach-imx/Makefile
> +++ b/arch/arm/mach-imx/Makefile
> @@ -49,7 +49,6 @@ obj-$(CONFIG_MACH_IMX1_DT) += imx1-dt.o
>  obj-$(CONFIG_MACH_MX21ADS) += mach-mx21ads.o
>  
>  # i.MX25 based machines
> -obj-$(CONFIG_MACH_MX25_3DS) += mach-mx25_3ds.o
>  obj-$(CONFIG_MACH_EUKREA_CPUIMX25SD) += mach-eukrea_cpuimx25.o
>  obj-$(CONFIG_MACH_EUKREA_MBIMXSD25_BASEBOARD) += eukrea_mbimxsd25-baseboard.o
>  obj-$(CONFIG_MACH_IMX25_DT) += imx25-dt.o
> diff --git a/arch/arm/mach-imx/mach-mx25_3ds.c b/arch/arm/mach-imx/mach-mx25_3ds.c
> deleted file mode 100644
> index 0d01e36..0000000
> -- 
> 1.9.1
>
Shawn Guo March 3, 2015, 4:10 a.m. UTC | #4
On Tue, Mar 3, 2015 at 11:54 AM, Shawn Guo <shawn.guo@linaro.org> wrote:
> On Sat, Feb 28, 2015 at 12:40:35PM -0300, Fabio Estevam wrote:
>> From: Fabio Estevam <fabio.estevam@freescale.com>
>>
>> imx25-pdk.dts provides a more complete support than the board file version, so
>> let's get rid of the board file.
>>
>> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
>
> It applies neither my branch nor v4.0-rc1.  Can you please generate the
> patch against imx/soc branch?

Sorry, I meant imx/cleanup branch.

Shawn
diff mbox

Patch

diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
index e8627e0..aaea738 100644
--- a/arch/arm/mach-imx/Kconfig
+++ b/arch/arm/mach-imx/Kconfig
@@ -151,21 +151,6 @@  config MACH_MX21ADS
 
 comment "MX25 platforms:"
 
-config MACH_MX25_3DS
-	bool "Support MX25PDK (3DS) Platform"
-	select IMX_HAVE_PLATFORM_FLEXCAN
-	select IMX_HAVE_PLATFORM_FSL_USB2_UDC
-	select IMX_HAVE_PLATFORM_IMX2_WDT
-	select IMX_HAVE_PLATFORM_IMXDI_RTC
-	select IMX_HAVE_PLATFORM_IMX_FB
-	select IMX_HAVE_PLATFORM_IMX_I2C
-	select IMX_HAVE_PLATFORM_IMX_KEYPAD
-	select IMX_HAVE_PLATFORM_IMX_UART
-	select IMX_HAVE_PLATFORM_MXC_EHCI
-	select IMX_HAVE_PLATFORM_MXC_NAND
-	select IMX_HAVE_PLATFORM_SDHCI_ESDHC_IMX
-	select SOC_IMX25
-
 config MACH_EUKREA_CPUIMX25SD
 	bool "Support Eukrea CPUIMX25 Platform"
 	select IMX_HAVE_PLATFORM_FLEXCAN
diff --git a/arch/arm/mach-imx/Makefile b/arch/arm/mach-imx/Makefile
index 8d1b101..e92ede9 100644
--- a/arch/arm/mach-imx/Makefile
+++ b/arch/arm/mach-imx/Makefile
@@ -49,7 +49,6 @@  obj-$(CONFIG_MACH_IMX1_DT) += imx1-dt.o
 obj-$(CONFIG_MACH_MX21ADS) += mach-mx21ads.o
 
 # i.MX25 based machines
-obj-$(CONFIG_MACH_MX25_3DS) += mach-mx25_3ds.o
 obj-$(CONFIG_MACH_EUKREA_CPUIMX25SD) += mach-eukrea_cpuimx25.o
 obj-$(CONFIG_MACH_EUKREA_MBIMXSD25_BASEBOARD) += eukrea_mbimxsd25-baseboard.o
 obj-$(CONFIG_MACH_IMX25_DT) += imx25-dt.o