Commit c6d822c5 authored by Alaa Emad's avatar Alaa Emad Committed by Greg Kroah-Hartman

media: dvb: Add check on sp8870_readreg return

The function sp8870_readreg returns a negative value when i2c_transfer
fails so properly check for this and return the error if it happens.

Cc: Sean Young <sean@mess.org>
Cc: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Signed-off-by: default avatarAlaa Emad <alaaemadhossney.ae@gmail.com>
Link: https://lore.kernel.org/r/20210503115736.2104747-60-gregkh@linuxfoundation.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 47e4ff06
...@@ -280,7 +280,9 @@ static int sp8870_set_frontend_parameters(struct dvb_frontend *fe) ...@@ -280,7 +280,9 @@ static int sp8870_set_frontend_parameters(struct dvb_frontend *fe)
sp8870_writereg(state, 0xc05, reg0xc05); sp8870_writereg(state, 0xc05, reg0xc05);
// read status reg in order to clear pending irqs // read status reg in order to clear pending irqs
sp8870_readreg(state, 0x200); err = sp8870_readreg(state, 0x200);
if (err < 0)
return err;
// system controller start // system controller start
sp8870_microcontroller_start(state); sp8870_microcontroller_start(state);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment