]> git.karo-electronics.de Git - linux-beck.git/log
linux-beck.git
14 years agoStaging: batman-adv: Fix resizing of broadcast seqno buffers on if deletion
Linus Lüssing [Tue, 19 Oct 2010 09:59:11 +0000 (11:59 +0200)]
Staging: batman-adv: Fix resizing of broadcast seqno buffers on if deletion

Not only the entries of the deleted interface got erased, but also all
ones with a lower if_num. This commit fixes this issue by setting the
destination appropriately.

Signed-off-by: Linus Lüssing <linus.luessing@web.de>
Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: batman-adv: Don't dereference unchecked incoming soft_iface
Sven Eckelmann [Tue, 19 Oct 2010 09:59:10 +0000 (11:59 +0200)]
Staging: batman-adv: Don't dereference unchecked incoming soft_iface

send_packet derefenced forw_packet->if_incoming and checked if
forw_packet->if_incoming is NULL.

This cannot happen, but still makes irritates when reading through the
functions.

Reported-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: batman-adv: Add hash recommendations to TODO
Sven Eckelmann [Tue, 19 Oct 2010 09:59:09 +0000 (11:59 +0200)]
Staging: batman-adv: Add hash recommendations to TODO

David Miller <davem@davemloft.net> recommended in
20100924.134334.28812338.davem@davemloft.net that we must make the hash
abstraction helper more efficient and may remove it completely.

Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: sst: more copy_to_user() changes
Dan Carpenter [Fri, 15 Oct 2010 19:10:44 +0000 (21:10 +0200)]
Staging: sst: more copy_to_user() changes

Vinod wanted the copy_to_user() calls in this format:
if (copy_to_user())
retval = -EFAULT;
instead of this:
retval = copy_to_user();
if (retval)
retval = -EFAULt;

I've done that for the whole intel_sst_app_interface.c file.  In the
process I noticed a couple more places that returned -EBUSY or -EAGAIN
instead of -EFAULT.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: intel_sst: off by one bug
Dan Carpenter [Fri, 15 Oct 2010 20:36:48 +0000 (22:36 +0200)]
Staging: intel_sst: off by one bug

This should be >= instead of > or we go passed the end of the array.

Also the arrays are declared with size MAX_NUM_STREAMS.  This is the
only place that uses MAX_NUM_STREAMS_MFLD.  It seems like asking for
trouble to use two variables for the same information.  I've changed
everything to use MAX_NUM_STREAMS.

This bug isn't really harmful.  In the worst case, if you enabled
debugging then you would see a message.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: intel_sst: return -EFAULT if copy_to_user() fails
Dan Carpenter [Fri, 15 Oct 2010 03:44:07 +0000 (05:44 +0200)]
Staging: intel_sst: return -EFAULT if copy_to_user() fails

copy_to_user() returns the number of bytes remaining to be copied but we
want to return an error code.  And that error code is -EFAULT not -EIO.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: Convert camel cased functions in channel_mgmt.c to lower case
Haiyang Zhang [Fri, 15 Oct 2010 17:14:07 +0000 (10:14 -0700)]
staging: hv: Convert camel cased functions in channel_mgmt.c to lower case

Convert camel cased functions in channel_mgmt.c to lower case

Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: Convert camel cased local variable names in channel_mgmt.c to lower...
Haiyang Zhang [Fri, 15 Oct 2010 17:14:06 +0000 (10:14 -0700)]
staging: hv: Convert camel cased local variable names in channel_mgmt.c to lower case

Convert camel cased local variable names in channel_mgmt.c to lower case

Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: Convert camel cased parameter in channel_interface.h to lower case
Haiyang Zhang [Fri, 15 Oct 2010 17:14:05 +0000 (10:14 -0700)]
staging: hv: Convert camel cased parameter in channel_interface.h to lower case

Convert camel cased parameter in channel_interface.h to lower case

Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: Convert camel cased function names in channel_interface.c to lower cases
Haiyang Zhang [Fri, 15 Oct 2010 17:14:04 +0000 (10:14 -0700)]
staging: hv: Convert camel cased function names in channel_interface.c to lower cases

Convert camel cased function names in channel_interface.c to lower cases

Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: Convert camel cased local variables in channel_interface.c to lower...
Haiyang Zhang [Fri, 15 Oct 2010 17:14:03 +0000 (10:14 -0700)]
staging: hv: Convert camel cased local variables in channel_interface.c to lower cases

Convert camel cased local variables in channel_interface.c to lower cases

Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: tidspbridge: Clean up error-handling code
Julia Lawall [Fri, 15 Oct 2010 13:00:10 +0000 (15:00 +0200)]
staging: tidspbridge: Clean up error-handling code

In the first hunk, 0 is returned on memory allocation failure, even though
other failures return -ENOMEM or other similar values.

In the second hunk, there was error handling code that returned without
freeing psz_path_name.

A simplified version of the semantic match that finds the first problem is as
follows: (http://coccinelle.lip6.fr/)

// <smpl>
@@
expression ret;
expression x,e1,e2,e3;
@@

ret = 0
... when != ret = e1
*x = \(kmalloc\|kcalloc\|kzalloc\)(...)
... when != ret = e2
if (x == NULL) { ... when != ret = e3
  return ret;
}
// </smpl>

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Use specific error return code for reg_ft1000_netdev().
Marek Belisko [Fri, 15 Oct 2010 07:41:45 +0000 (09:41 +0200)]
staging: ft1000: Use specific error return code for reg_ft1000_netdev().

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Use specific return error codes.
Marek Belisko [Fri, 15 Oct 2010 06:57:25 +0000 (08:57 +0200)]
staging: ft1000: Use specific return error codes.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Check return value when init net device.
Marek Belisko [Fri, 15 Oct 2010 06:57:24 +0000 (08:57 +0200)]
staging: ft1000: Check return value when init net device.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Remove DRVMGS typedef usage.
Marek Belisko [Fri, 15 Oct 2010 12:13:03 +0000 (14:13 +0200)]
staging: ft1000: Remove DRVMGS typedef usage.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Remove PSEUDO_HDR typedef usage.
Marek Belisko [Fri, 15 Oct 2010 12:13:02 +0000 (14:13 +0200)]
staging: ft1000: Remove PSEUDO_HDR typedef usage.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Remove PROV_RECORD typedef usage.
Marek Belisko [Fri, 15 Oct 2010 12:13:01 +0000 (14:13 +0200)]
staging: ft1000: Remove PROV_RECORD typedef usage.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Remove APP_INFO_BLOCK typedef usage.
Marek Belisko [Fri, 15 Oct 2010 12:13:00 +0000 (14:13 +0200)]
staging: ft1000: Remove APP_INFO_BLOCK typedef usage.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Remove DSPINITMSG typedef usage.
Marek Belisko [Fri, 15 Oct 2010 12:12:59 +0000 (14:12 +0200)]
staging: ft1000: Remove DSPINITMSG typedef usage.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Remove MEDIAMSG typedef usage.
Marek Belisko [Fri, 15 Oct 2010 12:12:58 +0000 (14:12 +0200)]
staging: ft1000: Remove MEDIAMSG typedef usage.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Remove unused function ft1000_hbchk()
Marek Belisko [Fri, 15 Oct 2010 08:51:22 +0000 (10:51 +0200)]
staging: ft1000: Remove unused function ft1000_hbchk()

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: intel_sst: cleanup naming a little
Dan Carpenter [Fri, 15 Oct 2010 03:44:58 +0000 (05:44 +0200)]
Staging: intel_sst: cleanup naming a little

&sst_drv_ctx->streams[str_id].lock and &stream->lock are the same.
The mutex_lock() uses &stream->lock so this makes things consistent and
it's nicer to read as well.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: ft1000-usb: use the correct list head
Dan Carpenter [Fri, 15 Oct 2010 03:40:42 +0000 (05:40 +0200)]
Staging: ft1000-usb: use the correct list head

Using pdpram_blk here would cause an oops.  We intended to use
&freercvpool.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Tested by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: lirc: fix PTR_ERR() check
Dan Carpenter [Fri, 15 Oct 2010 03:42:35 +0000 (05:42 +0200)]
Staging: lirc: fix PTR_ERR() check

This is a typo.  It's supposed to be IS_ERR() here instead of PTR_ERR().

Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Jarod Wilson <jarod@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: ft1000-usb: fix array overflow
Dan Carpenter [Fri, 15 Oct 2010 03:42:00 +0000 (05:42 +0200)]
Staging: ft1000-usb: fix array overflow

The code here is supposed checking if we exited the loop without hitting
a break.  The problem is that in the error handling "i" is out of bounds
and it corrupts memory when we do an info->app_info[i].nRxMsgMiss++.
>From the comments, it looks like someone noticed this corruption and
updated the code, but didn't totally fix the problem.  The correct fix
is just to remove nRxMsgMiss++ from the error path.

I believe this bug can be triggered remotely.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_dectlk.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:24:00 +0000 (19:24 -0500)]
staging: speakup: speakup_dectlk.c: style fixes

* Clean this file based on reports from checkpatch.pl.
* Replace a function-like macro with an inline function.
* Remove an incorrect use of the volatile qualifier.  The
previously-volatile variable is now always protected by a spinlock.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_txprt.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:59 +0000 (19:23 -0500)]
staging: speakup: speakup_txprt.c: style fixes

* Clean this file based on reports from checkpatch.pl.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_spkout.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:58 +0000 (19:23 -0500)]
staging: speakup: speakup_spkout.c: style fixes

* Clean this file based on reports from checkpatch.pl.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_soft.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:57 +0000 (19:23 -0500)]
staging: speakup: speakup_soft.c: style fixes

* Clean this file based on reports from checkpatch.pl.
* Make the file_operations structure const.
* Use strict_strtoul instead of simple_strtoul.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_ltlk.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:56 +0000 (19:23 -0500)]
staging: speakup: speakup_ltlk.c: style fixes

* Clean this file based on reports from checkpatch.pl.
* Remove an unused function-like macro.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_keypc.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:55 +0000 (19:23 -0500)]
staging: speakup: speakup_keypc.c: style fixes

* Clean this file based on reports from checkpatch.pl.
* Eliminate the unused synth_readable macro.
* Convert function-like macros to inline functions.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_dummy.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:54 +0000 (19:23 -0500)]
staging: speakup: speakup_dummy.c: style fixes

* Clean this file based on reports from checkpatch.pl.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_dtlk.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:53 +0000 (19:23 -0500)]
staging: speakup: speakup_dtlk.c: style fixes

* Clean this file based on reports from checkpatch.pl.
* Replace function-like macros with inline functions.
* Simplify some boolean expressions.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_decpc.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:52 +0000 (19:23 -0500)]
staging: speakup: speakup_decpc.c: style fixes

* Clean this file based on reports from checkpatch.pl.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_decext.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:51 +0000 (19:23 -0500)]
staging: speakup: speakup_decext.c: style fixes

* Clean this file based on reports from checkpatch.pl.
* Replace function-like macros with inline macros.
* Eliminate a use of the ternary operator, for readability.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_bns.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:50 +0000 (19:23 -0500)]
staging: speakup: speakup_bns.c: style fixes

* Clean this file based on reports from checkpatch.pl.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_audptr.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:49 +0000 (19:23 -0500)]
staging: speakup: speakup_audptr.c: style fixes

* Clean this file based on reports from checkpatch.pl.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_apollo.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:48 +0000 (19:23 -0500)]
staging: speakup: speakup_apollo.c: style fixes

* Clean this file based on reports from checkpatch.pl.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_acntsa.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:47 +0000 (19:23 -0500)]
staging: speakup: speakup_acntsa.c: style fixes

* Clean this file based on reports from checkpatch.pl.
* Eliminate an unused function-like macro.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: speakup_acntpc.c: style fixes
Christopher Brannon [Fri, 15 Oct 2010 00:23:46 +0000 (19:23 -0500)]
staging: speakup: speakup_acntpc.c: style fixes

* Clean this file based on reports from checkpatch.pl.
* Eliminate the unused synth_readable macro.
* Convert function-like macros to inline functions.

Signed-off-by: Christopher Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agodrivers: misc: ti-st: fix review comments
Pavan Savoy [Tue, 12 Oct 2010 20:27:38 +0000 (16:27 -0400)]
drivers: misc: ti-st: fix review comments

Based on comments from Jiri Slaby, drop the register
storage specifier, remove the unused code, cleanup
the const to non-const type casting.
Also make the line discipline ops structure static, since
its a singleton, unmodified structure which need not be
in heap.

Reported-by: Jiri Slaby <jirislaby@gmail.com>
Signed-off-by: Pavan Savoy <pavan_savoy@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove unrefrenced malloced field
mike.rapoport@gmail.com [Tue, 12 Oct 2010 22:09:13 +0000 (00:09 +0200)]
staging: brcm80211: remove unrefrenced malloced field

from 'struct osl_info' structure

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: replace MFREE with kfree
mike.rapoport@gmail.com [Tue, 12 Oct 2010 22:09:12 +0000 (00:09 +0200)]
staging: brcm80211: replace MFREE with kfree

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove unused NATIVE_{MALLOC,MFREE}
mike.rapoport@gmail.com [Tue, 12 Oct 2010 22:09:11 +0000 (00:09 +0200)]
staging: brcm80211: remove unused NATIVE_{MALLOC,MFREE}

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: replace MALLOC() with k[zm]alloc
mike.rapoport@gmail.com [Tue, 12 Oct 2010 22:09:10 +0000 (00:09 +0200)]
staging: brcm80211: replace MALLOC() with k[zm]alloc

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove osl_malloced()/MALLOCED()
mike.rapoport@gmail.com [Tue, 12 Oct 2010 22:09:09 +0000 (00:09 +0200)]
staging: brcm80211: remove osl_malloced()/MALLOCED()

because there are other means to track memory leaks in kernel

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove DHD_USE_STATIC_BUF
mike.rapoport@gmail.com [Tue, 12 Oct 2010 22:09:08 +0000 (00:09 +0200)]
staging: brcm80211: remove DHD_USE_STATIC_BUF

The DHD_USE_STATIC_BUF cannot be enabled in the build configuration,
remove it.

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: brcm80211: fix up compiler warnings I created
Greg Kroah-Hartman [Thu, 14 Oct 2010 19:26:08 +0000 (12:26 -0700)]
Staging: brcm80211: fix up compiler warnings I created

I messed up on some of the uintptr conversions, this fixes them up
properly.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove OSL_DELAY
mike.rapoport@gmail.com [Tue, 12 Oct 2010 22:09:07 +0000 (00:09 +0200)]
staging: brcm80211: remove OSL_DELAY

and use udelay and mdelay instead

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: brcmfmac: bug fix - scan result report
nohee ko [Wed, 13 Oct 2010 21:26:52 +0000 (14:26 -0700)]
staging: brcm80211: brcmfmac: bug fix - scan result report

Scan result was reported with some duplicated information.
Those were about SSID/ rate. With this patch, the duplication
can be erased.

Signed-off-by: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: brcmfmac: replace KILL_PROC with send_sig()
nohee ko [Wed, 13 Oct 2010 21:24:25 +0000 (14:24 -0700)]
staging: brcm80211: brcmfmac: replace KILL_PROC with send_sig()

logic optimization : replace KILL_PROC with send_sig()

Signed-off-by: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: Purge unused bcmwpa.[ch]
Brett Rudley [Tue, 12 Oct 2010 22:01:58 +0000 (15:01 -0700)]
staging: brcm80211: Purge unused bcmwpa.[ch]

Move two macros into wlc_mac80211.h and purge the rest.

Signed-off-by: Brett Rudley <brudley@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: fix printk format warning
Randy Dunlap [Wed, 13 Oct 2010 20:10:49 +0000 (13:10 -0700)]
staging: speakup: fix printk format warning

Fix printk format warning:

drivers/staging/speakup/serialio.c:44: warning: format '%x' expects type 'unsigned int', but argument 2 has type 'long unsigned int'

Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Cc: <speakup@braille.uwo.ca>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: octeon: Remove USB code
Bernhard Walle [Wed, 13 Oct 2010 16:38:38 +0000 (18:38 +0200)]
staging: octeon: Remove USB code

Remove functions and definitions that refer to USB since the driver
doesn't support USB and it's not planned to add USB support to that
driver. So the code is just dead code which can be removed.

Cc: David Daney <ddaney@caviumnetworks.com>
Signed-off-by: Bernhard Walle <walle@corscience.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Use specific error return value.
Marek Belisko [Thu, 14 Oct 2010 11:36:11 +0000 (13:36 +0200)]
staging: ft1000: Use specific error return value.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Get rid of using atoi.
Marek Belisko [Thu, 14 Oct 2010 12:14:33 +0000 (14:14 +0200)]
staging: ft1000: Get rid of using atoi.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: sst: Remove the aava bits
Alan Cox [Thu, 14 Oct 2010 15:59:53 +0000 (16:59 +0100)]
staging: sst: Remove the aava bits

We don't need these special cases any more.

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: sst: remove the kernel locking in ioctl
Vinod Koul [Thu, 14 Oct 2010 15:59:38 +0000 (16:59 +0100)]
staging: sst: remove the kernel locking in ioctl

kernel locking in ioctl was introduced when ioctl function was
moved from ioctl to unlocked ioctl
This is no longer required hence removed

Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: remove dead code.
Marek Belisko [Thu, 14 Oct 2010 09:41:00 +0000 (11:41 +0200)]
staging: ft1000: remove dead code.

was #ifdef so remove code which was under #else to make
code more readable.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: correct indentations for ft1000_usb.c file.
Marek Belisko [Thu, 14 Oct 2010 09:40:59 +0000 (11:40 +0200)]
staging: ft1000: correct indentations for ft1000_usb.c file.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: replace and remove c99 comments.
Marek Belisko [Thu, 14 Oct 2010 09:40:58 +0000 (11:40 +0200)]
staging: ft1000: replace and remove c99 comments.

Comments for probe, disconnect, init and exit functions
removed completely (functions names are self explaining).

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: lines over 80 characters corrected.
Marek Belisko [Thu, 14 Oct 2010 09:40:57 +0000 (11:40 +0200)]
staging: ft1000: lines over 80 characters corrected.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: various checkpatch.pl fixes.
Marek Belisko [Thu, 14 Oct 2010 09:40:56 +0000 (11:40 +0200)]
staging: ft1000: various checkpatch.pl fixes.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: iio: ad7476: fix !CONFIG_IIO_RING_BUFFER build issue reported by Randy Dunlap
Michael Hennerich [Thu, 14 Oct 2010 08:17:18 +0000 (10:17 +0200)]
staging: iio: ad7476: fix !CONFIG_IIO_RING_BUFFER build issue reported by Randy Dunlap

Fix function function decalration
Enable CONFIG_IIO_RING_BUFFER for the ad7476 driver

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: bug fix- rmmod hang problem
nohee ko [Tue, 12 Oct 2010 20:33:29 +0000 (13:33 -0700)]
staging: brcm80211: bug fix- rmmod hang problem

Bug fix - rmmod hang problem.
Can keep both of kthread & down_interruptible.
And in the meantime, can terminate the threads
properly during rmmod process.

Signed-off-by: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: brcm80211: remove sbpciregs_t typedef
Greg Kroah-Hartman [Tue, 12 Oct 2010 21:04:26 +0000 (14:04 -0700)]
Staging: brcm80211: remove sbpciregs_t typedef

don't use a typedef, use a 'struct'.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: brcm80211: util: remove unneeded use of uintptr
Greg Kroah-Hartman [Tue, 12 Oct 2010 20:20:38 +0000 (13:20 -0700)]
Staging: brcm80211: util: remove unneeded use of uintptr

uintptr shouldn't be used when you are just casting to a pointer anyway,
so remove it from these usages.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: brcm80211: sys: remove unneeded use of uintptr
Greg Kroah-Hartman [Tue, 12 Oct 2010 20:20:19 +0000 (13:20 -0700)]
Staging: brcm80211: sys: remove unneeded use of uintptr

uintptr shouldn't be used when you are just casting to a pointer anyway,
so remove it from these usages.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: brcm80211: phy: remove unneeded use of uintptr
Greg Kroah-Hartman [Tue, 12 Oct 2010 20:20:00 +0000 (13:20 -0700)]
Staging: brcm80211: phy: remove unneeded use of uintptr

uintptr shouldn't be used when you are just casting to a pointer anyway,
so remove it from these usages.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: brcm80211: include: remove unneeded use of uintptr
Greg Kroah-Hartman [Tue, 12 Oct 2010 20:18:17 +0000 (13:18 -0700)]
Staging: brcm80211: include: remove unneeded use of uintptr

uintptr shouldn't be used when you are just casting to a pointer anyway,
so remove it from these usages.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: brcm80211: bcmsdpcm: remove a bunch of unused macros
Greg Kroah-Hartman [Tue, 12 Oct 2010 20:04:58 +0000 (13:04 -0700)]
Staging: brcm80211: bcmsdpcm: remove a bunch of unused macros

These direct memory reading macros are not being used, so remove them.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: brcm80211: remove FALSE #define
Greg Kroah-Hartman [Tue, 12 Oct 2010 19:50:15 +0000 (12:50 -0700)]
Staging: brcm80211: remove FALSE #define

use the kernel provided 'false' value instead, especially
as we are using a 'bool' for these variables.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: brcm80211: remove TRUE #define
Greg Kroah-Hartman [Tue, 12 Oct 2010 19:15:18 +0000 (12:15 -0700)]
Staging: brcm80211: remove TRUE #define

use the kernel provided 'true' value instead, especially
as we are using a 'bool' for these variables.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: kobjects.c: style fixes
William Hubbs [Tue, 12 Oct 2010 16:39:30 +0000 (11:39 -0500)]
staging: speakup: kobjects.c: style fixes

clean upd this file based on reports from checkpatch.pl

Signed-off-by: William Hubbs <w.d.hubbs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: keyhelp.c: style fixes
William Hubbs [Tue, 12 Oct 2010 16:39:29 +0000 (11:39 -0500)]
staging: speakup: keyhelp.c: style fixes

Clean up this file based on reports from checkpatch.pl.

Signed-off-by: William Hubbs <w.d.hubbs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: i18n.c: style fixes
William Hubbs [Tue, 12 Oct 2010 16:39:28 +0000 (11:39 -0500)]
staging: speakup: i18n.c: style fixes

Clean up this file based on reports from checkpatch.pl.

Signed-off-by: William Hubbs <w.d.hubbs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: speakup: devsynth.c: style fixes
William Hubbs [Tue, 12 Oct 2010 16:39:27 +0000 (11:39 -0500)]
staging: speakup: devsynth.c: style fixes

clean up this file based on reports from checkpatch.pl.

Signed-off-by: William Hubbs <w.d.hubbs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: Doubled ringbuffer size for Hyper-v network driver
Hank Janssen [Tue, 12 Oct 2010 17:45:23 +0000 (10:45 -0700)]
staging: hv: Doubled ringbuffer size for Hyper-v network driver

Double the default network ringsize buffer for Hyper-V network driver.
In very heavily loaded systems the there is a chance you run out of
ringbuffer space and error out.

Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: get rid of typedef usage.
Marek Belisko [Tue, 12 Oct 2010 08:26:43 +0000 (10:26 +0200)]
staging: ft1000: get rid of typedef usage.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Remove unused/unnecessary comments.
Marek Belisko [Tue, 12 Oct 2010 08:26:42 +0000 (10:26 +0200)]
staging: ft1000: Remove unused/unnecessary comments.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000: Fix memory leak when polling fail.
Marek Belisko [Tue, 12 Oct 2010 08:26:41 +0000 (10:26 +0200)]
staging: ft1000: Fix memory leak when polling fail.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: ft1000-usb: Remove global flag DSP_loading.
Marek Belisko [Tue, 12 Oct 2010 08:26:40 +0000 (10:26 +0200)]
staging: ft1000-usb: Remove global flag DSP_loading.

This patch remove global flag DSP_loading which was checked
only on one place. Instead check return value for dsp_reload() function.

Signed-off-by: Marek Belisko <marek.belisko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: wlags49_h2: remove custom macros min() and max()
Andy Shevchenko [Tue, 12 Oct 2010 13:22:49 +0000 (16:22 +0300)]
staging: wlags49_h2: remove custom macros min() and max()

Signed-off-by: Andy Shevchenko <ext-andriy.shevchenko@nokia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: rt2860: remove custom min() and max()
Andy Shevchenko [Tue, 12 Oct 2010 13:22:48 +0000 (16:22 +0300)]
staging: rt2860: remove custom min() and max()

Signed-off-by: Andy Shevchenko <ext-andriy.shevchenko@nokia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove useless bcm_ether_ntoa()
Andy Shevchenko [Mon, 11 Oct 2010 13:58:33 +0000 (16:58 +0300)]
staging: brcm80211: remove useless bcm_ether_ntoa()

Since we have '%pM' modifier in the kernel's *printf() we don't need to
convert address to the string beforehand.

Signed-off-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Nohee Ko <noheek@broadcom.com>
Cc: Greg Kroah-Hartman <gregkh@suse.de>
Cc: linux-wireless@vger.kernel.org
Cc: devel@driverdev.osuosl.org
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove duplicated function, wf_channel2mhz
nohee ko [Tue, 12 Oct 2010 01:33:45 +0000 (18:33 -0700)]
staging: brcm80211: remove duplicated function, wf_channel2mhz

remove duplicated function, wf_channel2mhz

Signed-off-by: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: iio: adc: ad799x miscellaneous cleanups
Michael Hennerich [Tue, 12 Oct 2010 08:30:51 +0000 (10:30 +0200)]
staging: iio: adc: ad799x miscellaneous cleanups

-Simplify by using iio_alloc_pollfunc helper function
-Print in_scale in decimal format.
-Simplify ring pointer dereferences

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: iio: adc: ad7476 more list review feedback by Jonathan Cameron
Michael Hennerich [Mon, 11 Oct 2010 16:05:56 +0000 (18:05 +0200)]
staging: iio: adc: ad7476 more list review feedback by Jonathan Cameron

Print in_scale in decimal format.
Kconfig list the names of the parts in full, to ease seraching.
Use st->spi in the call as it is only used the once.

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: iio: adc: ad7476 new SPI ADC driver
Michael Hennerich [Mon, 11 Oct 2010 16:05:37 +0000 (18:05 +0200)]
staging: iio: adc: ad7476 new SPI ADC driver

New driver handling:
AD7475, AD7476, AD7477, AD7478, AD7466, AD7467, AD7468, AD7495
SPI micropower and high speed 12-/10-/8-Bit ADCs

staging: iio: adc: ad7476 apply list review feedback by Jonathan Cameron

Changes since last RFC post V1:
Mainly list review feedback by Jonathan Cameron

-Remove scan_attrs from chip info structure.
-Remove name from chip info structure, use new spi_device_id instead.
-Allow transfer buffers to live in their own cache lines, to avoid DMA/cache coherency issues.
-Move scan el code into the ring buffer file.
-Use helper function to alloc the pollfunc.
-Use regulator framework and get vref_mv from the regulator in case not specified by pdata.
-Devices with buit-in reference use vref from the chip info structure
-Don't error on missing platform_data

-Make vref_mv type unsigned short
-Print in_scale "Vref / 2^(bits)" if fractional.

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: intel_sst: include delay.h to fix build error
Randy Dunlap [Tue, 12 Oct 2010 00:47:04 +0000 (17:47 -0700)]
staging: intel_sst: include delay.h to fix build error

intel_sst_drv_interface.c uses msleep() so it needs to include delay.h:

drivers/staging/intel_sst/intel_sst_drv_interface.c:88: error: implicit declaration of function 'msleep'

Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove BCMNMIATTACHFN() macro.
Jason Cooper [Mon, 11 Oct 2010 14:03:00 +0000 (10:03 -0400)]
staging: brcm80211: remove BCMNMIATTACHFN() macro.

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove BCMUNINITFN() macro.
Jason Cooper [Mon, 11 Oct 2010 14:02:59 +0000 (10:02 -0400)]
staging: brcm80211: remove BCMUNINITFN() macro.

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove BCMINITFN() macro.
Jason Cooper [Mon, 11 Oct 2010 14:02:58 +0000 (10:02 -0400)]
staging: brcm80211: remove BCMINITFN() macro.

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove BCMINITDATA() macro.
Jason Cooper [Mon, 11 Oct 2010 14:02:57 +0000 (10:02 -0400)]
staging: brcm80211: remove BCMINITDATA() macro.

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: fix various checkpatch errors.
Jason Cooper [Mon, 11 Oct 2010 14:02:56 +0000 (10:02 -0400)]
staging: brcm80211: fix various checkpatch errors.

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: s/unsigned char/size_t/ for min_t()
Jason Cooper [Mon, 11 Oct 2010 14:02:55 +0000 (10:02 -0400)]
staging: brcm80211: s/unsigned char/size_t/ for min_t()

Avoid potential truncation.

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: remove custom string library again
Andy Shevchenko [Mon, 11 Oct 2010 09:07:35 +0000 (12:07 +0300)]
staging: brcm80211: remove custom string library again

There was a clean up commit for softmac driver. Do the same for fullmac
implementation.

Here:
 - strtoul and bcm_strtoul are changed to simple_strtoul
 - bcmstrtok -> strsep

All unused functions are deleted.

Signed-off-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: brcm80211: clean up custom ctype library again
Andy Shevchenko [Mon, 11 Oct 2010 09:07:34 +0000 (12:07 +0300)]
staging: brcm80211: clean up custom ctype library again

The similar cleanup was done before for softmac implementation.  But the
fullmac driver brings this back. Remove it again.

Signed-off-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: speakup: i18n.c: cleaned up file
Tracey Dent [Mon, 11 Oct 2010 04:00:44 +0000 (00:00 -0400)]
Staging: speakup: i18n.c: cleaned up file

Cleaned up file with the use of checkpatch.pl.

Signed-off-by: Tracey Dent <tdent48227@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>