On 01/03/20 14:09:51, Dan Carpenter wrote:
> On Fri, Dec 20, 2019 at 09:15:09PM +0000, Malcolm Priestley wrote:
> > The driver standard error returns remove bool false conditions.
> > 
> > Cc: stable <sta...@vger.kernel.org> # v5.3+
> > Signed-off-by: Malcolm Priestley <tvbox...@gmail.com>
> 
> Fixes: 07ba60a15843 ("staging: vt6656: clean-up registers initialization 
> error path")
> 
> The other part of that bug was fixed silently in commit 987d864a2363
> ("staging: vt6656: manage error path during device initialization").
> I'm quite embarrassed that I didn't catch these during review...  It's
> the obvious bug right?  "You have reversed the return values but not
> updated any of the callers."  *Egg on my face*.
> 
> regards,
> dan carpenter
> 

That one is ugly, my bad.

The fix looks good to me, and as Dan said the other part has already
been fixed.

Reviewed-by: Quentin Deslandes <quentin.deslan...@itdev.co.uk>

Regards,
Quentin Deslandes
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to