On Thu, 18 May 2017 11:06:43 -0300 Mauro Carvalho Chehab <mche...@s-opensource.com> wrote:
> diff --git a/drivers/media/tuners/fc0011.c b/drivers/media/tuners/fc0011.c
> index 192b1c7740df..145407dee3db 100644
> --- a/drivers/media/tuners/fc0011.c
> +++ b/drivers/media/tuners/fc0011.c
> @@ -342,6 +342,7 @@ static int fc0011_set_params(struct dvb_frontend *fe)
> switch (vco_sel) {
> default:
> WARN_ON(1);
> + return -EINVAL;
> case 0:
> if (vco_cal < 8) {
> regs[FC11_REG_VCOSEL] &= ~(FC11_VCOSEL_1 |
> FC11_VCOSEL_2);
This fall through is intentional, but I guess returning an error is OK,
too. This should not happen anyway.
I cannot test this, though.
Acked-by: Michael Büsch <m...@bues.ch>
--
Michael
pgpfAl6Qz6id8.pgp
Description: OpenPGP digital signature
