Am 18.12.2011 00:39, schrieb Oliver Endriss:
On Saturday 17 December 2011 21:57:16linuxtv@xxxxxxxxxxxxxxx wrote:
From: Stefan Ringel<linuxtv@xxxxxxxxxxxxxxx>
all drxk have dvb-t, but not dvb-c.
Signed-off-by: Stefan Ringel<linuxtv@xxxxxxxxxxxxxxx>
---
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?
broking by attaching or deattaching the frontend driver, if no dvb-c
frontend drxk can use(i.e. drx-3916k). Do you add a frontend which is
not physically present? Look to my 3rd patch. I tested many ways to
attach that.
Stefan
If you really need this, please add an option to the config struct
with default that does not change anything for existing drivers.
CU
Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
