Am Samstag 14 Februar 2009 19:43:52 schrieb [email protected]: > Hi, > I managed to build & test the module with linux 2.6.29-rc4, configured > using v4l2 only. > The following patch is required against > e22293994c6019579e24d789be203729699306dc > git diff e22293994c6019579e24d789be203729699306dc > diff --git a/sn9c20x-v4l2.c b/sn9c20x-v4l2.c > index cecd2bf..b93a9b7 100644 > --- a/sn9c20x-v4l2.c > +++ b/sn9c20x-v4l2.c > @@ -42,7 +42,7 @@ > #include <media/v4l2-ioctl.h> > #endif > > -static struct file_operations v4l_sn9c20x_fops; > +static struct v4l2_file_operations v4l_sn9c20x_fops; > /** > * @var sn9c20x_controls > * List of all V4Lv2 controls supported by the driver > @@ -397,7 +397,7 @@ int v4l2_enable_video(struct usb_sn9c20x *dev, int > mode) > * > * This function permits to open a video device (/dev/videoX) > */ > -static int v4l_sn9c20x_open(struct inode *inode, struct file *fp) > +static int v4l_sn9c20x_open(struct file *fp) > { > int ret = 0; > > @@ -428,7 +428,7 @@ static int v4l_sn9c20x_open(struct inode *inode, > struct file *fp) > * > * This function permits to release an opened file with the 'open' > method. > */ > -static int v4l_sn9c20x_release(struct inode *inode, struct file *fp) > +static int v4l_sn9c20x_release( struct file *fp) > { > struct usb_sn9c20x *dev; > struct video_device *vdev; > @@ -1304,7 +1304,7 @@ int sn9c20x_vidioc_s_param(struct file *file, > void *priv, > * > * This function permits to manage all the IOCTL from the > application. > */ > -static int v4l_sn9c20x_ioctl(struct inode *inode, struct file *fp, > +static long v4l_sn9c20x_ioctl(struct file *fp, > unsigned int cmd, unsigned long arg) > { > int err; > @@ -1319,7 +1319,7 @@ static int v4l_sn9c20x_ioctl(struct inode > *inode, struct file *fp, > if (dev == NULL || vdev == NULL) > return -EFAULT; > > - err = video_ioctl2(inode, fp, cmd, arg); > + err = video_ioctl2(fp, cmd, arg); > > return err; > } > @@ -1449,7 +1449,7 @@ int v4l_sn9c20x_unregister_video_device(struct > usb_sn9c20x *dev) > * > * This variable contains some callback > */ > -static struct file_operations v4l_sn9c20x_fops = { > +static struct v4l2_file_operations v4l_sn9c20x_fops = { > .owner = THIS_MODULE, > .open = v4l_sn9c20x_open, > .release = v4l_sn9c20x_release, > @@ -1460,6 +1460,6 @@ static struct file_operations v4l_sn9c20x_fops = > { > #ifdef CONFIG_COMPAT > .compat_ioctl = v4l_compat_ioctl32, > #endif > - .llseek = no_llseek > + //.llseek = no_llseek > }; > > I'm sorry nobody answered right away.
Thanks alot for the patch (btw could you send it again as an attachment - easier to preserve cryptographic signature that way.) So, do we want to make a 2.6.29-rc branch for all those testers out there? (I myself am stuck with 2.6.27 since the fedora-packagers seem to have some arguments about 2.6.28, anyway my new toy kde42 and especially kmail make me overlook that small non-bleeding-edge kernel.) GWater --~--~---------~--~----~------------~-------~--~----~ Lets make microdia webcams plug'n play, (currently plug'n pray) To post to this group, send email to [email protected] Visit us online https://groups.google.com/group/microdia -~----------~----~----~----~------~----~------~--~---
