Re: easycapdc60 driver for staging

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

 



On Fri, Jun 04, 2010 at 02:07:15PM +0100, R.M. Thomas wrote:
> The tarball containing the easycap driver is available for
> collection at your convenience from:
> 
> http://sourceforge.net/projects/easycapdc60/files/tmp/KandR4staging.tar.gz
> 
> 
> When you unzip it you will find a directory named easycap which I
> think may be suitable for dropping directly into drivers/staging.
> There is also a file patch.txt which shows the one-line changes I
> made to files drivers/staging/Kconfig and drivers/staging/Makefile
> in order to achieve an in-tree build of the driver on kernel 2.6.34
> cloned from git.
> 
> I have reformatted all the source code to meet the requirements of
> scripts/checkpatch.pl.  The reformatted code builds without
> complaint against 2.6.32 and the test runs I have done with mplayer,
> tvtime and vlc give results indistinguishable from those of the
> pre-reformatted version 0.8 of the driver still available at
> Sourceforge.  (This could not be taken for granted because the
> reformatting was done mostly by hand editing and one cannot rule out
> the possibility of typos which do not break the compilation but
> nevetheless trash the logic.)
> 
> I have not been able to do test runs of the module built in-tree on
> 2.6.34 because this kernel refuses to boot on the old machine I use
> for easycap stuff.  Probably I did something wrong during/after the
> kernel build, but since each build takes nearly four hours I don't
> have time at the moment to investigate what.
> 
> Let me know if there are any immediate problems I need to fix.

Looks good to start with, I've added it to my tree.  Odds are there are
going to be come Kconfig changes needed to handle the dependancies
properly, but I know Randy will be sending me patches for that come
Monday morning :)

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux