[brcm63xx] give some credits to Tanguy Bouzeloc in the spi driver
[openwrt/svn-archive/archive.git] / target / linux / brcm63xx / patches-2.6.35 / 240-spi.patch
index 4661486bb15db9083f23d50b5c63f2677a6f4e34..32c2d05c1c56445fc87882e6d7f28a1e17728e04 100644 (file)
        [IRQ_DSL]               = BCM_6358_DSL_IRQ,
 --- /dev/null
 +++ b/arch/mips/bcm63xx/dev-spi.c
-@@ -0,0 +1,127 @@
+@@ -0,0 +1,128 @@
 +/*
 + * 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) 2009 Florian Fainelli <florian@openwrt.org>
++ * Copyright (C) 2010 Tanguy Bouzeloc <tanguy.bouzeloc@efixo.com>
 + */
 +
 +#include <linux/init.h>
  #endif /* BCM63XX_REGS_H_ */
 --- /dev/null
 +++ b/drivers/spi/bcm63xx_spi.c
-@@ -0,0 +1,479 @@
+@@ -0,0 +1,496 @@
 +/*
 + * Broadcom BCM63xx SPI controller support
 + *
 + * Copyright (C) 2009 Florian Fainelli <florian@openwrt.org>
++ * Copyright (C) 2010 Tanguy Bouzeloc <tanguy.bouzeloc@efixo.com>
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License
 +
 +struct bcm63xx_spi {
 +      spinlock_t              lock;
-+      int                     stopping;
++      int                     stopping;
 +        struct completion     done;
 +
-+        void __iomem          *regs;
-+        int                   irq;
++      void __iomem            *regs;
++      int                     irq;
 +
 +      /* Platform data */
 +        u32                   speed_hz;
 +      /* Data buffers */
 +      const unsigned char     *tx_ptr;
 +      unsigned char           *rx_ptr;
++
++      /* data iomem */
++      u8 __iomem              *tx_io;
++      const u8 __iomem        *rx_io;
++
 +      int                     remaining_bytes;
 +
 +      struct clk              *clk;
 +      struct platform_device  *pdev;
 +};
 +
++static inline u8 bcm_spi_readb(struct bcm63xx_spi *bs,
++                              unsigned int offset)
++{
++      return bcm_readw(bs->regs + bcm63xx_spireg(offset));
++}
++
++static inline u16 bcm_spi_readw(struct bcm63xx_spi *bs,
++                              unsigned int offset)
++{
++      return bcm_readw(bs->regs + bcm63xx_spireg(offset));
++}
++
++static inline void bcm_spi_writeb(struct bcm63xx_spi *bs,
++                                u8 value, unsigned int offset)
++{
++      bcm_writeb(value, bs->regs + bcm63xx_spireg(offset));
++}
++
++static inline void bcm_spi_writew(struct bcm63xx_spi *bs,
++                                u16 value, unsigned int offset)
++{
++      bcm_writew(value, bs->regs + bcm63xx_spireg(offset));
++}
++
 +static int bcm63xx_spi_setup_transfer(struct spi_device *spi,
-+                                      struct spi_transfer *t)
++                                    struct spi_transfer *t)
 +{
 +      u8 bits_per_word;
 +      u8 clk_cfg;
 +      case 32:
 +              clk_cfg = SPI_CLK_1_563MHZ;
 +              break;
-+      case 128:
++      case 64:
 +              clk_cfg = SPI_CLK_0_781MHZ;
 +              break;
-+      case 64:
++      case 128:
 +      default:
 +              /* Set to slowest mode for compatibility */
-+              clk_cfg = SPI_CLK_0_781MHZ;
++              clk_cfg = SPI_CLK_0_391MHZ;
 +              break;
 +      }
 +
-+      bcm_spi_writeb(clk_cfg, SPI_CLK_CFG);
++      bcm_spi_writeb(bs, clk_cfg, SPI_CLK_CFG);
 +      dev_dbg(&spi->dev, "Setting clock register to %d (hz %d, cmd %02x)\n",
 +              div, hz, clk_cfg);
 +
 +static int bcm63xx_spi_setup(struct spi_device *spi)
 +{
 +      struct bcm63xx_spi *bs;
-+      int retval;
++      int ret;
 +
 +      bs = spi_master_get_devdata(spi->master);
 +
 +              return -EINVAL;
 +      }
 +
-+      retval = bcm63xx_spi_setup_transfer(spi, NULL);
-+      if (retval < 0) {
++      ret = bcm63xx_spi_setup_transfer(spi, NULL);
++      if (ret < 0) {
 +              dev_err(&spi->dev, "setup: unsupported mode bits %x\n",
 +                      spi->mode & ~MODEBITS);
-+              return retval;
++              return ret;
 +      }
 +
 +      dev_dbg(&spi->dev, "%s, mode %d, %u bits/w, %u nsec/bit\n",
 +/* Fill the TX FIFO with as many bytes as possible */
 +static void bcm63xx_spi_fill_tx_fifo(struct bcm63xx_spi *bs)
 +{
-+        u8 tail;
++      u8 size;
 +
 +        /* Fill the Tx FIFO with as many bytes as possible */
-+      tail = bcm_spi_readb(SPI_MSG_TAIL);
-+
-+        while ((tail < bs->fifo_size) && (bs->remaining_bytes > 0)) {
-+                if (bs->tx_ptr)
-+                        bcm_spi_writeb(*bs->tx_ptr++, SPI_MSG_DATA);
-+              else
-+                      bcm_spi_writeb(0, SPI_MSG_DATA);
-+
-+                bs->remaining_bytes--;
-+              tail = bcm_spi_readb(SPI_MSG_TAIL);
-+        }
++      size = bs->remaining_bytes < bs->fifo_size ? bs->remaining_bytes :
++              bs->fifo_size;
++      memcpy_toio(bs->tx_io, bs->tx_ptr, size);
++      bs->remaining_bytes -= size;
 +}
 +
 +static int bcm63xx_txrx_bufs(struct spi_device *spi, struct spi_transfer *t)
 +
 +      /* Enable the command done interrupt which
 +       * we use to determine completion of a command */
-+      bcm_spi_writeb(SPI_INTR_CMD_DONE, SPI_INT_MASK);
++      bcm_spi_writeb(bs, SPI_INTR_CMD_DONE, SPI_INT_MASK);
 +
 +      /* Fill in the Message control register */
 +      msg_ctl = (t->len << SPI_BYTE_CNT_SHIFT);
 +      else if (t->tx_buf)
 +              msg_ctl |= (SPI_HD_W << SPI_MSG_TYPE_SHIFT);
 +
-+      bcm_spi_writew(msg_ctl, SPI_MSG_CTL);
++      bcm_spi_writew(bs, msg_ctl, SPI_MSG_CTL);
 +
 +      /* Issue the transfer */
 +      cmd = SPI_CMD_START_IMMEDIATE;
 +      cmd |= (0 << SPI_CMD_PREPEND_BYTE_CNT_SHIFT);
-+      bcm_spi_writew(cmd, SPI_CMD);
++      cmd |= (spi->chip_select << SPI_CMD_DEVICE_ID_SHIFT);
++      bcm_spi_writew(bs, cmd, SPI_CMD);
 +      wait_for_completion(&bs->done);
 +
 +      /* Disable the CMD_DONE interrupt */
-+      bcm_spi_writeb(0, SPI_INT_MASK);
++      bcm_spi_writeb(bs, 0, SPI_INT_MASK);
 +
 +      return t->len - bs->remaining_bytes;
 +}
 +
-+static int bcm63xx_transfer(struct spi_device *spi, struct spi_message *msg)
++static int bcm63xx_transfer(struct spi_device *spi, struct spi_message *m)
 +{
 +      struct bcm63xx_spi *bs = spi_master_get_devdata(spi->master);
-+      struct spi_transfer *xfer;
++      struct spi_transfer *t;
 +      int ret = 0;
 +
-+      if (unlikely(list_empty(&msg->transfers)))
++      if (unlikely(list_empty(&m->transfers)))
 +              return -EINVAL;
 +
 +      if (bs->stopping)
 +              return -ESHUTDOWN;
 +
-+      list_for_each_entry(xfer, &msg->transfers, transfer_list) {
-+              ret += bcm63xx_txrx_bufs(spi, xfer);
++      list_for_each_entry(t, &m->transfers, transfer_list) {
++              ret += bcm63xx_txrx_bufs(spi, t);
 +      }
 +
-+      msg->complete(msg->context);
++      m->complete(m->context);
 +
 +      return ret;
 +}
 +      u16 cmd;
 +
 +      /* Read interupts and clear them immediately */
-+      intr = bcm_spi_readb(SPI_INT_STATUS);
-+      bcm_spi_writeb(SPI_INTR_CLEAR_ALL, SPI_INT_STATUS);
-+      bcm_spi_writeb(0, SPI_INT_MASK);
++      intr = bcm_spi_readb(bs, SPI_INT_STATUS);
++      bcm_spi_writeb(bs, SPI_INTR_CLEAR_ALL, SPI_INT_STATUS);
++      bcm_spi_writeb(bs, 0, SPI_INT_MASK);
 +
 +      /* A tansfer completed */
 +      if (intr & SPI_INTR_CMD_DONE) {
 +              u8 rx_tail;
 +
-+              rx_tail = bcm_spi_readb(SPI_RX_TAIL);
++              rx_tail = bcm_spi_readb(bs, SPI_RX_TAIL);
 +
 +              /* Read out all the data */
-+              if (rx_tail) {
-+                      u8 data;
-+                      u8 i = 0;
-+
-+                      for(i = 0; i < rx_tail; i++) {
-+                              data = bcm_spi_readb(SPI_RX_DATA);
-+                              if (bs->rx_ptr)
-+                                      *bs->rx_ptr++ = data;
-+                      }
-+              }
++              if (rx_tail)
++                      memcpy_fromio(bs->rx_ptr, bs->rx_io, rx_tail);
 +
 +              /* See if there is more data to send */
 +              if (bs->remaining_bytes > 0) {
 +                      bcm63xx_spi_fill_tx_fifo(bs);
 +
 +                      /* Start the transfer */
-+                      bcm_spi_writew(SPI_HD_W << SPI_MSG_TYPE_SHIFT,
++                      bcm_spi_writew(bs, SPI_HD_W << SPI_MSG_TYPE_SHIFT,
 +                                     SPI_MSG_CTL);
-+                      cmd = bcm_spi_readw(SPI_CMD);
++                      cmd = bcm_spi_readw(bs, SPI_CMD);
 +                      cmd |= SPI_CMD_START_IMMEDIATE;
 +                      cmd |= (0 << SPI_CMD_PREPEND_BYTE_CNT_SHIFT);
-+                      bcm_spi_writeb(SPI_INTR_CMD_DONE, SPI_INT_MASK);
-+                      bcm_spi_writew(cmd, SPI_CMD);
++                      bcm_spi_writeb(bs, SPI_INTR_CMD_DONE, SPI_INT_MASK);
++                      bcm_spi_writew(bs, cmd, SPI_CMD);
 +              } else {
 +                      complete(&bs->done);
 +              }
 +              goto out;
 +      }
 +
-+      clk = clk_get(&pdev->dev, "spi");
++      clk = clk_get(dev, "spi");
 +      if (IS_ERR(clk)) {
 +              dev_err(dev, "no clock for device\n");
 +              ret = -ENODEV;
 +              goto out;
 +      }
 +
-+      master = spi_alloc_master(&pdev->dev, sizeof(struct bcm63xx_spi));
++      master = spi_alloc_master(dev, sizeof(*bs));
 +      if (!master) {
 +              dev_err(dev, "out of memory\n");
 +              ret = -ENOMEM;
 +      init_completion(&bs->done);
 +
 +      platform_set_drvdata(pdev, master);
-+        bs->pdev = pdev;
++      bs->pdev = pdev;
 +
-+      if (!request_mem_region(r->start,
-+                      r->end - r->start, PFX)) {
++      if (!request_mem_region(r->start, r->end - r->start, PFX)) {
 +              dev_err(dev, "iomem request failed\n");
 +              ret = -ENXIO;
 +              goto out_put_master;
 +      }
 +
-+        bs->regs = ioremap_nocache(r->start, r->end - r->start);
++      bs->regs = ioremap_nocache(r->start, r->end - r->start);
 +      if (!bs->regs) {
 +              dev_err(dev, "unable to ioremap regs\n");
 +              ret = -ENOMEM;
 +      bs->clk = clk;
 +      bs->fifo_size = pdata->fifo_size;
 +
-+      ret = request_irq(irq, bcm63xx_spi_interrupt, 0,
-+                        pdev->name, master);
++      ret = request_irq(irq, bcm63xx_spi_interrupt, 0, pdev->name, master);
 +      if (ret) {
 +              dev_err(dev, "unable to request irq\n");
 +              goto out_unmap;
 +      master->transfer = bcm63xx_transfer;
 +      bs->speed_hz = pdata->speed_hz;
 +      bs->stopping = 0;
++      bs->tx_io = (u8*)(bs->regs + bcm63xx_spireg(SPI_MSG_DATA));
++      bs->rx_io = (const u8*)(bs->regs + bcm63xx_spireg(SPI_RX_DATA));
 +      spin_lock_init(&bs->lock);
 +
 +      /* Initialize hardware */
 +      clk_enable(bs->clk);
-+      bcm_spi_writeb(SPI_INTR_CLEAR_ALL, SPI_INT_STATUS);
++      bcm_spi_writeb(bs, SPI_INTR_CLEAR_ALL, SPI_INT_STATUS);
 +
 +      /* register and we are done */
 +      ret = spi_register_master(master);
 +      dev_info(dev, "at 0x%08x (irq %d, FIFOs size %d) v%s\n",
 +               r->start, irq, bs->fifo_size, DRV_VER);
 +
-+      return ret;
++      return 0;
 +
 +out_reset_hw:
 +      clk_disable(clk);
 +
 +static int __exit bcm63xx_spi_remove(struct platform_device *pdev)
 +{
-+      struct spi_master  *master = platform_get_drvdata(pdev);
-+      struct bcm63xx_spi *bs = spi_master_get_devdata(master);
-+      struct resource    *r = platform_get_resource(pdev, IORESOURCE_MEM, 0);
++      struct spi_master       *master = platform_get_drvdata(pdev);
++      struct bcm63xx_spi      *bs = spi_master_get_devdata(master);
++      struct resource         *r = platform_get_resource(pdev, IORESOURCE_MEM, 0);
 +
 +      /* reset spi block */
-+      bcm_spi_writeb(0, SPI_INT_MASK);
++      bcm_spi_writeb(bs, 0, SPI_INT_MASK);
 +      spin_lock(&bs->lock);
 +      bs->stopping = 1;
 +
 +              .owner  = THIS_MODULE,
 +      },
 +      .probe          = bcm63xx_spi_probe,
-+      .remove         = bcm63xx_spi_remove,
++      .remove         = __exit_p(bcm63xx_spi_remove),
 +      .suspend        = bcm63xx_spi_suspend,
 +      .resume         = bcm63xx_spi_resume,
 +};
 +
 +MODULE_ALIAS("platform:bcm63xx_spi");
 +MODULE_AUTHOR("Florian Fainelli <florian@openwrt.org>");
++MODULE_AUTHOR("Tanguy Bouzeloc <tanguy.bouzeloc@efixo.com>");
 +MODULE_DESCRIPTION("Broadcom BCM63xx SPI Controller driver");
 +MODULE_LICENSE("GPL");
 +MODULE_VERSION(DRV_VER);
  spi_s3c24xx_hw-y                      := spi_s3c24xx.o
 --- /dev/null
 +++ b/arch/mips/include/asm/mach-bcm63xx/bcm63xx_dev_spi.h
-@@ -0,0 +1,136 @@
+@@ -0,0 +1,126 @@
 +#ifndef BCM63XX_DEV_SPI_H
 +#define BCM63XX_DEV_SPI_H
 +
 +      return 0;
 +}
 +
-+/*
-+ * helpers for the SPI register sets
-+ */
-+#define bcm_spi_readb(o)    bcm_rset_readb(RSET_SPI, bcm63xx_spireg(o))
-+#define bcm_spi_readw(o)    bcm_rset_readw(RSET_SPI, bcm63xx_spireg(o))
-+#define bcm_spi_readl(o)    bcm_rset_readl(RSET_SPI, bcm63xx_spireg(o))
-+#define bcm_spi_writeb(v,o) bcm_rset_writeb(RSET_SPI, (v), bcm63xx_spireg(o))
-+#define bcm_spi_writew(v,o) bcm_rset_writew(RSET_SPI, (v), bcm63xx_spireg(o))
-+#define bcm_spi_writel(v,o) bcm_rset_writel(RSET_SPI, (v), bcm63xx_spireg(o))
-+
 +#endif /* BCM63XX_DEV_SPI_H */
 --- a/arch/mips/bcm63xx/Makefile
 +++ b/arch/mips/bcm63xx/Makefile
  #include <board_bcm963xx.h>
  
  #define PFX   "board_bcm963xx: "
-@@ -1489,6 +1490,9 @@ void __init board_prom_init(void)
-       bcm_gpio_writel(val, GPIO_MODE_REG);
+@@ -1587,6 +1588,9 @@ int __init board_register_devices(void)
+       if (board.num_spis)
+               spi_register_board_info(board.spis, board.num_spis);
  
 +      if (!BCMCPU_IS_6345())
 +              bcm63xx_spi_register();
 +
-       /* Generate MAC address for WLAN and
-        * register our SPROM */
- #ifdef CONFIG_SSB_PCIHOST
+       /* read base address of boot chip select (0) */
+       if (BCMCPU_IS_6345())
+               val = 0x1fc00000;