Skip to contentSkip to conversation

Add the DTV_ISDB_TS_ID property for ISDB_S

PATCH In review [email protected] 2 participants 2 messages Aug 23, 2009 – Sep 18, 2009 Media (V4L/DVB)

lore
Share & tools
lore
#0 lore raw

PATCH 1/2 Add the DTV_ISDB_TS_ID property for ISDB_S

Add the DTV_ISDB_TS_ID property for ISDB-S

In ISDB-S, time-devision duplex is used to multiplexing several waves in the same frequency. Each wave is identified by its own transport stream ID, or TS ID. We need to provide some way to specify this ID from user applications to handle ISDB-S frontends.

This code has been tested with the Earthsoft PT1 driver.

linux/drivers/media/dvb/dvb-core/dvb_frontend.c +11 -0
diff -r 948b12c08e2b -r bc34617eca49 linux/drivers/media/dvb/dvb-core/dvb_frontend.c
@@ -948,6 +948,11 @@
 		.cmd	= DTV_TRANSMISSION_MODE,
 		.set	= 1,
 	},
+	[DTV_ISDB_TS_ID] = {
+		.name	= "DTV_ISDB_TS_ID",
+		.cmd	= DTV_ISDB_TS_ID,
+		.set	= 1,
+	},
 	/* Get */
 	[DTV_DISEQC_SLAVE_REPLY] = {
 		.name	= "DTV_DISEQC_SLAVE_REPLY",
@@ -1356,6 +1361,9 @@
 	case DTV_HIERARCHY:
 		tvp->u.data = fe->dtv_property_cache.hierarchy;
 		break;
+	case DTV_ISDB_TS_ID:
+		tvp->u.data = fe->dtv_property_cache.isdb_ts_id;
+		break;
 	default:
 		r = -1;
 	}
@@ -1462,6 +1470,9 @@
 	case DTV_HIERARCHY:
 		fe->dtv_property_cache.hierarchy = tvp->u.data;
 		break;
+	case DTV_ISDB_TS_ID:
+		fe->dtv_property_cache.isdb_ts_id = tvp->u.data;
+		break;
 	default:
 		r = -1;
 	}
linux/drivers/media/dvb/dvb-core/dvb_frontend.h +1 -0
diff -r 948b12c08e2b -r bc34617eca49 linux/drivers/media/dvb/dvb-core/dvb_frontend.h
@@ -355,6 +355,7 @@
 	fe_modulation_t		isdb_layerc_modulation;
 	u32			isdb_layerc_segment_width;
 #endif
+	u32			isdb_ts_id;
 };
 
 struct dvb_frontend {
linux/include/linux/dvb/frontend.h +3 -1
diff -r 948b12c08e2b -r bc34617eca49 linux/include/linux/dvb/frontend.h
@@ -307,7 +307,9 @@
 #define DTV_TRANSMISSION_MODE			39
 #define DTV_HIERARCHY				40
 
-#define DTV_MAX_COMMAND				DTV_HIERARCHY
+#define DTV_ISDB_TS_ID				41
+
+#define DTV_MAX_COMMAND				DTV_ISDB_TS_ID
 
 typedef enum fe_pilot {
 	PILOT_ON,
#1 lore raw

Em Sun, 23 Aug 2009 12:49:49 +0900
[email protected] escreveu:

Quoted 9 lines

Add the DTV_ISDB_TS_ID property for ISDB-S

In ISDB-S, time-devision duplex is used to multiplexing several waves in the same frequency. Each wave is identified by its own transport stream ID, or TS ID. We need to provide some way to specify this ID from user applications to handle ISDB-S frontends.

This code has been tested with the Earthsoft PT1 driver.

Committed, thanks. I had to fix a merge conflict with the ISDB-T API additions.

Review

Quoted 2 lines
(quoted diff) +2 -0
+	u32			isdb_ts_id;
+#define DTV_ISDB_TS_ID				41

I also added an "s" after ISDB to use the same name convention used on other API functions.

Cheers,
Mauro