linux-dev/drivers/staging/dt3155, branch master

Staging: dt3155: remove the driver

2010-08-03T01:11:34Z

There is now a proper V4L driver for this device in the tree, so remove this one. Cc: Scott Smedley Cc: H Hartley Sweeten Signed-off-by: Greg Kroah-Hartman

Staging: dt3155: make local functions static

2010-07-26T22:34:44Z

The functions adjust_4MB and allocate_buffers are only called locally in dt3155_isr.c and should be static. Signed-off-by: H Hartley Sweeten Cc: Scott Smedley Signed-off-by: Greg Kroah-Hartman

Staging: dt3155: make module_{init/exit} functions static

2010-07-26T22:34:04Z

The module_init() and module_exit() functions should be static and marked with __init and __exit. Signed-off-by: H Hartley Sweeten Cc: Scott Smedley Signed-off-by: Greg Kroah-Hartman

Staging: dt3155: don't use default init_module/cleanup_module function names

2010-07-22T21:49:19Z

Fromy: Randy Dunlap Fix the dt3155 driver to use module_init()/module_exit() instead of default init_module() and cleanup_module() function names. Signed-off-by: Randy Dunlap Cc: Scott Smedley Cc: Marek Lindner Cc: Simon Wunderlich Cc: Andrew Lunn Signed-off-by: Greg Kroah-Hartman

Staging: dt3155: Fixed all printk warnings found by checkpatch.pl

2010-07-22T21:40:05Z

Added all KERN_ levels in printk found by checkpatch.pl Signed-off-by: Joe Eloff Signed-off-by: Greg Kroah-Hartman

Staging: dt3155: Change include to linux header

2010-07-22T21:38:54Z

Changed to Signed-off-by: Joe Eloff Signed-off-by: Greg Kroah-Hartman

Staging: autoconvert trivial BKL users to private mutex

2010-07-22T18:10:29Z

All these files use the big kernel lock in a trivial way to serialize their private file operations, typically resulting from an earlier semi-automatic pushdown from VFS. None of these drivers appears to want to lock against other code, and they all use the BKL as the top-level lock in their file operations, meaning that there is no lock-order inversion problem. Consequently, we can remove the BKL completely, replacing it with a per-file mutex in every case. Using a scripted approach means we can avoid typos. file=$1 name=$2 if grep -q lock_kernel ${file} ; then if grep -q 'include.*linux.mutex.h' ${file} ; then sed -i '/include.*/d' ${file} else sed -i 's/include.*.*$/include /g' ${file} fi sed -i ${file} \ -e "/^#include.*linux.mutex.h/,$ { 1,/^$static\|int\|long$/ { /^$static\|int\|long$/istatic DEFINE_MUTEX(${name}_mutex); } }" \ -e "s/$un$*lock_kernel\>[ ]*()/mutex_\1lock(\&${name}_mutex)/g" \ -e '/[ ]*cycle_kernel_lock();/d' else sed -i -e '/include.*\/d' ${file} \ -e '/cycle_kernel_lock()/d' fi Signed-off-by: Arnd Bergmann Signed-off-by: Greg Kroah-Hartman

Staging: dt3155: use the fbuffer pointer instead of looking it up

2010-07-22T18:04:00Z

Instead of passing the minor number and having to look up the fbuffer, just pass the fbuffer directly to the buffer management code. Also, to make the code more consistent, change the push_empty() call so that the fbuffer is passed as the first parameter. Prototype the printques routine to avoid having to declare it as extern. Cleanup some of the comments in dt3155_isr.h. Signed-off-by: H Hartley Sweeten Cc: Scott Smedley Signed-off-by: Greg Kroah-Hartman

Staging: dt3155_isr: cleanup fbuffer usage

2010-07-08T19:33:44Z

Now that dt3155_drv.c is not dependent on the global symbol dt3155_fbuffer[], declared in dt3155_isr.c, remove it. This also fixes many of the coding style problems in dt3155_isr.c. Signed-off-by: H Hartley Sweeten Cc: Scott Smedley Signed-off-by: Greg Kroah-Hartman

Staging: dt3155: remove remaining 32-bit and 8-bit global registers

2010-07-08T19:33:03Z

Remove the remaining global 32-bit and 8-bit i2c registers. Create a local variable of the correct type where they are needed. Signed-off-by: H Hartley Sweeten Cc: Scott Smedley Signed-off-by: Greg Kroah-Hartman