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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Beat Michel Liechti <bml303@xxxxxxxxx>

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@xxxxxxxxx>

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@xxxxxxxxxxxxxxxxx>
---

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(+)

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;

-- 
Stefan Richter
-=====-==--= --=- ==---
http://arcgraph.de/sr/

--
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

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux