dmaengine: dw-edma: remove a macro conditional with similar branches
authorVladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
Fri, 10 Jun 2022 10:07:00 +0000 (13:07 +0300)
committerVinod Koul <vkoul@kernel.org>
Fri, 10 Jun 2022 13:49:24 +0000 (19:19 +0530)
After adding commit 8fc5133d6d4d ("dmaengine: dw-edma: Fix unaligned
64bit access") two branches under macro conditional become identical,
thus the code can be simplified without any functional change.

Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
Acked-by: Herve Codina <herve.codina@bootlin.com>
Link: https://lore.kernel.org/r/20220610100700.2295522-1-vladimir.zapolskiy@linaro.org
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/dma/dw-edma/dw-edma-v0-core.c

index 33bc1e6c4cf2e74db9fbc54d4cfa8daadaff2ea8..c73b9ed1ce74e056c4da1ee2b5e3aaede13fd9ac 100644 (file)
@@ -414,19 +414,11 @@ void dw_edma_v0_core_start(struct dw_edma_chunk *chunk, bool first)
                SET_CH_32(dw, chan->dir, chan->id, ch_control1,
                          (DW_EDMA_V0_CCS | DW_EDMA_V0_LLE));
                /* Linked list */
-
-               #ifdef CONFIG_64BIT
                /* llp is not aligned on 64bit -> keep 32bit accesses */
                SET_CH_32(dw, chan->dir, chan->id, llp.lsb,
                          lower_32_bits(chunk->ll_region.paddr));
                SET_CH_32(dw, chan->dir, chan->id, llp.msb,
                          upper_32_bits(chunk->ll_region.paddr));
-               #else /* CONFIG_64BIT */
-               SET_CH_32(dw, chan->dir, chan->id, llp.lsb,
-                         lower_32_bits(chunk->ll_region.paddr));
-               SET_CH_32(dw, chan->dir, chan->id, llp.msb,
-                         upper_32_bits(chunk->ll_region.paddr));
-               #endif /* CONFIG_64BIT */
        }
        /* Doorbell */
        SET_RW_32(dw, chan->dir, doorbell,