Hi Dan, > -----Original Message----- > From: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > On Sat, Jun 27, 2020 at 10:51:44PM +0530, Alim Akhtar wrote: > > Hi Dan > > > > > -----Original Message----- > > > The "head" pointer can't be NULL because it points to an address in the > > middle > > > of a ufs_hba struct. Looking at this code, probably someone would wonder > > if > > > the intent was to check whether "hba" is NULL, but "hba" > > > isn't NULL and the check can just be removed. > > > > > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > > --- > > Please add Fixes: tag > > With that > > Acked-by: Alim Akhtar <alim.akhtar@xxxxxxxxxxx> > > It's not a bug fix it's just a cleanup. > Acked-by: Alim Akhtar <alim.akhtar@xxxxxxxxxxx> Thanks! > regards, > dan carpenter