summaryrefslogtreecommitdiff
path: root/src/drivers
diff options
context:
space:
mode:
authorFurquan Shaikh <furquan@chromium.org>2016-12-01 07:25:31 -0800
committerMartin Roth <martinroth@google.com>2016-12-07 20:19:07 +0100
commitb5d41cb063a54d2a90e0480ede18d3b9c1ae8474 (patch)
treec0a34038912a8aa8d8eae9a20ee5cbadb6dc067e /src/drivers
parent2dc8b77d0e95dbb90a99ff5dc2b162b805faba18 (diff)
spi: Clean up SPI driver interface
1. Add new structure spi_ctrlr_buses that allows platform to define a mapping from SPI controller to buses managed by the controller. 2. Provide weak implementations of spi_init and spi_setup_slave that will be used by platforms using the new interface. BUG=chrome-os-partner:59832 BRANCH=None TEST=Compiles successfully Change-Id: Ia6f47941b786299f4d823895898ffb1b36e02f73 Signed-off-by: Furquan Shaikh <furquan@chromium.org> Reviewed-on: https://review.coreboot.org/17561 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/drivers')
-rw-r--r--src/drivers/spi/spi-generic.c35
1 files changed, 35 insertions, 0 deletions
diff --git a/src/drivers/spi/spi-generic.c b/src/drivers/spi/spi-generic.c
index 7398ee928f..4fcd04c13a 100644
--- a/src/drivers/spi/spi-generic.c
+++ b/src/drivers/spi/spi-generic.c
@@ -41,3 +41,38 @@ int spi_xfer(const struct spi_slave *slave, const void *dout, size_t bytesout,
return -1;
}
+
+void __attribute__((weak)) spi_init(void)
+{
+ /* Default weak implementation - do nothing. */
+}
+
+const struct spi_ctrlr_buses spi_ctrlr_bus_map[0] __attribute__((weak));
+const size_t spi_ctrlr_bus_map_count __attribute__((weak));
+
+int __attribute__((weak)) spi_setup_slave(unsigned int bus, unsigned int cs,
+ struct spi_slave *slave)
+{
+ size_t i;
+
+ memset(slave, 0, sizeof(*slave));
+
+ for (i = 0; i < spi_ctrlr_bus_map_count; i++) {
+ if ((spi_ctrlr_bus_map[i].bus_start <= bus) &&
+ (spi_ctrlr_bus_map[i].bus_end >= bus)) {
+ slave->ctrlr = spi_ctrlr_bus_map[i].ctrlr;
+ break;
+ }
+ }
+
+ if (slave->ctrlr == NULL)
+ return -1;
+
+ slave->bus = bus;
+ slave->cs = cs;
+
+ if (slave->ctrlr->setup)
+ return slave->ctrlr->setup(slave);
+
+ return 0;
+}