Minimize differences between sun4i and sun6i driver without code
changes.

Signed-off-by: Michal Suchanek <hramr...@gmail.com>
---
 drivers/spi/spi-sun4i.c | 11 ++++++-----
 drivers/spi/spi-sun6i.c |  8 +++-----
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/drivers/spi/spi-sun4i.c b/drivers/spi/spi-sun4i.c
index b7f8de1..bbb0996 100644
--- a/drivers/spi/spi-sun4i.c
+++ b/drivers/spi/spi-sun4i.c
@@ -19,6 +19,7 @@
 #include <linux/module.h>
 #include <linux/platform_device.h>
 #include <linux/pm_runtime.h>
+#include <linux/reset.h>
 
 #include <linux/spi/spi.h>
 
@@ -56,7 +57,7 @@
 
 #define SUNXI_CLK_CTL_REG              0x1c
 #define SUNXI_CLK_CTL_CDR2_MASK                0xff
-#define SUNXI_CLK_CTL_CDR2(div)                ((div) & 
SUNXI_CLK_CTL_CDR2_MASK)
+#define SUNXI_CLK_CTL_CDR2(div)                (((div) & 
SUNXI_CLK_CTL_CDR2_MASK) << 0)
 #define SUNXI_CLK_CTL_CDR1_MASK                0xf
 #define SUNXI_CLK_CTL_CDR1(div)                (((div) & 
SUNXI_CLK_CTL_CDR1_MASK) << 8)
 #define SUNXI_CLK_CTL_DRS              BIT(12)
@@ -78,6 +79,7 @@ struct sunxi_spi {
        void __iomem            *base_addr;
        struct clk              *hclk;
        struct clk              *mclk;
+       struct reset_control    *rstc;
 
        struct completion       done;
 
@@ -136,7 +138,6 @@ static void sunxi_spi_set_cs(struct spi_device *spi, bool 
enable)
        u32 reg;
 
        reg = sunxi_spi_read(sspi, SUNXI_TFR_CTL_REG);
-
        reg &= ~SUNXI_TFR_CTL_CS_MASK;
        reg |= SUNXI_TFR_CTL_CS(spi->chip_select);
 
@@ -198,7 +199,6 @@ static int sunxi_spi_transfer_one(struct spi_master *master,
        /* Clear pending interrupts */
        sunxi_spi_write(sspi, SUNXI_INT_STA_REG, ~0);
 
-
        reg = sunxi_spi_read(sspi, SUNXI_TFR_CTL_REG);
 
        /* Reset FIFOs */
@@ -224,7 +224,6 @@ static int sunxi_spi_transfer_one(struct spi_master *master,
        else
                reg &= ~SUNXI_TFR_CTL_FBS;
 
-
        /*
         * If it's a TX only transfer, we don't want to fill the RX
         * FIFO with bogus data
@@ -248,7 +247,7 @@ static int sunxi_spi_transfer_one(struct spi_master *master,
         *
         * We have two choices there. Either we can use the clock
         * divide rate 1, which is calculated thanks to this formula:
-        * SPI_CLK = MOD_CLK / (2 ^ (cdr + 1))
+        * SPI_CLK = MOD_CLK / (2 ^ cdr)
         * Or we can use CDR2, which is calculated with the formula:
         * SPI_CLK = MOD_CLK / (2 * (cdr + 1))
         * Wether we use the former or the latter is set through the
@@ -352,6 +351,8 @@ static int sunxi_spi_runtime_resume(struct device *dev)
 
        return 0;
 
+err2:
+       clk_disable_unprepare(sspi->mclk);
 err:
        clk_disable_unprepare(sspi->hclk);
 out:
diff --git a/drivers/spi/spi-sun6i.c b/drivers/spi/spi-sun6i.c
index f26b52a..d14a953 100644
--- a/drivers/spi/spi-sun6i.c
+++ b/drivers/spi/spi-sun6i.c
@@ -29,7 +29,6 @@
 #define SUNXI_CTL_ENABLE               BIT(0)
 #define SUNXI_CTL_MASTER               BIT(1)
 #define SUNXI_CTL_TP                   BIT(7)
-#define SUNXI_GBL_CTL_RST              BIT(31)
 
 #define SUNXI_TFR_CTL_REG              0x08
 #define SUNXI_TFR_CTL_CPHA             BIT(0)
@@ -44,7 +43,6 @@
 #define SUNXI_TFR_CTL_XCH              BIT(31)
 
 #define SUNXI_INT_CTL_REG              0x10
-#define SUNXI_INT_CTL_RF_OVF           BIT(8)
 #define SUNXI_INT_CTL_TC               BIT(12)
 
 #define SUNXI_INT_STA_REG              0x14
@@ -200,7 +198,9 @@ static int sunxi_spi_transfer_one(struct spi_master *master,
        /* Clear pending interrupts */
        sunxi_spi_write(sspi, SUNXI_INT_STA_REG, ~0);
 
-       /* Reset FIFO */
+       reg = sunxi_spi_read(sspi, SUNXI_TFR_CTL_REG);
+
+       /* Reset FIFOs */
        sunxi_spi_write(sspi, SUNXI_FIFO_CTL_REG,
                        SUNXI_CTL_RF_RST | SUNXI_CTL_TF_RST);
 
@@ -208,8 +208,6 @@ static int sunxi_spi_transfer_one(struct spi_master *master,
         * Setup the transfer control register: Chip Select,
         * polarities, etc.
         */
-       reg = sunxi_spi_read(sspi, SUNXI_TFR_CTL_REG);
-
        if (spi->mode & SPI_CPOL)
                reg |= SUNXI_TFR_CTL_CPOL;
        else
-- 
2.8.1

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to