Hi, Sakari, > -----Original Message----- > From: Sakari Ailus [mailto:[email protected]] > Sent: Friday, February 1, 2019 2:43 AM > To: Zhi, Yong <[email protected]> > Cc: [email protected]; Mani, Rajmohan > <[email protected]>; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; Qiu, Tian Shu > <[email protected]>; Cao, Bingbu <[email protected]> > Subject: Re: [PATCH v2 2/2] media: ipu3-imgu: Remove dead code for NULL > check > > Hi Yong, > > On Wed, Jan 16, 2019 at 09:18:47AM -0800, Yong Zhi wrote: > > Since ipu3_css_buf_dequeue() never returns NULL, remove the dead code > > to fix static checker warning: > > > > drivers/staging/media/ipu3/ipu3.c:493 imgu_isr_threaded() > > warn: 'b' is an error pointer or valid > > > > Reported-by: Dan Carpenter <[email protected]> [Bug report: > > https://lore.kernel.org/linux-media/20190104122856.GA1169@kadam/] > > Signed-off-by: Yong Zhi <[email protected]> > > Reviewed-by: Tomasz Figa <[email protected]> > > Reviewed-by: Laurent Pinchart <[email protected]> > > I don't see Laurent's Reviewed-by: tag on the list. Did you get that from him > off-list? If he hasn't given one, please send v3 without that tag. >
The bug report link was suggested by Laurent, so that I assume he reviewed the patch, I can re-send without the tag if this does not count. > Thanks. > > -- > Sakari Ailus > [email protected]
