: NUC900: Add spi driver support for nuc900

Dear sirs,

Thanks a lot for your help,and I fix the spi driver and re-submit it.

Signed-off-by Wan ZongShun <[email protected]>

---

?arch/arm/mach-w90x900/include/mach/nuc900_spi.h | ? 35 ++
?drivers/spi/Kconfig ? ? ? ? ? ? ? ? ? ? ? ? ? ? | ? ?7 +
?drivers/spi/Makefile ? ? ? ? ? ? ? ? ? ? ? ? ? ?| ? ?1 +
?drivers/spi/spi_nuc900.c ? ? ? ? ? ? ? ? ? ? ? ?| ?509 +++++++++++++++++++++++

?4 files changed, 552 insertions(+), 0 deletions(-) ?create mode 100644 arch/arm/mach-w90x900/include/mach/nuc900_spi.h ?create mode 100644 drivers/spi/spi_nuc900.c

arch/arm/mach-w90x900/include/mach/nuc900_spi.h new file +35 -0

new file mode 100644
index 0000000..24ea23b
@@ -0,0 +1,35 @@
+/*
+ * arch/arm/mach-w90x900/include/mach/nuc900_spi.h
+ *
+ * Copyright (c) 2009 Nuvoton technology corporation.
+ *
+ * Wan ZongShun <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation;version 2 of the License.
+ *
+ */
+
+#ifndef __ASM_ARCH_SPI_H
+#define __ASM_ARCH_SPI_H
+
+extern void mfp_set_groupg(struct device *dev);
+
+struct w90p910_spi_info {
+ ? ? ? unsigned int num_cs;
+ ? ? ? unsigned int lsb;
+ ? ? ? unsigned int txneg;
+ ? ? ? unsigned int rxneg;
+ ? ? ? unsigned int divider;
+ ? ? ? unsigned int sleep;
+ ? ? ? unsigned int txnum;
+ ? ? ? unsigned int txbitlen;
+ ? ? ? int bus_num;
+};
+
+struct w90p910_spi_chip {
+ ? ? ? unsigned char bits_per_word;
+};
+
+#endif /* __ASM_ARCH_SPI_H */

drivers/spi/Kconfig +0 -0

index 4b6f7cb..2e1b20c 100644
@@ -244,6 +244,13 @@ config SPI_XILINX

? ? ? ? ?See the "OPB Serial Peripheral Interface (SPI) (v1.00e)" ? ? ? ? ?Product Specification document (DS464) for hardware details.

drivers/spi/Kconfig +7 -0

+config SPI_NUC900
+ ? ? ? tristate "Nuvoton NUC900 series SPI"
+ ? ? ? depends on ARCH_W90X900 && EXPERIMENTAL
+ ? ? ? select SPI_BITBANG
+ ? ? ? help
+ ? ? ? ? SPI driver for Nuvoton NUC900 series ARM SoCs
+

?#
?# Add new SPI master controllers in alphabetical order above this line ?#

drivers/spi/Makefile +0 -0

index 21a1182..694a4cb 100644
@@ -33,6 +33,7 @@ obj-$(CONFIG_SPI_TXX9) ? ? ? ? ? ? ? ? ? ? ? ?+= spi_txx9.o

?obj-$(CONFIG_SPI_XILINX) ? ? ? ? ? ? ? += xilinx_spi.o ?obj-$(CONFIG_SPI_SH_SCI) ? ? ? ? ? ? ? += spi_sh_sci.o ?obj-$(CONFIG_SPI_STMP3XXX) ? ? ? ? ? ? += spi_stmp.o +obj-$(CONFIG_SPI_NUC900) ? ? ? ? ? ? ? += spi_nuc900.o ?# ? ? ?... add above this line ...

?# SPI protocol drivers (device/link on bus)

drivers/spi/spi_nuc900.c new file +442 -0

new file mode 100644
index 0000000..6836fd1
@@ -0,0 +1,509 @@
+/* linux/drivers/spi/spi_nuc900.c
+ *
+ * Copyright (c) 2009 Nuvoton technology.
+ * Wan ZongShun <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+*/
+
+#include <linux/init.h>
+#include <linux/spinlock.h>
+#include <linux/workqueue.h>
+#include <linux/interrupt.h>
+#include <linux/delay.h>
+#include <linux/errno.h>
+#include <linux/err.h>
+#include <linux/clk.h>
+#include <linux/device.h>
+#include <linux/platform_device.h>
+#include <linux/gpio.h>
+#include <linux/io.h>
+
+#include <linux/spi/spi.h>
+#include <linux/spi/spi_bitbang.h>
+
+#include <mach/nuc900_spi.h>
+
+/* usi registers offset */
+#define USI_CNT ? ? ? ? ? ? ? ?0x00
+#define USI_DIV ? ? ? ? ? ? ? ?0x04
+#define USI_SSR ? ? ? ? ? ? ? ?0x08
+#define USI_RX0 ? ? ? ? ? ? ? ?0x10
+#define USI_TX0 ? ? ? ? ? ? ? ?0x10
+
+/* usi register bit */
+#define ENINT ? ? ? ? ?(0x01 << 17)
+#define ENFLG ? ? ? ? ?(0x01 << 16)
+#define TXNUM ? ? ? ? ?(0x03 << 8)
+#define TXNEG ? ? ? ? ?(0x01 << 2)
+#define RXNEG ? ? ? ? ?(0x01 << 1)
+#define LSB ? ? ? ? ? ?(0x01 << 10)
+#define SELECTLEV ? ? ?(0x01 << 2)
+#define SELECTPOL ? ? ?(0x01 << 31)
+#define SELECTSLAVE ? ?0x01
+#define GOBUSY ? ? ? ? 0x01
+
+struct w90p910_spi {
+ ? ? ? struct spi_bitbang ? ? ? bitbang;
+ ? ? ? struct completion ? ? ? ?done;
+ ? ? ? void __iomem ? ? ? ? ? ?*regs;
+ ? ? ? int ? ? ? ? ? ? ? ? ? ? ?irq;
+ ? ? ? int ? ? ? ? ? ? ? ? ? ? ?len;
+ ? ? ? int ? ? ? ? ? ? ? ? ? ? ?count;
+ ? ? ? const unsigned char ? ? *tx;
+ ? ? ? unsigned char ? ? ? ? ? *rx;
+ ? ? ? struct clk ? ? ? ? ? ? ?*clk;
+ ? ? ? struct resource ? ? ? ? *ioarea;
+ ? ? ? struct spi_master ? ? ? *master;
+ ? ? ? struct spi_device ? ? ? *curdev;
+ ? ? ? struct device ? ? ? ? ? *dev;
+ ? ? ? struct w90p910_spi_info *pdata;
+ ? ? ? spinlock_t ? ? ? ? ? ? ?lock;
+};
+
+static inline struct w90p910_spi *to_hw(struct spi_device *sdev)
+{
+ ? ? ? return spi_master_get_devdata(sdev->master);
+}
+
+static void w90p910_slave_select(struct spi_device *spi, unsigned int ssr)
+{
+ ? ? ? struct w90p910_spi *hw = to_hw(spi);
+ ? ? ? unsigned int val;
+ ? ? ? unsigned int cs = spi->mode & SPI_CS_HIGH ? 1 : 0;
+ ? ? ? unsigned int cpol = spi->mode & SPI_CPOL ? 1 : 0;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_SSR);
+
+ ? ? ? if (!cs)
+ ? ? ? ? ? ? ? val &= ~SELECTLEV;
+ ? ? ? else
+ ? ? ? ? ? ? ? val |= SELECTLEV;
+
+ ? ? ? if (!ssr)
+ ? ? ? ? ? ? ? val &= ~SELECTSLAVE;
+ ? ? ? else
+ ? ? ? ? ? ? ? val |= SELECTSLAVE;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_SSR);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (!cpol)
+ ? ? ? ? ? ? ? val &= ~SELECTPOL;
+ ? ? ? else
+ ? ? ? ? ? ? ? val |= SELECTPOL;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_spi_chipsel(struct spi_device *spi, int value)
+{
+ ? ? ? switch (value) {
+ ? ? ? case BITBANG_CS_INACTIVE:
+ ? ? ? ? ? ? ? w90p910_slave_select(spi, 0);
+ ? ? ? ? ? ? ? break;
+
+ ? ? ? case BITBANG_CS_ACTIVE:
+ ? ? ? ? ? ? ? w90p910_slave_select(spi, 1);
+ ? ? ? ? ? ? ? break;
+ ? ? ? }
+}
+
+static void w90p910_spi_setup_txnum(struct w90p910_spi *hw,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned int txnum)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (!txnum)
+ ? ? ? ? ? ? ? val &= ~TXNUM;
+ ? ? ? else
+ ? ? ? ? ? ? ? val |= txnum << 0x08;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+
+}
+
+static void w90p910_spi_setup_txbitlen(struct w90p910_spi *hw,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned int txbitlen)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? val |= (txbitlen << 0x03);
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_spi_gobusy(struct w90p910_spi *hw)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? val |= GOBUSY;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static int w90p910_spi_setupxfer(struct spi_device *spi,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?struct spi_transfer *t)
+{
+ ? ? ? return 0;
+}
+
+static int w90p910_spi_setup(struct spi_device *spi)
+{
+ ? ? ? return 0;
+}
+
+static inline unsigned int hw_txbyte(struct w90p910_spi *hw, int count)
+{
+ ? ? ? return hw->tx ? hw->tx[count] : 0;
+}
+
+static int w90p910_spi_txrx(struct spi_device *spi, struct spi_transfer *t)
+{
+ ? ? ? struct w90p910_spi *hw = to_hw(spi);
+
+ ? ? ? hw->tx = t->tx_buf;
+ ? ? ? hw->rx = t->rx_buf;
+ ? ? ? hw->len = t->len;
+ ? ? ? hw->count = 0;
+
+ ? ? ? __raw_writel(hw_txbyte(hw, 0x0), hw->regs + USI_TX0);
+
+ ? ? ? w90p910_spi_gobusy(hw);
+
+ ? ? ? wait_for_completion(&hw->done);
+
+ ? ? ? return hw->count;
+}
+
+static irqreturn_t w90p910_spi_irq(int irq, void *dev)
+{
+ ? ? ? struct w90p910_spi *hw = dev;
+ ? ? ? unsigned int status;
+ ? ? ? unsigned int count = hw->count;
+
+ ? ? ? status = __raw_readl(hw->regs + USI_CNT);
+ ? ? ? __raw_writel(status, hw->regs + USI_CNT);
+
+ ? ? ? if (status & ENFLG) {
+ ? ? ? ? ? ? ? hw->count++;
+
+ ? ? ? ? ? ? ? if (hw->rx)
+ ? ? ? ? ? ? ? ? ? ? ? hw->rx[count] = __raw_readl(hw->regs + USI_RX0);
+ ? ? ? ? ? ? ? count++;
+
+ ? ? ? ? ? ? ? if (count < hw->len) {
+ ? ? ? ? ? ? ? ? ? ? ? __raw_writel(hw_txbyte(hw, count), hw->regs + USI_TX0);
+ ? ? ? ? ? ? ? ? ? ? ? w90p910_spi_gobusy(hw);
+ ? ? ? ? ? ? ? } else {
+ ? ? ? ? ? ? ? ? ? ? ? complete(&hw->done);
+ ? ? ? ? ? ? ? }
+
+ ? ? ? ? ? ? ? return IRQ_HANDLED;
+ ? ? ? }
+
+ ? ? ? complete(&hw->done);
+ ? ? ? return IRQ_HANDLED;
+}
+
+static void w90p910_tx_edge(struct w90p910_spi *hw, unsigned int edge)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (edge)
+ ? ? ? ? ? ? ? val |= TXNEG;
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~TXNEG;
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_rx_edge(struct w90p910_spi *hw, unsigned int edge)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (edge)
+ ? ? ? ? ? ? ? val |= RXNEG;
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~RXNEG;
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_send_first(struct w90p910_spi *hw, unsigned int lsb)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (lsb)
+ ? ? ? ? ? ? ? val |= LSB;
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~LSB;
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_set_sleep(struct w90p910_spi *hw, unsigned int sleep)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (sleep)
+ ? ? ? ? ? ? ? val |= (sleep << 12);
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~(0x0f << 12);
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_enable_int(struct w90p910_spi *hw, unsigned int enable)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (enable)
+ ? ? ? ? ? ? ? val |= ENINT;
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~ENINT;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_set_divider(struct w90p910_spi *hw)
+{
+ ? ? ? __raw_writel(hw->pdata->divider, hw->regs + USI_DIV);
+}
+
+static void w90p910_init_spi(struct w90p910_spi *hw)
+{
+ ? ? ? clk_enable(hw->clk);
+ ? ? ? spin_lock_init(&hw->lock);
+
+ ? ? ? w90p910_tx_edge(hw, hw->pdata->txneg);
+ ? ? ? w90p910_rx_edge(hw, hw->pdata->rxneg);
+ ? ? ? w90p910_send_first(hw, hw->pdata->lsb);
+ ? ? ? w90p910_set_sleep(hw, hw->pdata->sleep);
+ ? ? ? w90p910_spi_setup_txbitlen(hw, hw->pdata->txbitlen);
+ ? ? ? w90p910_spi_setup_txnum(hw, hw->pdata->txnum);
+ ? ? ? w90p910_set_divider(hw);
+ ? ? ? w90p910_enable_int(hw, 1);
+}
+
+static int __devinit w90p910_spi_probe(struct platform_device *pdev)
+{
+ ? ? ? struct w90p910_spi *hw;
+ ? ? ? struct spi_master *master;
+ ? ? ? struct resource *res;
+ ? ? ? int err = 0;
+
+ ? ? ? master = spi_alloc_master(&pdev->dev, sizeof(struct w90p910_spi));
+ ? ? ? if (master == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No memory for spi_master\n");
+ ? ? ? ? ? ? ? err = -ENOMEM;
+ ? ? ? ? ? ? ? goto err_nomem;
+ ? ? ? }
+
+ ? ? ? hw = spi_master_get_devdata(master);
+ ? ? ? memset(hw, 0, sizeof(struct w90p910_spi));
+
+ ? ? ? hw->master = spi_master_get(master);
+ ? ? ? hw->pdata ?= pdev->dev.platform_data;
+ ? ? ? hw->dev = &pdev->dev;
+
+ ? ? ? if (hw->pdata == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No platform data supplied\n");
+ ? ? ? ? ? ? ? err = -ENOENT;
+ ? ? ? ? ? ? ? goto err_pdata;
+ ? ? ? }
+
+ ? ? ? platform_set_drvdata(pdev, hw);
+ ? ? ? init_completion(&hw->done);
+
+ ? ? ? master->mode_bits ? ? ? ? ?= SPI_MODE_0;
+ ? ? ? master->num_chipselect ? ? = hw->pdata->num_cs;
+ ? ? ? master->bus_num ? ? ? ? ? ?= hw->pdata->bus_num;
+ ? ? ? hw->bitbang.master ? ? ? ? = hw->master;
+ ? ? ? hw->bitbang.setup_transfer = w90p910_spi_setupxfer;
+ ? ? ? hw->bitbang.chipselect ? ? = w90p910_spi_chipsel;
+ ? ? ? hw->bitbang.txrx_bufs ? ? ?= w90p910_spi_txrx;
+ ? ? ? hw->bitbang.master->setup ?= w90p910_spi_setup;
+
+ ? ? ? res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ ? ? ? if (res == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Cannot get IORESOURCE_MEM\n");
+ ? ? ? ? ? ? ? err = -ENOENT;
+ ? ? ? ? ? ? ? goto err_pdata;
+ ? ? ? }
+
+ ? ? ? hw->ioarea = request_mem_region(res->start,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? resource_size(res), pdev->name);
+
+ ? ? ? if (hw->ioarea == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Cannot reserve region\n");
+ ? ? ? ? ? ? ? err = -ENXIO;
+ ? ? ? ? ? ? ? goto err_pdata;
+ ? ? ? }
+
+ ? ? ? hw->regs = ioremap(res->start, resource_size(res));
+ ? ? ? if (hw->regs == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Cannot map IO\n");
+ ? ? ? ? ? ? ? err = -ENXIO;
+ ? ? ? ? ? ? ? goto err_iomap;
+ ? ? ? }
+
+ ? ? ? hw->irq = platform_get_irq(pdev, 0);
+ ? ? ? if (hw->irq < 0) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No IRQ specified\n");
+ ? ? ? ? ? ? ? err = -ENOENT;
+ ? ? ? ? ? ? ? goto err_irq;
+ ? ? ? }
+
+ ? ? ? err = request_irq(hw->irq, w90p910_spi_irq, 0, pdev->name, hw);
+ ? ? ? if (err) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Cannot claim IRQ\n");
+ ? ? ? ? ? ? ? goto err_irq;
+ ? ? ? }
+
+ ? ? ? hw->clk = clk_get(&pdev->dev, "spi");
+ ? ? ? if (IS_ERR(hw->clk)) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No clock for device\n");
+ ? ? ? ? ? ? ? err = PTR_ERR(hw->clk);
+ ? ? ? ? ? ? ? goto err_clk;
+ ? ? ? }
+
+ ? ? ? mfp_set_groupg(&pdev->dev);
+ ? ? ? w90p910_init_spi(hw);
+
+ ? ? ? err = spi_bitbang_start(&hw->bitbang);
+ ? ? ? if (err) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Failed to register SPI master\n");
+ ? ? ? ? ? ? ? goto err_register;
+ ? ? ? }
+
+ ? ? ? return 0;
+

drivers/spi/spi_nuc900.c new file +57 -1

+err_pdata:
+ ? ? ? spi_master_put(hw->master);;
+
+err_nomem:
+ ? ? ? return err;
+}
+
+static int __devexit w90p910_spi_remove(struct platform_device *dev)
+{
+ ? ? ? struct w90p910_spi *hw = platform_get_drvdata(dev);
+
+ ? ? ? w90p910_enable_int(hw, 0);
+
+ ? ? ? free_irq(hw->irq, hw);
+
+ ? ? ? platform_set_drvdata(dev, NULL);
+
+ ? ? ? spi_unregister_master(hw->master);
+
+ ? ? ? clk_disable(hw->clk);
+ ? ? ? clk_put(hw->clk);
+
+ ? ? ? iounmap(hw->regs);
+
+ ? ? ? release_resource(hw->ioarea);
+ ? ? ? kfree(hw->ioarea);
+
+ ? ? ? spi_master_put(hw->master);
+ ? ? ? return 0;
+}
+
+static struct platform_driver w90p910_spi_driver = {
+ ? ? ? .probe ? ? ? ? ?= w90p910_spi_probe,
+ ? ? ? .remove ? ? ? ? = __devexit_p(w90p910_spi_remove),
+ ? ? ? .driver ? ? ? ? = {
+ ? ? ? ? ? ? ? .name ? = "w90p910-spi",
+ ? ? ? ? ? ? ? .owner ?= THIS_MODULE,
+ ? ? ? },
+};
+
+static int __init w90p910_spi_init(void)
+{
+ ? ? ? return platform_driver_register(&w90p910_spi_driver);
+}
+
+static void __exit w90p910_spi_exit(void)
+{
+ ? ? ? platform_driver_unregister(&w90p910_spi_driver);
+}
+
+module_init(w90p910_spi_init);
+module_exit(w90p910_spi_exit);
+
+MODULE_AUTHOR("Wan ZongShun <[email protected]>");
+MODULE_DESCRIPTION("w90p910 spi driver!");
+MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:w90p910-spi");
--

1.5.6.3

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Quoted 31 lines

Dear sirs,
+ ? ? ? err = request_irq(hw->irq, w90p910_spi_irq, 0, pdev->name, hw);
+ ? ? ? if (err) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Cannot claim IRQ\n");
+ ? ? ? ? ? ? ? goto err_irq;
+ ? ? ? }

(quoted diff) +25 -0

+
+ ? ? ? hw->clk = clk_get(&pdev->dev, "spi");
+ ? ? ? if (IS_ERR(hw->clk)) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No clock for device\n");
+ ? ? ? ? ? ? ? err = PTR_ERR(hw->clk);
+ ? ? ? ? ? ? ? goto err_clk;
+ ? ? ? }
+
+ ? ? ? mfp_set_groupg(&pdev->dev);
+ ? ? ? w90p910_init_spi(hw);
+
+ ? ? ? err = spi_bitbang_start(&hw->bitbang);
+ ? ? ? if (err) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Failed to register SPI master\n");
+ ? ? ? ? ? ? ? goto err_register;
+ ? ? ? }
+
+ ? ? ? return 0;
+
+err_register:
+ ? ? ? clk_disable(hw->clk);
+ ? ? ? clk_put(hw->clk);
+err_clk:
+ ? ? ? free_irq(hw->irq, hw);
+err_irq:

Quoted 4 lines

+ ? ? ? iounmap(hw->regs);
+err_iomap:
+ ? ? ? release_resource(hw->ioarea);
+ ? ? ? kfree(hw->ioarea);

I think release_mem_region() is better than release_resource() after request_mem_region().
After all, they are a couple :)

Quoted 7 lines

drivers/spi/spi_nuc900.c new file +7 -0

+err_pdata:
+ ? ? ? spi_master_put(hw->master);;
+
+err_nomem:
+ ? ? ? return err;
+}
+

drivers/spi/spi_nuc900.c new file +24 -0

+static void w90p910_init_spi(struct w90p910_spi *hw)
+{
+       clk_enable(hw->clk);
+       spin_lock_init(&hw->lock);
+
+       w90p910_tx_edge(hw, hw->pdata->txneg);
+       w90p910_rx_edge(hw, hw->pdata->rxneg);
+       w90p910_send_first(hw, hw->pdata->lsb);
+       w90p910_set_sleep(hw, hw->pdata->sleep);
+       w90p910_spi_setup_txbitlen(hw, hw->pdata->txbitlen);
+       w90p910_spi_setup_txnum(hw, hw->pdata->txnum);
+       w90p910_set_divider(hw);
+       w90p910_enable_int(hw, 1);
+}
+
+static int __devinit w90p910_spi_probe(struct platform_device *pdev)
+{
+ ? ? ? struct w90p910_spi *hw;
+ ? ? ? struct spi_master *master;
+ ? ? ? struct resource *res;
+ ? ? ? int err = 0;
+
+ ? ? ? master = spi_alloc_master(&pdev->dev, sizeof(struct w90p910_spi));
+ ? ? ? if (master == NULL) {

Quoted 25 lines

(quoted diff) +24 -0

+static void w90p910_init_spi(struct w90p910_spi *hw)
+{
+ ? ? ? clk_enable(hw->clk);
+ ? ? ? spin_lock_init(&hw->lock);
+
+ ? ? ? w90p910_tx_edge(hw, hw->pdata->txneg);
+ ? ? ? w90p910_rx_edge(hw, hw->pdata->rxneg);
+ ? ? ? w90p910_send_first(hw, hw->pdata->lsb);
+ ? ? ? w90p910_set_sleep(hw, hw->pdata->sleep);
+ ? ? ? w90p910_spi_setup_txbitlen(hw, hw->pdata->txbitlen);
+ ? ? ? w90p910_spi_setup_txnum(hw, hw->pdata->txnum);
+ ? ? ? w90p910_set_divider(hw);
+ ? ? ? w90p910_enable_int(hw, 1);
+}
+
+static int __devinit w90p910_spi_probe(struct platform_device *pdev)
+{
+ ? ? ? struct w90p910_spi *hw;
+ ? ? ? struct spi_master *master;
+ ? ? ? struct resource *res;
+ ? ? ? int err = 0;
+
+ ? ? ? master = spi_alloc_master(&pdev->dev, sizeof(struct w90p910_spi));
+ ? ? ? if (master == NULL) {

Review

Quoted 10 lines

(quoted diff) +10 -0

+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No memory for spi_master\n");
+ ? ? ? ? ? ? ? err = -ENOMEM;
+ ? ? ? ? ? ? ? goto err_nomem;
+ ? ? ? }
+
+ ? ? ? hw = spi_master_get_devdata(master);
+ ? ? ? memset(hw, 0, sizeof(struct w90p910_spi));
+
+ ? ? ? hw->master = spi_master_get(master);
+ ? ? ? hw->pdata ?= pdev->dev.platform_data;

Seems that platform_data is necessary in your driver, but I found it is not initialized in mach-w90x900/dev.c:

static struct platform_device nuc900_device_spi = { ? ? ? ?.name ? ? ? ? ? = "nuc900-spi",
? ? ? ?.id ? ? ? ? ? ? = -1,
? ? ? ?.num_resources ?= ARRAY_SIZE(nuc900_spi_resource), ? ? ? ?.resource ? ? ? = nuc900_spi_resource,
};

--
Regards
Li Jie

Dear sirs,

Thanks a lot for your help,and I fix the spi driver and re-submit it.

Signed-off-by Wan ZongShun <[email protected]>

---

?arch/arm/mach-w90x900/include/mach/nuc900_spi.h | ? 35 ++
?drivers/spi/Kconfig ? ? ? ? ? ? ? ? ? ? ? ? ? ? | ? ?7 +
?drivers/spi/Makefile ? ? ? ? ? ? ? ? ? ? ? ? ? ?| ? ?1 +
?drivers/spi/spi_nuc900.c ? ? ? ? ? ? ? ? ? ? ? ?| ?509 +++++++++++++++++++++++

?4 files changed, 552 insertions(+), 0 deletions(-) ?create mode 100644 arch/arm/mach-w90x900/include/mach/nuc900_spi.h ?create mode 100644 drivers/spi/spi_nuc900.c

arch/arm/mach-w90x900/include/mach/nuc900_spi.h new file +35 -0

new file mode 100644
index 0000000..24ea23b
@@ -0,0 +1,35 @@
+/*
+ * arch/arm/mach-w90x900/include/mach/nuc900_spi.h
+ *
+ * Copyright (c) 2009 Nuvoton technology corporation.
+ *
+ * Wan ZongShun <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation;version 2 of the License.
+ *
+ */
+
+#ifndef __ASM_ARCH_SPI_H
+#define __ASM_ARCH_SPI_H
+
+extern void mfp_set_groupg(struct device *dev);
+
+struct w90p910_spi_info {
+ ? ? ? unsigned int num_cs;
+ ? ? ? unsigned int lsb;
+ ? ? ? unsigned int txneg;
+ ? ? ? unsigned int rxneg;
+ ? ? ? unsigned int divider;
+ ? ? ? unsigned int sleep;
+ ? ? ? unsigned int txnum;
+ ? ? ? unsigned int txbitlen;
+ ? ? ? int bus_num;
+};
+
+struct w90p910_spi_chip {
+ ? ? ? unsigned char bits_per_word;
+};
+
+#endif /* __ASM_ARCH_SPI_H */

drivers/spi/Kconfig +0 -0

index 4b6f7cb..2e1b20c 100644
@@ -244,6 +244,13 @@ config SPI_XILINX

? ? ? ? ?See the "OPB Serial Peripheral Interface (SPI) (v1.00e)" ? ? ? ? ?Product Specification document (DS464) for hardware details.

drivers/spi/Kconfig +7 -0

+config SPI_NUC900
+ ? ? ? tristate "Nuvoton NUC900 series SPI"
+ ? ? ? depends on ARCH_W90X900 && EXPERIMENTAL
+ ? ? ? select SPI_BITBANG
+ ? ? ? help
+ ? ? ? ? SPI driver for Nuvoton NUC900 series ARM SoCs
+

?#
?# Add new SPI master controllers in alphabetical order above this line ?#

drivers/spi/Makefile +0 -0

index 21a1182..694a4cb 100644
@@ -33,6 +33,7 @@ obj-$(CONFIG_SPI_TXX9) ? ? ? ? ? ? ? ? ? ? ? ?+= spi_txx9.o

?obj-$(CONFIG_SPI_XILINX) ? ? ? ? ? ? ? += xilinx_spi.o ?obj-$(CONFIG_SPI_SH_SCI) ? ? ? ? ? ? ? += spi_sh_sci.o ?obj-$(CONFIG_SPI_STMP3XXX) ? ? ? ? ? ? += spi_stmp.o +obj-$(CONFIG_SPI_NUC900) ? ? ? ? ? ? ? += spi_nuc900.o ?# ? ? ?... add above this line ...

?# SPI protocol drivers (device/link on bus)

drivers/spi/spi_nuc900.c new file +509 -1

new file mode 100644
index 0000000..6836fd1
@@ -0,0 +1,509 @@
+/* linux/drivers/spi/spi_nuc900.c
+ *
+ * Copyright (c) 2009 Nuvoton technology.
+ * Wan ZongShun <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+*/
+
+#include <linux/init.h>
+#include <linux/spinlock.h>
+#include <linux/workqueue.h>
+#include <linux/interrupt.h>
+#include <linux/delay.h>
+#include <linux/errno.h>
+#include <linux/err.h>
+#include <linux/clk.h>
+#include <linux/device.h>
+#include <linux/platform_device.h>
+#include <linux/gpio.h>
+#include <linux/io.h>
+
+#include <linux/spi/spi.h>
+#include <linux/spi/spi_bitbang.h>
+
+#include <mach/nuc900_spi.h>
+
+/* usi registers offset */
+#define USI_CNT ? ? ? ? ? ? ? ?0x00
+#define USI_DIV ? ? ? ? ? ? ? ?0x04
+#define USI_SSR ? ? ? ? ? ? ? ?0x08
+#define USI_RX0 ? ? ? ? ? ? ? ?0x10
+#define USI_TX0 ? ? ? ? ? ? ? ?0x10
+
+/* usi register bit */
+#define ENINT ? ? ? ? ?(0x01 << 17)
+#define ENFLG ? ? ? ? ?(0x01 << 16)
+#define TXNUM ? ? ? ? ?(0x03 << 8)
+#define TXNEG ? ? ? ? ?(0x01 << 2)
+#define RXNEG ? ? ? ? ?(0x01 << 1)
+#define LSB ? ? ? ? ? ?(0x01 << 10)
+#define SELECTLEV ? ? ?(0x01 << 2)
+#define SELECTPOL ? ? ?(0x01 << 31)
+#define SELECTSLAVE ? ?0x01
+#define GOBUSY ? ? ? ? 0x01
+
+struct w90p910_spi {
+ ? ? ? struct spi_bitbang ? ? ? bitbang;
+ ? ? ? struct completion ? ? ? ?done;
+ ? ? ? void __iomem ? ? ? ? ? ?*regs;
+ ? ? ? int ? ? ? ? ? ? ? ? ? ? ?irq;
+ ? ? ? int ? ? ? ? ? ? ? ? ? ? ?len;
+ ? ? ? int ? ? ? ? ? ? ? ? ? ? ?count;
+ ? ? ? const unsigned char ? ? *tx;
+ ? ? ? unsigned char ? ? ? ? ? *rx;
+ ? ? ? struct clk ? ? ? ? ? ? ?*clk;
+ ? ? ? struct resource ? ? ? ? *ioarea;
+ ? ? ? struct spi_master ? ? ? *master;
+ ? ? ? struct spi_device ? ? ? *curdev;
+ ? ? ? struct device ? ? ? ? ? *dev;
+ ? ? ? struct w90p910_spi_info *pdata;
+ ? ? ? spinlock_t ? ? ? ? ? ? ?lock;
+};
+
+static inline struct w90p910_spi *to_hw(struct spi_device *sdev)
+{
+ ? ? ? return spi_master_get_devdata(sdev->master);
+}
+
+static void w90p910_slave_select(struct spi_device *spi, unsigned int ssr)
+{
+ ? ? ? struct w90p910_spi *hw = to_hw(spi);
+ ? ? ? unsigned int val;
+ ? ? ? unsigned int cs = spi->mode & SPI_CS_HIGH ? 1 : 0;
+ ? ? ? unsigned int cpol = spi->mode & SPI_CPOL ? 1 : 0;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_SSR);
+
+ ? ? ? if (!cs)
+ ? ? ? ? ? ? ? val &= ~SELECTLEV;
+ ? ? ? else
+ ? ? ? ? ? ? ? val |= SELECTLEV;
+
+ ? ? ? if (!ssr)
+ ? ? ? ? ? ? ? val &= ~SELECTSLAVE;
+ ? ? ? else
+ ? ? ? ? ? ? ? val |= SELECTSLAVE;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_SSR);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (!cpol)
+ ? ? ? ? ? ? ? val &= ~SELECTPOL;
+ ? ? ? else
+ ? ? ? ? ? ? ? val |= SELECTPOL;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_spi_chipsel(struct spi_device *spi, int value)
+{
+ ? ? ? switch (value) {
+ ? ? ? case BITBANG_CS_INACTIVE:
+ ? ? ? ? ? ? ? w90p910_slave_select(spi, 0);
+ ? ? ? ? ? ? ? break;
+
+ ? ? ? case BITBANG_CS_ACTIVE:
+ ? ? ? ? ? ? ? w90p910_slave_select(spi, 1);
+ ? ? ? ? ? ? ? break;
+ ? ? ? }
+}
+
+static void w90p910_spi_setup_txnum(struct w90p910_spi *hw,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned int txnum)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (!txnum)
+ ? ? ? ? ? ? ? val &= ~TXNUM;
+ ? ? ? else
+ ? ? ? ? ? ? ? val |= txnum << 0x08;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+
+}
+
+static void w90p910_spi_setup_txbitlen(struct w90p910_spi *hw,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned int txbitlen)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? val |= (txbitlen << 0x03);
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_spi_gobusy(struct w90p910_spi *hw)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? val |= GOBUSY;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static int w90p910_spi_setupxfer(struct spi_device *spi,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?struct spi_transfer *t)
+{
+ ? ? ? return 0;
+}
+
+static int w90p910_spi_setup(struct spi_device *spi)
+{
+ ? ? ? return 0;
+}
+
+static inline unsigned int hw_txbyte(struct w90p910_spi *hw, int count)
+{
+ ? ? ? return hw->tx ? hw->tx[count] : 0;
+}
+
+static int w90p910_spi_txrx(struct spi_device *spi, struct spi_transfer *t)
+{
+ ? ? ? struct w90p910_spi *hw = to_hw(spi);
+
+ ? ? ? hw->tx = t->tx_buf;
+ ? ? ? hw->rx = t->rx_buf;
+ ? ? ? hw->len = t->len;
+ ? ? ? hw->count = 0;
+
+ ? ? ? __raw_writel(hw_txbyte(hw, 0x0), hw->regs + USI_TX0);
+
+ ? ? ? w90p910_spi_gobusy(hw);
+
+ ? ? ? wait_for_completion(&hw->done);
+
+ ? ? ? return hw->count;
+}
+
+static irqreturn_t w90p910_spi_irq(int irq, void *dev)
+{
+ ? ? ? struct w90p910_spi *hw = dev;
+ ? ? ? unsigned int status;
+ ? ? ? unsigned int count = hw->count;
+
+ ? ? ? status = __raw_readl(hw->regs + USI_CNT);
+ ? ? ? __raw_writel(status, hw->regs + USI_CNT);
+
+ ? ? ? if (status & ENFLG) {
+ ? ? ? ? ? ? ? hw->count++;
+
+ ? ? ? ? ? ? ? if (hw->rx)
+ ? ? ? ? ? ? ? ? ? ? ? hw->rx[count] = __raw_readl(hw->regs + USI_RX0);
+ ? ? ? ? ? ? ? count++;
+
+ ? ? ? ? ? ? ? if (count < hw->len) {
+ ? ? ? ? ? ? ? ? ? ? ? __raw_writel(hw_txbyte(hw, count), hw->regs + USI_TX0);
+ ? ? ? ? ? ? ? ? ? ? ? w90p910_spi_gobusy(hw);
+ ? ? ? ? ? ? ? } else {
+ ? ? ? ? ? ? ? ? ? ? ? complete(&hw->done);
+ ? ? ? ? ? ? ? }
+
+ ? ? ? ? ? ? ? return IRQ_HANDLED;
+ ? ? ? }
+
+ ? ? ? complete(&hw->done);
+ ? ? ? return IRQ_HANDLED;
+}
+
+static void w90p910_tx_edge(struct w90p910_spi *hw, unsigned int edge)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (edge)
+ ? ? ? ? ? ? ? val |= TXNEG;
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~TXNEG;
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_rx_edge(struct w90p910_spi *hw, unsigned int edge)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (edge)
+ ? ? ? ? ? ? ? val |= RXNEG;
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~RXNEG;
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_send_first(struct w90p910_spi *hw, unsigned int lsb)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (lsb)
+ ? ? ? ? ? ? ? val |= LSB;
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~LSB;
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_set_sleep(struct w90p910_spi *hw, unsigned int sleep)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (sleep)
+ ? ? ? ? ? ? ? val |= (sleep << 12);
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~(0x0f << 12);
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_enable_int(struct w90p910_spi *hw, unsigned int enable)
+{
+ ? ? ? unsigned int val;
+ ? ? ? unsigned long flags;
+
+ ? ? ? spin_lock_irqsave(&hw->lock, flags);
+
+ ? ? ? val = __raw_readl(hw->regs + USI_CNT);
+
+ ? ? ? if (enable)
+ ? ? ? ? ? ? ? val |= ENINT;
+ ? ? ? else
+ ? ? ? ? ? ? ? val &= ~ENINT;
+
+ ? ? ? __raw_writel(val, hw->regs + USI_CNT);
+
+ ? ? ? spin_unlock_irqrestore(&hw->lock, flags);
+}
+
+static void w90p910_set_divider(struct w90p910_spi *hw)
+{
+ ? ? ? __raw_writel(hw->pdata->divider, hw->regs + USI_DIV);
+}
+
+static void w90p910_init_spi(struct w90p910_spi *hw)
+{
+ ? ? ? clk_enable(hw->clk);
+ ? ? ? spin_lock_init(&hw->lock);
+
+ ? ? ? w90p910_tx_edge(hw, hw->pdata->txneg);
+ ? ? ? w90p910_rx_edge(hw, hw->pdata->rxneg);
+ ? ? ? w90p910_send_first(hw, hw->pdata->lsb);
+ ? ? ? w90p910_set_sleep(hw, hw->pdata->sleep);
+ ? ? ? w90p910_spi_setup_txbitlen(hw, hw->pdata->txbitlen);
+ ? ? ? w90p910_spi_setup_txnum(hw, hw->pdata->txnum);
+ ? ? ? w90p910_set_divider(hw);
+ ? ? ? w90p910_enable_int(hw, 1);
+}
+
+static int __devinit w90p910_spi_probe(struct platform_device *pdev)
+{
+ ? ? ? struct w90p910_spi *hw;
+ ? ? ? struct spi_master *master;
+ ? ? ? struct resource *res;
+ ? ? ? int err = 0;
+
+ ? ? ? master = spi_alloc_master(&pdev->dev, sizeof(struct w90p910_spi));
+ ? ? ? if (master == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No memory for spi_master\n");
+ ? ? ? ? ? ? ? err = -ENOMEM;
+ ? ? ? ? ? ? ? goto err_nomem;
+ ? ? ? }
+
+ ? ? ? hw = spi_master_get_devdata(master);
+ ? ? ? memset(hw, 0, sizeof(struct w90p910_spi));
+
+ ? ? ? hw->master = spi_master_get(master);
+ ? ? ? hw->pdata ?= pdev->dev.platform_data;
+ ? ? ? hw->dev = &pdev->dev;
+
+ ? ? ? if (hw->pdata == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No platform data supplied\n");
+ ? ? ? ? ? ? ? err = -ENOENT;
+ ? ? ? ? ? ? ? goto err_pdata;
+ ? ? ? }
+
+ ? ? ? platform_set_drvdata(pdev, hw);
+ ? ? ? init_completion(&hw->done);
+
+ ? ? ? master->mode_bits ? ? ? ? ?= SPI_MODE_0;
+ ? ? ? master->num_chipselect ? ? = hw->pdata->num_cs;
+ ? ? ? master->bus_num ? ? ? ? ? ?= hw->pdata->bus_num;
+ ? ? ? hw->bitbang.master ? ? ? ? = hw->master;
+ ? ? ? hw->bitbang.setup_transfer = w90p910_spi_setupxfer;
+ ? ? ? hw->bitbang.chipselect ? ? = w90p910_spi_chipsel;
+ ? ? ? hw->bitbang.txrx_bufs ? ? ?= w90p910_spi_txrx;
+ ? ? ? hw->bitbang.master->setup ?= w90p910_spi_setup;
+
+ ? ? ? res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ ? ? ? if (res == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Cannot get IORESOURCE_MEM\n");
+ ? ? ? ? ? ? ? err = -ENOENT;
+ ? ? ? ? ? ? ? goto err_pdata;
+ ? ? ? }
+
+ ? ? ? hw->ioarea = request_mem_region(res->start,
+ ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? resource_size(res), pdev->name);
+
+ ? ? ? if (hw->ioarea == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Cannot reserve region\n");
+ ? ? ? ? ? ? ? err = -ENXIO;
+ ? ? ? ? ? ? ? goto err_pdata;
+ ? ? ? }
+
+ ? ? ? hw->regs = ioremap(res->start, resource_size(res));
+ ? ? ? if (hw->regs == NULL) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Cannot map IO\n");
+ ? ? ? ? ? ? ? err = -ENXIO;
+ ? ? ? ? ? ? ? goto err_iomap;
+ ? ? ? }
+
+ ? ? ? hw->irq = platform_get_irq(pdev, 0);
+ ? ? ? if (hw->irq < 0) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No IRQ specified\n");
+ ? ? ? ? ? ? ? err = -ENOENT;
+ ? ? ? ? ? ? ? goto err_irq;
+ ? ? ? }
+
+ ? ? ? err = request_irq(hw->irq, w90p910_spi_irq, 0, pdev->name, hw);
+ ? ? ? if (err) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Cannot claim IRQ\n");
+ ? ? ? ? ? ? ? goto err_irq;
+ ? ? ? }
+
+ ? ? ? hw->clk = clk_get(&pdev->dev, "spi");
+ ? ? ? if (IS_ERR(hw->clk)) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "No clock for device\n");
+ ? ? ? ? ? ? ? err = PTR_ERR(hw->clk);
+ ? ? ? ? ? ? ? goto err_clk;
+ ? ? ? }
+
+ ? ? ? mfp_set_groupg(&pdev->dev);
+ ? ? ? w90p910_init_spi(hw);
+
+ ? ? ? err = spi_bitbang_start(&hw->bitbang);
+ ? ? ? if (err) {
+ ? ? ? ? ? ? ? dev_err(&pdev->dev, "Failed to register SPI master\n");
+ ? ? ? ? ? ? ? goto err_register;
+ ? ? ? }
+
+ ? ? ? return 0;
+
+err_register:
+ ? ? ? clk_disable(hw->clk);
+ ? ? ? clk_put(hw->clk);
+err_clk:
+ ? ? ? free_irq(hw->irq, hw);
+err_irq:
+ ? ? ? iounmap(hw->regs);
+err_iomap:
+ ? ? ? release_resource(hw->ioarea);
+ ? ? ? kfree(hw->ioarea);
+err_pdata:
+ ? ? ? spi_master_put(hw->master);;
+
+err_nomem:
+ ? ? ? return err;
+}
+
+static int __devexit w90p910_spi_remove(struct platform_device *dev)
+{
+ ? ? ? struct w90p910_spi *hw = platform_get_drvdata(dev);
+
+ ? ? ? w90p910_enable_int(hw, 0);
+
+ ? ? ? free_irq(hw->irq, hw);
+
+ ? ? ? platform_set_drvdata(dev, NULL);
+
+ ? ? ? spi_unregister_master(hw->master);
+
+ ? ? ? clk_disable(hw->clk);
+ ? ? ? clk_put(hw->clk);
+
+ ? ? ? iounmap(hw->regs);
+
+ ? ? ? release_resource(hw->ioarea);
+ ? ? ? kfree(hw->ioarea);
+
+ ? ? ? spi_master_put(hw->master);
+ ? ? ? return 0;
+}
+
+static struct platform_driver w90p910_spi_driver = {
+ ? ? ? .probe ? ? ? ? ?= w90p910_spi_probe,
+ ? ? ? .remove ? ? ? ? = __devexit_p(w90p910_spi_remove),
+ ? ? ? .driver ? ? ? ? = {
+ ? ? ? ? ? ? ? .name ? = "w90p910-spi",
+ ? ? ? ? ? ? ? .owner ?= THIS_MODULE,
+ ? ? ? },
+};
+
+static int __init w90p910_spi_init(void)
+{
+ ? ? ? return platform_driver_register(&w90p910_spi_driver);
+}
+
+static void __exit w90p910_spi_exit(void)
+{
+ ? ? ? platform_driver_unregister(&w90p910_spi_driver);
+}
+
+module_init(w90p910_spi_init);
+module_exit(w90p910_spi_exit);
+
+MODULE_AUTHOR("Wan ZongShun <[email protected]>");
+MODULE_DESCRIPTION("w90p910 spi driver!");
+MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:w90p910-spi");
--

1.5.6.3

Hi Andrew

How about this spi driver, Could it be applied?