From 682e852e2638ed0aff84aa51181c9e5d2f939562 Mon Sep 17 00:00:00 2001 From: Alexey Dobriyan Date: Tue, 10 Jan 2006 00:09:16 +0300 Subject: [PATCH] Fix more "if ((err = foo() < 0))" typos Another reason to use: ret = foo(); if (ret < 0) goto out; Signed-off-by: Alexey Dobriyan Signed-off-by: Linus Torvalds --- drivers/media/dvb/frontends/mt312.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'drivers/media/dvb/frontends/mt312.c') diff --git a/drivers/media/dvb/frontends/mt312.c b/drivers/media/dvb/frontends/mt312.c index 8d672283c93d..ec4e641acc64 100644 --- a/drivers/media/dvb/frontends/mt312.c +++ b/drivers/media/dvb/frontends/mt312.c @@ -501,7 +501,8 @@ static int mt312_set_frontend(struct dvb_frontend* fe, case ID_VP310: // For now we will do this only for the VP310. // It should be better for the mt312 as well, but tunning will be slower. ACCJr 09/29/03 - if ((ret = mt312_readreg(state, CONFIG, &config_val) < 0)) + ret = mt312_readreg(state, CONFIG, &config_val); + if (ret < 0) return ret; if (p->u.qpsk.symbol_rate >= 30000000) //Note that 30MS/s should use 90MHz { -- cgit v1.2.3-59-g8ed1b