Commit 5fecc841 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab Committed by Greg Kroah-Hartman

cx231xx: don't return error on success

commit 1871d718 upstream.

The cx231xx_set_agc_analog_digital_mux_select() callers
expect it to return 0 or an error. Returning a positive value
makes the first attempt to switch between analog/digital to fail.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 68bf9d10
...@@ -1264,7 +1264,10 @@ int cx231xx_set_agc_analog_digital_mux_select(struct cx231xx *dev, ...@@ -1264,7 +1264,10 @@ int cx231xx_set_agc_analog_digital_mux_select(struct cx231xx *dev,
dev->board.agc_analog_digital_select_gpio, dev->board.agc_analog_digital_select_gpio,
analog_or_digital); analog_or_digital);
if (status < 0)
return status; return status;
return 0;
} }
int cx231xx_enable_i2c_port_3(struct cx231xx *dev, bool is_port_3) int cx231xx_enable_i2c_port_3(struct cx231xx *dev, bool is_port_3)
......
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