* [PATCH 1/2] stb0899: set FE_HAS_SIGNAL flag in read_status
@ 2012-02-28 18:40 Andreas Regel
2012-02-29 8:25 ` [linux-media] " Klaus Schmidinger
0 siblings, 1 reply; 2+ messages in thread
From: Andreas Regel @ 2012-02-28 18:40 UTC (permalink / raw)
To: Manu Abraham; +Cc: Linux Media Mailing List
In stb0899_read_status the FE_HAS_SIGNAL flag was not set in case of a
successful carrier lock. This change fixes that.
Signed-off-by: Andreas Regel <andreas.regel@gmx.de>
---
drivers/media/dvb/frontends/stb0899_drv.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/dvb/frontends/stb0899_drv.c
b/drivers/media/dvb/frontends/stb0899_drv.c
index 38565be..4a58afc 100644
--- a/drivers/media/dvb/frontends/stb0899_drv.c
+++ b/drivers/media/dvb/frontends/stb0899_drv.c
@@ -1071,7 +1071,7 @@ static int stb0899_read_status(struct dvb_frontend
*fe, enum fe_status *status)
reg = stb0899_read_reg(state, STB0899_VSTATUS);
if (STB0899_GETFIELD(VSTATUS_LOCKEDVIT, reg)) {
dprintk(state->verbose, FE_DEBUG, 1, "--------> FE_HAS_CARRIER |
FE_HAS_LOCK");
- *status |= FE_HAS_CARRIER | FE_HAS_LOCK;
+ *status |= FE_HAS_SIGNAL | FE_HAS_CARRIER | FE_HAS_LOCK;
reg = stb0899_read_reg(state, STB0899_PLPARM);
if (STB0899_GETFIELD(VITCURPUN, reg)) {
@@ -1088,7 +1088,7 @@ static int stb0899_read_status(struct dvb_frontend
*fe, enum fe_status *status)
if (internal->lock) {
reg = STB0899_READ_S2REG(STB0899_S2DEMOD, DMD_STAT2);
if (STB0899_GETFIELD(UWP_LOCK, reg) && STB0899_GETFIELD(CSM_LOCK,
reg)) {
- *status |= FE_HAS_CARRIER;
+ *status |= FE_HAS_SIGNAL | FE_HAS_CARRIER;
dprintk(state->verbose, FE_DEBUG, 1,
"UWP & CSM Lock ! ---> DVB-S2 FE_HAS_CARRIER");
-- 1.7.2.5
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [linux-media] [PATCH 1/2] stb0899: set FE_HAS_SIGNAL flag in read_status
2012-02-28 18:40 [PATCH 1/2] stb0899: set FE_HAS_SIGNAL flag in read_status Andreas Regel
@ 2012-02-29 8:25 ` Klaus Schmidinger
0 siblings, 0 replies; 2+ messages in thread
From: Klaus Schmidinger @ 2012-02-29 8:25 UTC (permalink / raw)
To: linux-media
On 28.02.2012 19:40, Andreas Regel wrote:
> In stb0899_read_status the FE_HAS_SIGNAL flag was not set in case of a
> successful carrier lock. This change fixes that.
>
> Signed-off-by: Andreas Regel <andreas.regel@gmx.de>
> ---
> drivers/media/dvb/frontends/stb0899_drv.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/dvb/frontends/stb0899_drv.c b/drivers/media/dvb/frontends/stb0899_drv.c
> index 38565be..4a58afc 100644
> --- a/drivers/media/dvb/frontends/stb0899_drv.c
> +++ b/drivers/media/dvb/frontends/stb0899_drv.c
> @@ -1071,7 +1071,7 @@ static int stb0899_read_status(struct dvb_frontend *fe, enum fe_status *status)
> reg = stb0899_read_reg(state, STB0899_VSTATUS);
> if (STB0899_GETFIELD(VSTATUS_LOCKEDVIT, reg)) {
> dprintk(state->verbose, FE_DEBUG, 1, "--------> FE_HAS_CARRIER | FE_HAS_LOCK");
> - *status |= FE_HAS_CARRIER | FE_HAS_LOCK;
> + *status |= FE_HAS_SIGNAL | FE_HAS_CARRIER | FE_HAS_LOCK;
> reg = stb0899_read_reg(state, STB0899_PLPARM);
> if (STB0899_GETFIELD(VITCURPUN, reg)) {
> @@ -1088,7 +1088,7 @@ static int stb0899_read_status(struct dvb_frontend *fe, enum fe_status *status)
> if (internal->lock) {
> reg = STB0899_READ_S2REG(STB0899_S2DEMOD, DMD_STAT2);
> if (STB0899_GETFIELD(UWP_LOCK, reg) && STB0899_GETFIELD(CSM_LOCK, reg)) {
> - *status |= FE_HAS_CARRIER;
> + *status |= FE_HAS_SIGNAL | FE_HAS_CARRIER;
> dprintk(state->verbose, FE_DEBUG, 1,
> "UWP & CSM Lock ! ---> DVB-S2 FE_HAS_CARRIER");
Acked-by: Klaus Schmidinger <Klaus.Schmidinger@tvdr.de>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-02-29 8:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-28 18:40 [PATCH 1/2] stb0899: set FE_HAS_SIGNAL flag in read_status Andreas Regel
2012-02-29 8:25 ` [linux-media] " Klaus Schmidinger
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.