Message ID | 1302675881-18862-2-git-send-email-b32955@freescale.com |
---|---|
State | Not Applicable |
Headers | show |
Hi, Huang Shijie writes: > add the clock and iomux initialization for GPMI in the imx23 and imx28. > and register the gpmi driver. > > Signed-off-by: Huang Shijie <b32955@freescale.com> > --- > arch/arm/mach-mxs/Kconfig | 2 + > arch/arm/mach-mxs/clock-mx23.c | 3 + > arch/arm/mach-mxs/clock-mx28.c | 3 + > arch/arm/mach-mxs/devices-mx23.h | 3 + > arch/arm/mach-mxs/devices-mx28.h | 3 + > arch/arm/mach-mxs/devices/Kconfig | 3 + > arch/arm/mach-mxs/devices/Makefile | 1 + > arch/arm/mach-mxs/devices/platform-gpmi.c | 134 +++++++++++++++++++++++ > arch/arm/mach-mxs/include/mach/devices-common.h | 4 + > arch/arm/mach-mxs/include/mach/gpmi-nfc.h | 65 +++++++++++ > arch/arm/mach-mxs/mach-mx23evk.c | 37 ++++++ > arch/arm/mach-mxs/mach-mx28evk.c | 37 ++++++ > 12 files changed, 295 insertions(+), 0 deletions(-) > create mode 100644 arch/arm/mach-mxs/devices/platform-gpmi.c > create mode 100644 arch/arm/mach-mxs/include/mach/gpmi-nfc.h > > diff --git a/arch/arm/mach-mxs/Kconfig b/arch/arm/mach-mxs/Kconfig > index 4f6f174..e034666 100644 > --- a/arch/arm/mach-mxs/Kconfig > +++ b/arch/arm/mach-mxs/Kconfig > @@ -22,6 +22,7 @@ config MACH_MX23EVK > select SOC_IMX23 > select MXS_HAVE_AMBA_DUART > select MXS_HAVE_PLATFORM_AUART > + select MXS_HAVE_PLATFORM_GPMI > select MXS_HAVE_PLATFORM_MXSFB > default y > help > @@ -35,6 +36,7 @@ config MACH_MX28EVK > select MXS_HAVE_PLATFORM_AUART > select MXS_HAVE_PLATFORM_FEC > select MXS_HAVE_PLATFORM_FLEXCAN > + select MXS_HAVE_PLATFORM_GPMI > select MXS_HAVE_PLATFORM_MXSFB > select MXS_OCOTP > default y > diff --git a/arch/arm/mach-mxs/clock-mx23.c b/arch/arm/mach-mxs/clock-mx23.c > index d133c7f..f3c9653 100644 > --- a/arch/arm/mach-mxs/clock-mx23.c > +++ b/arch/arm/mach-mxs/clock-mx23.c > @@ -29,6 +29,7 @@ > #include <mach/mx23.h> > #include <mach/common.h> > #include <mach/clock.h> > +#include <mach/gpmi-nfc.h> > > #include "regs-clkctrl-mx23.h" > > @@ -442,6 +443,7 @@ static struct clk_lookup lookups[] = { > _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) > /* for amba-pl011 driver */ > _REGISTER_CLOCK("duart", NULL, uart_clk) > + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX23, NULL, gpmi_clk) > _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) > _REGISTER_CLOCK("rtc", NULL, rtc_clk) > _REGISTER_CLOCK("mxs-dma-apbh", NULL, hbus_clk) > @@ -533,6 +535,7 @@ int __init mx23_clocks_init(void) > clk_enable(&xbus_clk); > clk_enable(&emi_clk); > clk_enable(&uart_clk); > + clk_enable(&gpmi_clk); > > clkdev_add_table(lookups, ARRAY_SIZE(lookups)); > > diff --git a/arch/arm/mach-mxs/clock-mx28.c b/arch/arm/mach-mxs/clock-mx28.c > index 5e489a2..7868d42 100644 > --- a/arch/arm/mach-mxs/clock-mx28.c > +++ b/arch/arm/mach-mxs/clock-mx28.c > @@ -29,6 +29,7 @@ > #include <mach/mx28.h> > #include <mach/common.h> > #include <mach/clock.h> > +#include <mach/gpmi-nfc.h> > > #include "regs-clkctrl-mx28.h" > > @@ -607,6 +608,7 @@ static struct clk_lookup lookups[] = { > _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) > /* for amba-pl011 driver */ > _REGISTER_CLOCK("duart", NULL, uart_clk) > + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX28, NULL, gpmi_clk) > _REGISTER_CLOCK("imx28-fec.0", NULL, fec_clk) > _REGISTER_CLOCK("imx28-fec.1", NULL, fec_clk) > _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) > @@ -749,6 +751,7 @@ int __init mx28_clocks_init(void) > clk_enable(&xbus_clk); > clk_enable(&emi_clk); > clk_enable(&uart_clk); > + clk_enable(&gpmi_clk); > > clk_set_parent(&lcdif_clk, &ref_pix_clk); > > diff --git a/arch/arm/mach-mxs/devices-mx23.h b/arch/arm/mach-mxs/devices-mx23.h > index c7e14f4..349fb1d 100644 > --- a/arch/arm/mach-mxs/devices-mx23.h > +++ b/arch/arm/mach-mxs/devices-mx23.h > @@ -16,6 +16,9 @@ extern const struct amba_device mx23_duart_device __initconst; > #define mx23_add_duart() \ > mxs_add_duart(&mx23_duart_device) > > +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst; > +#define mx23_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx23) > platform_data should be passed as parameter to the mx23_add_gpmi() call, so that it can be set per platform, not per SoC. > diff --git a/arch/arm/mach-mxs/devices-mx28.h b/arch/arm/mach-mxs/devices-mx28.h > index 9d08555..ed6427b 100644 > --- a/arch/arm/mach-mxs/devices-mx28.h > +++ b/arch/arm/mach-mxs/devices-mx28.h > @@ -16,6 +16,9 @@ extern const struct amba_device mx28_duart_device __initconst; > #define mx28_add_duart() \ > mxs_add_duart(&mx28_duart_device) > > +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst; > +#define mx28_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx28) > + dito. > diff --git a/arch/arm/mach-mxs/devices/platform-gpmi.c b/arch/arm/mach-mxs/devices/platform-gpmi.c > new file mode 100644 > index 0000000..74ebe22 > --- /dev/null > +++ b/arch/arm/mach-mxs/devices/platform-gpmi.c > @@ -0,0 +1,134 @@ > +/* > + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License along > + * with this program; if not, write to the Free Software Foundation, Inc., > + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. > + */ > +#include <asm/sizes.h> > +#include <mach/mx23.h> > +#include <mach/mx28.h> > +#include <mach/gpmi-nfc.h> > +#include <mach/devices-common.h> > + > +#ifdef CONFIG_SOC_IMX23 > +const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst = { > + .min_prop_delay_in_ns = 5, > + .max_prop_delay_in_ns = 9, > + .max_chip_count = 1, > +}; > + This is platform-dependent and thus should be placed in the mach-* files (and passed to mx23_add_gpmi()). > +const struct resource res_imx23[] __initconst = { > + { /* GPMI */ > + .start = MX23_GPMI_BASE_ADDR, > + .end = MX23_GPMI_BASE_ADDR + SZ_8K - 1, > + .name = GPMI_NFC_GPMI_REGS_ADDR_RES_NAME, > + .flags = IORESOURCE_MEM, > + }, { > + .start = MX23_INT_GPMI_ATTENTION, > + .end = MX23_INT_GPMI_ATTENTION, > + .name = GPMI_NFC_GPMI_INTERRUPT_RES_NAME, > + .flags = IORESOURCE_IRQ, > + }, { /* BCH */ > + .start = MX23_BCH_BASE_ADDR, > + .end = MX23_BCH_BASE_ADDR + SZ_8K - 1, > + .name = GPMI_NFC_BCH_REGS_ADDR_RES_NAME, > + .flags = IORESOURCE_MEM, > + }, { > + .start = MX23_INT_BCH, > + .end = MX23_INT_BCH, > + .name = GPMI_NFC_BCH_INTERRUPT_RES_NAME, > + .flags = IORESOURCE_IRQ, > + }, { /* DMA */ > + .start = MX23_DMA_GPMI0, > + .end = MX23_DMA_GPMI3, > + .name = GPMI_NFC_DMA_CHANNELS_RES_NAME, > + .flags = IORESOURCE_DMA, > + }, { > + .start = MX23_INT_GPMI_DMA, > + .end = MX23_INT_GPMI_DMA, > + .name = GPMI_NFC_DMA_INTERRUPT_RES_NAME, > + .flags = IORESOURCE_IRQ, > + }, > +}; > +#endif > + > +#ifdef CONFIG_SOC_IMX28 > +const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst = { > + .min_prop_delay_in_ns = 5, > + .max_prop_delay_in_ns = 9, > + .max_chip_count = 1, > +}; > + dito. Lothar Waßmann
On Wed, Apr 13, 2011 at 02:24:38PM +0800, Huang Shijie wrote: > add the clock and iomux initialization for GPMI in the imx23 and imx28. > and register the gpmi driver. > > Signed-off-by: Huang Shijie <b32955@freescale.com> > --- > arch/arm/mach-mxs/Kconfig | 2 + > arch/arm/mach-mxs/clock-mx23.c | 3 + > arch/arm/mach-mxs/clock-mx28.c | 3 + > arch/arm/mach-mxs/devices-mx23.h | 3 + > arch/arm/mach-mxs/devices-mx28.h | 3 + > arch/arm/mach-mxs/devices/Kconfig | 3 + > arch/arm/mach-mxs/devices/Makefile | 1 + > arch/arm/mach-mxs/devices/platform-gpmi.c | 134 +++++++++++++++++++++++ > arch/arm/mach-mxs/include/mach/devices-common.h | 4 + > arch/arm/mach-mxs/include/mach/gpmi-nfc.h | 65 +++++++++++ > arch/arm/mach-mxs/mach-mx23evk.c | 37 ++++++ > arch/arm/mach-mxs/mach-mx28evk.c | 37 ++++++ > 12 files changed, 295 insertions(+), 0 deletions(-) > create mode 100644 arch/arm/mach-mxs/devices/platform-gpmi.c > create mode 100644 arch/arm/mach-mxs/include/mach/gpmi-nfc.h > > diff --git a/arch/arm/mach-mxs/Kconfig b/arch/arm/mach-mxs/Kconfig > index 4f6f174..e034666 100644 > --- a/arch/arm/mach-mxs/Kconfig > +++ b/arch/arm/mach-mxs/Kconfig > @@ -22,6 +22,7 @@ config MACH_MX23EVK > select SOC_IMX23 > select MXS_HAVE_AMBA_DUART > select MXS_HAVE_PLATFORM_AUART > + select MXS_HAVE_PLATFORM_GPMI > select MXS_HAVE_PLATFORM_MXSFB > default y > help > @@ -35,6 +36,7 @@ config MACH_MX28EVK > select MXS_HAVE_PLATFORM_AUART > select MXS_HAVE_PLATFORM_FEC > select MXS_HAVE_PLATFORM_FLEXCAN > + select MXS_HAVE_PLATFORM_GPMI > select MXS_HAVE_PLATFORM_MXSFB > select MXS_OCOTP > default y > diff --git a/arch/arm/mach-mxs/clock-mx23.c b/arch/arm/mach-mxs/clock-mx23.c > index d133c7f..f3c9653 100644 > --- a/arch/arm/mach-mxs/clock-mx23.c > +++ b/arch/arm/mach-mxs/clock-mx23.c > @@ -29,6 +29,7 @@ > #include <mach/mx23.h> > #include <mach/common.h> > #include <mach/clock.h> > +#include <mach/gpmi-nfc.h> This include is only needed to get GPMI_NFC_DRIVER_MX23? Then I suggest to hardcode the clock name. > > #include "regs-clkctrl-mx23.h" > > @@ -442,6 +443,7 @@ static struct clk_lookup lookups[] = { > _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) > /* for amba-pl011 driver */ > _REGISTER_CLOCK("duart", NULL, uart_clk) > + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX23, NULL, gpmi_clk) > _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) > _REGISTER_CLOCK("rtc", NULL, rtc_clk) > _REGISTER_CLOCK("mxs-dma-apbh", NULL, hbus_clk) > @@ -533,6 +535,7 @@ int __init mx23_clocks_init(void) > clk_enable(&xbus_clk); > clk_enable(&emi_clk); > clk_enable(&uart_clk); > + clk_enable(&gpmi_clk); Doesn't the driver handle enabling the clock? > clkdev_add_table(lookups, ARRAY_SIZE(lookups)); > > diff --git a/arch/arm/mach-mxs/clock-mx28.c b/arch/arm/mach-mxs/clock-mx28.c > index 5e489a2..7868d42 100644 > --- a/arch/arm/mach-mxs/clock-mx28.c > +++ b/arch/arm/mach-mxs/clock-mx28.c > @@ -29,6 +29,7 @@ > #include <mach/mx28.h> > #include <mach/common.h> > #include <mach/clock.h> > +#include <mach/gpmi-nfc.h> > > #include "regs-clkctrl-mx28.h" > > @@ -607,6 +608,7 @@ static struct clk_lookup lookups[] = { > _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) > /* for amba-pl011 driver */ > _REGISTER_CLOCK("duart", NULL, uart_clk) > + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX28, NULL, gpmi_clk) > _REGISTER_CLOCK("imx28-fec.0", NULL, fec_clk) > _REGISTER_CLOCK("imx28-fec.1", NULL, fec_clk) > _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) > @@ -749,6 +751,7 @@ int __init mx28_clocks_init(void) > clk_enable(&xbus_clk); > clk_enable(&emi_clk); > clk_enable(&uart_clk); > + clk_enable(&gpmi_clk); ditto^2 > clk_set_parent(&lcdif_clk, &ref_pix_clk); > > diff --git a/arch/arm/mach-mxs/devices-mx23.h b/arch/arm/mach-mxs/devices-mx23.h > index c7e14f4..349fb1d 100644 > --- a/arch/arm/mach-mxs/devices-mx23.h > +++ b/arch/arm/mach-mxs/devices-mx23.h > @@ -16,6 +16,9 @@ extern const struct amba_device mx23_duart_device __initconst; > #define mx23_add_duart() \ > mxs_add_duart(&mx23_duart_device) > > +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst; > +#define mx23_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx23) > + > extern const struct mxs_auart_data mx23_auart_data[] __initconst; > #define mx23_add_auart(id) mxs_add_auart(&mx23_auart_data[id]) > #define mx23_add_auart0() mx23_add_auart(0) > diff --git a/arch/arm/mach-mxs/devices-mx28.h b/arch/arm/mach-mxs/devices-mx28.h > index 9d08555..ed6427b 100644 > --- a/arch/arm/mach-mxs/devices-mx28.h > +++ b/arch/arm/mach-mxs/devices-mx28.h > @@ -16,6 +16,9 @@ extern const struct amba_device mx28_duart_device __initconst; > #define mx28_add_duart() \ > mxs_add_duart(&mx28_duart_device) > > +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst; > +#define mx28_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx28) > + Can you please keep this file sorted? > extern const struct mxs_auart_data mx28_auart_data[] __initconst; > #define mx28_add_auart(id) mxs_add_auart(&mx28_auart_data[id]) > #define mx28_add_auart0() mx28_add_auart(0) > diff --git a/arch/arm/mach-mxs/devices/Kconfig b/arch/arm/mach-mxs/devices/Kconfig > index 1451ad0..81e99ce 100644 > --- a/arch/arm/mach-mxs/devices/Kconfig > +++ b/arch/arm/mach-mxs/devices/Kconfig > @@ -5,6 +5,9 @@ config MXS_HAVE_AMBA_DUART > config MXS_HAVE_PLATFORM_AUART > bool > > +config MXS_HAVE_PLATFORM_GPMI > + bool > + > config MXS_HAVE_PLATFORM_FEC > bool Can you please keep this file sorted? > > diff --git a/arch/arm/mach-mxs/devices/Makefile b/arch/arm/mach-mxs/devices/Makefile > index 0d9bea3..8a7c2c8 100644 > --- a/arch/arm/mach-mxs/devices/Makefile > +++ b/arch/arm/mach-mxs/devices/Makefile > @@ -2,6 +2,7 @@ obj-$(CONFIG_MXS_HAVE_AMBA_DUART) += amba-duart.o > obj-$(CONFIG_MXS_HAVE_PLATFORM_AUART) += platform-auart.o > obj-y += platform-dma.o > obj-$(CONFIG_MXS_HAVE_PLATFORM_FEC) += platform-fec.o > +obj-$(CONFIG_MXS_HAVE_PLATFORM_GPMI) += platform-gpmi.o > obj-$(CONFIG_MXS_HAVE_PLATFORM_FLEXCAN) += platform-flexcan.o Can you please keep this file sorted? > obj-$(CONFIG_MXS_HAVE_PLATFORM_MXS_I2C) += platform-mxs-i2c.o > obj-$(CONFIG_MXS_HAVE_PLATFORM_MXS_PWM) += platform-mxs-pwm.o > diff --git a/arch/arm/mach-mxs/devices/platform-gpmi.c b/arch/arm/mach-mxs/devices/platform-gpmi.c > new file mode 100644 > index 0000000..74ebe22 > --- /dev/null > +++ b/arch/arm/mach-mxs/devices/platform-gpmi.c > @@ -0,0 +1,134 @@ > +/* > + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License along > + * with this program; if not, write to the Free Software Foundation, Inc., > + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. > + */ > +#include <asm/sizes.h> > +#include <mach/mx23.h> > +#include <mach/mx28.h> > +#include <mach/gpmi-nfc.h> > +#include <mach/devices-common.h> > + > +#ifdef CONFIG_SOC_IMX23 > +const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst = { > + .min_prop_delay_in_ns = 5, > + .max_prop_delay_in_ns = 9, > + .max_chip_count = 1, > +}; > + > +const struct resource res_imx23[] __initconst = { > + { /* GPMI */ > + .start = MX23_GPMI_BASE_ADDR, > + .end = MX23_GPMI_BASE_ADDR + SZ_8K - 1, > + .name = GPMI_NFC_GPMI_REGS_ADDR_RES_NAME, > + .flags = IORESOURCE_MEM, > + }, { > + .start = MX23_INT_GPMI_ATTENTION, > + .end = MX23_INT_GPMI_ATTENTION, > + .name = GPMI_NFC_GPMI_INTERRUPT_RES_NAME, > + .flags = IORESOURCE_IRQ, > + }, { /* BCH */ > + .start = MX23_BCH_BASE_ADDR, > + .end = MX23_BCH_BASE_ADDR + SZ_8K - 1, > + .name = GPMI_NFC_BCH_REGS_ADDR_RES_NAME, > + .flags = IORESOURCE_MEM, > + }, { > + .start = MX23_INT_BCH, > + .end = MX23_INT_BCH, > + .name = GPMI_NFC_BCH_INTERRUPT_RES_NAME, > + .flags = IORESOURCE_IRQ, > + }, { /* DMA */ > + .start = MX23_DMA_GPMI0, > + .end = MX23_DMA_GPMI3, > + .name = GPMI_NFC_DMA_CHANNELS_RES_NAME, > + .flags = IORESOURCE_DMA, > + }, { > + .start = MX23_INT_GPMI_DMA, > + .end = MX23_INT_GPMI_DMA, > + .name = GPMI_NFC_DMA_INTERRUPT_RES_NAME, > + .flags = IORESOURCE_IRQ, > + }, > +}; maybe use a macro to define this and res_imx28? > +#endif > + > +#ifdef CONFIG_SOC_IMX28 > +const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst = { > + .min_prop_delay_in_ns = 5, > + .max_prop_delay_in_ns = 9, > + .max_chip_count = 1, > +}; > + > +const struct resource res_imx28[] __initconst = { > + { /* GPMI */ > + .start = MX28_GPMI_BASE_ADDR, > + .end = MX28_GPMI_BASE_ADDR + SZ_8K - 1, > + .name = GPMI_NFC_GPMI_REGS_ADDR_RES_NAME, > + .flags = IORESOURCE_MEM, > + }, { > + .start = MX28_INT_GPMI, > + .end = MX28_INT_GPMI, > + .name = GPMI_NFC_GPMI_INTERRUPT_RES_NAME, > + .flags = IORESOURCE_IRQ, > + }, { /* BCH */ > + .start = MX28_BCH_BASE_ADDR, > + .end = MX28_BCH_BASE_ADDR + SZ_8K - 1, > + .name = GPMI_NFC_BCH_REGS_ADDR_RES_NAME, > + .flags = IORESOURCE_MEM, > + }, { > + .start = MX28_INT_BCH, > + .end = MX28_INT_BCH, > + .name = GPMI_NFC_BCH_INTERRUPT_RES_NAME, > + .flags = IORESOURCE_IRQ, > + }, { /* DMA */ > + .start = MX28_DMA_GPMI0, > + .end = MX28_DMA_GPMI7, > + .name = GPMI_NFC_DMA_CHANNELS_RES_NAME, > + .flags = IORESOURCE_DMA, > + }, { > + .start = MX28_INT_GPMI_DMA, > + .end = MX28_INT_GPMI_DMA, > + .name = GPMI_NFC_DMA_INTERRUPT_RES_NAME, > + .flags = IORESOURCE_IRQ, > + }, > +}; > +#endif > + > +struct platform_device *__init > +mxs_add_gpmi(const struct gpmi_nfc_platform_data *data) > +{ > + const struct resource *res; > + int res_size; > + const char *name = NULL; > + > +#ifdef CONFIG_SOC_IMX23 > + if (cpu_is_mx23()) { You can save these cpu_is_mx23 calls by letting mxs23_add_gpmi pass res_imx23 (with a better name) to mxs_add_gpmi as is done by the other functions. (Or alternatively implement mxs23_add_gpmi as function.) > + res_size = ARRAY_SIZE(res_imx23); > + res = res_imx23; > + name = GPMI_NFC_DRIVER_MX23; > + } > +#endif > + > +#ifdef CONFIG_SOC_IMX28 > + if (cpu_is_mx28()) { > + res_size = ARRAY_SIZE(res_imx28); > + res = res_imx28; > + name = GPMI_NFC_DRIVER_MX28; > + } > +#endif > + > + return mxs_add_platform_device_dmamask(name, -1, > + res, res_size, > + data, sizeof(*data), DMA_BIT_MASK(32)); > +} > diff --git a/arch/arm/mach-mxs/include/mach/devices-common.h b/arch/arm/mach-mxs/include/mach/devices-common.h > index 71f2448..1f05503 100644 > --- a/arch/arm/mach-mxs/include/mach/devices-common.h > +++ b/arch/arm/mach-mxs/include/mach/devices-common.h > @@ -30,6 +30,10 @@ int __init mxs_add_amba_device(const struct amba_device *dev); > /* duart */ > int __init mxs_add_duart(const struct amba_device *dev); > > +/* GPMI */ > +#include <mach/gpmi-nfc.h> > +struct platform_device *__init mxs_add_gpmi( > + const struct gpmi_nfc_platform_data *data); > /* auart */ > struct mxs_auart_data { > int id; > diff --git a/arch/arm/mach-mxs/include/mach/gpmi-nfc.h b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h > new file mode 100644 > index 0000000..cf8e8c3 > --- /dev/null > +++ b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h > @@ -0,0 +1,65 @@ > +/* > + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License along > + * with this program; if not, write to the Free Software Foundation, Inc., > + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. > + */ > + > +#ifndef __INCLUDE_LINUX_DEVICE_H > +#define __INCLUDE_LINUX_DEVICE_H > + > +#define GPMI_NFC_DRIVER_NAME "gpmi-nfc" > +#define GPMI_NFC_DRIVER_MX23 "gpmi-nfc-mx23" > +#define GPMI_NFC_DRIVER_MX28 "gpmi-nfc-mx28" > +#define GPMI_NFC_DRIVER_MX50 "gpmi-nfc-mx508" I'd prefer "imx23-gpmi" et al here. Is the 8 in "gpmi-nfc-mx508" intended? > + > +/* Resource names for the GPMI NFC driver. */ > +#define GPMI_NFC_GPMI_REGS_ADDR_RES_NAME "GPMI NFC GPMI Registers" > +#define GPMI_NFC_GPMI_INTERRUPT_RES_NAME "GPMI NFC GPMI Interrupt" > +#define GPMI_NFC_BCH_REGS_ADDR_RES_NAME "GPMI NFC BCH Registers" > +#define GPMI_NFC_BCH_INTERRUPT_RES_NAME "GPMI NFC BCH Interrupt" > +#define GPMI_NFC_DMA_CHANNELS_RES_NAME "GPMI NFC DMA Channels" > +#define GPMI_NFC_DMA_INTERRUPT_RES_NAME "GPMI NFC DMA Interrupt" > + > +/** > + * struct gpmi_nfc_platform_data - GPMI NFC driver platform data. > + * > + * This structure communicates platform-specific information to the GPMI NFC > + * driver that can't be expressed as resources. > + * > + * @min_prop_delay_in_ns: Minimum propagation delay of GPMI signals to and > + * from the NAND Flash device, in nanoseconds. > + * @max_prop_delay_in_ns: Maximum propagation delay of GPMI signals to and > + * from the NAND Flash device, in nanoseconds. > + * @max_chip_count: The maximum number of chips for which the driver > + * should configure the hardware. This value most > + * likely reflects the number of pins that are > + * connected to a NAND Flash device. If this is > + * greater than the SoC hardware can support, the > + * driver will print a message and fail to initialize. > + * @partitions: An optional pointer to an array of partition > + * descriptions. > + * @partition_count: The number of elements in the partitions array. > + */ > +struct gpmi_nfc_platform_data { > + /* NAND Flash information. */ > + unsigned int min_prop_delay_in_ns; > + unsigned int max_prop_delay_in_ns; > + unsigned int max_chip_count; > + > + /* soc */ > + struct mtd_partition *partitions; > + unsigned partition_count; > +}; > +#endif > diff --git a/arch/arm/mach-mxs/mach-mx23evk.c b/arch/arm/mach-mxs/mach-mx23evk.c > index a66994f..db715f9 100644 > --- a/arch/arm/mach-mxs/mach-mx23evk.c > +++ b/arch/arm/mach-mxs/mach-mx23evk.c > @@ -34,6 +34,42 @@ static const iomux_cfg_t mx23evk_pads[] __initconst = { > MX23_PAD_PWM0__DUART_RX | MXS_PAD_CTRL, > MX23_PAD_PWM1__DUART_TX | MXS_PAD_CTRL, > > + /* gpmi */ > + MX23_PAD_GPMI_D00__GPMI_D00 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D01__GPMI_D01 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D02__GPMI_D02 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D03__GPMI_D03 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D04__GPMI_D04 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D05__GPMI_D05 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D06__GPMI_D06 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D07__GPMI_D07 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_CLE__GPMI_CLE | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_ALE__GPMI_ALE | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_WPN__GPMI_WPN | > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_WRN__GPMI_WRN | > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_RDN__GPMI_RDN | > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_RDY0__GPMI_RDY0 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_RDY1__GPMI_RDY1 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_CE0N__GPMI_CE0N | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_CE1N__GPMI_CE1N | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + > /* auart */ > MX23_PAD_AUART1_RX__AUART1_RX | MXS_PAD_CTRL, > MX23_PAD_AUART1_TX__AUART1_TX | MXS_PAD_CTRL, > @@ -108,6 +144,7 @@ static void __init mx23evk_init(void) > mxs_iomux_setup_multiple_pads(mx23evk_pads, ARRAY_SIZE(mx23evk_pads)); > > mx23_add_duart(); > + mx23_add_gpmi(); > mx23_add_auart0(); > > ret = gpio_request_one(MX23EVK_LCD_ENABLE, GPIOF_DIR_OUT, "lcd-enable"); IMHO adding the device to machines should be a seperate patch. > diff --git a/arch/arm/mach-mxs/mach-mx28evk.c b/arch/arm/mach-mxs/mach-mx28evk.c > index 08002d0..cf8ddcb 100644 > --- a/arch/arm/mach-mxs/mach-mx28evk.c > +++ b/arch/arm/mach-mxs/mach-mx28evk.c > @@ -39,6 +39,42 @@ static const iomux_cfg_t mx28evk_pads[] __initconst = { > MX28_PAD_PWM0__DUART_RX | MXS_PAD_CTRL, > MX28_PAD_PWM1__DUART_TX | MXS_PAD_CTRL, > > + /* gpmi */ > + MX28_PAD_GPMI_D00__GPMI_D0 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_D01__GPMI_D1 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_D02__GPMI_D2 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_D03__GPMI_D3 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_D04__GPMI_D4 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_D05__GPMI_D5 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_D06__GPMI_D6 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_D07__GPMI_D7 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_CE0N__GPMI_CE0N | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_CE1N__GPMI_CE1N | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_RDY0__GPMI_READY0 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_RDY1__GPMI_READY1 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_RDN__GPMI_RDN | > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_WRN__GPMI_WRN | > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_ALE__GPMI_ALE | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_CLE__GPMI_CLE | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX28_PAD_GPMI_RESETN__GPMI_RESETN | > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + > /* auart0 */ > MX28_PAD_AUART0_RX__AUART0_RX | MXS_PAD_CTRL, > MX28_PAD_AUART0_TX__AUART0_TX | MXS_PAD_CTRL, > @@ -265,6 +301,7 @@ static void __init mx28evk_init(void) > mxs_iomux_setup_multiple_pads(mx28evk_pads, ARRAY_SIZE(mx28evk_pads)); > > mx28_add_duart(); > + mx28_add_gpmi(); > mx28_add_auart0(); > mx28_add_auart3(); Best regards Uwe
Hi: > Hi, > > > Huang Shijie writes: >> add the clock and iomux initialization for GPMI in the imx23 and imx28. >> and register the gpmi driver. >> >> Signed-off-by: Huang Shijie<b32955@freescale.com> >> --- >> arch/arm/mach-mxs/Kconfig | 2 + >> arch/arm/mach-mxs/clock-mx23.c | 3 + >> arch/arm/mach-mxs/clock-mx28.c | 3 + >> arch/arm/mach-mxs/devices-mx23.h | 3 + >> arch/arm/mach-mxs/devices-mx28.h | 3 + >> arch/arm/mach-mxs/devices/Kconfig | 3 + >> arch/arm/mach-mxs/devices/Makefile | 1 + >> arch/arm/mach-mxs/devices/platform-gpmi.c | 134 +++++++++++++++++++++++ >> arch/arm/mach-mxs/include/mach/devices-common.h | 4 + >> arch/arm/mach-mxs/include/mach/gpmi-nfc.h | 65 +++++++++++ >> arch/arm/mach-mxs/mach-mx23evk.c | 37 ++++++ >> arch/arm/mach-mxs/mach-mx28evk.c | 37 ++++++ >> 12 files changed, 295 insertions(+), 0 deletions(-) >> create mode 100644 arch/arm/mach-mxs/devices/platform-gpmi.c >> create mode 100644 arch/arm/mach-mxs/include/mach/gpmi-nfc.h >> >> diff --git a/arch/arm/mach-mxs/Kconfig b/arch/arm/mach-mxs/Kconfig >> index 4f6f174..e034666 100644 >> --- a/arch/arm/mach-mxs/Kconfig >> +++ b/arch/arm/mach-mxs/Kconfig >> @@ -22,6 +22,7 @@ config MACH_MX23EVK >> select SOC_IMX23 >> select MXS_HAVE_AMBA_DUART >> select MXS_HAVE_PLATFORM_AUART >> + select MXS_HAVE_PLATFORM_GPMI >> select MXS_HAVE_PLATFORM_MXSFB >> default y >> help >> @@ -35,6 +36,7 @@ config MACH_MX28EVK >> select MXS_HAVE_PLATFORM_AUART >> select MXS_HAVE_PLATFORM_FEC >> select MXS_HAVE_PLATFORM_FLEXCAN >> + select MXS_HAVE_PLATFORM_GPMI >> select MXS_HAVE_PLATFORM_MXSFB >> select MXS_OCOTP >> default y >> diff --git a/arch/arm/mach-mxs/clock-mx23.c b/arch/arm/mach-mxs/clock-mx23.c >> index d133c7f..f3c9653 100644 >> --- a/arch/arm/mach-mxs/clock-mx23.c >> +++ b/arch/arm/mach-mxs/clock-mx23.c >> @@ -29,6 +29,7 @@ >> #include<mach/mx23.h> >> #include<mach/common.h> >> #include<mach/clock.h> >> +#include<mach/gpmi-nfc.h> >> >> #include "regs-clkctrl-mx23.h" >> >> @@ -442,6 +443,7 @@ static struct clk_lookup lookups[] = { >> _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) >> /* for amba-pl011 driver */ >> _REGISTER_CLOCK("duart", NULL, uart_clk) >> + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX23, NULL, gpmi_clk) >> _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) >> _REGISTER_CLOCK("rtc", NULL, rtc_clk) >> _REGISTER_CLOCK("mxs-dma-apbh", NULL, hbus_clk) >> @@ -533,6 +535,7 @@ int __init mx23_clocks_init(void) >> clk_enable(&xbus_clk); >> clk_enable(&emi_clk); >> clk_enable(&uart_clk); >> + clk_enable(&gpmi_clk); >> >> clkdev_add_table(lookups, ARRAY_SIZE(lookups)); >> >> diff --git a/arch/arm/mach-mxs/clock-mx28.c b/arch/arm/mach-mxs/clock-mx28.c >> index 5e489a2..7868d42 100644 >> --- a/arch/arm/mach-mxs/clock-mx28.c >> +++ b/arch/arm/mach-mxs/clock-mx28.c >> @@ -29,6 +29,7 @@ >> #include<mach/mx28.h> >> #include<mach/common.h> >> #include<mach/clock.h> >> +#include<mach/gpmi-nfc.h> >> >> #include "regs-clkctrl-mx28.h" >> >> @@ -607,6 +608,7 @@ static struct clk_lookup lookups[] = { >> _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) >> /* for amba-pl011 driver */ >> _REGISTER_CLOCK("duart", NULL, uart_clk) >> + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX28, NULL, gpmi_clk) >> _REGISTER_CLOCK("imx28-fec.0", NULL, fec_clk) >> _REGISTER_CLOCK("imx28-fec.1", NULL, fec_clk) >> _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) >> @@ -749,6 +751,7 @@ int __init mx28_clocks_init(void) >> clk_enable(&xbus_clk); >> clk_enable(&emi_clk); >> clk_enable(&uart_clk); >> + clk_enable(&gpmi_clk); >> >> clk_set_parent(&lcdif_clk,&ref_pix_clk); >> >> diff --git a/arch/arm/mach-mxs/devices-mx23.h b/arch/arm/mach-mxs/devices-mx23.h >> index c7e14f4..349fb1d 100644 >> --- a/arch/arm/mach-mxs/devices-mx23.h >> +++ b/arch/arm/mach-mxs/devices-mx23.h >> @@ -16,6 +16,9 @@ extern const struct amba_device mx23_duart_device __initconst; >> #define mx23_add_duart() \ >> mxs_add_duart(&mx23_duart_device) >> >> +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst; >> +#define mx23_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx23) >> > platform_data should be passed as parameter to the mx23_add_gpmi() > call, so that it can be set per platform, not per SoC. > > ok. thanks >> diff --git a/arch/arm/mach-mxs/devices-mx28.h b/arch/arm/mach-mxs/devices-mx28.h >> index 9d08555..ed6427b 100644 >> --- a/arch/arm/mach-mxs/devices-mx28.h >> +++ b/arch/arm/mach-mxs/devices-mx28.h >> @@ -16,6 +16,9 @@ extern const struct amba_device mx28_duart_device __initconst; >> #define mx28_add_duart() \ >> mxs_add_duart(&mx28_duart_device) >> >> +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst; >> +#define mx28_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx28) >> + > dito. > >> diff --git a/arch/arm/mach-mxs/devices/platform-gpmi.c b/arch/arm/mach-mxs/devices/platform-gpmi.c >> new file mode 100644 >> index 0000000..74ebe22 >> --- /dev/null >> +++ b/arch/arm/mach-mxs/devices/platform-gpmi.c >> @@ -0,0 +1,134 @@ >> +/* >> + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License as published by >> + * the Free Software Foundation; either version 2 of the License, or >> + * (at your option) any later version. >> + * >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + * >> + * You should have received a copy of the GNU General Public License along >> + * with this program; if not, write to the Free Software Foundation, Inc., >> + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >> + */ >> +#include<asm/sizes.h> >> +#include<mach/mx23.h> >> +#include<mach/mx28.h> >> +#include<mach/gpmi-nfc.h> >> +#include<mach/devices-common.h> >> + >> +#ifdef CONFIG_SOC_IMX23 >> +const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst = { >> + .min_prop_delay_in_ns = 5, >> + .max_prop_delay_in_ns = 9, >> + .max_chip_count = 1, >> +}; >> + > This is platform-dependent and thus should be placed in the mach-* > files (and passed to mx23_add_gpmi()). > >> +const struct resource res_imx23[] __initconst = { >> + { /* GPMI */ >> + .start = MX23_GPMI_BASE_ADDR, >> + .end = MX23_GPMI_BASE_ADDR + SZ_8K - 1, >> + .name = GPMI_NFC_GPMI_REGS_ADDR_RES_NAME, >> + .flags = IORESOURCE_MEM, >> + }, { >> + .start = MX23_INT_GPMI_ATTENTION, >> + .end = MX23_INT_GPMI_ATTENTION, >> + .name = GPMI_NFC_GPMI_INTERRUPT_RES_NAME, >> + .flags = IORESOURCE_IRQ, >> + }, { /* BCH */ >> + .start = MX23_BCH_BASE_ADDR, >> + .end = MX23_BCH_BASE_ADDR + SZ_8K - 1, >> + .name = GPMI_NFC_BCH_REGS_ADDR_RES_NAME, >> + .flags = IORESOURCE_MEM, >> + }, { >> + .start = MX23_INT_BCH, >> + .end = MX23_INT_BCH, >> + .name = GPMI_NFC_BCH_INTERRUPT_RES_NAME, >> + .flags = IORESOURCE_IRQ, >> + }, { /* DMA */ >> + .start = MX23_DMA_GPMI0, >> + .end = MX23_DMA_GPMI3, >> + .name = GPMI_NFC_DMA_CHANNELS_RES_NAME, >> + .flags = IORESOURCE_DMA, >> + }, { >> + .start = MX23_INT_GPMI_DMA, >> + .end = MX23_INT_GPMI_DMA, >> + .name = GPMI_NFC_DMA_INTERRUPT_RES_NAME, >> + .flags = IORESOURCE_IRQ, >> + }, >> +}; >> +#endif >> + >> +#ifdef CONFIG_SOC_IMX28 >> +const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst = { >> + .min_prop_delay_in_ns = 5, >> + .max_prop_delay_in_ns = 9, >> + .max_chip_count = 1, >> +}; >> + > dito. > Best Regards Huang Shijie > Lothar Waßmann
Hi. > On Wed, Apr 13, 2011 at 02:24:38PM +0800, Huang Shijie wrote: >> add the clock and iomux initialization for GPMI in the imx23 and imx28. >> and register the gpmi driver. >> >> Signed-off-by: Huang Shijie<b32955@freescale.com> >> --- >> arch/arm/mach-mxs/Kconfig | 2 + >> arch/arm/mach-mxs/clock-mx23.c | 3 + >> arch/arm/mach-mxs/clock-mx28.c | 3 + >> arch/arm/mach-mxs/devices-mx23.h | 3 + >> arch/arm/mach-mxs/devices-mx28.h | 3 + >> arch/arm/mach-mxs/devices/Kconfig | 3 + >> arch/arm/mach-mxs/devices/Makefile | 1 + >> arch/arm/mach-mxs/devices/platform-gpmi.c | 134 +++++++++++++++++++++++ >> arch/arm/mach-mxs/include/mach/devices-common.h | 4 + >> arch/arm/mach-mxs/include/mach/gpmi-nfc.h | 65 +++++++++++ >> arch/arm/mach-mxs/mach-mx23evk.c | 37 ++++++ >> arch/arm/mach-mxs/mach-mx28evk.c | 37 ++++++ >> 12 files changed, 295 insertions(+), 0 deletions(-) >> create mode 100644 arch/arm/mach-mxs/devices/platform-gpmi.c >> create mode 100644 arch/arm/mach-mxs/include/mach/gpmi-nfc.h >> >> diff --git a/arch/arm/mach-mxs/Kconfig b/arch/arm/mach-mxs/Kconfig >> index 4f6f174..e034666 100644 >> --- a/arch/arm/mach-mxs/Kconfig >> +++ b/arch/arm/mach-mxs/Kconfig >> @@ -22,6 +22,7 @@ config MACH_MX23EVK >> select SOC_IMX23 >> select MXS_HAVE_AMBA_DUART >> select MXS_HAVE_PLATFORM_AUART >> + select MXS_HAVE_PLATFORM_GPMI >> select MXS_HAVE_PLATFORM_MXSFB >> default y >> help >> @@ -35,6 +36,7 @@ config MACH_MX28EVK >> select MXS_HAVE_PLATFORM_AUART >> select MXS_HAVE_PLATFORM_FEC >> select MXS_HAVE_PLATFORM_FLEXCAN >> + select MXS_HAVE_PLATFORM_GPMI >> select MXS_HAVE_PLATFORM_MXSFB >> select MXS_OCOTP >> default y >> diff --git a/arch/arm/mach-mxs/clock-mx23.c b/arch/arm/mach-mxs/clock-mx23.c >> index d133c7f..f3c9653 100644 >> --- a/arch/arm/mach-mxs/clock-mx23.c >> +++ b/arch/arm/mach-mxs/clock-mx23.c >> @@ -29,6 +29,7 @@ >> #include<mach/mx23.h> >> #include<mach/common.h> >> #include<mach/clock.h> >> +#include<mach/gpmi-nfc.h> > This include is only needed to get GPMI_NFC_DRIVER_MX23? Then I suggest > to hardcode the clock name. > ok. >> >> #include "regs-clkctrl-mx23.h" >> >> @@ -442,6 +443,7 @@ static struct clk_lookup lookups[] = { >> _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) >> /* for amba-pl011 driver */ >> _REGISTER_CLOCK("duart", NULL, uart_clk) >> + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX23, NULL, gpmi_clk) >> _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) >> _REGISTER_CLOCK("rtc", NULL, rtc_clk) >> _REGISTER_CLOCK("mxs-dma-apbh", NULL, hbus_clk) >> @@ -533,6 +535,7 @@ int __init mx23_clocks_init(void) >> clk_enable(&xbus_clk); >> clk_enable(&emi_clk); >> clk_enable(&uart_clk); >> + clk_enable(&gpmi_clk); > Doesn't the driver handle enabling the clock? > >> clkdev_add_table(lookups, ARRAY_SIZE(lookups)); >> >> diff --git a/arch/arm/mach-mxs/clock-mx28.c b/arch/arm/mach-mxs/clock-mx28.c >> index 5e489a2..7868d42 100644 >> --- a/arch/arm/mach-mxs/clock-mx28.c >> +++ b/arch/arm/mach-mxs/clock-mx28.c >> @@ -29,6 +29,7 @@ >> #include<mach/mx28.h> >> #include<mach/common.h> >> #include<mach/clock.h> >> +#include<mach/gpmi-nfc.h> >> >> #include "regs-clkctrl-mx28.h" >> >> @@ -607,6 +608,7 @@ static struct clk_lookup lookups[] = { >> _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) >> /* for amba-pl011 driver */ >> _REGISTER_CLOCK("duart", NULL, uart_clk) >> + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX28, NULL, gpmi_clk) >> _REGISTER_CLOCK("imx28-fec.0", NULL, fec_clk) >> _REGISTER_CLOCK("imx28-fec.1", NULL, fec_clk) >> _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) >> @@ -749,6 +751,7 @@ int __init mx28_clocks_init(void) >> clk_enable(&xbus_clk); >> clk_enable(&emi_clk); >> clk_enable(&uart_clk); >> + clk_enable(&gpmi_clk); > ditto^2 > >> clk_set_parent(&lcdif_clk,&ref_pix_clk); >> >> diff --git a/arch/arm/mach-mxs/devices-mx23.h b/arch/arm/mach-mxs/devices-mx23.h >> index c7e14f4..349fb1d 100644 >> --- a/arch/arm/mach-mxs/devices-mx23.h >> +++ b/arch/arm/mach-mxs/devices-mx23.h >> @@ -16,6 +16,9 @@ extern const struct amba_device mx23_duart_device __initconst; >> #define mx23_add_duart() \ >> mxs_add_duart(&mx23_duart_device) >> >> +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst; >> +#define mx23_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx23) >> + >> extern const struct mxs_auart_data mx23_auart_data[] __initconst; >> #define mx23_add_auart(id) mxs_add_auart(&mx23_auart_data[id]) >> #define mx23_add_auart0() mx23_add_auart(0) >> diff --git a/arch/arm/mach-mxs/devices-mx28.h b/arch/arm/mach-mxs/devices-mx28.h >> index 9d08555..ed6427b 100644 >> --- a/arch/arm/mach-mxs/devices-mx28.h >> +++ b/arch/arm/mach-mxs/devices-mx28.h >> @@ -16,6 +16,9 @@ extern const struct amba_device mx28_duart_device __initconst; >> #define mx28_add_duart() \ >> mxs_add_duart(&mx28_duart_device) >> >> +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst; >> +#define mx28_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx28) >> + > Can you please keep this file sorted? > >> extern const struct mxs_auart_data mx28_auart_data[] __initconst; >> #define mx28_add_auart(id) mxs_add_auart(&mx28_auart_data[id]) >> #define mx28_add_auart0() mx28_add_auart(0) >> diff --git a/arch/arm/mach-mxs/devices/Kconfig b/arch/arm/mach-mxs/devices/Kconfig >> index 1451ad0..81e99ce 100644 >> --- a/arch/arm/mach-mxs/devices/Kconfig >> +++ b/arch/arm/mach-mxs/devices/Kconfig >> @@ -5,6 +5,9 @@ config MXS_HAVE_AMBA_DUART >> config MXS_HAVE_PLATFORM_AUART >> bool >> >> +config MXS_HAVE_PLATFORM_GPMI >> + bool >> + >> config MXS_HAVE_PLATFORM_FEC >> bool > Can you please keep this file sorted? > >> >> diff --git a/arch/arm/mach-mxs/devices/Makefile b/arch/arm/mach-mxs/devices/Makefile >> index 0d9bea3..8a7c2c8 100644 >> --- a/arch/arm/mach-mxs/devices/Makefile >> +++ b/arch/arm/mach-mxs/devices/Makefile >> @@ -2,6 +2,7 @@ obj-$(CONFIG_MXS_HAVE_AMBA_DUART) += amba-duart.o >> obj-$(CONFIG_MXS_HAVE_PLATFORM_AUART) += platform-auart.o >> obj-y += platform-dma.o >> obj-$(CONFIG_MXS_HAVE_PLATFORM_FEC) += platform-fec.o >> +obj-$(CONFIG_MXS_HAVE_PLATFORM_GPMI) += platform-gpmi.o >> obj-$(CONFIG_MXS_HAVE_PLATFORM_FLEXCAN) += platform-flexcan.o > Can you please keep this file sorted? > >> obj-$(CONFIG_MXS_HAVE_PLATFORM_MXS_I2C) += platform-mxs-i2c.o >> obj-$(CONFIG_MXS_HAVE_PLATFORM_MXS_PWM) += platform-mxs-pwm.o >> diff --git a/arch/arm/mach-mxs/devices/platform-gpmi.c b/arch/arm/mach-mxs/devices/platform-gpmi.c >> new file mode 100644 >> index 0000000..74ebe22 >> --- /dev/null >> +++ b/arch/arm/mach-mxs/devices/platform-gpmi.c >> @@ -0,0 +1,134 @@ >> +/* >> + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License as published by >> + * the Free Software Foundation; either version 2 of the License, or >> + * (at your option) any later version. >> + * >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + * >> + * You should have received a copy of the GNU General Public License along >> + * with this program; if not, write to the Free Software Foundation, Inc., >> + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >> + */ >> +#include<asm/sizes.h> >> +#include<mach/mx23.h> >> +#include<mach/mx28.h> >> +#include<mach/gpmi-nfc.h> >> +#include<mach/devices-common.h> >> + >> +#ifdef CONFIG_SOC_IMX23 >> +const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst = { >> + .min_prop_delay_in_ns = 5, >> + .max_prop_delay_in_ns = 9, >> + .max_chip_count = 1, >> +}; >> + >> +const struct resource res_imx23[] __initconst = { >> + { /* GPMI */ >> + .start = MX23_GPMI_BASE_ADDR, >> + .end = MX23_GPMI_BASE_ADDR + SZ_8K - 1, >> + .name = GPMI_NFC_GPMI_REGS_ADDR_RES_NAME, >> + .flags = IORESOURCE_MEM, >> + }, { >> + .start = MX23_INT_GPMI_ATTENTION, >> + .end = MX23_INT_GPMI_ATTENTION, >> + .name = GPMI_NFC_GPMI_INTERRUPT_RES_NAME, >> + .flags = IORESOURCE_IRQ, >> + }, { /* BCH */ >> + .start = MX23_BCH_BASE_ADDR, >> + .end = MX23_BCH_BASE_ADDR + SZ_8K - 1, >> + .name = GPMI_NFC_BCH_REGS_ADDR_RES_NAME, >> + .flags = IORESOURCE_MEM, >> + }, { >> + .start = MX23_INT_BCH, >> + .end = MX23_INT_BCH, >> + .name = GPMI_NFC_BCH_INTERRUPT_RES_NAME, >> + .flags = IORESOURCE_IRQ, >> + }, { /* DMA */ >> + .start = MX23_DMA_GPMI0, >> + .end = MX23_DMA_GPMI3, >> + .name = GPMI_NFC_DMA_CHANNELS_RES_NAME, >> + .flags = IORESOURCE_DMA, >> + }, { >> + .start = MX23_INT_GPMI_DMA, >> + .end = MX23_INT_GPMI_DMA, >> + .name = GPMI_NFC_DMA_INTERRUPT_RES_NAME, >> + .flags = IORESOURCE_IRQ, >> + }, >> +}; > maybe use a macro to define this and res_imx28? > ok. >> +#endif >> + >> +#ifdef CONFIG_SOC_IMX28 >> +const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst = { >> + .min_prop_delay_in_ns = 5, >> + .max_prop_delay_in_ns = 9, >> + .max_chip_count = 1, >> +}; >> + >> +const struct resource res_imx28[] __initconst = { >> + { /* GPMI */ >> + .start = MX28_GPMI_BASE_ADDR, >> + .end = MX28_GPMI_BASE_ADDR + SZ_8K - 1, >> + .name = GPMI_NFC_GPMI_REGS_ADDR_RES_NAME, >> + .flags = IORESOURCE_MEM, >> + }, { >> + .start = MX28_INT_GPMI, >> + .end = MX28_INT_GPMI, >> + .name = GPMI_NFC_GPMI_INTERRUPT_RES_NAME, >> + .flags = IORESOURCE_IRQ, >> + }, { /* BCH */ >> + .start = MX28_BCH_BASE_ADDR, >> + .end = MX28_BCH_BASE_ADDR + SZ_8K - 1, >> + .name = GPMI_NFC_BCH_REGS_ADDR_RES_NAME, >> + .flags = IORESOURCE_MEM, >> + }, { >> + .start = MX28_INT_BCH, >> + .end = MX28_INT_BCH, >> + .name = GPMI_NFC_BCH_INTERRUPT_RES_NAME, >> + .flags = IORESOURCE_IRQ, >> + }, { /* DMA */ >> + .start = MX28_DMA_GPMI0, >> + .end = MX28_DMA_GPMI7, >> + .name = GPMI_NFC_DMA_CHANNELS_RES_NAME, >> + .flags = IORESOURCE_DMA, >> + }, { >> + .start = MX28_INT_GPMI_DMA, >> + .end = MX28_INT_GPMI_DMA, >> + .name = GPMI_NFC_DMA_INTERRUPT_RES_NAME, >> + .flags = IORESOURCE_IRQ, >> + }, >> +}; >> +#endif >> + >> +struct platform_device *__init >> +mxs_add_gpmi(const struct gpmi_nfc_platform_data *data) >> +{ >> + const struct resource *res; >> + int res_size; >> + const char *name = NULL; >> + >> +#ifdef CONFIG_SOC_IMX23 >> + if (cpu_is_mx23()) { > You can save these cpu_is_mx23 calls by letting mxs23_add_gpmi pass > res_imx23 (with a better name) to mxs_add_gpmi as is done by the other > functions. (Or alternatively implement mxs23_add_gpmi as function.) > thanks. >> + res_size = ARRAY_SIZE(res_imx23); >> + res = res_imx23; >> + name = GPMI_NFC_DRIVER_MX23; >> + } >> +#endif >> + >> +#ifdef CONFIG_SOC_IMX28 >> + if (cpu_is_mx28()) { >> + res_size = ARRAY_SIZE(res_imx28); >> + res = res_imx28; >> + name = GPMI_NFC_DRIVER_MX28; >> + } >> +#endif >> + >> + return mxs_add_platform_device_dmamask(name, -1, >> + res, res_size, >> + data, sizeof(*data), DMA_BIT_MASK(32)); >> +} >> diff --git a/arch/arm/mach-mxs/include/mach/devices-common.h b/arch/arm/mach-mxs/include/mach/devices-common.h >> index 71f2448..1f05503 100644 >> --- a/arch/arm/mach-mxs/include/mach/devices-common.h >> +++ b/arch/arm/mach-mxs/include/mach/devices-common.h >> @@ -30,6 +30,10 @@ int __init mxs_add_amba_device(const struct amba_device *dev); >> /* duart */ >> int __init mxs_add_duart(const struct amba_device *dev); >> >> +/* GPMI */ >> +#include<mach/gpmi-nfc.h> >> +struct platform_device *__init mxs_add_gpmi( >> + const struct gpmi_nfc_platform_data *data); >> /* auart */ >> struct mxs_auart_data { >> int id; >> diff --git a/arch/arm/mach-mxs/include/mach/gpmi-nfc.h b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h >> new file mode 100644 >> index 0000000..cf8e8c3 >> --- /dev/null >> +++ b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h >> @@ -0,0 +1,65 @@ >> +/* >> + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License as published by >> + * the Free Software Foundation; either version 2 of the License, or >> + * (at your option) any later version. >> + * >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + * >> + * You should have received a copy of the GNU General Public License along >> + * with this program; if not, write to the Free Software Foundation, Inc., >> + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >> + */ >> + >> +#ifndef __INCLUDE_LINUX_DEVICE_H >> +#define __INCLUDE_LINUX_DEVICE_H >> + >> +#define GPMI_NFC_DRIVER_NAME "gpmi-nfc" >> +#define GPMI_NFC_DRIVER_MX23 "gpmi-nfc-mx23" >> +#define GPMI_NFC_DRIVER_MX28 "gpmi-nfc-mx28" >> +#define GPMI_NFC_DRIVER_MX50 "gpmi-nfc-mx508" > I'd prefer "imx23-gpmi" et al here. Is the 8 in "gpmi-nfc-mx508" > intended? > The '508' is full-name of the mx50. But it seems strange. ok, I will remove it. >> + >> +/* Resource names for the GPMI NFC driver. */ >> +#define GPMI_NFC_GPMI_REGS_ADDR_RES_NAME "GPMI NFC GPMI Registers" >> +#define GPMI_NFC_GPMI_INTERRUPT_RES_NAME "GPMI NFC GPMI Interrupt" >> +#define GPMI_NFC_BCH_REGS_ADDR_RES_NAME "GPMI NFC BCH Registers" >> +#define GPMI_NFC_BCH_INTERRUPT_RES_NAME "GPMI NFC BCH Interrupt" >> +#define GPMI_NFC_DMA_CHANNELS_RES_NAME "GPMI NFC DMA Channels" >> +#define GPMI_NFC_DMA_INTERRUPT_RES_NAME "GPMI NFC DMA Interrupt" >> + >> +/** >> + * struct gpmi_nfc_platform_data - GPMI NFC driver platform data. >> + * >> + * This structure communicates platform-specific information to the GPMI NFC >> + * driver that can't be expressed as resources. >> + * >> + * @min_prop_delay_in_ns: Minimum propagation delay of GPMI signals to and >> + * from the NAND Flash device, in nanoseconds. >> + * @max_prop_delay_in_ns: Maximum propagation delay of GPMI signals to and >> + * from the NAND Flash device, in nanoseconds. >> + * @max_chip_count: The maximum number of chips for which the driver >> + * should configure the hardware. This value most >> + * likely reflects the number of pins that are >> + * connected to a NAND Flash device. If this is >> + * greater than the SoC hardware can support, the >> + * driver will print a message and fail to initialize. >> + * @partitions: An optional pointer to an array of partition >> + * descriptions. >> + * @partition_count: The number of elements in the partitions array. >> + */ >> +struct gpmi_nfc_platform_data { >> + /* NAND Flash information. */ >> + unsigned int min_prop_delay_in_ns; >> + unsigned int max_prop_delay_in_ns; >> + unsigned int max_chip_count; >> + >> + /* soc */ >> + struct mtd_partition *partitions; >> + unsigned partition_count; >> +}; >> +#endif >> diff --git a/arch/arm/mach-mxs/mach-mx23evk.c b/arch/arm/mach-mxs/mach-mx23evk.c >> index a66994f..db715f9 100644 >> --- a/arch/arm/mach-mxs/mach-mx23evk.c >> +++ b/arch/arm/mach-mxs/mach-mx23evk.c >> @@ -34,6 +34,42 @@ static const iomux_cfg_t mx23evk_pads[] __initconst = { >> MX23_PAD_PWM0__DUART_RX | MXS_PAD_CTRL, >> MX23_PAD_PWM1__DUART_TX | MXS_PAD_CTRL, >> >> + /* gpmi */ >> + MX23_PAD_GPMI_D00__GPMI_D00 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D01__GPMI_D01 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D02__GPMI_D02 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D03__GPMI_D03 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D04__GPMI_D04 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D05__GPMI_D05 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D06__GPMI_D06 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D07__GPMI_D07 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_CLE__GPMI_CLE | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_ALE__GPMI_ALE | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_WPN__GPMI_WPN | >> + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_WRN__GPMI_WRN | >> + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_RDN__GPMI_RDN | >> + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_RDY0__GPMI_RDY0 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_RDY1__GPMI_RDY1 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_CE0N__GPMI_CE0N | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_CE1N__GPMI_CE1N | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + >> /* auart */ >> MX23_PAD_AUART1_RX__AUART1_RX | MXS_PAD_CTRL, >> MX23_PAD_AUART1_TX__AUART1_TX | MXS_PAD_CTRL, >> @@ -108,6 +144,7 @@ static void __init mx23evk_init(void) >> mxs_iomux_setup_multiple_pads(mx23evk_pads, ARRAY_SIZE(mx23evk_pads)); >> >> mx23_add_duart(); >> + mx23_add_gpmi(); >> mx23_add_auart0(); >> >> ret = gpio_request_one(MX23EVK_LCD_ENABLE, GPIOF_DIR_OUT, "lcd-enable"); > IMHO adding the device to machines should be a seperate patch. Do I have to split the driver into two parts? one for ARM-KERNEL, and another for MTD? If I have to do it, I will split the driver. Best Regards >> diff --git a/arch/arm/mach-mxs/mach-mx28evk.c b/arch/arm/mach-mxs/mach-mx28evk.c >> index 08002d0..cf8ddcb 100644 >> --- a/arch/arm/mach-mxs/mach-mx28evk.c >> +++ b/arch/arm/mach-mxs/mach-mx28evk.c >> @@ -39,6 +39,42 @@ static const iomux_cfg_t mx28evk_pads[] __initconst = { >> MX28_PAD_PWM0__DUART_RX | MXS_PAD_CTRL, >> MX28_PAD_PWM1__DUART_TX | MXS_PAD_CTRL, >> >> + /* gpmi */ >> + MX28_PAD_GPMI_D00__GPMI_D0 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_D01__GPMI_D1 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_D02__GPMI_D2 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_D03__GPMI_D3 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_D04__GPMI_D4 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_D05__GPMI_D5 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_D06__GPMI_D6 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_D07__GPMI_D7 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_CE0N__GPMI_CE0N | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_CE1N__GPMI_CE1N | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_RDY0__GPMI_READY0 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_RDY1__GPMI_READY1 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_RDN__GPMI_RDN | >> + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_WRN__GPMI_WRN | >> + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_ALE__GPMI_ALE | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_CLE__GPMI_CLE | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX28_PAD_GPMI_RESETN__GPMI_RESETN | >> + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + >> /* auart0 */ >> MX28_PAD_AUART0_RX__AUART0_RX | MXS_PAD_CTRL, >> MX28_PAD_AUART0_TX__AUART0_TX | MXS_PAD_CTRL, >> @@ -265,6 +301,7 @@ static void __init mx28evk_init(void) >> mxs_iomux_setup_multiple_pads(mx28evk_pads, ARRAY_SIZE(mx28evk_pads)); >> >> mx28_add_duart(); >> + mx28_add_gpmi(); >> mx28_add_auart0(); >> mx28_add_auart3(); > Best regards > Uwe >
Hello, > >>diff --git a/arch/arm/mach-mxs/mach-mx23evk.c b/arch/arm/mach-mxs/mach-mx23evk.c > >IMHO adding the device to machines should be a seperate patch. > Do I have to split the driver into two parts? one for ARM-KERNEL, > and another for MTD? > If I have to do it, I will split the driver. I'm not sure I understood what you mean here. I havn't looked at the mtd part of this series, but I think having: - ARM: mxs: add GPMI support for imx23/imx28 - ARM: mxs: add gpmi device for mx2[38]evk - MTD: what - MTD: ever - MTD: is - MTD: needed is OK. Best regards Uwe
hi: > Hello, > >>>> diff --git a/arch/arm/mach-mxs/mach-mx23evk.c b/arch/arm/mach-mxs/mach-mx23evk.c >>> IMHO adding the device to machines should be a seperate patch. >> Do I have to split the driver into two parts? one for ARM-KERNEL, >> and another for MTD? >> If I have to do it, I will split the driver. > I'm not sure I understood what you mean here. I havn't looked at the mtd > part of this series, but I think having: > > - ARM: mxs: add GPMI support for imx23/imx28 > - ARM: mxs: add gpmi device for mx2[38]evk > - MTD: what > - MTD: ever > - MTD: is > - MTD: needed > > is OK. > ok, thanks. Best Regards Huang Shijie > Best regards > Uwe >
On Wed, Apr 13, 2011 at 02:24:38PM +0800, Huang Shijie wrote: > add the clock and iomux initialization for GPMI in the imx23 and imx28. > and register the gpmi driver. > > Signed-off-by: Huang Shijie <b32955@freescale.com> > --- > arch/arm/mach-mxs/Kconfig | 2 + > arch/arm/mach-mxs/clock-mx23.c | 3 + > arch/arm/mach-mxs/clock-mx28.c | 3 + > arch/arm/mach-mxs/devices-mx23.h | 3 + > arch/arm/mach-mxs/devices-mx28.h | 3 + > arch/arm/mach-mxs/devices/Kconfig | 3 + > arch/arm/mach-mxs/devices/Makefile | 1 + > arch/arm/mach-mxs/devices/platform-gpmi.c | 134 +++++++++++++++++++++++ > arch/arm/mach-mxs/include/mach/devices-common.h | 4 + > arch/arm/mach-mxs/include/mach/gpmi-nfc.h | 65 +++++++++++ > arch/arm/mach-mxs/mach-mx23evk.c | 37 ++++++ > arch/arm/mach-mxs/mach-mx28evk.c | 37 ++++++ > 12 files changed, 295 insertions(+), 0 deletions(-) > create mode 100644 arch/arm/mach-mxs/devices/platform-gpmi.c > create mode 100644 arch/arm/mach-mxs/include/mach/gpmi-nfc.h > [...] > diff --git a/arch/arm/mach-mxs/devices/platform-gpmi.c b/arch/arm/mach-mxs/devices/platform-gpmi.c > new file mode 100644 > index 0000000..74ebe22 > --- /dev/null > +++ b/arch/arm/mach-mxs/devices/platform-gpmi.c > @@ -0,0 +1,134 @@ > +/* > + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License along > + * with this program; if not, write to the Free Software Foundation, Inc., > + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. > + */ > +#include <asm/sizes.h> > +#include <mach/mx23.h> > +#include <mach/mx28.h> > +#include <mach/gpmi-nfc.h> > +#include <mach/devices-common.h> devices-common.h already includes gpmi-nfc.h [...] > diff --git a/arch/arm/mach-mxs/include/mach/gpmi-nfc.h b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h > new file mode 100644 > index 0000000..cf8e8c3 > --- /dev/null > +++ b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h > @@ -0,0 +1,65 @@ > +/* > + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License along > + * with this program; if not, write to the Free Software Foundation, Inc., > + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. > + */ > + > +#ifndef __INCLUDE_LINUX_DEVICE_H > +#define __INCLUDE_LINUX_DEVICE_H __INCLUDE_LINUX_DEVICE_H is too generic to protect the inclusion of gpmi-nfc.h [...] > diff --git a/arch/arm/mach-mxs/mach-mx23evk.c b/arch/arm/mach-mxs/mach-mx23evk.c > index a66994f..db715f9 100644 > --- a/arch/arm/mach-mxs/mach-mx23evk.c > +++ b/arch/arm/mach-mxs/mach-mx23evk.c > @@ -34,6 +34,42 @@ static const iomux_cfg_t mx23evk_pads[] __initconst = { > MX23_PAD_PWM0__DUART_RX | MXS_PAD_CTRL, > MX23_PAD_PWM1__DUART_TX | MXS_PAD_CTRL, > > + /* gpmi */ > + MX23_PAD_GPMI_D00__GPMI_D00 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), You can use MXS_PAD_CTRL here. > + MX23_PAD_GPMI_D01__GPMI_D01 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D02__GPMI_D02 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D03__GPMI_D03 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D04__GPMI_D04 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D05__GPMI_D05 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D06__GPMI_D06 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_D07__GPMI_D07 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_CLE__GPMI_CLE | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_ALE__GPMI_ALE | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_WPN__GPMI_WPN | > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_WRN__GPMI_WRN | > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_RDN__GPMI_RDN | > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_RDY0__GPMI_RDY0 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_RDY1__GPMI_RDY1 | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_CE0N__GPMI_CE0N | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + MX23_PAD_GPMI_CE1N__GPMI_CE1N | > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > + mmc (ssp) has pin conflict with gpmi on both mx23evk and mx28evk. Uwe, How do you usually handle such pin conflicts on board? > /* auart */ > MX23_PAD_AUART1_RX__AUART1_RX | MXS_PAD_CTRL, > MX23_PAD_AUART1_TX__AUART1_TX | MXS_PAD_CTRL, > @@ -108,6 +144,7 @@ static void __init mx23evk_init(void) > mxs_iomux_setup_multiple_pads(mx23evk_pads, ARRAY_SIZE(mx23evk_pads)); > > mx23_add_duart(); > + mx23_add_gpmi(); > mx23_add_auart0(); > Can you please add gpmi stuff like this and pad configuration at the end to reflect the order that devices get supported? > ret = gpio_request_one(MX23EVK_LCD_ENABLE, GPIOF_DIR_OUT, "lcd-enable"); > diff --git a/arch/arm/mach-mxs/mach-mx28evk.c b/arch/arm/mach-mxs/mach-mx28evk.c > index 08002d0..cf8ddcb 100644 > --- a/arch/arm/mach-mxs/mach-mx28evk.c > +++ b/arch/arm/mach-mxs/mach-mx28evk.c Ditto
2011/4/13 Shawn Guo <shawn.guo@freescale.com>: > On Wed, Apr 13, 2011 at 02:24:38PM +0800, Huang Shijie wrote: >> add the clock and iomux initialization for GPMI in the imx23 and imx28. >> and register the gpmi driver. >> >> Signed-off-by: Huang Shijie <b32955@freescale.com> >> --- >> arch/arm/mach-mxs/Kconfig | 2 + >> arch/arm/mach-mxs/clock-mx23.c | 3 + >> arch/arm/mach-mxs/clock-mx28.c | 3 + >> arch/arm/mach-mxs/devices-mx23.h | 3 + >> arch/arm/mach-mxs/devices-mx28.h | 3 + >> arch/arm/mach-mxs/devices/Kconfig | 3 + >> arch/arm/mach-mxs/devices/Makefile | 1 + >> arch/arm/mach-mxs/devices/platform-gpmi.c | 134 +++++++++++++++++++++++ >> arch/arm/mach-mxs/include/mach/devices-common.h | 4 + >> arch/arm/mach-mxs/include/mach/gpmi-nfc.h | 65 +++++++++++ >> arch/arm/mach-mxs/mach-mx23evk.c | 37 ++++++ >> arch/arm/mach-mxs/mach-mx28evk.c | 37 ++++++ >> 12 files changed, 295 insertions(+), 0 deletions(-) >> create mode 100644 arch/arm/mach-mxs/devices/platform-gpmi.c >> create mode 100644 arch/arm/mach-mxs/include/mach/gpmi-nfc.h >> > [...] >> diff --git a/arch/arm/mach-mxs/devices/platform-gpmi.c b/arch/arm/mach-mxs/devices/platform-gpmi.c >> new file mode 100644 >> index 0000000..74ebe22 >> --- /dev/null >> +++ b/arch/arm/mach-mxs/devices/platform-gpmi.c >> @@ -0,0 +1,134 @@ >> +/* >> + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License as published by >> + * the Free Software Foundation; either version 2 of the License, or >> + * (at your option) any later version. >> + * >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + * >> + * You should have received a copy of the GNU General Public License along >> + * with this program; if not, write to the Free Software Foundation, Inc., >> + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >> + */ >> +#include <asm/sizes.h> >> +#include <mach/mx23.h> >> +#include <mach/mx28.h> >> +#include <mach/gpmi-nfc.h> >> +#include <mach/devices-common.h> > > devices-common.h already includes gpmi-nfc.h ok. thanks > > [...] >> diff --git a/arch/arm/mach-mxs/include/mach/gpmi-nfc.h b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h >> new file mode 100644 >> index 0000000..cf8e8c3 >> --- /dev/null >> +++ b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h >> @@ -0,0 +1,65 @@ >> +/* >> + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License as published by >> + * the Free Software Foundation; either version 2 of the License, or >> + * (at your option) any later version. >> + * >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + * >> + * You should have received a copy of the GNU General Public License along >> + * with this program; if not, write to the Free Software Foundation, Inc., >> + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >> + */ >> + >> +#ifndef __INCLUDE_LINUX_DEVICE_H >> +#define __INCLUDE_LINUX_DEVICE_H > > __INCLUDE_LINUX_DEVICE_H is too generic to protect the inclusion of > gpmi-nfc.h ok, thanks > > [...] >> diff --git a/arch/arm/mach-mxs/mach-mx23evk.c b/arch/arm/mach-mxs/mach-mx23evk.c >> index a66994f..db715f9 100644 >> --- a/arch/arm/mach-mxs/mach-mx23evk.c >> +++ b/arch/arm/mach-mxs/mach-mx23evk.c >> @@ -34,6 +34,42 @@ static const iomux_cfg_t mx23evk_pads[] __initconst = { >> MX23_PAD_PWM0__DUART_RX | MXS_PAD_CTRL, >> MX23_PAD_PWM1__DUART_TX | MXS_PAD_CTRL, >> >> + /* gpmi */ >> + MX23_PAD_GPMI_D00__GPMI_D00 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > You can use MXS_PAD_CTRL here. > >> + MX23_PAD_GPMI_D01__GPMI_D01 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D02__GPMI_D02 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D03__GPMI_D03 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D04__GPMI_D04 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D05__GPMI_D05 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D06__GPMI_D06 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_D07__GPMI_D07 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_CLE__GPMI_CLE | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_ALE__GPMI_ALE | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_WPN__GPMI_WPN | >> + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_WRN__GPMI_WRN | >> + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_RDN__GPMI_RDN | >> + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_RDY0__GPMI_RDY0 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_RDY1__GPMI_RDY1 | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_CE0N__GPMI_CE0N | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + MX23_PAD_GPMI_CE1N__GPMI_CE1N | >> + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), >> + > > mmc (ssp) has pin conflict with gpmi on both mx23evk and mx28evk. > > Uwe, > > How do you usually handle such pin conflicts on board? In the our imx508 code, we add a initialization hook in the platform data , and calls the hook in the driver's probe(). Does this a good solution? > >> /* auart */ >> MX23_PAD_AUART1_RX__AUART1_RX | MXS_PAD_CTRL, >> MX23_PAD_AUART1_TX__AUART1_TX | MXS_PAD_CTRL, >> @@ -108,6 +144,7 @@ static void __init mx23evk_init(void) >> mxs_iomux_setup_multiple_pads(mx23evk_pads, ARRAY_SIZE(mx23evk_pads)); >> >> mx23_add_duart(); >> + mx23_add_gpmi(); >> mx23_add_auart0(); >> > Can you please add gpmi stuff like this and pad configuration at the > end to reflect the order that devices get supported? ok. Best Regards Huang Shijie > >> ret = gpio_request_one(MX23EVK_LCD_ENABLE, GPIOF_DIR_OUT, "lcd-enable"); >> diff --git a/arch/arm/mach-mxs/mach-mx28evk.c b/arch/arm/mach-mxs/mach-mx28evk.c >> index 08002d0..cf8ddcb 100644 >> --- a/arch/arm/mach-mxs/mach-mx28evk.c >> +++ b/arch/arm/mach-mxs/mach-mx28evk.c > Ditto > > -- > Regards, > Shawn > >
Hi Shawn, On Wed, Apr 13, 2011 at 07:43:05PM +0800, Shawn Guo wrote: > On Wed, Apr 13, 2011 at 02:24:38PM +0800, Huang Shijie wrote: > > + MX23_PAD_GPMI_D01__GPMI_D01 | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_D02__GPMI_D02 | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_D03__GPMI_D03 | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_D04__GPMI_D04 | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_D05__GPMI_D05 | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_D06__GPMI_D06 | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_D07__GPMI_D07 | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_CLE__GPMI_CLE | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_ALE__GPMI_ALE | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_WPN__GPMI_WPN | > > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_WRN__GPMI_WRN | > > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_RDN__GPMI_RDN | > > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_RDY0__GPMI_RDY0 | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_RDY1__GPMI_RDY1 | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_CE0N__GPMI_CE0N | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + MX23_PAD_GPMI_CE1N__GPMI_CE1N | > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > + > > mmc (ssp) has pin conflict with gpmi on both mx23evk and mx28evk. hmm, $ git grep GPMI linus/master -- arch/arm/mach-mxs/mach-mx23evk.c linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D08__LCD_D18 | MXS linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D09__LCD_D19 | MXS linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D10__LCD_D20 | MXS linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D11__LCD_D21 | MXS linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D12__LCD_D22 | MXS linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D13__LCD_D23 | MXS linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D08__SSP1_DATA4 | linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D09__SSP1_DATA5 | linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D10__SSP1_DATA6 | linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D11__SSP1_DATA7 | so MX23_PAD_GPMI_D08 - MX23_PAD_GPMI_D13 is used (D08-D11 twice). So the problem is already in linus/master. Is it that what you mean? > How do you usually handle such pin conflicts on board? I don't understand this question. Maybe: deciding which pad mux to use? Best regards Uwe
On Wed, Apr 13, 2011 at 03:51:52PM +0200, Uwe Kleine-König wrote: > Hi Shawn, > > On Wed, Apr 13, 2011 at 07:43:05PM +0800, Shawn Guo wrote: > > On Wed, Apr 13, 2011 at 02:24:38PM +0800, Huang Shijie wrote: > > > + MX23_PAD_GPMI_D01__GPMI_D01 | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_D02__GPMI_D02 | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_D03__GPMI_D03 | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_D04__GPMI_D04 | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_D05__GPMI_D05 | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_D06__GPMI_D06 | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_D07__GPMI_D07 | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_CLE__GPMI_CLE | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_ALE__GPMI_ALE | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_WPN__GPMI_WPN | > > > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_WRN__GPMI_WRN | > > > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_RDN__GPMI_RDN | > > > + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_RDY0__GPMI_RDY0 | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_RDY1__GPMI_RDY1 | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_CE0N__GPMI_CE0N | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + MX23_PAD_GPMI_CE1N__GPMI_CE1N | > > > + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), > > > + > > > > mmc (ssp) has pin conflict with gpmi on both mx23evk and mx28evk. > hmm, > > $ git grep GPMI linus/master -- arch/arm/mach-mxs/mach-mx23evk.c > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D08__LCD_D18 | MXS > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D09__LCD_D19 | MXS > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D10__LCD_D20 | MXS > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D11__LCD_D21 | MXS > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D12__LCD_D22 | MXS > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D13__LCD_D23 | MXS > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D08__SSP1_DATA4 | > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D09__SSP1_DATA5 | > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D10__SSP1_DATA6 | > linus/master:arch/arm/mach-mxs/mach-mx23evk.c: MX23_PAD_GPMI_D11__SSP1_DATA7 | > > so MX23_PAD_GPMI_D08 - MX23_PAD_GPMI_D13 is used (D08-D11 twice). So the > problem is already in linus/master. Is it that what you mean? > Yeah, you are right. > > How do you usually handle such pin conflicts on board? > I don't understand this question. Maybe: deciding which pad mux to use? > Yes, the question is how you decide which pad mux to use? The first thing I think of is to ask CONFIG options to help. But I'm unsure if it's the right way.
diff --git a/arch/arm/mach-mxs/Kconfig b/arch/arm/mach-mxs/Kconfig index 4f6f174..e034666 100644 --- a/arch/arm/mach-mxs/Kconfig +++ b/arch/arm/mach-mxs/Kconfig @@ -22,6 +22,7 @@ config MACH_MX23EVK select SOC_IMX23 select MXS_HAVE_AMBA_DUART select MXS_HAVE_PLATFORM_AUART + select MXS_HAVE_PLATFORM_GPMI select MXS_HAVE_PLATFORM_MXSFB default y help @@ -35,6 +36,7 @@ config MACH_MX28EVK select MXS_HAVE_PLATFORM_AUART select MXS_HAVE_PLATFORM_FEC select MXS_HAVE_PLATFORM_FLEXCAN + select MXS_HAVE_PLATFORM_GPMI select MXS_HAVE_PLATFORM_MXSFB select MXS_OCOTP default y diff --git a/arch/arm/mach-mxs/clock-mx23.c b/arch/arm/mach-mxs/clock-mx23.c index d133c7f..f3c9653 100644 --- a/arch/arm/mach-mxs/clock-mx23.c +++ b/arch/arm/mach-mxs/clock-mx23.c @@ -29,6 +29,7 @@ #include <mach/mx23.h> #include <mach/common.h> #include <mach/clock.h> +#include <mach/gpmi-nfc.h> #include "regs-clkctrl-mx23.h" @@ -442,6 +443,7 @@ static struct clk_lookup lookups[] = { _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) /* for amba-pl011 driver */ _REGISTER_CLOCK("duart", NULL, uart_clk) + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX23, NULL, gpmi_clk) _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) _REGISTER_CLOCK("rtc", NULL, rtc_clk) _REGISTER_CLOCK("mxs-dma-apbh", NULL, hbus_clk) @@ -533,6 +535,7 @@ int __init mx23_clocks_init(void) clk_enable(&xbus_clk); clk_enable(&emi_clk); clk_enable(&uart_clk); + clk_enable(&gpmi_clk); clkdev_add_table(lookups, ARRAY_SIZE(lookups)); diff --git a/arch/arm/mach-mxs/clock-mx28.c b/arch/arm/mach-mxs/clock-mx28.c index 5e489a2..7868d42 100644 --- a/arch/arm/mach-mxs/clock-mx28.c +++ b/arch/arm/mach-mxs/clock-mx28.c @@ -29,6 +29,7 @@ #include <mach/mx28.h> #include <mach/common.h> #include <mach/clock.h> +#include <mach/gpmi-nfc.h> #include "regs-clkctrl-mx28.h" @@ -607,6 +608,7 @@ static struct clk_lookup lookups[] = { _REGISTER_CLOCK("duart", "apb_pclk", xbus_clk) /* for amba-pl011 driver */ _REGISTER_CLOCK("duart", NULL, uart_clk) + _REGISTER_CLOCK(GPMI_NFC_DRIVER_MX28, NULL, gpmi_clk) _REGISTER_CLOCK("imx28-fec.0", NULL, fec_clk) _REGISTER_CLOCK("imx28-fec.1", NULL, fec_clk) _REGISTER_CLOCK("mxs-auart.0", NULL, uart_clk) @@ -749,6 +751,7 @@ int __init mx28_clocks_init(void) clk_enable(&xbus_clk); clk_enable(&emi_clk); clk_enable(&uart_clk); + clk_enable(&gpmi_clk); clk_set_parent(&lcdif_clk, &ref_pix_clk); diff --git a/arch/arm/mach-mxs/devices-mx23.h b/arch/arm/mach-mxs/devices-mx23.h index c7e14f4..349fb1d 100644 --- a/arch/arm/mach-mxs/devices-mx23.h +++ b/arch/arm/mach-mxs/devices-mx23.h @@ -16,6 +16,9 @@ extern const struct amba_device mx23_duart_device __initconst; #define mx23_add_duart() \ mxs_add_duart(&mx23_duart_device) +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst; +#define mx23_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx23) + extern const struct mxs_auart_data mx23_auart_data[] __initconst; #define mx23_add_auart(id) mxs_add_auart(&mx23_auart_data[id]) #define mx23_add_auart0() mx23_add_auart(0) diff --git a/arch/arm/mach-mxs/devices-mx28.h b/arch/arm/mach-mxs/devices-mx28.h index 9d08555..ed6427b 100644 --- a/arch/arm/mach-mxs/devices-mx28.h +++ b/arch/arm/mach-mxs/devices-mx28.h @@ -16,6 +16,9 @@ extern const struct amba_device mx28_duart_device __initconst; #define mx28_add_duart() \ mxs_add_duart(&mx28_duart_device) +extern const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst; +#define mx28_add_gpmi() mxs_add_gpmi(&gpmi_platform_data_imx28) + extern const struct mxs_auart_data mx28_auart_data[] __initconst; #define mx28_add_auart(id) mxs_add_auart(&mx28_auart_data[id]) #define mx28_add_auart0() mx28_add_auart(0) diff --git a/arch/arm/mach-mxs/devices/Kconfig b/arch/arm/mach-mxs/devices/Kconfig index 1451ad0..81e99ce 100644 --- a/arch/arm/mach-mxs/devices/Kconfig +++ b/arch/arm/mach-mxs/devices/Kconfig @@ -5,6 +5,9 @@ config MXS_HAVE_AMBA_DUART config MXS_HAVE_PLATFORM_AUART bool +config MXS_HAVE_PLATFORM_GPMI + bool + config MXS_HAVE_PLATFORM_FEC bool diff --git a/arch/arm/mach-mxs/devices/Makefile b/arch/arm/mach-mxs/devices/Makefile index 0d9bea3..8a7c2c8 100644 --- a/arch/arm/mach-mxs/devices/Makefile +++ b/arch/arm/mach-mxs/devices/Makefile @@ -2,6 +2,7 @@ obj-$(CONFIG_MXS_HAVE_AMBA_DUART) += amba-duart.o obj-$(CONFIG_MXS_HAVE_PLATFORM_AUART) += platform-auart.o obj-y += platform-dma.o obj-$(CONFIG_MXS_HAVE_PLATFORM_FEC) += platform-fec.o +obj-$(CONFIG_MXS_HAVE_PLATFORM_GPMI) += platform-gpmi.o obj-$(CONFIG_MXS_HAVE_PLATFORM_FLEXCAN) += platform-flexcan.o obj-$(CONFIG_MXS_HAVE_PLATFORM_MXS_I2C) += platform-mxs-i2c.o obj-$(CONFIG_MXS_HAVE_PLATFORM_MXS_PWM) += platform-mxs-pwm.o diff --git a/arch/arm/mach-mxs/devices/platform-gpmi.c b/arch/arm/mach-mxs/devices/platform-gpmi.c new file mode 100644 index 0000000..74ebe22 --- /dev/null +++ b/arch/arm/mach-mxs/devices/platform-gpmi.c @@ -0,0 +1,134 @@ +/* + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + */ +#include <asm/sizes.h> +#include <mach/mx23.h> +#include <mach/mx28.h> +#include <mach/gpmi-nfc.h> +#include <mach/devices-common.h> + +#ifdef CONFIG_SOC_IMX23 +const struct gpmi_nfc_platform_data gpmi_platform_data_imx23 __initconst = { + .min_prop_delay_in_ns = 5, + .max_prop_delay_in_ns = 9, + .max_chip_count = 1, +}; + +const struct resource res_imx23[] __initconst = { + { /* GPMI */ + .start = MX23_GPMI_BASE_ADDR, + .end = MX23_GPMI_BASE_ADDR + SZ_8K - 1, + .name = GPMI_NFC_GPMI_REGS_ADDR_RES_NAME, + .flags = IORESOURCE_MEM, + }, { + .start = MX23_INT_GPMI_ATTENTION, + .end = MX23_INT_GPMI_ATTENTION, + .name = GPMI_NFC_GPMI_INTERRUPT_RES_NAME, + .flags = IORESOURCE_IRQ, + }, { /* BCH */ + .start = MX23_BCH_BASE_ADDR, + .end = MX23_BCH_BASE_ADDR + SZ_8K - 1, + .name = GPMI_NFC_BCH_REGS_ADDR_RES_NAME, + .flags = IORESOURCE_MEM, + }, { + .start = MX23_INT_BCH, + .end = MX23_INT_BCH, + .name = GPMI_NFC_BCH_INTERRUPT_RES_NAME, + .flags = IORESOURCE_IRQ, + }, { /* DMA */ + .start = MX23_DMA_GPMI0, + .end = MX23_DMA_GPMI3, + .name = GPMI_NFC_DMA_CHANNELS_RES_NAME, + .flags = IORESOURCE_DMA, + }, { + .start = MX23_INT_GPMI_DMA, + .end = MX23_INT_GPMI_DMA, + .name = GPMI_NFC_DMA_INTERRUPT_RES_NAME, + .flags = IORESOURCE_IRQ, + }, +}; +#endif + +#ifdef CONFIG_SOC_IMX28 +const struct gpmi_nfc_platform_data gpmi_platform_data_imx28 __initconst = { + .min_prop_delay_in_ns = 5, + .max_prop_delay_in_ns = 9, + .max_chip_count = 1, +}; + +const struct resource res_imx28[] __initconst = { + { /* GPMI */ + .start = MX28_GPMI_BASE_ADDR, + .end = MX28_GPMI_BASE_ADDR + SZ_8K - 1, + .name = GPMI_NFC_GPMI_REGS_ADDR_RES_NAME, + .flags = IORESOURCE_MEM, + }, { + .start = MX28_INT_GPMI, + .end = MX28_INT_GPMI, + .name = GPMI_NFC_GPMI_INTERRUPT_RES_NAME, + .flags = IORESOURCE_IRQ, + }, { /* BCH */ + .start = MX28_BCH_BASE_ADDR, + .end = MX28_BCH_BASE_ADDR + SZ_8K - 1, + .name = GPMI_NFC_BCH_REGS_ADDR_RES_NAME, + .flags = IORESOURCE_MEM, + }, { + .start = MX28_INT_BCH, + .end = MX28_INT_BCH, + .name = GPMI_NFC_BCH_INTERRUPT_RES_NAME, + .flags = IORESOURCE_IRQ, + }, { /* DMA */ + .start = MX28_DMA_GPMI0, + .end = MX28_DMA_GPMI7, + .name = GPMI_NFC_DMA_CHANNELS_RES_NAME, + .flags = IORESOURCE_DMA, + }, { + .start = MX28_INT_GPMI_DMA, + .end = MX28_INT_GPMI_DMA, + .name = GPMI_NFC_DMA_INTERRUPT_RES_NAME, + .flags = IORESOURCE_IRQ, + }, +}; +#endif + +struct platform_device *__init +mxs_add_gpmi(const struct gpmi_nfc_platform_data *data) +{ + const struct resource *res; + int res_size; + const char *name = NULL; + +#ifdef CONFIG_SOC_IMX23 + if (cpu_is_mx23()) { + res_size = ARRAY_SIZE(res_imx23); + res = res_imx23; + name = GPMI_NFC_DRIVER_MX23; + } +#endif + +#ifdef CONFIG_SOC_IMX28 + if (cpu_is_mx28()) { + res_size = ARRAY_SIZE(res_imx28); + res = res_imx28; + name = GPMI_NFC_DRIVER_MX28; + } +#endif + + return mxs_add_platform_device_dmamask(name, -1, + res, res_size, + data, sizeof(*data), DMA_BIT_MASK(32)); +} diff --git a/arch/arm/mach-mxs/include/mach/devices-common.h b/arch/arm/mach-mxs/include/mach/devices-common.h index 71f2448..1f05503 100644 --- a/arch/arm/mach-mxs/include/mach/devices-common.h +++ b/arch/arm/mach-mxs/include/mach/devices-common.h @@ -30,6 +30,10 @@ int __init mxs_add_amba_device(const struct amba_device *dev); /* duart */ int __init mxs_add_duart(const struct amba_device *dev); +/* GPMI */ +#include <mach/gpmi-nfc.h> +struct platform_device *__init mxs_add_gpmi( + const struct gpmi_nfc_platform_data *data); /* auart */ struct mxs_auart_data { int id; diff --git a/arch/arm/mach-mxs/include/mach/gpmi-nfc.h b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h new file mode 100644 index 0000000..cf8e8c3 --- /dev/null +++ b/arch/arm/mach-mxs/include/mach/gpmi-nfc.h @@ -0,0 +1,65 @@ +/* + * Copyright (C) 2011 Freescale Semiconductor, Inc. All Rights Reserved. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + */ + +#ifndef __INCLUDE_LINUX_DEVICE_H +#define __INCLUDE_LINUX_DEVICE_H + +#define GPMI_NFC_DRIVER_NAME "gpmi-nfc" +#define GPMI_NFC_DRIVER_MX23 "gpmi-nfc-mx23" +#define GPMI_NFC_DRIVER_MX28 "gpmi-nfc-mx28" +#define GPMI_NFC_DRIVER_MX50 "gpmi-nfc-mx508" + +/* Resource names for the GPMI NFC driver. */ +#define GPMI_NFC_GPMI_REGS_ADDR_RES_NAME "GPMI NFC GPMI Registers" +#define GPMI_NFC_GPMI_INTERRUPT_RES_NAME "GPMI NFC GPMI Interrupt" +#define GPMI_NFC_BCH_REGS_ADDR_RES_NAME "GPMI NFC BCH Registers" +#define GPMI_NFC_BCH_INTERRUPT_RES_NAME "GPMI NFC BCH Interrupt" +#define GPMI_NFC_DMA_CHANNELS_RES_NAME "GPMI NFC DMA Channels" +#define GPMI_NFC_DMA_INTERRUPT_RES_NAME "GPMI NFC DMA Interrupt" + +/** + * struct gpmi_nfc_platform_data - GPMI NFC driver platform data. + * + * This structure communicates platform-specific information to the GPMI NFC + * driver that can't be expressed as resources. + * + * @min_prop_delay_in_ns: Minimum propagation delay of GPMI signals to and + * from the NAND Flash device, in nanoseconds. + * @max_prop_delay_in_ns: Maximum propagation delay of GPMI signals to and + * from the NAND Flash device, in nanoseconds. + * @max_chip_count: The maximum number of chips for which the driver + * should configure the hardware. This value most + * likely reflects the number of pins that are + * connected to a NAND Flash device. If this is + * greater than the SoC hardware can support, the + * driver will print a message and fail to initialize. + * @partitions: An optional pointer to an array of partition + * descriptions. + * @partition_count: The number of elements in the partitions array. + */ +struct gpmi_nfc_platform_data { + /* NAND Flash information. */ + unsigned int min_prop_delay_in_ns; + unsigned int max_prop_delay_in_ns; + unsigned int max_chip_count; + + /* soc */ + struct mtd_partition *partitions; + unsigned partition_count; +}; +#endif diff --git a/arch/arm/mach-mxs/mach-mx23evk.c b/arch/arm/mach-mxs/mach-mx23evk.c index a66994f..db715f9 100644 --- a/arch/arm/mach-mxs/mach-mx23evk.c +++ b/arch/arm/mach-mxs/mach-mx23evk.c @@ -34,6 +34,42 @@ static const iomux_cfg_t mx23evk_pads[] __initconst = { MX23_PAD_PWM0__DUART_RX | MXS_PAD_CTRL, MX23_PAD_PWM1__DUART_TX | MXS_PAD_CTRL, + /* gpmi */ + MX23_PAD_GPMI_D00__GPMI_D00 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_D01__GPMI_D01 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_D02__GPMI_D02 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_D03__GPMI_D03 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_D04__GPMI_D04 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_D05__GPMI_D05 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_D06__GPMI_D06 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_D07__GPMI_D07 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_CLE__GPMI_CLE | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_ALE__GPMI_ALE | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_WPN__GPMI_WPN | + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_WRN__GPMI_WRN | + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_RDN__GPMI_RDN | + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_RDY0__GPMI_RDY0 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_RDY1__GPMI_RDY1 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_CE0N__GPMI_CE0N | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX23_PAD_GPMI_CE1N__GPMI_CE1N | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + /* auart */ MX23_PAD_AUART1_RX__AUART1_RX | MXS_PAD_CTRL, MX23_PAD_AUART1_TX__AUART1_TX | MXS_PAD_CTRL, @@ -108,6 +144,7 @@ static void __init mx23evk_init(void) mxs_iomux_setup_multiple_pads(mx23evk_pads, ARRAY_SIZE(mx23evk_pads)); mx23_add_duart(); + mx23_add_gpmi(); mx23_add_auart0(); ret = gpio_request_one(MX23EVK_LCD_ENABLE, GPIOF_DIR_OUT, "lcd-enable"); diff --git a/arch/arm/mach-mxs/mach-mx28evk.c b/arch/arm/mach-mxs/mach-mx28evk.c index 08002d0..cf8ddcb 100644 --- a/arch/arm/mach-mxs/mach-mx28evk.c +++ b/arch/arm/mach-mxs/mach-mx28evk.c @@ -39,6 +39,42 @@ static const iomux_cfg_t mx28evk_pads[] __initconst = { MX28_PAD_PWM0__DUART_RX | MXS_PAD_CTRL, MX28_PAD_PWM1__DUART_TX | MXS_PAD_CTRL, + /* gpmi */ + MX28_PAD_GPMI_D00__GPMI_D0 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_D01__GPMI_D1 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_D02__GPMI_D2 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_D03__GPMI_D3 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_D04__GPMI_D4 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_D05__GPMI_D5 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_D06__GPMI_D6 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_D07__GPMI_D7 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_CE0N__GPMI_CE0N | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_CE1N__GPMI_CE1N | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_RDY0__GPMI_READY0 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_RDY1__GPMI_READY1 | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_RDN__GPMI_RDN | + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_WRN__GPMI_WRN | + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_ALE__GPMI_ALE | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_CLE__GPMI_CLE | + (MXS_PAD_4MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + MX28_PAD_GPMI_RESETN__GPMI_RESETN | + (MXS_PAD_12MA | MXS_PAD_3V3 | MXS_PAD_NOPULL), + /* auart0 */ MX28_PAD_AUART0_RX__AUART0_RX | MXS_PAD_CTRL, MX28_PAD_AUART0_TX__AUART0_TX | MXS_PAD_CTRL, @@ -265,6 +301,7 @@ static void __init mx28evk_init(void) mxs_iomux_setup_multiple_pads(mx28evk_pads, ARRAY_SIZE(mx28evk_pads)); mx28_add_duart(); + mx28_add_gpmi(); mx28_add_auart0(); mx28_add_auart3();
add the clock and iomux initialization for GPMI in the imx23 and imx28. and register the gpmi driver. Signed-off-by: Huang Shijie <b32955@freescale.com> --- arch/arm/mach-mxs/Kconfig | 2 + arch/arm/mach-mxs/clock-mx23.c | 3 + arch/arm/mach-mxs/clock-mx28.c | 3 + arch/arm/mach-mxs/devices-mx23.h | 3 + arch/arm/mach-mxs/devices-mx28.h | 3 + arch/arm/mach-mxs/devices/Kconfig | 3 + arch/arm/mach-mxs/devices/Makefile | 1 + arch/arm/mach-mxs/devices/platform-gpmi.c | 134 +++++++++++++++++++++++ arch/arm/mach-mxs/include/mach/devices-common.h | 4 + arch/arm/mach-mxs/include/mach/gpmi-nfc.h | 65 +++++++++++ arch/arm/mach-mxs/mach-mx23evk.c | 37 ++++++ arch/arm/mach-mxs/mach-mx28evk.c | 37 ++++++ 12 files changed, 295 insertions(+), 0 deletions(-) create mode 100644 arch/arm/mach-mxs/devices/platform-gpmi.c create mode 100644 arch/arm/mach-mxs/include/mach/gpmi-nfc.h