Re: [PATCH v5 2/5] dmaengine: arm-dma350: Add slave and cyclic transfer support

[Jelly Jia <Jelly.Jia@cixtech.com>]

Hi Frank,

> Please check both issues.

I've checked both and made the following changes locally:

1. Initialize dch->req to U32_MAX, and reset it and clear dch->sconfig
   when channel resources are freed. This lets the existing request
   range check reject slave/cyclic preparation without a request
   assignment. Valid request line 0 and memcpy/memset remain unaffected.
   Request/configuration state is retained across terminate_all().

2. Keep clearing dch->desc on error, since the descriptor is handed to
   virt-dma for completion or termination. Instead, prevent
   d350_start_next() from starting another transfer while the channel
   is in DMA_ERROR. This preserves the failed cookie's status and
   residue until termination. Also reset the channel status in
   terminate_all() even when dch->desc is NULL, so the channel can be
   used again after termination and synchronization.

I've tested the updated tree on Sky1 and will include both fixes
in the next revision.

Best regards,
Jelly