diff options
| author | Bryan Wu <bryan.wu@analog.com> | 2007-12-05 10:45:22 +0300 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-12-05 20:21:20 +0300 |
| commit | 003d922618150eaab53936f57ba8a61f2b601486 (patch) | |
| tree | 54bd61e6176b0447d2df30dd7b3aab505b0df6ec | |
| parent | f452126c2e4b8bbfd8e41ebdf1e734e3bf18f8e9 (diff) | |
| download | linux-003d922618150eaab53936f57ba8a61f2b601486.tar.xz | |
Blackfin SPI driver: move hard coded pin_req to board file
Remove some sort of bloaty code, try to get these pin_req arrays built at compile-time
- move this static things to the blackfin board file
- add pin_req array to struct bfin5xx_spi_master
- tested on BF537/BF548 with SPI flash
Signed-off-by: Bryan Wu <bryan.wu@analog.com>
Cc: David Brownell <david-b@pacbell.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | drivers/spi/spi_bfin5xx.c | 28 | ||||
| -rw-r--r-- | include/asm-blackfin/bfin5xx_spi.h | 1 |
2 files changed, 8 insertions, 21 deletions
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c index 6a02bd3ef9fd..d6e9812538c4 100644 --- a/drivers/spi/spi_bfin5xx.c +++ b/drivers/spi/spi_bfin5xx.c @@ -80,6 +80,9 @@ struct driver_data { /* Regs base of SPI controller */ void __iomem *regs_base; + /* Pin request list */ + u16 *pin_req; + /* BFIN hookup */ struct bfin5xx_spi_master *master_info; @@ -1255,25 +1258,6 @@ static inline int destroy_queue(struct driver_data *drv_data) return 0; } -static int setup_pin_mux(int action, int bus_num) -{ - - u16 pin_req[3][4] = { - {P_SPI0_SCK, P_SPI0_MISO, P_SPI0_MOSI, 0}, - {P_SPI1_SCK, P_SPI1_MISO, P_SPI1_MOSI, 0}, - {P_SPI2_SCK, P_SPI2_MISO, P_SPI2_MOSI, 0}, - }; - - if (action) { - if (peripheral_request_list(pin_req[bus_num], DRV_NAME)) - return -EFAULT; - } else { - peripheral_free_list(pin_req[bus_num]); - } - - return 0; -} - static int __init bfin5xx_spi_probe(struct platform_device *pdev) { struct device *dev = &pdev->dev; @@ -1296,6 +1280,7 @@ static int __init bfin5xx_spi_probe(struct platform_device *pdev) drv_data->master = master; drv_data->master_info = platform_info; drv_data->pdev = pdev; + drv_data->pin_req = platform_info->pin_req; master->bus_num = pdev->id; master->num_chipselect = platform_info->num_chipselect; @@ -1346,7 +1331,8 @@ static int __init bfin5xx_spi_probe(struct platform_device *pdev) goto out_error_queue_alloc; } - if (setup_pin_mux(1, master->bus_num)) { + status = peripheral_request_list(drv_data->pin_req, DRV_NAME); + if (status != 0) { dev_err(&pdev->dev, ": Requesting Peripherals failed\n"); goto out_error; } @@ -1394,7 +1380,7 @@ static int __devexit bfin5xx_spi_remove(struct platform_device *pdev) /* Disconnect from the SPI framework */ spi_unregister_master(drv_data->master); - setup_pin_mux(0, drv_data->master->bus_num); + peripheral_free_list(drv_data->pin_req); /* Prevent double remove */ platform_set_drvdata(pdev, NULL); diff --git a/include/asm-blackfin/bfin5xx_spi.h b/include/asm-blackfin/bfin5xx_spi.h index d4485b37d8e2..1a0b57f6a3d4 100644 --- a/include/asm-blackfin/bfin5xx_spi.h +++ b/include/asm-blackfin/bfin5xx_spi.h @@ -152,6 +152,7 @@ struct bfin5xx_spi_master { u16 num_chipselect; u8 enable_dma; + u16 pin_req[4]; }; /* spi_board_info.controller_data for SPI slave devices, |