Hi Vinod, Thanks for pointing it out, I missed it. Will send V2 with correction. Regards, Surendran. -----Original Message----- From: Vinod Koul [mailto:vkoul@xxxxxxxxxx] Sent: Tuesday, October 13, 2020 9:40 PM To: Surendran K <surendran.k@xxxxxxxxxxx> Cc: dmaengine@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; shaik.ameer@xxxxxxxxxxx; alim.akhtar@xxxxxxxxxxx; pankaj.dubey@xxxxxxxxxxx Subject: Re: [PATCH] DMA: PL330: Remove unreachable code On 13-10-20, 17:17, Surendran K wrote: > _setup_req(..) never returns negative value. > Hence the condition ret < 0 is never met The subsystem is "dmaengine", git log would tell you the tags to use > > Signed-off-by: Surendran K <surendran.k@xxxxxxxxxxx> > --- > drivers/dma/pl330.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c index > e9f0101d92fa..8355586c9788 100644 > --- a/drivers/dma/pl330.c > +++ b/drivers/dma/pl330.c > @@ -1527,8 +1527,6 @@ static int pl330_submit_req(struct pl330_thread > *thrd, > > /* First dry run to check if req is acceptable */ > ret = _setup_req(pl330, 1, thrd, idx, &xs); > - if (ret < 0) > - goto xfer_exit; > > if (ret > pl330->mcbufsz / 2) { > dev_info(pl330->ddma.dev, "%s:%d Try increasing mcbufsz (%i/%i)\n", > -- > 2.17.1 -- ~Vinod
