* [PATCH 2/2] af9033: fix snr value not correct issue
@ 2014-09-29 8:47 Bimow Chen
2014-09-29 12:06 ` Antti Palosaari
0 siblings, 1 reply; 2+ messages in thread
From: Bimow Chen @ 2014-09-29 8:47 UTC (permalink / raw)
To: linux-media; +Cc: crope
[-- Attachment #1: Type: text/plain, Size: 39 bytes --]
Snr returns value not correct. Fix it.
[-- Attachment #2: 0002-af9033-fix-snr-value-not-correct-issue.patch --]
[-- Type: text/x-patch, Size: 3022 bytes --]
>From 427a5c6ef49e3235ac35a0464c375f2a2706619e Mon Sep 17 00:00:00 2001
From: Bimow Chen <Bimow.Chen@ite.com.tw>
Date: Mon, 29 Sep 2014 16:30:52 +0800
Subject: [PATCH 2/2] af9033: fix snr value not correct issue
Snr returns value not correct. Fix it.
Signed-off-by: Bimow Chen <Bimow.Chen@ite.com.tw>
---
drivers/media/dvb-frontends/af9033.c | 52 ++++++++++++++++++++++++++---
drivers/media/dvb-frontends/af9033_priv.h | 5 ++-
2 files changed, 51 insertions(+), 6 deletions(-)
diff --git a/drivers/media/dvb-frontends/af9033.c b/drivers/media/dvb-frontends/af9033.c
index e191bd5..30dc366 100644
--- a/drivers/media/dvb-frontends/af9033.c
+++ b/drivers/media/dvb-frontends/af9033.c
@@ -851,8 +851,8 @@ static int af9033_read_snr(struct dvb_frontend *fe, u16 *snr)
struct dtv_frontend_properties *c = &dev->fe.dtv_property_cache;
/* use DVBv5 CNR */
- if (c->cnr.stat[0].scale == FE_SCALE_DECIBEL)
- *snr = div_s64(c->cnr.stat[0].svalue, 100); /* 1000x => 10x */
+ if (c->cnr.stat[0].scale == FE_SCALE_RELATIVE)
+ *snr = c->cnr.stat[0].uvalue;
else
*snr = 0;
@@ -1025,6 +1025,33 @@ static void af9033_stat_work(struct work_struct *work)
snr_val = (buf[2] << 16) | (buf[1] << 8) | (buf[0] << 0);
+ /* read superframe number */
+ ret = af9033_rd_reg(dev, 0x80f78b, &u8tmp);
+ if (ret)
+ goto err;
+
+ if (u8tmp)
+ snr_val /= u8tmp;
+
+ /* read current transmission mode */
+ ret = af9033_rd_reg(dev, 0x80f900, &u8tmp);
+ if (ret)
+ goto err;
+
+ switch ((u8tmp >> 0) & 3) {
+ case 0:
+ snr_val *= 4;
+ break;
+ case 1:
+ snr_val *= 1;
+ break;
+ case 2:
+ snr_val *= 2;
+ break;
+ default:
+ goto err;
+ }
+
/* read current modulation */
ret = af9033_rd_reg(dev, 0x80f903, &u8tmp);
if (ret)
@@ -1048,14 +1075,29 @@ static void af9033_stat_work(struct work_struct *work)
}
for (i = 0; i < len; i++) {
- tmp = snr_lut[i].snr * 1000;
+ tmp = snr_lut[i].snr;
if (snr_val < snr_lut[i].val)
break;
}
+ /* scale value to 0x0000-0xffff */
+ switch ((u8tmp >> 0) & 3) {
+ case 0:
+ tmp = tmp * 0xFFFF / 23;
+ break;
+ case 1:
+ tmp = tmp * 0xFFFF / 26;
+ break;
+ case 2:
+ tmp = tmp * 0xFFFF / 32;
+ break;
+ default:
+ goto err;
+ }
+
c->cnr.len = 1;
- c->cnr.stat[0].scale = FE_SCALE_DECIBEL;
- c->cnr.stat[0].svalue = tmp;
+ c->cnr.stat[0].scale = FE_SCALE_RELATIVE;
+ c->cnr.stat[0].uvalue = tmp;
} else {
c->cnr.len = 1;
c->cnr.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
diff --git a/drivers/media/dvb-frontends/af9033_priv.h b/drivers/media/dvb-frontends/af9033_priv.h
index c9c8798..8e23275 100644
--- a/drivers/media/dvb-frontends/af9033_priv.h
+++ b/drivers/media/dvb-frontends/af9033_priv.h
@@ -181,7 +181,10 @@ static const struct val_snr qam64_snr_lut[] = {
{ 0x05570d, 26 },
{ 0x059feb, 27 },
{ 0x05bf38, 28 },
- { 0xffffff, 29 },
+ { 0x05f78f, 29 },
+ { 0x0612c3, 30 },
+ { 0x0626be, 31 },
+ { 0xffffff, 32 },
};
static const struct reg_val ofsm_init[] = {
--
1.7.0.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 2/2] af9033: fix snr value not correct issue
2014-09-29 8:47 [PATCH 2/2] af9033: fix snr value not correct issue Bimow Chen
@ 2014-09-29 12:06 ` Antti Palosaari
0 siblings, 0 replies; 2+ messages in thread
From: Antti Palosaari @ 2014-09-29 12:06 UTC (permalink / raw)
To: Bimow Chen, linux-media
On 09/29/2014 11:47 AM, Bimow Chen wrote:
> Snr returns value not correct. Fix it.
And same here. You change SNR, which is reported by dB to relative
0-0xffff. If you like to do it for some reason, then do it for DVBv3 API
only. Leave new DVBv5 statistic API reports to dB.
I am pretty sure we should report DVBv5 signals using dB if possible,
not relative.
regards
Antti
>
>
> 0002-af9033-fix-snr-value-not-correct-issue.patch
>
>
>>From 427a5c6ef49e3235ac35a0464c375f2a2706619e Mon Sep 17 00:00:00 2001
> From: Bimow Chen<Bimow.Chen@ite.com.tw>
> Date: Mon, 29 Sep 2014 16:30:52 +0800
> Subject: [PATCH 2/2] af9033: fix snr value not correct issue
>
> Snr returns value not correct. Fix it.
>
> Signed-off-by: Bimow Chen<Bimow.Chen@ite.com.tw>
> ---
> drivers/media/dvb-frontends/af9033.c | 52 ++++++++++++++++++++++++++---
> drivers/media/dvb-frontends/af9033_priv.h | 5 ++-
> 2 files changed, 51 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/media/dvb-frontends/af9033.c b/drivers/media/dvb-frontends/af9033.c
> index e191bd5..30dc366 100644
> --- a/drivers/media/dvb-frontends/af9033.c
> +++ b/drivers/media/dvb-frontends/af9033.c
> @@ -851,8 +851,8 @@ static int af9033_read_snr(struct dvb_frontend *fe, u16 *snr)
> struct dtv_frontend_properties *c = &dev->fe.dtv_property_cache;
>
> /* use DVBv5 CNR */
> - if (c->cnr.stat[0].scale == FE_SCALE_DECIBEL)
> - *snr = div_s64(c->cnr.stat[0].svalue, 100); /* 1000x => 10x */
> + if (c->cnr.stat[0].scale == FE_SCALE_RELATIVE)
> + *snr = c->cnr.stat[0].uvalue;
> else
> *snr = 0;
>
> @@ -1025,6 +1025,33 @@ static void af9033_stat_work(struct work_struct *work)
>
> snr_val = (buf[2] << 16) | (buf[1] << 8) | (buf[0] << 0);
>
> + /* read superframe number */
> + ret = af9033_rd_reg(dev, 0x80f78b, &u8tmp);
> + if (ret)
> + goto err;
> +
> + if (u8tmp)
> + snr_val /= u8tmp;
> +
> + /* read current transmission mode */
> + ret = af9033_rd_reg(dev, 0x80f900, &u8tmp);
> + if (ret)
> + goto err;
> +
> + switch ((u8tmp >> 0) & 3) {
> + case 0:
> + snr_val *= 4;
> + break;
> + case 1:
> + snr_val *= 1;
> + break;
> + case 2:
> + snr_val *= 2;
> + break;
> + default:
> + goto err;
> + }
> +
> /* read current modulation */
> ret = af9033_rd_reg(dev, 0x80f903, &u8tmp);
> if (ret)
> @@ -1048,14 +1075,29 @@ static void af9033_stat_work(struct work_struct *work)
> }
>
> for (i = 0; i < len; i++) {
> - tmp = snr_lut[i].snr * 1000;
> + tmp = snr_lut[i].snr;
> if (snr_val < snr_lut[i].val)
> break;
> }
>
> + /* scale value to 0x0000-0xffff */
> + switch ((u8tmp >> 0) & 3) {
> + case 0:
> + tmp = tmp * 0xFFFF / 23;
> + break;
> + case 1:
> + tmp = tmp * 0xFFFF / 26;
> + break;
> + case 2:
> + tmp = tmp * 0xFFFF / 32;
> + break;
> + default:
> + goto err;
> + }
> +
> c->cnr.len = 1;
> - c->cnr.stat[0].scale = FE_SCALE_DECIBEL;
> - c->cnr.stat[0].svalue = tmp;
> + c->cnr.stat[0].scale = FE_SCALE_RELATIVE;
> + c->cnr.stat[0].uvalue = tmp;
> } else {
> c->cnr.len = 1;
> c->cnr.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
> diff --git a/drivers/media/dvb-frontends/af9033_priv.h b/drivers/media/dvb-frontends/af9033_priv.h
> index c9c8798..8e23275 100644
> --- a/drivers/media/dvb-frontends/af9033_priv.h
> +++ b/drivers/media/dvb-frontends/af9033_priv.h
> @@ -181,7 +181,10 @@ static const struct val_snr qam64_snr_lut[] = {
> { 0x05570d, 26 },
> { 0x059feb, 27 },
> { 0x05bf38, 28 },
> - { 0xffffff, 29 },
> + { 0x05f78f, 29 },
> + { 0x0612c3, 30 },
> + { 0x0626be, 31 },
> + { 0xffffff, 32 },
> };
>
> static const struct reg_val ofsm_init[] = {
> -- 1.7.0.4
>
--
http://palosaari.fi/
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-09-29 12:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-29 8:47 [PATCH 2/2] af9033: fix snr value not correct issue Bimow Chen
2014-09-29 12:06 ` Antti Palosaari
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.