[PATCH] dma: ti: k3-udma: Fix error handling for setup_resources() in udma_probe()
Siddharth Vadapalli
s-vadapalli at ti.com
Tue Feb 20 10:54:26 CET 2024
On 24/02/20 12:51PM, Dan Carpenter wrote:
> On Fri, Feb 16, 2024 at 04:11:05PM +0530, Siddharth Vadapalli wrote:
> > diff --git a/drivers/dma/ti/k3-udma.c b/drivers/dma/ti/k3-udma.c
> > index eea9ec9659..8a6625f034 100644
> > --- a/drivers/dma/ti/k3-udma.c
> > +++ b/drivers/dma/ti/k3-udma.c
> > @@ -1770,9 +1770,11 @@ static int udma_probe(struct udevice *dev)
> > return PTR_ERR(ud->ringacc);
> >
> > ud->dev = dev;
> > - ud->ch_count = setup_resources(ud);
> > - if (ud->ch_count <= 0)
> > - return ud->ch_count;
> > + ret = setup_resources(ud);
> > + if (ret <= 0)
> > + return ret;
>
> The code was like this originally, but setup_resources() can't actually
> return zero so it would be nicer to say:
> ret = setup_resources(ud);
> if (ret < 0)
> return ret;
Thank you for reviewing the patch and pointing this out. I will fix it
and post the v2 patch.
Regards,
Siddharth.
More information about the U-Boot
mailing list