From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Date: Mon, 08 Feb 2021 15:11:29 +0000 Subject: Re: [PATCH] bus: fsl-mc: Fix test for end of loop Message-Id: <20210208151129.GJ2696@kadam> List-Id: References: <20210208141803.bqbnbgvprtlo3vs6@skbuf> In-Reply-To: <20210208141803.bqbnbgvprtlo3vs6@skbuf> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Ioana Ciornei Cc: Stuart Yoder , Laurentiu Tudor , "linux-kernel@vger.kernel.org" , "kernel-janitors@vger.kernel.org" On Mon, Feb 08, 2021 at 02:18:04PM +0000, 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. Fix this by testing if > > "i = FSL_MC_NUM_ACCEPTED_CMDS" instead. > > > > Fixes: 2cf1e703f066 ("bus: fsl-mc: add fsl-mc userspace support") > > Signed-off-by: Dan Carpenter > > Hi, > > I just noticed that Greg wasn't copied on the initial email. > > If you don't mind I will re-submit your patch along with other updates > to the fsl-mc bus so that you don't have to bother. > Thanks for doing that! Was Greg supposed to have been copied, though? He's not listed in the ./scripts/get_maintainer.pl output. regards, dan carpenter