diff options
author | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-03-11 06:34:44 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-03-11 06:37:21 -0400 |
commit | ee0f4a144423cdb5744ceca4ed9e91582c39becf (patch) | |
tree | e4df9bfd32068e6fe24525fc535c10e8984a17be | |
parent | d591590e1b5bba2f96b44b1e05f65bc8269f0a68 (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.c | 2 |
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) */ |