The patch number 8988 was added via Steven Toth <[EMAIL PROTECTED]>
to http://linuxtv.org/hg/v4l-dvb master development tree.
Kernel patches in this development tree may be modified to be backward
compatible with older kernels. Compatibility modifications will be
removed before inclusion into the mainstream Kernel
If anyone has any objections, please let us know by sending a message to:
[EMAIL PROTECTED]
------
From: Steven Toth <[EMAIL PROTECTED]>
S2API: Allow the properties to call legacy ioctls
SET_TONE and SET_VOLTAGE were not previously implemented. Two options
existed. Either cut/paste from the previous ioctl handler into the
process_properties function, which is code duplication. Or, split
the current ioctl handler into it's two major pieces.
Piece 1, responsible for input validation and semaphore acquiring
Piece 2 the processing of the previous ioctls
and finally, a new third pieces where the array of properties
is processed, and can freely call the legacy ioctl handler without
having to re-acquire the fepriv->sem semaphore. This is a clean approach
and ensures the existing legacy ioctls are processed as they were previously
(but with an extra function call) and allows the new API to share code
without duplication.
Priority: normal
Signed-off-by: Steven Toth <[EMAIL PROTECTED]>
---
linux/drivers/media/dvb/dvb-core/dvb_frontend.c | 71 +++++++++++++---
1 file changed, 61 insertions(+), 10 deletions(-)
diff -r 3a4c28521d43 -r f03dc5bfc40f
linux/drivers/media/dvb/dvb-core/dvb_frontend.c
--- a/linux/drivers/media/dvb/dvb-core/dvb_frontend.c Thu Sep 04 00:17:33
2008 -0400
+++ b/linux/drivers/media/dvb/dvb-core/dvb_frontend.c Thu Sep 04 20:19:43
2008 -0400
@@ -1167,9 +1167,16 @@ int tv_property_cache_submit(struct dvb_
return r;
}
-int tv_property_process(struct dvb_frontend *fe, tv_property_t *tvp)
+static int dvb_frontend_ioctl_legacy(struct inode *inode, struct file *file,
+ unsigned int cmd, void *parg);
+static int dvb_frontend_ioctl_properties(struct inode *inode, struct file
*file,
+ unsigned int cmd, void *parg);
+
+int tv_property_process(struct dvb_frontend *fe, tv_property_t *tvp,
+ struct inode *inode, struct file *file)
{
int r = 0;
+ struct dvb_frontend_private *fepriv = fe->frontend_priv;
printk("%s()\n", __FUNCTION__);
tv_property_dump(tvp);
@@ -1192,7 +1199,9 @@ int tv_property_process(struct dvb_front
*/
fe->tv_property_cache.state = TV_SEQ_COMPLETE;
printk("%s() Finalised property cache\n", __FUNCTION__);
- r = tv_property_cache_submit(fe);
+ r |= tv_property_cache_submit(fe);
+ r |= dvb_frontend_ioctl_legacy(inode, file, FE_SET_FRONTEND,
+ &fepriv->parameters);
break;
case TV_SET_FREQUENCY:
fe->tv_property_cache.frequency = tvp->u.data;
@@ -1289,10 +1298,25 @@ int tv_property_process(struct dvb_front
case TV_GET_ISDB_LAYERC_SEGMENT_WIDTH:
tvp->u.data = fe->tv_property_cache.isdb_layerc_segment_width;
break;
-
- }
-
- return 0;
+ case TV_SET_VOLTAGE:
+ fe->tv_property_cache.voltage = tvp->u.data;
+ r = dvb_frontend_ioctl_legacy(inode, file, FE_SET_VOLTAGE,
+ &fe->tv_property_cache.voltage);
+ break;
+ case TV_GET_VOLTAGE:
+ tvp->u.data = fe->tv_property_cache.voltage;
+ break;
+ case TV_SET_TONE:
+ fe->tv_property_cache.sectone = tvp->u.data;
+ r = dvb_frontend_ioctl_legacy(inode, file, FE_SET_TONE,
+ (void *)fe->tv_property_cache.sectone);
+ break;
+ case TV_GET_TONE:
+ tvp->u.data = fe->tv_property_cache.sectone;
+ break;
+ }
+
+ return r;
}
static int dvb_frontend_ioctl(struct inode *inode, struct file *file,
@@ -1302,7 +1326,6 @@ static int dvb_frontend_ioctl(struct ino
struct dvb_frontend *fe = dvbdev->priv;
struct dvb_frontend_private *fepriv = fe->frontend_priv;
int err = -EOPNOTSUPP;
- tv_property_t* tvp;
dprintk ("%s\n", __func__);
@@ -1317,6 +1340,25 @@ static int dvb_frontend_ioctl(struct ino
if (down_interruptible (&fepriv->sem))
return -ERESTARTSYS;
+ if ((cmd == FE_SET_PROPERTY) || (cmd == FE_GET_PROPERTY))
+ err = dvb_frontend_ioctl_properties(inode, file, cmd, parg);
+ else
+ err = dvb_frontend_ioctl_legacy(inode, file, cmd, parg);
+
+ up(&fepriv->sem);
+ return err;
+}
+
+static int dvb_frontend_ioctl_properties(struct inode *inode, struct file
*file,
+ unsigned int cmd, void *parg)
+{
+ struct dvb_device *dvbdev = file->private_data;
+ struct dvb_frontend *fe = dvbdev->priv;
+ int err = -EOPNOTSUPP;
+ tv_property_t *tvp;
+
+ dprintk("%s\n", __func__);
+
if(cmd == FE_SET_PROPERTY) {
printk("%s() FE_SET_PROPERTY\n", __FUNCTION__);
@@ -1325,7 +1367,7 @@ static int dvb_frontend_ioctl(struct ino
/* TODO: ioctl userdata out of range check here */
tvp = parg;
while(tvp->cmd != TV_SEQ_UNDEFINED) {
- tv_property_process(fe, tvp);
+ tv_property_process(fe, tvp, inode, file);
if( (tvp->cmd == TV_SEQ_TERMINATE) || (tvp->cmd ==
TV_SEQ_COMPLETE) )
break;
tvp++;
@@ -1333,10 +1375,20 @@ static int dvb_frontend_ioctl(struct ino
if(fe->tv_property_cache.state == TV_SEQ_COMPLETE) {
printk("%s() Property cache is full, tuning\n",
__FUNCTION__);
- cmd = FE_SET_FRONTEND;
}
err = 0;
}
+
+ return err;
+}
+
+static int dvb_frontend_ioctl_legacy(struct inode *inode, struct file *file,
+ unsigned int cmd, void *parg)
+{
+ struct dvb_device *dvbdev = file->private_data;
+ struct dvb_frontend *fe = dvbdev->priv;
+ struct dvb_frontend_private *fepriv = fe->frontend_priv;
+ int err = -EOPNOTSUPP;
switch (cmd) {
case FE_GET_INFO: {
@@ -1596,7 +1648,6 @@ static int dvb_frontend_ioctl(struct ino
break;
};
- up (&fepriv->sem);
return err;
}
---
Patch is available at:
http://linuxtv.org/hg/v4l-dvb/rev/f03dc5bfc40f59e87f0be01c3acd08134ef8f757
_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits
