]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
[media] lmedm04: use u32 instead of u64 for relative stats
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Tue, 23 Jun 2015 15:22:06 +0000 (12:22 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Wed, 24 Jun 2015 11:38:30 +0000 (08:38 -0300)
Cleanup this sparse warning:
drivers/media/usb/dvb-usb-v2/lmedm04.c:302 lme2510_update_stats() warn: should '((255 - st->signal_sn - 161) * 3) << 8' be a 64 bit type?

Both c_tmp and s_tmp actually stores a u16 stat. Using a u64 data
there is a waste, specially on u32 archs, as 64 ints there are more
expensive.

So, change the types to u32 and do the typecast only when storing
the result.

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/usb/dvb-usb-v2/lmedm04.c

index 57fb184184bf3539656315fb6ddb8a3c03b8c640..fcef2a33ef3d1ab634b79c97b7f6c1c88d2d9676 100644 (file)
@@ -262,7 +262,7 @@ static void lme2510_update_stats(struct dvb_usb_adapter *adap)
        struct lme2510_state *st = adap_to_priv(adap);
        struct dvb_frontend *fe = adap->fe[0];
        struct dtv_frontend_properties *c;
-       u64 s_tmp = 0, c_tmp = 0;
+       u32 s_tmp = 0, c_tmp = 0;
 
        if (!fe)
                return;
@@ -309,11 +309,11 @@ static void lme2510_update_stats(struct dvb_usb_adapter *adap)
 
        c->strength.len = 1;
        c->strength.stat[0].scale = FE_SCALE_RELATIVE;
-       c->strength.stat[0].uvalue = s_tmp;
+       c->strength.stat[0].uvalue = (u64)s_tmp;
 
        c->cnr.len = 1;
        c->cnr.stat[0].scale = FE_SCALE_RELATIVE;
-       c->cnr.stat[0].uvalue = c_tmp;
+       c->cnr.stat[0].uvalue = (u64)c_tmp;
 }
 
 static void lme2510_int_response(struct urb *lme_urb)