[update] firedtv: dvb_frontend_info for FireDTV S2, fix "frequency limits undefined" error

Message ID tkrat.f46f5cba7bb5dd9f@s5r6.in-berlin.de (mailing list archive)
State Superseded, archived
Headers

Commit Message

Stefan Richter Feb. 24, 2009, 2:52 p.m. UTC
  From: Beat Michel Liechti <bml303@gmail.com>

I found that the function fdtv_frontend_init in the file firedtv-fe.c was
missing a case for FIREDTV_DVB_S2 which resulted in "frequency limits
undefined" errors in syslog.

Signed-off-by: Beat Michel Liechti <bml303@gmail.com>

Change by Stefan R: combine it with case case FIREDTV_DVB_S as
originally suggested by Beat Michel.  This enables FE_CAN_FEC_AUTO also
for FireDTV-S2 devices which is possible as long as only DVB-S channels
are used.  FE_CAN_FEC_AUTO would be wrong for DVB-S2 channels, but those
cannot be used yet since the driver is not yet converted to S2API.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
---

If you don't mind, I commit this for now (want to send a mainline pull
request later today) and we work out any remaining loose DVB-S2 ends
from there.

 drivers/media/dvb/firewire/firedtv-fe.c |    1 +
 1 file changed, 1 insertion(+)
  

Patch

Index: linux/drivers/media/dvb/firewire/firedtv-fe.c
===================================================================
--- linux.orig/drivers/media/dvb/firewire/firedtv-fe.c
+++ linux/drivers/media/dvb/firewire/firedtv-fe.c
@@ -185,6 +185,7 @@  void fdtv_frontend_init(struct firedtv *
 
 	switch (fdtv->type) {
 	case FIREDTV_DVB_S:
+	case FIREDTV_DVB_S2:
 		fi->type		= FE_QPSK;
 
 		fi->frequency_min	= 950000;