Am 18.12.2011 00:47, schrieb Oliver Endriss:
On Sunday 18 December 2011 00:39:49 Oliver Endriss wrote:
On Saturday 17 December 2011 21:57:16 linu...@stefanringel.de wrote:
From: Stefan Ringel<linu...@stefanringel.de>
all drxk have dvb-t, but not dvb-c.
Signed-off-by: Stefan Ringel<linu...@stefanringel.de>
---
drivers/media/dvb/frontends/drxk_hard.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/media/dvb/frontends/drxk_hard.c
b/drivers/media/dvb/frontends/drxk_hard.c
index 038e470..8a59801 100644
--- a/drivers/media/dvb/frontends/drxk_hard.c
+++ b/drivers/media/dvb/frontends/drxk_hard.c
@@ -6460,9 +6460,11 @@ struct dvb_frontend *drxk_attach(const struct
drxk_config *config,
init_state(state);
if (init_drxk(state)< 0)
goto error;
- *fe_t =&state->t_frontend;
^^^^^^^^^^^^^^^^^^^^^^^^^^^
- return&state->c_frontend;
^^^^^^^^^^^^^^^^^^^^^^^^^^
+ if (state->m_hasDVBC)
+ *fe_t =&state->c_frontend;
^^^^^^^^^^^^^^^^^^^^^^^^^^^
+
+ return&state->t_frontend;
^^^^^^^^^^^^^^^^^^^
error:
printk(KERN_ERR "drxk: not found\n");
NAK, this changes the behaviour for existing drivers.
What is the point to swap DVB-T and DVB-C frontends?
If you really need this, please add an option to the config struct
with default that does not change anything for existing drivers.
Correction:
Better do something like this (untested):
if (state->m_hasDVBC) {
*fe_t =&state->t_frontend;
return state->c_frontend;
} else
return&state->t_frontend;
CU
Oliver
What shall be that, explain? For me not practicable.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
