spi: octeon: Split driver into Octeon specific and common parts
authorJan Glauber <jglauber@cavium.com>
Sat, 23 Jul 2016 10:42:54 +0000 (12:42 +0200)
committerMark Brown <broonie@kernel.org>
Sun, 24 Jul 2016 20:54:29 +0000 (21:54 +0100)
Separate driver probing from SPI transfer functions.

Signed-off-by: Jan Glauber <jglauber@cavium.com>
Tested-by: Steven J. Hill <steven.hill@cavium.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/Makefile
drivers/spi/spi-cavium-octeon.c [new file with mode: 0644]
drivers/spi/spi-cavium.c [new file with mode: 0644]
drivers/spi/spi-cavium.h
drivers/spi/spi-octeon.c [deleted file]

index 3c74d00..185367e 100644 (file)
@@ -56,6 +56,7 @@ obj-$(CONFIG_SPI_MT65XX)                += spi-mt65xx.o
 obj-$(CONFIG_SPI_MXS)                  += spi-mxs.o
 obj-$(CONFIG_SPI_NUC900)               += spi-nuc900.o
 obj-$(CONFIG_SPI_OC_TINY)              += spi-oc-tiny.o
+spi-octeon-objs                                := spi-cavium.o spi-cavium-octeon.o
 obj-$(CONFIG_SPI_OCTEON)               += spi-octeon.o
 obj-$(CONFIG_SPI_OMAP_UWIRE)           += spi-omap-uwire.o
 obj-$(CONFIG_SPI_OMAP_100K)            += spi-omap-100k.o
diff --git a/drivers/spi/spi-cavium-octeon.c b/drivers/spi/spi-cavium-octeon.c
new file mode 100644 (file)
index 0000000..ee4703e
--- /dev/null
@@ -0,0 +1,104 @@
+/*
+ * This file is subject to the terms and conditions of the GNU General Public
+ * License.  See the file "COPYING" in the main directory of this archive
+ * for more details.
+ *
+ * Copyright (C) 2011, 2012 Cavium, Inc.
+ */
+
+#include <linux/platform_device.h>
+#include <linux/spi/spi.h>
+#include <linux/module.h>
+#include <linux/io.h>
+#include <linux/of.h>
+
+#include <asm/octeon/octeon.h>
+
+#include "spi-cavium.h"
+
+static int octeon_spi_probe(struct platform_device *pdev)
+{
+       struct resource *res_mem;
+       void __iomem *reg_base;
+       struct spi_master *master;
+       struct octeon_spi *p;
+       int err = -ENOENT;
+
+       master = spi_alloc_master(&pdev->dev, sizeof(struct octeon_spi));
+       if (!master)
+               return -ENOMEM;
+       p = spi_master_get_devdata(master);
+       platform_set_drvdata(pdev, master);
+
+       res_mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+       reg_base = devm_ioremap_resource(&pdev->dev, res_mem);
+       if (IS_ERR(reg_base)) {
+               err = PTR_ERR(reg_base);
+               goto fail;
+       }
+
+       p->register_base = reg_base;
+       p->sys_freq = octeon_get_io_clock_rate();
+
+       p->regs.config = 0;
+       p->regs.status = 0x08;
+       p->regs.tx = 0x10;
+       p->regs.data = 0x80;
+
+       master->num_chipselect = 4;
+       master->mode_bits = SPI_CPHA |
+                           SPI_CPOL |
+                           SPI_CS_HIGH |
+                           SPI_LSB_FIRST |
+                           SPI_3WIRE;
+
+       master->transfer_one_message = octeon_spi_transfer_one_message;
+       master->bits_per_word_mask = SPI_BPW_MASK(8);
+       master->max_speed_hz = OCTEON_SPI_MAX_CLOCK_HZ;
+
+       master->dev.of_node = pdev->dev.of_node;
+       err = devm_spi_register_master(&pdev->dev, master);
+       if (err) {
+               dev_err(&pdev->dev, "register master failed: %d\n", err);
+               goto fail;
+       }
+
+       dev_info(&pdev->dev, "OCTEON SPI bus driver\n");
+
+       return 0;
+fail:
+       spi_master_put(master);
+       return err;
+}
+
+static int octeon_spi_remove(struct platform_device *pdev)
+{
+       struct spi_master *master = platform_get_drvdata(pdev);
+       struct octeon_spi *p = spi_master_get_devdata(master);
+
+       /* Clear the CSENA* and put everything in a known state. */
+       writeq(0, p->register_base + OCTEON_SPI_CFG(p));
+
+       return 0;
+}
+
+static const struct of_device_id octeon_spi_match[] = {
+       { .compatible = "cavium,octeon-3010-spi", },
+       {},
+};
+MODULE_DEVICE_TABLE(of, octeon_spi_match);
+
+static struct platform_driver octeon_spi_driver = {
+       .driver = {
+               .name           = "spi-octeon",
+               .of_match_table = octeon_spi_match,
+       },
+       .probe          = octeon_spi_probe,
+       .remove         = octeon_spi_remove,
+};
+
+module_platform_driver(octeon_spi_driver);
+
+MODULE_DESCRIPTION("Cavium, Inc. OCTEON SPI bus driver");
+MODULE_AUTHOR("David Daney");
+MODULE_LICENSE("GPL");
diff --git a/drivers/spi/spi-cavium.c b/drivers/spi/spi-cavium.c
new file mode 100644 (file)
index 0000000..5aaf215
--- /dev/null
@@ -0,0 +1,151 @@
+/*
+ * This file is subject to the terms and conditions of the GNU General Public
+ * License.  See the file "COPYING" in the main directory of this archive
+ * for more details.
+ *
+ * Copyright (C) 2011, 2012 Cavium, Inc.
+ */
+
+#include <linux/spi/spi.h>
+#include <linux/module.h>
+#include <linux/delay.h>
+#include <linux/io.h>
+
+#include "spi-cavium.h"
+
+static void octeon_spi_wait_ready(struct octeon_spi *p)
+{
+       union cvmx_mpi_sts mpi_sts;
+       unsigned int loops = 0;
+
+       do {
+               if (loops++)
+                       __delay(500);
+               mpi_sts.u64 = readq(p->register_base + OCTEON_SPI_STS(p));
+       } while (mpi_sts.s.busy);
+}
+
+static int octeon_spi_do_transfer(struct octeon_spi *p,
+                                 struct spi_message *msg,
+                                 struct spi_transfer *xfer,
+                                 bool last_xfer)
+{
+       struct spi_device *spi = msg->spi;
+       union cvmx_mpi_cfg mpi_cfg;
+       union cvmx_mpi_tx mpi_tx;
+       unsigned int clkdiv;
+       int mode;
+       bool cpha, cpol;
+       const u8 *tx_buf;
+       u8 *rx_buf;
+       int len;
+       int i;
+
+       mode = spi->mode;
+       cpha = mode & SPI_CPHA;
+       cpol = mode & SPI_CPOL;
+
+       clkdiv = p->sys_freq / (2 * xfer->speed_hz);
+
+       mpi_cfg.u64 = 0;
+
+       mpi_cfg.s.clkdiv = clkdiv;
+       mpi_cfg.s.cshi = (mode & SPI_CS_HIGH) ? 1 : 0;
+       mpi_cfg.s.lsbfirst = (mode & SPI_LSB_FIRST) ? 1 : 0;
+       mpi_cfg.s.wireor = (mode & SPI_3WIRE) ? 1 : 0;
+       mpi_cfg.s.idlelo = cpha != cpol;
+       mpi_cfg.s.cslate = cpha ? 1 : 0;
+       mpi_cfg.s.enable = 1;
+
+       if (spi->chip_select < 4)
+               p->cs_enax |= 1ull << (12 + spi->chip_select);
+       mpi_cfg.u64 |= p->cs_enax;
+
+       if (mpi_cfg.u64 != p->last_cfg) {
+               p->last_cfg = mpi_cfg.u64;
+               writeq(mpi_cfg.u64, p->register_base + OCTEON_SPI_CFG(p));
+       }
+       tx_buf = xfer->tx_buf;
+       rx_buf = xfer->rx_buf;
+       len = xfer->len;
+       while (len > OCTEON_SPI_MAX_BYTES) {
+               for (i = 0; i < OCTEON_SPI_MAX_BYTES; i++) {
+                       u8 d;
+                       if (tx_buf)
+                               d = *tx_buf++;
+                       else
+                               d = 0;
+                       writeq(d, p->register_base + OCTEON_SPI_DAT0(p) + (8 * i));
+               }
+               mpi_tx.u64 = 0;
+               mpi_tx.s.csid = spi->chip_select;
+               mpi_tx.s.leavecs = 1;
+               mpi_tx.s.txnum = tx_buf ? OCTEON_SPI_MAX_BYTES : 0;
+               mpi_tx.s.totnum = OCTEON_SPI_MAX_BYTES;
+               writeq(mpi_tx.u64, p->register_base + OCTEON_SPI_TX(p));
+
+               octeon_spi_wait_ready(p);
+               if (rx_buf)
+                       for (i = 0; i < OCTEON_SPI_MAX_BYTES; i++) {
+                               u64 v = readq(p->register_base + OCTEON_SPI_DAT0(p) + (8 * i));
+                               *rx_buf++ = (u8)v;
+                       }
+               len -= OCTEON_SPI_MAX_BYTES;
+       }
+
+       for (i = 0; i < len; i++) {
+               u8 d;
+               if (tx_buf)
+                       d = *tx_buf++;
+               else
+                       d = 0;
+               writeq(d, p->register_base + OCTEON_SPI_DAT0(p) + (8 * i));
+       }
+
+       mpi_tx.u64 = 0;
+       mpi_tx.s.csid = spi->chip_select;
+       if (last_xfer)
+               mpi_tx.s.leavecs = xfer->cs_change;
+       else
+               mpi_tx.s.leavecs = !xfer->cs_change;
+       mpi_tx.s.txnum = tx_buf ? len : 0;
+       mpi_tx.s.totnum = len;
+       writeq(mpi_tx.u64, p->register_base + OCTEON_SPI_TX(p));
+
+       octeon_spi_wait_ready(p);
+       if (rx_buf)
+               for (i = 0; i < len; i++) {
+                       u64 v = readq(p->register_base + OCTEON_SPI_DAT0(p) + (8 * i));
+                       *rx_buf++ = (u8)v;
+               }
+
+       if (xfer->delay_usecs)
+               udelay(xfer->delay_usecs);
+
+       return xfer->len;
+}
+
+int octeon_spi_transfer_one_message(struct spi_master *master,
+                                   struct spi_message *msg)
+{
+       struct octeon_spi *p = spi_master_get_devdata(master);
+       unsigned int total_len = 0;
+       int status = 0;
+       struct spi_transfer *xfer;
+
+       list_for_each_entry(xfer, &msg->transfers, transfer_list) {
+               bool last_xfer = list_is_last(&xfer->transfer_list,
+                                             &msg->transfers);
+               int r = octeon_spi_do_transfer(p, msg, xfer, last_xfer);
+               if (r < 0) {
+                       status = r;
+                       goto err;
+               }
+               total_len += r;
+       }
+err:
+       msg->status = status;
+       msg->actual_length = total_len;
+       spi_finalize_current_message(master);
+       return status;
+}
index d41dba5..88c5f36 100644 (file)
@@ -1,3 +1,32 @@
+#ifndef __SPI_CAVIUM_H
+#define __SPI_CAVIUM_H
+
+#define OCTEON_SPI_MAX_BYTES 9
+#define OCTEON_SPI_MAX_CLOCK_HZ 16000000
+
+struct octeon_spi_regs {
+       int config;
+       int status;
+       int tx;
+       int data;
+};
+
+struct octeon_spi {
+       void __iomem *register_base;
+       u64 last_cfg;
+       u64 cs_enax;
+       int sys_freq;
+       struct octeon_spi_regs regs;
+};
+
+#define OCTEON_SPI_CFG(x)      (x->regs.config)
+#define OCTEON_SPI_STS(x)      (x->regs.status)
+#define OCTEON_SPI_TX(x)       (x->regs.tx)
+#define OCTEON_SPI_DAT0(x)     (x->regs.data)
+
+int octeon_spi_transfer_one_message(struct spi_master *master,
+                                   struct spi_message *msg);
+
 /* MPI register descriptions */
 
 #define CVMX_MPI_CFG (CVMX_ADD_IO_SEG(0x0001070000001000ull))
@@ -296,3 +325,5 @@ union cvmx_mpi_tx {
        struct cvmx_mpi_tx_s cn66xx;
        struct cvmx_mpi_tx_cn61xx cnf71xx;
 };
+
+#endif /* __SPI_CAVIUM_H */
diff --git a/drivers/spi/spi-octeon.c b/drivers/spi/spi-octeon.c
deleted file mode 100644 (file)
index 2180176..0000000
+++ /dev/null
@@ -1,267 +0,0 @@
-/*
- * This file is subject to the terms and conditions of the GNU General Public
- * License.  See the file "COPYING" in the main directory of this archive
- * for more details.
- *
- * Copyright (C) 2011, 2012 Cavium, Inc.
- */
-
-#include <linux/platform_device.h>
-#include <linux/interrupt.h>
-#include <linux/spi/spi.h>
-#include <linux/module.h>
-#include <linux/delay.h>
-#include <linux/io.h>
-#include <linux/of.h>
-
-#include <asm/octeon/octeon.h>
-
-#include "spi-cavium.h"
-
-#define OCTEON_SPI_MAX_BYTES 9
-
-#define OCTEON_SPI_MAX_CLOCK_HZ 16000000
-
-struct octeon_spi_regs {
-       int config;
-       int status;
-       int tx;
-       int data;
-};
-
-struct octeon_spi {
-       void __iomem *register_base;
-       u64 last_cfg;
-       u64 cs_enax;
-       int sys_freq;
-       struct octeon_spi_regs regs;
-};
-
-#define OCTEON_SPI_CFG(x)      (x->regs.config)
-#define OCTEON_SPI_STS(x)      (x->regs.status)
-#define OCTEON_SPI_TX(x)       (x->regs.tx)
-#define OCTEON_SPI_DAT0(x)     (x->regs.data)
-
-static void octeon_spi_wait_ready(struct octeon_spi *p)
-{
-       union cvmx_mpi_sts mpi_sts;
-       unsigned int loops = 0;
-
-       do {
-               if (loops++)
-                       __delay(500);
-               mpi_sts.u64 = readq(p->register_base + OCTEON_SPI_STS(p));
-       } while (mpi_sts.s.busy);
-}
-
-static int octeon_spi_do_transfer(struct octeon_spi *p,
-                                 struct spi_message *msg,
-                                 struct spi_transfer *xfer,
-                                 bool last_xfer)
-{
-       struct spi_device *spi = msg->spi;
-       union cvmx_mpi_cfg mpi_cfg;
-       union cvmx_mpi_tx mpi_tx;
-       unsigned int clkdiv;
-       int mode;
-       bool cpha, cpol;
-       const u8 *tx_buf;
-       u8 *rx_buf;
-       int len;
-       int i;
-
-       mode = spi->mode;
-       cpha = mode & SPI_CPHA;
-       cpol = mode & SPI_CPOL;
-
-       clkdiv = p->sys_freq / (2 * xfer->speed_hz);
-
-       mpi_cfg.u64 = 0;
-
-       mpi_cfg.s.clkdiv = clkdiv;
-       mpi_cfg.s.cshi = (mode & SPI_CS_HIGH) ? 1 : 0;
-       mpi_cfg.s.lsbfirst = (mode & SPI_LSB_FIRST) ? 1 : 0;
-       mpi_cfg.s.wireor = (mode & SPI_3WIRE) ? 1 : 0;
-       mpi_cfg.s.idlelo = cpha != cpol;
-       mpi_cfg.s.cslate = cpha ? 1 : 0;
-       mpi_cfg.s.enable = 1;
-
-       if (spi->chip_select < 4)
-               p->cs_enax |= 1ull << (12 + spi->chip_select);
-       mpi_cfg.u64 |= p->cs_enax;
-
-       if (mpi_cfg.u64 != p->last_cfg) {
-               p->last_cfg = mpi_cfg.u64;
-               writeq(mpi_cfg.u64, p->register_base + OCTEON_SPI_CFG(p));
-       }
-       tx_buf = xfer->tx_buf;
-       rx_buf = xfer->rx_buf;
-       len = xfer->len;
-       while (len > OCTEON_SPI_MAX_BYTES) {
-               for (i = 0; i < OCTEON_SPI_MAX_BYTES; i++) {
-                       u8 d;
-                       if (tx_buf)
-                               d = *tx_buf++;
-                       else
-                               d = 0;
-                       writeq(d, p->register_base + OCTEON_SPI_DAT0(p) + (8 * i));
-               }
-               mpi_tx.u64 = 0;
-               mpi_tx.s.csid = spi->chip_select;
-               mpi_tx.s.leavecs = 1;
-               mpi_tx.s.txnum = tx_buf ? OCTEON_SPI_MAX_BYTES : 0;
-               mpi_tx.s.totnum = OCTEON_SPI_MAX_BYTES;
-               writeq(mpi_tx.u64, p->register_base + OCTEON_SPI_TX(p));
-
-               octeon_spi_wait_ready(p);
-               if (rx_buf)
-                       for (i = 0; i < OCTEON_SPI_MAX_BYTES; i++) {
-                               u64 v = readq(p->register_base + OCTEON_SPI_DAT0(p) + (8 * i));
-                               *rx_buf++ = (u8)v;
-                       }
-               len -= OCTEON_SPI_MAX_BYTES;
-       }
-
-       for (i = 0; i < len; i++) {
-               u8 d;
-               if (tx_buf)
-                       d = *tx_buf++;
-               else
-                       d = 0;
-               writeq(d, p->register_base + OCTEON_SPI_DAT0(p) + (8 * i));
-       }
-
-       mpi_tx.u64 = 0;
-       mpi_tx.s.csid = spi->chip_select;
-       if (last_xfer)
-               mpi_tx.s.leavecs = xfer->cs_change;
-       else
-               mpi_tx.s.leavecs = !xfer->cs_change;
-       mpi_tx.s.txnum = tx_buf ? len : 0;
-       mpi_tx.s.totnum = len;
-       writeq(mpi_tx.u64, p->register_base + OCTEON_SPI_TX(p));
-
-       octeon_spi_wait_ready(p);
-       if (rx_buf)
-               for (i = 0; i < len; i++) {
-                       u64 v = readq(p->register_base + OCTEON_SPI_DAT0(p) + (8 * i));
-                       *rx_buf++ = (u8)v;
-               }
-
-       if (xfer->delay_usecs)
-               udelay(xfer->delay_usecs);
-
-       return xfer->len;
-}
-
-static int octeon_spi_transfer_one_message(struct spi_master *master,
-                                          struct spi_message *msg)
-{
-       struct octeon_spi *p = spi_master_get_devdata(master);
-       unsigned int total_len = 0;
-       int status = 0;
-       struct spi_transfer *xfer;
-
-       list_for_each_entry(xfer, &msg->transfers, transfer_list) {
-               bool last_xfer = list_is_last(&xfer->transfer_list,
-                                             &msg->transfers);
-               int r = octeon_spi_do_transfer(p, msg, xfer, last_xfer);
-               if (r < 0) {
-                       status = r;
-                       goto err;
-               }
-               total_len += r;
-       }
-err:
-       msg->status = status;
-       msg->actual_length = total_len;
-       spi_finalize_current_message(master);
-       return status;
-}
-
-static int octeon_spi_probe(struct platform_device *pdev)
-{
-       struct resource *res_mem;
-       void __iomem *reg_base;
-       struct spi_master *master;
-       struct octeon_spi *p;
-       int err = -ENOENT;
-
-       master = spi_alloc_master(&pdev->dev, sizeof(struct octeon_spi));
-       if (!master)
-               return -ENOMEM;
-       p = spi_master_get_devdata(master);
-       platform_set_drvdata(pdev, master);
-
-       res_mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       reg_base = devm_ioremap_resource(&pdev->dev, res_mem);
-       if (IS_ERR(reg_base)) {
-               err = PTR_ERR(reg_base);
-               goto fail;
-       }
-
-       p->register_base = reg_base;
-       p->sys_freq = octeon_get_io_clock_rate();
-
-       p->regs.config = 0;
-       p->regs.status = 0x08;
-       p->regs.tx = 0x10;
-       p->regs.data = 0x80;
-
-       master->num_chipselect = 4;
-       master->mode_bits = SPI_CPHA |
-                           SPI_CPOL |
-                           SPI_CS_HIGH |
-                           SPI_LSB_FIRST |
-                           SPI_3WIRE;
-
-       master->transfer_one_message = octeon_spi_transfer_one_message;
-       master->bits_per_word_mask = SPI_BPW_MASK(8);
-       master->max_speed_hz = OCTEON_SPI_MAX_CLOCK_HZ;
-
-       master->dev.of_node = pdev->dev.of_node;
-       err = devm_spi_register_master(&pdev->dev, master);
-       if (err) {
-               dev_err(&pdev->dev, "register master failed: %d\n", err);
-               goto fail;
-       }
-
-       dev_info(&pdev->dev, "OCTEON SPI bus driver\n");
-
-       return 0;
-fail:
-       spi_master_put(master);
-       return err;
-}
-
-static int octeon_spi_remove(struct platform_device *pdev)
-{
-       struct spi_master *master = platform_get_drvdata(pdev);
-       struct octeon_spi *p = spi_master_get_devdata(master);
-
-       /* Clear the CSENA* and put everything in a known state. */
-       writeq(0, p->register_base + OCTEON_SPI_CFG(p));
-
-       return 0;
-}
-
-static const struct of_device_id octeon_spi_match[] = {
-       { .compatible = "cavium,octeon-3010-spi", },
-       {},
-};
-MODULE_DEVICE_TABLE(of, octeon_spi_match);
-
-static struct platform_driver octeon_spi_driver = {
-       .driver = {
-               .name           = "spi-octeon",
-               .of_match_table = octeon_spi_match,
-       },
-       .probe          = octeon_spi_probe,
-       .remove         = octeon_spi_remove,
-};
-
-module_platform_driver(octeon_spi_driver);
-
-MODULE_DESCRIPTION("Cavium, Inc. OCTEON SPI bus driver");
-MODULE_AUTHOR("David Daney");
-MODULE_LICENSE("GPL");