There are some boards that do not strictly follow SPI standard and use only 3 
wires
(SCLK, MOSI, SS) for connecting some simple auxiliary chips and controls them 
with GPIO
based 'spi controller'. In this configuration the MISO line is missing (it is 
not
required if the chip does not transfer any data back to host). The example of 
such
board is a NCP ARM S3C64XX based machine. This patch adds support for such 
non-standard
configuration in GPIO-based SPI controller.

Reviewed-by: Kyungmin Park <[email protected]>
Signed-off-by: Marek Szyprowski <[email protected]>

--- 

diff --git a/drivers/spi/spi_gpio.c b/drivers/spi/spi_gpio.c
index 26bd03e..16f74fd 100644
--- a/drivers/spi/spi_gpio.c
+++ b/drivers/spi/spi_gpio.c
@@ -114,7 +114,10 @@ static inline void setmosi(const struct spi_device *spi, 
int is_on)
 
 static inline int getmiso(const struct spi_device *spi)
 {
-       return !!gpio_get_value(SPI_MISO_GPIO);
+       if (gpio_is_valid(SPI_MISO_GPIO))
+               return !!gpio_get_value(SPI_MISO_GPIO);
+       else
+               return 0;
 }
 
 #undef pdata
@@ -243,9 +246,11 @@ spi_gpio_request(struct spi_gpio_platform_data *pdata, 
const char *label)
        if (value)
                goto done;
 
-       value = spi_gpio_alloc(SPI_MISO_GPIO, label, true);
-       if (value)
-               goto free_mosi;
+       if (gpio_is_valid(SPI_MISO_GPIO)) {
+               value = spi_gpio_alloc(SPI_MISO_GPIO, label, true);
+               if (value)
+                       goto free_mosi;
+       }
 
        value = spi_gpio_alloc(SPI_SCK_GPIO, label, false);
        if (value)
@@ -254,7 +259,8 @@ spi_gpio_request(struct spi_gpio_platform_data *pdata, 
const char *label)
        goto done;
 
 free_miso:
-       gpio_free(SPI_MISO_GPIO);
+       if (gpio_is_valid(SPI_MISO_GPIO))
+               gpio_free(SPI_MISO_GPIO);
 free_mosi:
        gpio_free(SPI_MOSI_GPIO);
 done:
@@ -308,7 +314,8 @@ static int __init spi_gpio_probe(struct platform_device 
*pdev)
        if (status < 0) {
                spi_master_put(spi_gpio->bitbang.master);
 gpio_free:
-               gpio_free(SPI_MISO_GPIO);
+               if (gpio_is_valid(SPI_MISO_GPIO))
+                       gpio_free(SPI_MISO_GPIO);
                gpio_free(SPI_MOSI_GPIO);
                gpio_free(SPI_SCK_GPIO);
                spi_master_put(master);
@@ -332,7 +339,8 @@ static int __exit spi_gpio_remove(struct platform_device 
*pdev)
 
        platform_set_drvdata(pdev, NULL);
 
-       gpio_free(SPI_MISO_GPIO);
+       if (gpio_is_valid(SPI_MISO_GPIO))
+               gpio_free(SPI_MISO_GPIO);
        gpio_free(SPI_MOSI_GPIO);
        gpio_free(SPI_SCK_GPIO);



------------------------------------------------------------------------------
Register Now for Creativity and Technology (CaT), June 3rd, NYC. CaT
is a gathering of tech-side developers & brand creativity professionals. Meet
the minds behind Google Creative Lab, Visual Complexity, Processing, & 
iPhoneDevCamp asthey present alongside digital heavyweights like Barbarian
Group, R/GA, & Big Spaceship. http://www.creativitycat.com 
_______________________________________________
spi-devel-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

Reply via email to