Commit 9fd5ca5f authored by Vinod Koul's avatar Vinod Koul

Merge branch 'topic/idma' into for-linus

parents d3f1e93c 390c49f7
...@@ -178,20 +178,12 @@ static irqreturn_t idma64_irq(int irq, void *dev) ...@@ -178,20 +178,12 @@ static irqreturn_t idma64_irq(int irq, void *dev)
if (!status) if (!status)
return IRQ_NONE; return IRQ_NONE;
/* Disable interrupts */
channel_clear_bit(idma64, MASK(XFER), idma64->all_chan_mask);
channel_clear_bit(idma64, MASK(ERROR), idma64->all_chan_mask);
status_xfer = dma_readl(idma64, RAW(XFER)); status_xfer = dma_readl(idma64, RAW(XFER));
status_err = dma_readl(idma64, RAW(ERROR)); status_err = dma_readl(idma64, RAW(ERROR));
for (i = 0; i < idma64->dma.chancnt; i++) for (i = 0; i < idma64->dma.chancnt; i++)
idma64_chan_irq(idma64, i, status_err, status_xfer); idma64_chan_irq(idma64, i, status_err, status_xfer);
/* Re-enable interrupts */
channel_set_bit(idma64, MASK(XFER), idma64->all_chan_mask);
channel_set_bit(idma64, MASK(ERROR), idma64->all_chan_mask);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
...@@ -239,7 +231,7 @@ static void idma64_vdesc_free(struct virt_dma_desc *vdesc) ...@@ -239,7 +231,7 @@ static void idma64_vdesc_free(struct virt_dma_desc *vdesc)
idma64_desc_free(idma64c, to_idma64_desc(vdesc)); idma64_desc_free(idma64c, to_idma64_desc(vdesc));
} }
static u64 idma64_hw_desc_fill(struct idma64_hw_desc *hw, static void idma64_hw_desc_fill(struct idma64_hw_desc *hw,
struct dma_slave_config *config, struct dma_slave_config *config,
enum dma_transfer_direction direction, u64 llp) enum dma_transfer_direction direction, u64 llp)
{ {
...@@ -276,26 +268,26 @@ static u64 idma64_hw_desc_fill(struct idma64_hw_desc *hw, ...@@ -276,26 +268,26 @@ static u64 idma64_hw_desc_fill(struct idma64_hw_desc *hw,
IDMA64C_CTLL_SRC_WIDTH(src_width); IDMA64C_CTLL_SRC_WIDTH(src_width);
lli->llp = llp; lli->llp = llp;
return hw->llp;
} }
static void idma64_desc_fill(struct idma64_chan *idma64c, static void idma64_desc_fill(struct idma64_chan *idma64c,
struct idma64_desc *desc) struct idma64_desc *desc)
{ {
struct dma_slave_config *config = &idma64c->config; struct dma_slave_config *config = &idma64c->config;
struct idma64_hw_desc *hw = &desc->hw[desc->ndesc - 1]; unsigned int i = desc->ndesc;
struct idma64_hw_desc *hw = &desc->hw[i - 1];
struct idma64_lli *lli = hw->lli; struct idma64_lli *lli = hw->lli;
u64 llp = 0; u64 llp = 0;
unsigned int i = desc->ndesc;
/* Fill the hardware descriptors and link them to a list */ /* Fill the hardware descriptors and link them to a list */
do { do {
hw = &desc->hw[--i]; hw = &desc->hw[--i];
llp = idma64_hw_desc_fill(hw, config, desc->direction, llp); idma64_hw_desc_fill(hw, config, desc->direction, llp);
llp = hw->llp;
desc->length += hw->len; desc->length += hw->len;
} while (i); } while (i);
/* Trigger interrupt after last block */ /* Trigger an interrupt after the last block is transfered */
lli->ctllo |= IDMA64C_CTLL_INT_EN; lli->ctllo |= IDMA64C_CTLL_INT_EN;
} }
...@@ -596,6 +588,8 @@ static int idma64_probe(struct idma64_chip *chip) ...@@ -596,6 +588,8 @@ static int idma64_probe(struct idma64_chip *chip)
idma64->dma.dev = chip->dev; idma64->dma.dev = chip->dev;
dma_set_max_seg_size(idma64->dma.dev, IDMA64C_CTLH_BLOCK_TS_MASK);
ret = dma_async_device_register(&idma64->dma); ret = dma_async_device_register(&idma64->dma);
if (ret) if (ret)
return ret; return ret;
......
...@@ -54,7 +54,8 @@ ...@@ -54,7 +54,8 @@
#define IDMA64C_CTLL_LLP_S_EN (1 << 28) /* src block chain */ #define IDMA64C_CTLL_LLP_S_EN (1 << 28) /* src block chain */
/* Bitfields in CTL_HI */ /* Bitfields in CTL_HI */
#define IDMA64C_CTLH_BLOCK_TS(x) ((x) & ((1 << 17) - 1)) #define IDMA64C_CTLH_BLOCK_TS_MASK ((1 << 17) - 1)
#define IDMA64C_CTLH_BLOCK_TS(x) ((x) & IDMA64C_CTLH_BLOCK_TS_MASK)
#define IDMA64C_CTLH_DONE (1 << 17) #define IDMA64C_CTLH_DONE (1 << 17)
/* Bitfields in CFG_LO */ /* Bitfields in CFG_LO */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment