253 lines
6.5 KiB
Diff
253 lines
6.5 KiB
Diff
From bdbd9b2861ba73557795915598bb276a8568d130 Mon Sep 17 00:00:00 2001
|
|
From: Gabor Juhos <juhosg@openwrt.org>
|
|
Date: Wed, 11 Jan 2012 22:25:11 +0100
|
|
Subject: [PATCH 7/7] spi/ath79: make chipselect logic more flexible
|
|
|
|
Signed-off-by: Gabor Juhos <juhosg@openwrt.org>
|
|
---
|
|
arch/mips/ath79/mach-ap121.c | 6 ++
|
|
arch/mips/ath79/mach-ap81.c | 6 ++
|
|
arch/mips/ath79/mach-pb44.c | 6 ++
|
|
arch/mips/ath79/mach-ubnt-xm.c | 6 ++
|
|
.../include/asm/mach-ath79/ath79_spi_platform.h | 8 ++-
|
|
drivers/spi/spi-ath79.c | 63 ++++++++++++--------
|
|
6 files changed, 69 insertions(+), 26 deletions(-)
|
|
|
|
--- a/arch/mips/ath79/mach-ap121.c
|
|
+++ b/arch/mips/ath79/mach-ap121.c
|
|
@@ -58,12 +58,18 @@ static struct gpio_keys_button ap121_gpi
|
|
}
|
|
};
|
|
|
|
+static struct ath79_spi_controller_data ap121_spi0_data = {
|
|
+ .cs_type = ATH79_SPI_CS_TYPE_INTERNAL,
|
|
+ .cs_line = 0,
|
|
+};
|
|
+
|
|
static struct spi_board_info ap121_spi_info[] = {
|
|
{
|
|
.bus_num = 0,
|
|
.chip_select = 0,
|
|
.max_speed_hz = 25000000,
|
|
.modalias = "mx25l1606e",
|
|
+ .controller_data = &ap121_spi0_data,
|
|
}
|
|
};
|
|
|
|
--- a/arch/mips/ath79/mach-ap81.c
|
|
+++ b/arch/mips/ath79/mach-ap81.c
|
|
@@ -67,12 +67,18 @@ static struct gpio_keys_button ap81_gpio
|
|
}
|
|
};
|
|
|
|
+static struct ath79_spi_controller_data ap81_spi0_data = {
|
|
+ .cs_type = ATH79_SPI_CS_TYPE_INTERNAL,
|
|
+ .cs_line = 0,
|
|
+};
|
|
+
|
|
static struct spi_board_info ap81_spi_info[] = {
|
|
{
|
|
.bus_num = 0,
|
|
.chip_select = 0,
|
|
.max_speed_hz = 25000000,
|
|
.modalias = "m25p64",
|
|
+ .controller_data = &ap81_spi0_data,
|
|
}
|
|
};
|
|
|
|
--- a/arch/mips/ath79/mach-pb44.c
|
|
+++ b/arch/mips/ath79/mach-pb44.c
|
|
@@ -87,12 +87,18 @@ static struct gpio_keys_button pb44_gpio
|
|
}
|
|
};
|
|
|
|
+static struct ath79_spi_controller_data pb44_spi0_data = {
|
|
+ .cs_type = ATH79_SPI_CS_TYPE_INTERNAL,
|
|
+ .cs_line = 0,
|
|
+};
|
|
+
|
|
static struct spi_board_info pb44_spi_info[] = {
|
|
{
|
|
.bus_num = 0,
|
|
.chip_select = 0,
|
|
.max_speed_hz = 25000000,
|
|
.modalias = "m25p64",
|
|
+ .controller_data = &pb44_spi0_data,
|
|
},
|
|
};
|
|
|
|
--- a/arch/mips/ath79/mach-ubnt-xm.c
|
|
+++ b/arch/mips/ath79/mach-ubnt-xm.c
|
|
@@ -65,12 +65,18 @@ static struct gpio_keys_button ubnt_xm_g
|
|
}
|
|
};
|
|
|
|
+static struct ath79_spi_controller_data ubnt_xm_spi0_data = {
|
|
+ .cs_type = ATH79_SPI_CS_TYPE_INTERNAL,
|
|
+ .cs_line = 0,
|
|
+};
|
|
+
|
|
static struct spi_board_info ubnt_xm_spi_info[] = {
|
|
{
|
|
.bus_num = 0,
|
|
.chip_select = 0,
|
|
.max_speed_hz = 25000000,
|
|
.modalias = "mx25l6405d",
|
|
+ .controller_data = &ubnt_xm_spi0_data,
|
|
}
|
|
};
|
|
|
|
--- a/arch/mips/include/asm/mach-ath79/ath79_spi_platform.h
|
|
+++ b/arch/mips/include/asm/mach-ath79/ath79_spi_platform.h
|
|
@@ -16,8 +16,14 @@ struct ath79_spi_platform_data {
|
|
unsigned num_chipselect;
|
|
};
|
|
|
|
+enum ath79_spi_cs_type {
|
|
+ ATH79_SPI_CS_TYPE_INTERNAL,
|
|
+ ATH79_SPI_CS_TYPE_GPIO,
|
|
+};
|
|
+
|
|
struct ath79_spi_controller_data {
|
|
- unsigned gpio;
|
|
+ enum ath79_spi_cs_type cs_type;
|
|
+ unsigned cs_line;
|
|
};
|
|
|
|
#endif /* _ATH79_SPI_PLATFORM_H */
|
|
--- a/drivers/spi/spi-ath79.c
|
|
+++ b/drivers/spi/spi-ath79.c
|
|
@@ -35,6 +35,8 @@
|
|
#define ATH79_SPI_RRW_DELAY_FACTOR 12000
|
|
#define MHZ (1000 * 1000)
|
|
|
|
+#define ATH79_SPI_CS_LINE_MAX 2
|
|
+
|
|
struct ath79_spi {
|
|
struct spi_bitbang bitbang;
|
|
u32 ioc_base;
|
|
@@ -69,6 +71,7 @@ static void ath79_spi_chipselect(struct
|
|
{
|
|
struct ath79_spi *sp = ath79_spidev_to_sp(spi);
|
|
int cs_high = (spi->mode & SPI_CS_HIGH) ? is_active : !is_active;
|
|
+ struct ath79_spi_controller_data *cdata = spi->controller_data;
|
|
|
|
if (is_active) {
|
|
/* set initial clock polarity */
|
|
@@ -80,20 +83,21 @@ static void ath79_spi_chipselect(struct
|
|
ath79_spi_wr(sp, AR71XX_SPI_REG_IOC, sp->ioc_base);
|
|
}
|
|
|
|
- if (spi->chip_select) {
|
|
- struct ath79_spi_controller_data *cdata = spi->controller_data;
|
|
-
|
|
- /* SPI is normally active-low */
|
|
- gpio_set_value(cdata->gpio, cs_high);
|
|
- } else {
|
|
+ switch (cdata->cs_type) {
|
|
+ case ATH79_SPI_CS_TYPE_INTERNAL:
|
|
if (cs_high)
|
|
- sp->ioc_base |= AR71XX_SPI_IOC_CS0;
|
|
+ sp->ioc_base |= AR71XX_SPI_IOC_CS(cdata->cs_line);
|
|
else
|
|
- sp->ioc_base &= ~AR71XX_SPI_IOC_CS0;
|
|
+ sp->ioc_base &= ~AR71XX_SPI_IOC_CS(cdata->cs_line);
|
|
|
|
ath79_spi_wr(sp, AR71XX_SPI_REG_IOC, sp->ioc_base);
|
|
- }
|
|
+ break;
|
|
|
|
+ case ATH79_SPI_CS_TYPE_GPIO:
|
|
+ /* SPI is normally active-low */
|
|
+ gpio_set_value(cdata->cs_line, cs_high);
|
|
+ break;
|
|
+ }
|
|
}
|
|
|
|
static void ath79_spi_enable(struct ath79_spi *sp)
|
|
@@ -120,24 +124,30 @@ static void ath79_spi_disable(struct ath
|
|
static int ath79_spi_setup_cs(struct spi_device *spi)
|
|
{
|
|
struct ath79_spi_controller_data *cdata;
|
|
+ unsigned long flags;
|
|
int status;
|
|
|
|
cdata = spi->controller_data;
|
|
- if (spi->chip_select && !cdata)
|
|
+ if (!cdata)
|
|
return -EINVAL;
|
|
|
|
status = 0;
|
|
- if (spi->chip_select) {
|
|
- unsigned long flags;
|
|
+ switch (cdata->cs_type) {
|
|
+ case ATH79_SPI_CS_TYPE_INTERNAL:
|
|
+ if (cdata->cs_line > ATH79_SPI_CS_LINE_MAX)
|
|
+ status = -EINVAL;
|
|
+ break;
|
|
|
|
+ case ATH79_SPI_CS_TYPE_GPIO:
|
|
flags = GPIOF_DIR_OUT;
|
|
if (spi->mode & SPI_CS_HIGH)
|
|
flags |= GPIOF_INIT_HIGH;
|
|
else
|
|
flags |= GPIOF_INIT_LOW;
|
|
|
|
- status = gpio_request_one(cdata->gpio, flags,
|
|
+ status = gpio_request_one(cdata->cs_line, flags,
|
|
dev_name(&spi->dev));
|
|
+ break;
|
|
}
|
|
|
|
return status;
|
|
@@ -145,9 +155,15 @@ static int ath79_spi_setup_cs(struct spi
|
|
|
|
static void ath79_spi_cleanup_cs(struct spi_device *spi)
|
|
{
|
|
- if (spi->chip_select) {
|
|
- struct ath79_spi_controller_data *cdata = spi->controller_data;
|
|
- gpio_free(cdata->gpio);
|
|
+ struct ath79_spi_controller_data *cdata = spi->controller_data;
|
|
+
|
|
+ switch (cdata->cs_type) {
|
|
+ case ATH79_SPI_CS_TYPE_INTERNAL:
|
|
+ /* nothing to do */
|
|
+ break;
|
|
+ case ATH79_SPI_CS_TYPE_GPIO:
|
|
+ gpio_free(cdata->cs_line);
|
|
+ break;
|
|
}
|
|
}
|
|
|
|
@@ -215,6 +231,10 @@ static __devinit int ath79_spi_probe(str
|
|
unsigned long rate;
|
|
int ret;
|
|
|
|
+ pdata = pdev->dev.platform_data;
|
|
+ if (!pdata)
|
|
+ return -EINVAL;
|
|
+
|
|
master = spi_alloc_master(&pdev->dev, sizeof(*sp));
|
|
if (master == NULL) {
|
|
dev_err(&pdev->dev, "failed to allocate spi master\n");
|
|
@@ -224,17 +244,10 @@ static __devinit int ath79_spi_probe(str
|
|
sp = spi_master_get_devdata(master);
|
|
platform_set_drvdata(pdev, sp);
|
|
|
|
- pdata = pdev->dev.platform_data;
|
|
-
|
|
master->setup = ath79_spi_setup;
|
|
master->cleanup = ath79_spi_cleanup;
|
|
- if (pdata) {
|
|
- master->bus_num = pdata->bus_num;
|
|
- master->num_chipselect = pdata->num_chipselect;
|
|
- } else {
|
|
- master->bus_num = -1;
|
|
- master->num_chipselect = 1;
|
|
- }
|
|
+ master->bus_num = pdata->bus_num;
|
|
+ master->num_chipselect = pdata->num_chipselect;
|
|
|
|
sp->bitbang.master = spi_master_get(master);
|
|
sp->bitbang.chipselect = ath79_spi_chipselect;
|