Re: [DVB patch 5/5] flexcop: add BCM3510 ATSC frontend support for Air2PC card

Johannes Stezenbach
2005-05-24T10:43:23Z

Hi Andrew,

here's an incremental patch which fixes the following warning
when compiling with gcc-3.4.4:

drivers/media/dvb/frontends/bcm3510.c: In function `bcm3510_read_signal_strength':
drivers/media/dvb/frontends/bcm3510.c:322: warning: ISO C90 forbids mixed declarations and code

Johannes

---
Fix compilation with gcc-2.95 (warning with gcc-3.4.4) by
not mixing code and declarations.

Signed-off-by: Johannes Stezenbach <js@linuxtv.org>

drivers/media/dvb/frontends/bcm3510.c | 4 +++-
1 files changed, 3 insertions(+), 1 deletion(-)

Index: linux-2.6.12-rc4-git3/drivers/media/dvb/frontends/bcm3510.c
===================================================================
--- linux-2.6.12-rc4-git3.orig/drivers/media/dvb/frontends/bcm3510.c 2005-05-23 01:27:27.000000000 +0200
+++ linux-2.6.12-rc4-git3/drivers/media/dvb/frontends/bcm3510.c 2005-05-24 12:33:50.000000000 +0200
@@ -318,8 +318,10 @@ static int bcm3510_read_unc(struct dvb_f
static int bcm3510_read_signal_strength(struct dvb_frontend* fe, u16* strength)
{
struct bcm3510_state* st = fe->demodulator_priv;
+ s32 t;
+
bcm3510_refresh_state(st);
- s32 t = st->status2.SIGNAL;
+ t = st->status2.SIGNAL;

if (t > 190)
t = 190;