Hi Pratyush,

On 3/30/21 8:33 PM, Pratyush Yadav wrote:
> The CSI2RX subsystem uses PSI-L DMA to transfer frames to memory. It can
> have up to 32 threads but the current driver only supports using one. So
> add an entry for that one thread.

If you are absolutely sure that the other threads are not going to be
used, then:
Acked-by: Peter Ujfalusi <[email protected]>

but I would consider adding the other threads if there is a chance that
the cs2rx will need to support it in the future.

> Signed-off-by: Pratyush Yadav <[email protected]>
> ---
>  drivers/dma/ti/k3-psil-j721e.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/drivers/dma/ti/k3-psil-j721e.c b/drivers/dma/ti/k3-psil-j721e.c
> index 7580870ed746..19ffa31e6dc6 100644
> --- a/drivers/dma/ti/k3-psil-j721e.c
> +++ b/drivers/dma/ti/k3-psil-j721e.c
> @@ -58,6 +58,14 @@
>               },                                      \
>       }
>  
> +#define PSIL_CSI2RX(x)                                       \
> +     {                                               \
> +             .thread_id = x,                         \
> +             .ep_config = {                          \
> +                     .ep_type = PSIL_EP_NATIVE,      \
> +             },                                      \
> +     }
> +
>  /* PSI-L source thread IDs, used for RX (DMA_DEV_TO_MEM) */
>  static struct psil_ep j721e_src_ep_map[] = {
>       /* SA2UL */
> @@ -138,6 +146,8 @@ static struct psil_ep j721e_src_ep_map[] = {
>       PSIL_PDMA_XY_PKT(0x4707),
>       PSIL_PDMA_XY_PKT(0x4708),
>       PSIL_PDMA_XY_PKT(0x4709),
> +     /* CSI2RX */
> +     PSIL_CSI2RX(0x4940),
>       /* CPSW9 */
>       PSIL_ETHERNET(0x4a00),
>       /* CPSW0 */
> 

-- 
Péter

Reply via email to