Re: [PATCH] bus: fsl-mc: Fix test for end of loop

2021-02-08 Thread Ioana Ciornei
On Mon, Feb 08, 2021 at 06:11:29PM +0300, Dan Carpenter wrote: > On Mon, Feb 08, 2021 at 02:18:04PM +, Ioana Ciornei wrote: > > On Mon, Feb 01, 2021 at 03:28:54PM +0300, Dan Carpenter wrote: > > > The "desc" pointer can't possibly be NULL here. If we can't find the > > > correct "desc" then tt

Re: [PATCH] bus: fsl-mc: Fix test for end of loop

2021-02-08 Thread Dan Carpenter
On Mon, Feb 08, 2021 at 02:18:04PM +, Ioana Ciornei wrote: > On Mon, Feb 01, 2021 at 03:28:54PM +0300, Dan Carpenter wrote: > > The "desc" pointer can't possibly be NULL here. If we can't find the > > correct "desc" then tt points to the last element of the > > fsl_mc_accepted_cmds[] array. F

Re: [PATCH] bus: fsl-mc: Fix test for end of loop

2021-02-08 Thread Ioana Ciornei
On Mon, Feb 01, 2021 at 03:28:54PM +0300, Dan Carpenter wrote: > The "desc" pointer can't possibly be NULL here. If we can't find the > correct "desc" then tt points to the last element of the > fsl_mc_accepted_cmds[] array. Fix this by testing if > "i == FSL_MC_NUM_ACCEPTED_CMDS" instead. > > F

Re: [PATCH] bus: fsl-mc: Fix test for end of loop

2021-02-02 Thread Ioana Ciornei
On Mon, Feb 01, 2021 at 03:28:54PM +0300, Dan Carpenter wrote: > The "desc" pointer can't possibly be NULL here. If we can't find the > correct "desc" then tt points to the last element of the > fsl_mc_accepted_cmds[] array. Fix this by testing if > "i == FSL_MC_NUM_ACCEPTED_CMDS" instead. > > F

[PATCH] bus: fsl-mc: Fix test for end of loop

2021-02-01 Thread Dan Carpenter
The "desc" pointer can't possibly be NULL here. If we can't find the correct "desc" then tt points to the last element of the fsl_mc_accepted_cmds[] array. Fix this by testing if "i == FSL_MC_NUM_ACCEPTED_CMDS" instead. Fixes: 2cf1e703f066 ("bus: fsl-mc: add fsl-mc userspace support") Signed-off