2024-01-29 16:33:14

by Kory Maincent

[permalink] [raw]
Subject: [PATCH v7 0/6] Fix support of dw-edma HDMA NATIVE IP in remote setup

This patch series fix the support of dw-edma HDMA NATIVE IP.
I can only test it in remote HDMA IP setup with single dma transfer, but
with these fixes it works properly.

Few fixes has also been added for eDMA version. Similarly to HDMA I have
tested only eDMA in remote setup.

Changes in v2:
- Update comments and fix typos.
- Removed patches that tackle hypothetical bug and then were not pertinent.
- Add the similar HDMA race condition in remote setup fix to eDMA IP driver.

Changes in v3:
- Fix comment style.
- Split a patch in two to differ bug fix and simple harmless typo.

Changes in v4:
- Update patch git commit message.
- Link to v3: https://lore.kernel.org/r/[email protected]

Changes in v5:
- No change
- Rebase to mainline 6.7-rc1
- Link to v4: https://lore.kernel.org/r/[email protected]

Changes in v6:
- Fix several commit messages and comments.
- Link to v5: https://lore.kernel.org/r/[email protected]

Changes in v7:
- No change, ready for merge
- Rebase to mainline 6.8-rc2
- Link to v6: https://lore.kernel.org/r/[email protected]

Signed-off-by: Kory Maincent <[email protected]>
---
Kory Maincent (6):
dmaengine: dw-edma: Fix the ch_count hdma callback
dmaengine: dw-edma: Fix wrong interrupt bit set for HDMA
dmaengine: dw-edma: HDMA_V0_REMOTEL_STOP_INT_EN typo fix
dmaengine: dw-edma: Add HDMA remote interrupt configuration
dmaengine: dw-edma: HDMA: Add sync read before starting the DMA transfer in remote setup
dmaengine: dw-edma: eDMA: Add sync read before starting the DMA transfer in remote setup

drivers/dma/dw-edma/dw-edma-v0-core.c | 17 +++++++++++++++
drivers/dma/dw-edma/dw-hdma-v0-core.c | 39 +++++++++++++++++++++++------------
drivers/dma/dw-edma/dw-hdma-v0-regs.h | 2 +-
3 files changed, 44 insertions(+), 14 deletions(-)
---
base-commit: 41bccc98fb7931d63d03f326a746ac4d429c1dd3
change-id: 20231011-b4-feature_hdma_mainline-b6c57f8e3b5d

Best regards,
--
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com



2024-02-02 07:54:22

by Manivannan Sadhasivam

[permalink] [raw]
Subject: Re: [PATCH v7 0/6] Fix support of dw-edma HDMA NATIVE IP in remote setup

On Mon, Jan 29, 2024 at 05:25:56PM +0100, Kory Maincent wrote:
> This patch series fix the support of dw-edma HDMA NATIVE IP.
> I can only test it in remote HDMA IP setup with single dma transfer, but
> with these fixes it works properly.
>
> Few fixes has also been added for eDMA version. Similarly to HDMA I have
> tested only eDMA in remote setup.
>

Vinod, could you please merge this series for v6.9? This already missed previous
release.

- Mani

> Changes in v2:
> - Update comments and fix typos.
> - Removed patches that tackle hypothetical bug and then were not pertinent.
> - Add the similar HDMA race condition in remote setup fix to eDMA IP driver.
>
> Changes in v3:
> - Fix comment style.
> - Split a patch in two to differ bug fix and simple harmless typo.
>
> Changes in v4:
> - Update patch git commit message.
> - Link to v3: https://lore.kernel.org/r/[email protected]
>
> Changes in v5:
> - No change
> - Rebase to mainline 6.7-rc1
> - Link to v4: https://lore.kernel.org/r/[email protected]
>
> Changes in v6:
> - Fix several commit messages and comments.
> - Link to v5: https://lore.kernel.org/r/[email protected]
>
> Changes in v7:
> - No change, ready for merge
> - Rebase to mainline 6.8-rc2
> - Link to v6: https://lore.kernel.org/r/[email protected]
>
> Signed-off-by: Kory Maincent <[email protected]>
> ---
> Kory Maincent (6):
> dmaengine: dw-edma: Fix the ch_count hdma callback
> dmaengine: dw-edma: Fix wrong interrupt bit set for HDMA
> dmaengine: dw-edma: HDMA_V0_REMOTEL_STOP_INT_EN typo fix
> dmaengine: dw-edma: Add HDMA remote interrupt configuration
> dmaengine: dw-edma: HDMA: Add sync read before starting the DMA transfer in remote setup
> dmaengine: dw-edma: eDMA: Add sync read before starting the DMA transfer in remote setup
>
> drivers/dma/dw-edma/dw-edma-v0-core.c | 17 +++++++++++++++
> drivers/dma/dw-edma/dw-hdma-v0-core.c | 39 +++++++++++++++++++++++------------
> drivers/dma/dw-edma/dw-hdma-v0-regs.h | 2 +-
> 3 files changed, 44 insertions(+), 14 deletions(-)
> ---
> base-commit: 41bccc98fb7931d63d03f326a746ac4d429c1dd3
> change-id: 20231011-b4-feature_hdma_mainline-b6c57f8e3b5d
>
> Best regards,
> --
> Köry Maincent, Bootlin
> Embedded Linux and kernel engineering
> https://bootlin.com
>

--
மணிவண்ணன் சதாசிவம்

2024-02-07 08:49:26

by Vinod Koul

[permalink] [raw]
Subject: Re: [PATCH v7 0/6] Fix support of dw-edma HDMA NATIVE IP in remote setup


On Mon, 29 Jan 2024 17:25:56 +0100, Kory Maincent wrote:
> This patch series fix the support of dw-edma HDMA NATIVE IP.
> I can only test it in remote HDMA IP setup with single dma transfer, but
> with these fixes it works properly.
>
> Few fixes has also been added for eDMA version. Similarly to HDMA I have
> tested only eDMA in remote setup.
>
> [...]

Applied, thanks!

[1/6] dmaengine: dw-edma: Fix the ch_count hdma callback
commit: cd665bfc757c71e9b7e0abff0f362d8abd38a805
[2/6] dmaengine: dw-edma: Fix wrong interrupt bit set for HDMA
commit: 7b52ba8616e978bf4f38f207f11a8176517244d0
[3/6] dmaengine: dw-edma: HDMA_V0_REMOTEL_STOP_INT_EN typo fix
commit: 930a8a015dcfde4b8906351ff081066dc277748c
[4/6] dmaengine: dw-edma: Add HDMA remote interrupt configuration
commit: e2f6a5789051ee9c632f27a12d0f01f0cbf78aac
[5/6] dmaengine: dw-edma: HDMA: Add sync read before starting the DMA transfer in remote setup
commit: 712a92a48158e02155b4b6b21e03a817f78c9b7e
[6/6] dmaengine: dw-edma: eDMA: Add sync read before starting the DMA transfer in remote setup
commit: bbcc1c83f343e580c3aa1f2a8593343bf7b55bba

Best regards,
--
~Vinod