aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <m.chehab@samsung.com>2014-03-11 06:34:44 -0400
committerMauro Carvalho Chehab <m.chehab@samsung.com>2014-03-11 06:37:21 -0400
commitee0f4a144423cdb5744ceca4ed9e91582c39becf (patch)
treee4df9bfd32068e6fe24525fc535c10e8984a17be
parentd591590e1b5bba2f96b44b1e05f65bc8269f0a68 (diff)
drx-j: use ber_count var
drivers/media/dvb-frontends/drx39xyj/drxj.c: In function 'ctrl_get_qam_sig_quality': drivers/media/dvb-frontends/drx39xyj/drxj.c:9468:6: warning: variable 'ber_cnt' set but not used [-Wunused-but-set-variable] u32 ber_cnt = 0; /* BER count */ ^ By reading the comment, it is said that BER should be calculated as: qam_pre_rs_ber = frac_times1e6( ber_cnt, rs_bit_cnt ); Also, it makes sense to take the mantissa into account, so fix the code to do what's commented. Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
-rw-r--r--drivers/media/dvb-frontends/drx39xyj/drxj.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/dvb-frontends/drx39xyj/drxj.c b/drivers/media/dvb-frontends/drx39xyj/drxj.c
index 0c0e9f3b108f..41d4bfe66764 100644
--- a/drivers/media/dvb-frontends/drx39xyj/drxj.c
+++ b/drivers/media/dvb-frontends/drx39xyj/drxj.c
@@ -9583,7 +9583,7 @@ ctrl_get_qam_sig_quality(struct drx_demod_instance *demod)
9583 if (m > (rs_bit_cnt >> (e + 1)) || (rs_bit_cnt >> e) == 0) 9583 if (m > (rs_bit_cnt >> (e + 1)) || (rs_bit_cnt >> e) == 0)
9584 qam_pre_rs_ber = 500000 * rs_bit_cnt >> e; 9584 qam_pre_rs_ber = 500000 * rs_bit_cnt >> e;
9585 else 9585 else
9586 qam_pre_rs_ber = m; 9586 qam_pre_rs_ber = ber_cnt;
9587 9587
9588 /* post RS BER = 1000000* (11.17 * FEC_OC_SNC_FAIL_COUNT__A) / */ 9588 /* post RS BER = 1000000* (11.17 * FEC_OC_SNC_FAIL_COUNT__A) / */
9589 /* (1504.0 * FEC_OC_SNC_FAIL_PERIOD__A) */ 9589 /* (1504.0 * FEC_OC_SNC_FAIL_PERIOD__A) */