dmaengine: ste_dma40: Remove unnecessary call to d40_phy_cfg()

The majority of configuration done in d40_phy_config() pertains
to physical channels. Move the call over to runtime config which
has different code paths for physical and logical channels already,
and make it an exclusive physical channel config function as the
name implies, and drop the is_log argument.

Since we moved the call to runtime_config() it only gets called
for device transfers, so encode the small snippet of configuration
pertaining to memcpy channels into the d40_config_memcpy()
function.

Acked-by: Vinod Koul <vinod.koul@intel.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
[rewrote the commit message]
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index 759293e4a1ccfc884d70a0d2c1ee94b651649e41..b7fe46bd6a8e735e64d027dc0a0d6b31f62e0c21 100644 (file)
--- a/drivers/dma/ste_dma40.c
+++ b/drivers/dma/ste_dma40.c
@@ -2043,6 +2043,14 @@ static int d40_config_memcpy(struct d40_chan *d40c)
        } else if (dma_has_cap(DMA_MEMCPY, cap) &&
                   dma_has_cap(DMA_SLAVE, cap)) {
                d40c->dma_cfg = dma40_memcpy_conf_phy;
+
+               /* Generate interrrupt at end of transfer or relink. */
+               d40c->dst_def_cfg |= BIT(D40_SREG_CFG_TIM_POS);
+
+               /* Generate interrupt on error. */
+               d40c->src_def_cfg |= BIT(D40_SREG_CFG_EIM_POS);
+               d40c->dst_def_cfg |= BIT(D40_SREG_CFG_EIM_POS);
+
        } else {
                chan_err(d40c, "No memcpy\n");
                return -EINVAL;
@@ -2496,9 +2504,6 @@ static int d40_alloc_chan_resources(struct dma_chan *chan)
        }
 
        pm_runtime_get_sync(d40c->base->dev);
-       /* Fill in basic CFG register values */
-       d40_phy_cfg(&d40c->dma_cfg, &d40c->src_def_cfg,
-                   &d40c->dst_def_cfg, chan_is_logical(d40c));
 
        d40_set_prio_realtime(d40c);
 
@@ -2862,8 +2867,7 @@ static int d40_set_runtime_config(struct dma_chan *chan,
        if (chan_is_logical(d40c))
                d40_log_cfg(cfg, &d40c->log_def.lcsp1, &d40c->log_def.lcsp3);
        else
-               d40_phy_cfg(cfg, &d40c->src_def_cfg,
-                           &d40c->dst_def_cfg, false);
+               d40_phy_cfg(cfg, &d40c->src_def_cfg, &d40c->dst_def_cfg);
 
        /* These settings will take precedence later */
        d40c->runtime_addr = config_addr;

diff --git a/drivers/dma/ste_dma40_ll.c b/drivers/dma/ste_dma40_ll.c
index 435a223c891f3b27a259ade16b099a373c04eda5..ab5a2a706f7aaeb0021a7a1c9007587f7a984cea 100644 (file)
--- a/drivers/dma/ste_dma40_ll.c
+++ b/drivers/dma/ste_dma40_ll.c
@@ -50,63 +50,58 @@ void d40_log_cfg(struct stedma40_chan_cfg *cfg,
 
 }
 
-/* Sets up SRC and DST CFG register for both logical and physical channels */
-void d40_phy_cfg(struct stedma40_chan_cfg *cfg,
-                u32 *src_cfg, u32 *dst_cfg, bool is_log)
+void d40_phy_cfg(struct stedma40_chan_cfg *cfg, u32 *src_cfg, u32 *dst_cfg)
 {
        u32 src = 0;
        u32 dst = 0;
 
-       if (!is_log) {
-               /* Physical channel */
-               if ((cfg->dir ==  STEDMA40_PERIPH_TO_MEM) ||
-                   (cfg->dir == STEDMA40_PERIPH_TO_PERIPH)) {
-                       /* Set master port to 1 */
-                       src |= 1 << D40_SREG_CFG_MST_POS;
-                       src |= D40_TYPE_TO_EVENT(cfg->dev_type);
-
-                       if (cfg->src_info.flow_ctrl == STEDMA40_NO_FLOW_CTRL)
-                               src |= 1 << D40_SREG_CFG_PHY_TM_POS;
-                       else
-                               src |= 3 << D40_SREG_CFG_PHY_TM_POS;
-               }
-               if ((cfg->dir ==  STEDMA40_MEM_TO_PERIPH) ||
-                   (cfg->dir == STEDMA40_PERIPH_TO_PERIPH)) {
-                       /* Set master port to 1 */
-                       dst |= 1 << D40_SREG_CFG_MST_POS;
-                       dst |= D40_TYPE_TO_EVENT(cfg->dev_type);
-
-                       if (cfg->dst_info.flow_ctrl == STEDMA40_NO_FLOW_CTRL)
-                               dst |= 1 << D40_SREG_CFG_PHY_TM_POS;
-                       else
-                               dst |= 3 << D40_SREG_CFG_PHY_TM_POS;
-               }
-               /* Interrupt on end of transfer for destination */
-               dst |= 1 << D40_SREG_CFG_TIM_POS;
-
-               /* Generate interrupt on error */
-               src |= 1 << D40_SREG_CFG_EIM_POS;
-               dst |= 1 << D40_SREG_CFG_EIM_POS;
-
-               /* PSIZE */
-               if (cfg->src_info.psize != STEDMA40_PSIZE_PHY_1) {
-                       src |= 1 << D40_SREG_CFG_PHY_PEN_POS;
-                       src |= cfg->src_info.psize << D40_SREG_CFG_PSIZE_POS;
-               }
-               if (cfg->dst_info.psize != STEDMA40_PSIZE_PHY_1) {
-                       dst |= 1 << D40_SREG_CFG_PHY_PEN_POS;
-                       dst |= cfg->dst_info.psize << D40_SREG_CFG_PSIZE_POS;
-               }
-
-               /* Element size */
-               src |= cfg->src_info.data_width << D40_SREG_CFG_ESIZE_POS;
-               dst |= cfg->dst_info.data_width << D40_SREG_CFG_ESIZE_POS;
-
-               /* Set the priority bit to high for the physical channel */
-               if (cfg->high_priority) {
-                       src |= 1 << D40_SREG_CFG_PRI_POS;
-                       dst |= 1 << D40_SREG_CFG_PRI_POS;
-               }
+       if ((cfg->dir ==  STEDMA40_PERIPH_TO_MEM) ||
+           (cfg->dir == STEDMA40_PERIPH_TO_PERIPH)) {
+               /* Set master port to 1 */
+               src |= 1 << D40_SREG_CFG_MST_POS;
+               src |= D40_TYPE_TO_EVENT(cfg->dev_type);
+
+               if (cfg->src_info.flow_ctrl == STEDMA40_NO_FLOW_CTRL)
+                       src |= 1 << D40_SREG_CFG_PHY_TM_POS;
+               else
+                       src |= 3 << D40_SREG_CFG_PHY_TM_POS;
+       }
+       if ((cfg->dir ==  STEDMA40_MEM_TO_PERIPH) ||
+           (cfg->dir == STEDMA40_PERIPH_TO_PERIPH)) {
+               /* Set master port to 1 */
+               dst |= 1 << D40_SREG_CFG_MST_POS;
+               dst |= D40_TYPE_TO_EVENT(cfg->dev_type);
+
+               if (cfg->dst_info.flow_ctrl == STEDMA40_NO_FLOW_CTRL)
+                       dst |= 1 << D40_SREG_CFG_PHY_TM_POS;
+               else
+                       dst |= 3 << D40_SREG_CFG_PHY_TM_POS;
+       }
+       /* Interrupt on end of transfer for destination */
+       dst |= 1 << D40_SREG_CFG_TIM_POS;
+
+       /* Generate interrupt on error */
+       src |= 1 << D40_SREG_CFG_EIM_POS;
+       dst |= 1 << D40_SREG_CFG_EIM_POS;
+
+       /* PSIZE */
+       if (cfg->src_info.psize != STEDMA40_PSIZE_PHY_1) {
+               src |= 1 << D40_SREG_CFG_PHY_PEN_POS;
+               src |= cfg->src_info.psize << D40_SREG_CFG_PSIZE_POS;
+       }
+       if (cfg->dst_info.psize != STEDMA40_PSIZE_PHY_1) {
+               dst |= 1 << D40_SREG_CFG_PHY_PEN_POS;
+               dst |= cfg->dst_info.psize << D40_SREG_CFG_PSIZE_POS;
+       }
+
+       /* Element size */
+       src |= cfg->src_info.data_width << D40_SREG_CFG_ESIZE_POS;
+       dst |= cfg->dst_info.data_width << D40_SREG_CFG_ESIZE_POS;
+
+       /* Set the priority bit to high for the physical channel */
+       if (cfg->high_priority) {
+               src |= 1 << D40_SREG_CFG_PRI_POS;
+               dst |= 1 << D40_SREG_CFG_PRI_POS;
        }
 
        if (cfg->src_info.big_endian)

diff --git a/drivers/dma/ste_dma40_ll.h b/drivers/dma/ste_dma40_ll.h
index fdde8ef775422fff777eff169a8b39733b72af43..1b47312bc574e81a77b0da7172d8ce65b777cec2 100644 (file)
--- a/drivers/dma/ste_dma40_ll.h
+++ b/drivers/dma/ste_dma40_ll.h
@@ -432,8 +432,7 @@ enum d40_lli_flags {
 
 void d40_phy_cfg(struct stedma40_chan_cfg *cfg,
                 u32 *src_cfg,
-                u32 *dst_cfg,
-                bool is_log);
+                u32 *dst_cfg);
 
 void d40_log_cfg(struct stedma40_chan_cfg *cfg,
                 u32 *lcsp1,