Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Message-ID: <7993bc10-fff1-bacb-d0c5-929b14f35244@ti.com>
Date:   Thu, 21 Apr 2022 20:45:36 +0530
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.8.1
Subject: Re: [PATCH] dmaengine: ti: k3-psil-am62: Update PSIL thread for saul.
Content-Language: en-US
To:     Jayesh Choudhary <j-choudhary@ti.com>, <dmaengine@vger.kernel.org>
CC:     <peter.ujfalusi@gmail.com>, <vkoul@kernel.org>,
        <linux-kernel@vger.kernel.org>
References: <20220421065323.16378-1-j-choudhary@ti.com>
From:   Vignesh Raghavendra <vigneshr@ti.com>
In-Reply-To: <20220421065323.16378-1-j-choudhary@ti.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Jayesh,

On 21/04/22 12:23 pm, Jayesh Choudhary wrote:
> Correct the RX PSIL thread for sa3ul.
> 

Commit message needs more info:
Threads are not wrong but, the first 4 threads are reserved
for secure side usage and the rest is available to be paired with main pktdma.

Also, add fixes tag:

Fixes: 5ac6bfb587772 ("dmaengine: ti: k3-psil: Add AM62x PSIL and PDMA data")


> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
> 
> The new updated PSIL threads have been tested on local am62x board.
> Log is available here:
> <https://gist.github.com/Jayesh2000/b0316190de3d9dbb8e98337106ebe24a>
> 
>  drivers/dma/ti/k3-psil-am62.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/dma/ti/k3-psil-am62.c b/drivers/dma/ti/k3-psil-am62.c
> index d431e2033237..2b6fd6e37c61 100644
> --- a/drivers/dma/ti/k3-psil-am62.c
> +++ b/drivers/dma/ti/k3-psil-am62.c
> @@ -70,10 +70,10 @@
>  /* PSI-L source thread IDs, used for RX (DMA_DEV_TO_MEM) */
>  static struct psil_ep am62_src_ep_map[] = {
>  	/* SAUL */
> -	PSIL_SAUL(0x7500, 20, 35, 8, 35, 0),
> -	PSIL_SAUL(0x7501, 21, 35, 8, 36, 0),
> -	PSIL_SAUL(0x7502, 22, 43, 8, 43, 0),
> -	PSIL_SAUL(0x7503, 23, 43, 8, 44, 0),
> +	PSIL_SAUL(0x7504, 20, 35, 8, 35, 0),
> +	PSIL_SAUL(0x7505, 21, 35, 8, 36, 0),
> +	PSIL_SAUL(0x7506, 22, 43, 8, 43, 0),
> +	PSIL_SAUL(0x7507, 23, 43, 8, 44, 0),
>  	/* PDMA_MAIN0 - SPI0-3 */
>  	PSIL_PDMA_XY_PKT(0x4302),
>  	PSIL_PDMA_XY_PKT(0x4303),


Regards
Vignesh