[PATCH v2] imx_spi: drop non usefull ifdef
Jean-Christophe PLAGNIOL-VILLARD
plagnioj at jcrosoft.com
Thu Aug 11 11:55:24 EDT 2011
drop CONFIG_DRIVER_SPI_IMX_<version>
this will not reduce barebox size as the compiler will do for us
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>
---
drivers/spi/Kconfig | 12 +--------
drivers/spi/imx_spi.c | 63 +++++++++---------------------------------------
2 files changed, 13 insertions(+), 62 deletions(-)
diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 9ab03f6..b48f9aa 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -6,17 +6,7 @@ config SPI
config DRIVER_SPI_IMX
bool "i.MX SPI Master driver"
- depends on ARCH_IMX
+ depends on ARCH_IMX && (ARCH_IMX27 || ARCH_IMX51 || ARCH_IMX53)
depends on SPI
-config DRIVER_SPI_IMX_0_0
- bool
- depends on ARCH_IMX27
- default y
-
-config DRIVER_SPI_IMX_2_3
- bool
- depends on ARCH_IMX51 || ARCH_IMX53
- default y
-
endmenu
diff --git a/drivers/spi/imx_spi.c b/drivers/spi/imx_spi.c
index 6dc41b9..0e2ea7e 100644
--- a/drivers/spi/imx_spi.c
+++ b/drivers/spi/imx_spi.c
@@ -90,24 +90,12 @@
#define CSPI_2_3_STAT_RR (1 << 3)
enum imx_spi_devtype {
-#ifdef CONFIG_DRIVER_SPI_IMX1
SPI_IMX_VER_IMX1,
-#endif
-#ifdef CONFIG_DRIVER_SPI_IMX_0_0
SPI_IMX_VER_0_0,
-#endif
-#ifdef CONFIG_DRIVER_SPI_IMX_0_4
SPI_IMX_VER_0_4,
-#endif
-#ifdef CONFIG_DRIVER_SPI_IMX_0_5
SPI_IMX_VER_0_5,
-#endif
-#ifdef CONFIG_DRIVER_SPI_IMX_0_7
SPI_IMX_VER_0_7,
-#endif
-#ifdef CONFIG_DRIVER_SPI_IMX_2_3
SPI_IMX_VER_2_3,
-#endif
};
struct imx_spi {
@@ -120,12 +108,6 @@ struct imx_spi {
void (*init)(struct imx_spi *imx);
};
-struct spi_imx_devtype_data {
- unsigned int (*xchg_single)(struct imx_spi *imx, u32 data);
- void (*chipselect)(struct spi_device *spi, int active);
- void (*init)(struct imx_spi *imx);
-};
-
static int imx_spi_setup(struct spi_device *spi)
{
debug("%s mode 0x%08x bits_per_word: %d speed: %d\n",
@@ -134,7 +116,6 @@ static int imx_spi_setup(struct spi_device *spi)
return 0;
}
-#ifdef CONFIG_DRIVER_SPI_IMX_0_0
static unsigned int cspi_0_0_xchg_single(struct imx_spi *imx, unsigned int data)
{
void __iomem *base = imx->regs;
@@ -204,9 +185,7 @@ static void cspi_0_0_init(struct imx_spi *imx)
readl(base + CSPI_0_0_RXDATA);
writel(0, base + CSPI_0_0_INT);
}
-#endif
-#ifdef CONFIG_DRIVER_SPI_IMX_2_3
static unsigned int cspi_2_3_xchg_single(struct imx_spi *imx, unsigned int data)
{
void __iomem *base = imx->regs;
@@ -308,7 +287,6 @@ static void cspi_2_3_chipselect(struct spi_device *spi, int is_active)
static void cspi_2_3_init(struct imx_spi *imx)
{
}
-#endif
static int imx_spi_transfer(struct spi_device *spi, struct spi_message *mesg)
{
@@ -334,29 +312,11 @@ static int imx_spi_transfer(struct spi_device *spi, struct spi_message *mesg)
return 0;
}
-static struct spi_imx_devtype_data spi_imx_devtype_data[] = {
-#ifdef CONFIG_DRIVER_SPI_IMX_0_0
- [SPI_IMX_VER_0_0] = {
- .chipselect = cspi_0_0_chipselect,
- .xchg_single = cspi_0_0_xchg_single,
- .init = cspi_0_0_init,
- },
-#endif
-#ifdef CONFIG_DRIVER_SPI_IMX_2_3
- [SPI_IMX_VER_2_3] = {
- .chipselect = cspi_2_3_chipselect,
- .xchg_single = cspi_2_3_xchg_single,
- .init = cspi_2_3_init,
- },
-#endif
-};
-
static int imx_spi_probe(struct device_d *dev)
{
struct spi_master *master;
struct imx_spi *imx;
struct spi_imx_master *pdata = dev->platform_data;
- enum imx_spi_devtype version;
imx = xzalloc(sizeof(*imx));
@@ -368,17 +328,18 @@ static int imx_spi_probe(struct device_d *dev)
master->num_chipselect = pdata->num_chipselect;
imx->cs_array = pdata->chipselect;
-#ifdef CONFIG_DRIVER_SPI_IMX_0_0
- if (cpu_is_mx27())
- version = SPI_IMX_VER_0_0;
-#endif
-#ifdef CONFIG_DRIVER_SPI_IMX_2_3
- if (cpu_is_mx51() || cpu_is_mx53())
- version = SPI_IMX_VER_2_3;
-#endif
- imx->chipselect = spi_imx_devtype_data[version].chipselect;
- imx->xchg_single = spi_imx_devtype_data[version].xchg_single;
- imx->init = spi_imx_devtype_data[version].init;
+ if (cpu_is_mx27()) {
+ imx->chipselect = cspi_0_0_chipselect;
+ imx->xchg_single = cspi_0_0_xchg_single;
+ imx->init = cspi_0_0_init;
+ }
+
+ if (cpu_is_mx51() || cpu_is_mx53()) {
+ imx->chipselect = cspi_2_3_chipselect;
+ imx->xchg_single = cspi_2_3_xchg_single;
+ imx->init = cspi_2_3_init;
+ }
+
imx->regs = dev_request_mem_region(dev, 0);
imx->init(imx);
--
1.7.5.4
More information about the barebox
mailing list