On Tue, Jul 23, 2024 at 06:49:32PM +0530, Mrinmay Sarkar wrote:
> DW_HDMA_V0_LIE and DW_HDMA_V0_RIE are initialized as BIT(3) and BIT(4)
> respectively in dw_hdma_control enum. But as per HDMA register these
> bits are corresponds to LWIE and RWIE bit i.e local watermark interrupt
> enable and remote watermarek interrupt enable. In linked list mode LWIE
> and RWIE bits only enable the local and remote watermark interrupt.
>
> Since the watermark interrupts are not used but enabled, this leads to
> spurious interrupts getting generated. So remove the code that enables
> them to avoid generating spurious watermark interrupts.
>
> And also rename DW_HDMA_V0_LIE to DW_HDMA_V0_LWIE and DW_HDMA_V0_RIE to
> DW_HDMA_V0_RWIE as there is no LIE and RIE bits in HDMA and those bits
> are corresponds to LWIE and RWIE bits.
>
> Fixes: e74c39573d35 ("dmaengine: dw-edma: Add support for native HDMA")
> cc: stable@xxxxxxxxxxxxxxx
> Signed-off-by: Mrinmay Sarkar <quic_msarkar@xxxxxxxxxxx>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>
- Mani
> ---
> drivers/dma/dw-edma/dw-hdma-v0-core.c | 17 +++--------------
> 1 file changed, 3 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/dma/dw-edma/dw-hdma-v0-core.c b/drivers/dma/dw-edma/dw-hdma-v0-core.c
> index fa89b3a..9ad2e28 100644
> --- a/drivers/dma/dw-edma/dw-hdma-v0-core.c
> +++ b/drivers/dma/dw-edma/dw-hdma-v0-core.c
> @@ -17,8 +17,8 @@ enum dw_hdma_control {
> DW_HDMA_V0_CB = BIT(0),
> DW_HDMA_V0_TCB = BIT(1),
> DW_HDMA_V0_LLP = BIT(2),
> - DW_HDMA_V0_LIE = BIT(3),
> - DW_HDMA_V0_RIE = BIT(4),
> + DW_HDMA_V0_LWIE = BIT(3),
> + DW_HDMA_V0_RWIE = BIT(4),
> DW_HDMA_V0_CCS = BIT(8),
> DW_HDMA_V0_LLE = BIT(9),
> };
> @@ -195,25 +195,14 @@ static void dw_hdma_v0_write_ll_link(struct dw_edma_chunk *chunk,
> static void dw_hdma_v0_core_write_chunk(struct dw_edma_chunk *chunk)
> {
> struct dw_edma_burst *child;
> - struct dw_edma_chan *chan = chunk->chan;
> u32 control = 0, i = 0;
> - int j;
>
> if (chunk->cb)
> control = DW_HDMA_V0_CB;
>
> - j = chunk->bursts_alloc;
> - list_for_each_entry(child, &chunk->burst->list, list) {
> - j--;
> - if (!j) {
> - control |= DW_HDMA_V0_LIE;
> - if (!(chan->dw->chip->flags & DW_EDMA_CHIP_LOCAL))
> - control |= DW_HDMA_V0_RIE;
> - }
> -
> + list_for_each_entry(child, &chunk->burst->list, list)
> dw_hdma_v0_write_ll_data(chunk, i++, control, child->sz,
> child->sar, child->dar);
> - }
>
> control = DW_HDMA_V0_LLP | DW_HDMA_V0_TCB;
> if (!chunk->cb)
> --
> 2.7.4
>
--
மணிவண்ணன் சதாசிவம்
