GitHub/LineageOS/android_kernel_motorola_exynos9610.git
14 years agoStaging: strip: delete the driver
Greg Kroah-Hartman [Thu, 6 May 2010 06:13:37 +0000 (23:13 -0700)]
Staging: strip: delete the driver

It has sat in the staging directory since October of 2009, and no one
has stepped up to take it over, so odds are, no one cares about it
anymore.  So, it is now deleted as scheduled, and documented in the TODO
file.

Cc: John W. Linville <linville@tuxdriver.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: arlan: delete the driver
Greg Kroah-Hartman [Thu, 6 May 2010 06:09:12 +0000 (23:09 -0700)]
Staging: arlan: delete the driver

It has sat in the staging directory since October of 2009, and no one
has stepped up to take it over, so odds are, no one cares about it
anymore.  So, it is now deleted as scheduled, and documented in the TODO
file.

Cc: John W. Linville <linville@tuxdriver.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename VersionInfo.h to version_info.h
Greg Kroah-Hartman [Thu, 6 May 2010 05:52:28 +0000 (22:52 -0700)]
Staging: hv: rename VersionInfo.h to version_info.h

The great renaming of the hv code is now complete.

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename VmbusPacketFormat.h to vmbus_packet_format.h
Greg Kroah-Hartman [Thu, 6 May 2010 05:50:11 +0000 (22:50 -0700)]
Staging: hv: rename VmbusPacketFormat.h to vmbus_packet_format.h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename VmbusChannelInterface.h to vmbus_channel_interface.h
Greg Kroah-Hartman [Thu, 6 May 2010 05:47:13 +0000 (22:47 -0700)]
Staging: hv: rename VmbusChannelInterface.h to vmbus_channel_interface.h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename VmbusPrivate.h to vmbus_private.h
Greg Kroah-Hartman [Thu, 6 May 2010 05:45:25 +0000 (22:45 -0700)]
Staging: hv: rename VmbusPrivate.h to vmbus_private.h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename VmbusApi.h to vmbus_api.h
Greg Kroah-Hartman [Thu, 6 May 2010 05:43:02 +0000 (22:43 -0700)]
Staging: hv: rename VmbusApi.h to vmbus_api.h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename StorVscApi.h to storvsc_api.h
Greg Kroah-Hartman [Thu, 6 May 2010 05:40:43 +0000 (22:40 -0700)]
Staging: hv: rename StorVscApi.h to storvsc_api.h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename NetVscApi.h to netvsc_api.h
Greg Kroah-Hartman [Thu, 6 May 2010 05:39:15 +0000 (22:39 -0700)]
Staging: hv: rename NetVscApi.h to netvsc_api.h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename ChannelInterface.c and .h to channel_interface.c and .h
Greg Kroah-Hartman [Thu, 6 May 2010 05:36:31 +0000 (22:36 -0700)]
Staging: hv: rename ChannelInterface.c and .h to channel_interface.c and .h

All of the uppercase .c files are now gone.

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename ChannelMgmt.c and .h to channel_mgmt.c and .h
Greg Kroah-Hartman [Thu, 6 May 2010 05:34:18 +0000 (22:34 -0700)]
Staging: hv: rename ChannelMgmt.c and .h to channel_mgmt.c and .h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename RndisFilter.c and .h to rndis_filter.c and .h
Greg Kroah-Hartman [Thu, 6 May 2010 05:32:08 +0000 (22:32 -0700)]
Staging: hv: rename RndisFilter.c and .h to rndis_filter.c and .h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename StorVsc.c to storvsc.c
Greg Kroah-Hartman [Thu, 6 May 2010 05:29:27 +0000 (22:29 -0700)]
Staging: hv: rename StorVsc.c to storvsc.c

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename NetVsc.c and .h to netvsc.c and .h
Greg Kroah-Hartman [Thu, 6 May 2010 05:27:31 +0000 (22:27 -0700)]
Staging: hv: rename NetVsc.c and .h to netvsc.c and .h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename Vmbus.c to vmbus.c
Greg Kroah-Hartman [Thu, 6 May 2010 05:25:23 +0000 (22:25 -0700)]
Staging: hv: rename Vmbus.c to vmbus.c

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename BlkVsc.c to blkvsc.c
Greg Kroah-Hartman [Thu, 6 May 2010 05:24:25 +0000 (22:24 -0700)]
Staging: hv: rename BlkVsc.c to blkvsc.c

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename RingBuffer.c and .h to ring_buffer.c and .h
Greg Kroah-Hartman [Thu, 6 May 2010 05:22:35 +0000 (22:22 -0700)]
Staging: hv: rename RingBuffer.c and .h to ring_buffer.c and .h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename Connection.c to connection.c
Greg Kroah-Hartman [Thu, 6 May 2010 05:19:40 +0000 (22:19 -0700)]
Staging: hv: rename Connection.c to connection.c

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename Channel.c and .h to channel.c and .h
Greg Kroah-Hartman [Thu, 6 May 2010 05:18:38 +0000 (22:18 -0700)]
Staging: hv: rename Channel.c and .h to channel.c and .h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename Hv.h to hv.h
Greg Kroah-Hartman [Thu, 6 May 2010 05:16:10 +0000 (22:16 -0700)]
Staging: hv: rename Hv.h to hv.h

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename Hv.c to hv.c
Greg Kroah-Hartman [Thu, 6 May 2010 05:14:22 +0000 (22:14 -0700)]
Staging: hv: rename Hv.c to hv.c

No CamelCase in file names.

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: rename hyperv_utils.c to hv_utils.c
Greg Kroah-Hartman [Thu, 6 May 2010 05:10:26 +0000 (22:10 -0700)]
Staging: hv: rename hyperv_utils.c to hv_utils.c

As the module only has one .c file in it, just name the file
the same as the desired module.

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: util.h: fix up space mess again
Greg Kroah-Hartman [Thu, 6 May 2010 05:03:22 +0000 (22:03 -0700)]
Staging: hv: util.h: fix up space mess again

Again, use tabs, not spaces, it's not difficult to remember...

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: remove typedef that was just added.
Greg Kroah-Hartman [Thu, 6 May 2010 05:00:45 +0000 (22:00 -0700)]
Staging: hv: remove typedef that was just added.

It's a u64, so use a u64, it's not some special typedef.

Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: Add Time Sync feature to hv_utils module.
Haiyang Zhang [Wed, 5 May 2010 19:23:46 +0000 (19:23 +0000)]
Staging: hv: Add Time Sync feature to hv_utils module.

The Time Sync feature synchronizes guest time to host UTC time after reboot,
and restore from saved/paused state.

Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: Channel.c: fix up compiler warning
Greg Kroah-Hartman [Thu, 6 May 2010 04:04:19 +0000 (21:04 -0700)]
Staging: hv: Channel.c: fix up compiler warning

In the series of ASSERT removals, somehow we ended up with a compiler
warning in Channel.c.  This patch fixes that up.

Cc: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT in logging.h
Bill Pemberton [Wed, 5 May 2010 19:27:57 +0000 (15:27 -0400)]
staging: hv: remove ASSERT in logging.h

ASSERT is no longer used in hv, so remove the define

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT() and return -EINVAL in NetVsc.c
Bill Pemberton [Wed, 5 May 2010 19:27:56 +0000 (15:27 -0400)]
staging: hv: remove ASSERT() and return -EINVAL in NetVsc.c

return -EINVAL instead of calling ASSERT()

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: replace ASSERT() with WARN_ON() in NetVsc.c
Bill Pemberton [Wed, 5 May 2010 19:27:55 +0000 (15:27 -0400)]
staging: hv: replace ASSERT() with WARN_ON() in NetVsc.c

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s
Bill Pemberton [Wed, 5 May 2010 19:27:54 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s

These ASSERT()s serve no purpose other than for debugging.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s in vmbus_drv.c
Bill Pemberton [Wed, 5 May 2010 19:27:51 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s in vmbus_drv.c

These ASSERT()s serve no purpose other than for debugging.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s and return -EINVAL in RingBuffer.c
Bill Pemberton [Wed, 5 May 2010 19:27:50 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s and return -EINVAL in RingBuffer.c

return -EINVAL instead of calling ASSERT() for these conditionals.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: check return value of RingBufferInit()
Bill Pemberton [Wed, 5 May 2010 19:27:49 +0000 (15:27 -0400)]
staging: hv: check return value of RingBufferInit()

RingBufferInit() would always return sucess and instead relied on an
ASSERT() to test for an error condition.  Remove the ASSERT() and
return -EINVAL instead.  The return value of RingBufferInit() was also
never checked, so check it.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s in RingBuffer.c
Bill Pemberton [Wed, 5 May 2010 19:27:48 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s in RingBuffer.c

These ASSERT()s serve no purpose other than for debugging.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s in RndisFilter.c
Bill Pemberton [Wed, 5 May 2010 19:27:47 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s in RndisFilter.c

return -EINVAL instead of calling ASSERT() for these conditionals.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s in RndisFilter.c
Bill Pemberton [Wed, 5 May 2010 19:27:46 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s in RndisFilter.c

These ASSERT()s serve no purpose other than for debugging.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s in StorVsc.c
Bill Pemberton [Wed, 5 May 2010 19:27:45 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s in StorVsc.c

These ASSERT()s serve no purpose other than for debugging.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s in blkvsc_drv.c
Bill Pemberton [Wed, 5 May 2010 19:27:44 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s in blkvsc_drv.c

These ASSERT()s serve no purpose other than for debugging.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: make the block driver depend on LBDAF
Bill Pemberton [Wed, 5 May 2010 19:27:43 +0000 (15:27 -0400)]
staging: hv: make the block driver depend on LBDAF

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: return error instead calling ASSERT in blkvsc_drv.c
Bill Pemberton [Wed, 5 May 2010 19:27:42 +0000 (15:27 -0400)]
staging: hv: return error instead calling ASSERT in blkvsc_drv.c

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s in storvsc_drv.c
Bill Pemberton [Wed, 5 May 2010 19:27:41 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s in storvsc_drv.c

These ASSERT()s serve no purpose other than for debugging.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agostaging: hv: remove ASSERT()s in Channel.c
Bill Pemberton [Wed, 5 May 2010 19:27:40 +0000 (15:27 -0400)]
staging: hv: remove ASSERT()s in Channel.c

These ASSERT()s serve no purpose other than for debugging.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: return -EINVAL instead of calling ASSERT()
Bill Pemberton [Wed, 5 May 2010 19:27:39 +0000 (15:27 -0400)]
Staging: hv: return -EINVAL instead of calling ASSERT()

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: remove ASSERT() in Channel.c
Bill Pemberton [Wed, 5 May 2010 19:27:38 +0000 (15:27 -0400)]
Staging: hv: remove ASSERT() in Channel.c

VmbusChannelOpen() will now return -EINVAL if UserDataLen is too big.
Previously this was handled by an assert.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: test return value of VmbusChannelEstablishGpadl()
Bill Pemberton [Wed, 5 May 2010 19:27:37 +0000 (15:27 -0400)]
Staging: hv: test return value of VmbusChannelEstablishGpadl()

The return value of VmbusChannelEstablishGpadl() was not examined in
Channel.c

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: remove ASSERT() in Channel.c
Bill Pemberton [Wed, 5 May 2010 19:27:36 +0000 (15:27 -0400)]
Staging: hv: remove ASSERT() in Channel.c

return an error instead of calling ASSERT() if VmbusPostMessage()
fails.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: remove ASSERT() in Channel.c
Bill Pemberton [Wed, 5 May 2010 19:27:35 +0000 (15:27 -0400)]
Staging: hv: remove ASSERT() in Channel.c

check memory allocation in VmbusChannelCreateGpadlHeader() and
return -ENOMEM if it fails

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: remove ASSERT()s in Channel.c
Bill Pemberton [Wed, 5 May 2010 19:27:34 +0000 (15:27 -0400)]
Staging: hv: remove ASSERT()s in Channel.c

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: test return value of osd_WaitEventCreate()
Bill Pemberton [Wed, 5 May 2010 19:27:33 +0000 (15:27 -0400)]
Staging: hv: test return value of osd_WaitEventCreate()

The return value of osd_WaitEventCreate() was not examined in some
places.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: return correct error values in Connection.c
Bill Pemberton [Wed, 5 May 2010 19:27:32 +0000 (15:27 -0400)]
Staging: hv: return correct error values in Connection.c

Also check the kzalloc call return value.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: check return value of osd_PageAlloc()
Bill Pemberton [Wed, 5 May 2010 19:27:30 +0000 (15:27 -0400)]
Staging: hv: check return value of osd_PageAlloc()

The return value of osd_PageAlloc() was checked using an ASSERT().
Change that to more useful behaviour.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: remove ASSERT()s in ChannelMgt.c
Bill Pemberton [Wed, 5 May 2010 19:27:31 +0000 (15:27 -0400)]
Staging: hv: remove ASSERT()s in ChannelMgt.c

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: Remove check for NULL before calling kfree()
Bill Pemberton [Wed, 5 May 2010 19:27:29 +0000 (15:27 -0400)]
Staging: hv: Remove check for NULL before calling kfree()

kfree() knows how to deal with NULL, so there's no reason to check for
NULL before passing something to it.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: dt3155v4l: fix some sparse warnings
Greg Kroah-Hartman [Wed, 5 May 2010 17:45:16 +0000 (10:45 -0700)]
Staging: dt3155v4l: fix some sparse warnings

Mostly some __iomem markings and some static functions as well.

Cc: Marin Mitov <mitov@issp.bas.bg
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: dt3155v4l: Adding the missing linux/delay.h
Marin Mitov [Wed, 5 May 2010 17:31:38 +0000 (20:31 +0300)]
Staging: dt3155v4l: Adding the missing linux/delay.h

The prototypes of udelay() and msleep() are in linux/delay.h,
so include it.

Reported-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Marin Mitov <mitov@issp.bas.bg>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: vt6656: code cleanup, resolved sparse finding
Andres More [Tue, 4 May 2010 23:40:11 +0000 (20:40 -0300)]
Staging: vt6656: code cleanup, resolved sparse finding

Cleared sparse warning 'Using plain integer as NULL pointer'

Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: vt6656: code cleanup, removed OUT definition
Andres More [Tue, 4 May 2010 23:40:10 +0000 (20:40 -0300)]
Staging: vt6656: code cleanup, removed OUT definition

Remoted empty OUT define in ttype.h and its usage across the code.

Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: vt6656: removed SUCCESS define as it is not used
Andres More [Tue, 4 May 2010 23:40:09 +0000 (20:40 -0300)]
Staging: vt6656: removed SUCCESS define as it is not used

Signed-off-by: Andres More <more.andres@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: arlan: fixed unnecessary whitespace style issue in arlan-main.c
Prashant P. Shah [Wed, 5 May 2010 14:16:28 +0000 (19:46 +0530)]
Staging: arlan: fixed unnecessary whitespace style issue in arlan-main.c

This is a patch to the arlan-main.c file that fixes the unnecessary whitespace
issues found by the checkpatch.pl tool.

Signed-off-by: Prashant P. Shah <pshah.mumbai@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: cxt1e1: fix up one remaining THIS_MODULE usage
Peter Hüwe [Wed, 5 May 2010 12:21:14 +0000 (14:21 +0200)]
Staging: cxt1e1: fix up one remaining THIS_MODULE usage

Copied from original Patch by Randy Dunlap

Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: rtl8192u: Check kmalloc return value before use the buffer in ieee80211_soft...
wzt.wzt@gmail.com [Wed, 5 May 2010 06:56:52 +0000 (14:56 +0800)]
Staging: rtl8192u: Check kmalloc return value before use the buffer in ieee80211_softmac.c

Check kmalloc return value before use the buffer.

Signed-off-by: Zhitong Wang <zhitong.wangzt@alibaba-inc.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: cb_pcidas64: fix up build warnings
Greg Kroah-Hartman [Wed, 5 May 2010 16:12:13 +0000 (09:12 -0700)]
Staging: comedi: cb_pcidas64: fix up build warnings

Now that the 8255.h file is being included properly, we were
using the incorrect function prototypes, which causes a build
warning now.  This fixes it up and preserves the __iomem markings that
sparse wants to see.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: 8255: fix up previous static markings.
Greg Kroah-Hartman [Wed, 5 May 2010 16:10:31 +0000 (09:10 -0700)]
Staging: comedi: 8255: fix up previous static markings.

Ian pointed out that exported symbols should not be marked as static :)

Fixed this up by properly including the 8255.h file, and fixing
the function prototypes there, as the CONFIG variables were not getting
set so users of the header file were seeing the incorrect function
prototypes.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: line6: driver: fix up sparse warnings
Greg Kroah-Hartman [Wed, 5 May 2010 05:45:08 +0000 (22:45 -0700)]
Staging: line6: driver: fix up sparse warnings

minor stuff.

Cc: Markus Grabner <grabner@icg.tugraz.at>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: industrialio-trigger.c: minor fixups
Greg Kroah-Hartman [Wed, 5 May 2010 05:34:00 +0000 (22:34 -0700)]
Staging: iio: industrialio-trigger.c: minor fixups

We needed to include a header file that declared the functions that are
being exported in this file.

Also fix up an indentation problem, and some sparse warnings.

Cc: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: ring_sw.c: fix up sparse warnings
Greg Kroah-Hartman [Wed, 5 May 2010 05:33:27 +0000 (22:33 -0700)]
Staging: iio: ring_sw.c: fix up sparse warnings

NULL usage, static stuff, etc.

Cc: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: industrialio-ring.c: fix up sparse warnings
Greg Kroah-Hartman [Wed, 5 May 2010 05:32:43 +0000 (22:32 -0700)]
Staging: iio: industrialio-ring.c: fix up sparse warnings

Cc: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: accel: fix up some sparse warnings.
Greg Kroah-Hartman [Wed, 5 May 2010 05:32:01 +0000 (22:32 -0700)]
Staging: iio: accel: fix up some sparse warnings.

Minor stuff (static, NULL, etc.)

Cc: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: trigger: fix up some global variables
Greg Kroah-Hartman [Wed, 5 May 2010 05:31:22 +0000 (22:31 -0700)]
Staging: iio: trigger: fix up some global variables

These should be static.

Cc: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: light: tsl2563: fix static sparse warning
Greg Kroah-Hartman [Wed, 5 May 2010 05:29:41 +0000 (22:29 -0700)]
Staging: iio: light: tsl2563: fix static sparse warning

Cc: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: adc: max1363_ring.c: fix up sparse warnings
Greg Kroah-Hartman [Wed, 5 May 2010 05:28:37 +0000 (22:28 -0700)]
Staging: iio: adc: max1363_ring.c: fix up sparse warnings

Cc: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: max1363_core: fix bug in kzalloc call
Greg Kroah-Hartman [Wed, 5 May 2010 05:26:19 +0000 (22:26 -0700)]
Staging: iio: max1363_core: fix bug in kzalloc call

The operands were switched around :(

Cc: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: adc: max1363_core: fix up some sparse warnings
Greg Kroah-Hartman [Wed, 5 May 2010 05:24:05 +0000 (22:24 -0700)]
Staging: iio: adc: max1363_core: fix up some sparse warnings

Also fix a minor coding style issue.

Cc: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: light trivial whitespace fix
Jonathan Cameron [Tue, 4 May 2010 21:34:37 +0000 (22:34 +0100)]
Staging: iio: light trivial whitespace fix

Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: accelerometers trivial checkpatch related fixes
Jonathan Cameron [Tue, 4 May 2010 21:34:36 +0000 (22:34 +0100)]
Staging: iio: accelerometers trivial checkpatch related fixes

Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: iio: Trivial - remove pointless semi colon (checkpatch found)
Jonathan Cameron [Tue, 4 May 2010 21:20:54 +0000 (22:20 +0100)]
Staging: iio: Trivial - remove pointless semi colon (checkpatch found)

Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: add basic ethtool support
Stephen Hemminger [Tue, 4 May 2010 16:58:57 +0000 (09:58 -0700)]
Staging: hv: add basic ethtool support

Ethtool allows querying device information and controlling parameters.
For now just add ability to turn on/off scatter/gather.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Acked-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: add transmit flow control
Stephen Hemminger [Tue, 4 May 2010 16:58:56 +0000 (09:58 -0700)]
Staging: hv: add transmit flow control

Keep track of the number of pages sent over transmit and stop
before going over.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Acked-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: transmit scatter gather support
Stephen Hemminger [Tue, 4 May 2010 16:58:55 +0000 (09:58 -0700)]
Staging: hv: transmit scatter gather support

The transmit management of pages was confusing for handling
fragmented SKB's. (But since NETIF_F_SG was never set, the code was never hit).

The parameter AdditionalRequestPageBufferCount is always one,
(and leads to ugly code), so just inline and add comments.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Acked-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: use existing Ethernet header size
Stephen Hemminger [Tue, 4 May 2010 16:58:54 +0000 (09:58 -0700)]
Staging: hv: use existing Ethernet header size

Use ETH_ALEN to indicate that MAC address is Ethernet.
Also use Linux printk format for mac addresses.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Acked-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: ring parameter
Stephen Hemminger [Tue, 4 May 2010 16:58:53 +0000 (09:58 -0700)]
Staging: hv: ring parameter

The ring size parameter should be number of pages (not bytes).
Add module parameter information as well.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Acked-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: hv: cleanup network driver
Stephen Hemminger [Tue, 4 May 2010 16:58:52 +0000 (09:58 -0700)]
Staging: hv: cleanup network driver

Minor stuff:
   * Add module description
   * Remove variable set but never used.
   * Move variable inside conditional

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Acked-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: hwdrv_apci3xxx.c: loads of sparse cleanups
Greg Kroah-Hartman [Tue, 4 May 2010 23:14:02 +0000 (16:14 -0700)]
Staging: comedi: hwdrv_apci3xxx.c: loads of sparse cleanups

__iomem pointer fixes, and static cleanups.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: hwdrv_apci3501.h: make a variable static.
Greg Kroah-Hartman [Tue, 4 May 2010 23:13:28 +0000 (16:13 -0700)]
Staging: comedi: hwdrv_apci3501.h: make a variable static.

Cleans up the sparse warning.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: hwdrv_apci2032.c: static sparse fix
Greg Kroah-Hartman [Tue, 4 May 2010 23:12:53 +0000 (16:12 -0700)]
Staging: comedi: hwdrv_apci2032.c: static sparse fix

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: hwdrv_apci1564.c: static sparse cleanups.
Greg Kroah-Hartman [Tue, 4 May 2010 23:12:29 +0000 (16:12 -0700)]
Staging: comedi: hwdrv_apci1564.c: static sparse cleanups.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: hwdrv_apci1500.c: sparse static cleanups
Greg Kroah-Hartman [Tue, 4 May 2010 23:12:00 +0000 (16:12 -0700)]
Staging: comedi: hwdrv_apci1500.c: sparse static cleanups

Mark a bunch of variables and functions as static.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: hwdrv_apci1032.c: sparse warning fixups
Greg Kroah-Hartman [Tue, 4 May 2010 23:11:26 +0000 (16:11 -0700)]
Staging: comedi: hwdrv_apci1032.c: sparse warning fixups

Mark a variable as static.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: hwdrv_apci035.h: fix sparse warnings
Greg Kroah-Hartman [Tue, 4 May 2010 23:10:36 +0000 (16:10 -0700)]
Staging: comedi: hwdrv_apci035.h: fix sparse warnings

This let us delete two variables and mark one as static.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: hwdrv_apci035.c: fix sparse warnings
Greg Kroah-Hartman [Tue, 4 May 2010 23:09:53 +0000 (16:09 -0700)]
Staging: comedi: hwdrv_apci035.c: fix sparse warnings

Some variables should be static.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: adl_pci9111.c: fix sparse warnings
Greg Kroah-Hartman [Tue, 4 May 2010 23:09:26 +0000 (16:09 -0700)]
Staging: comedi: adl_pci9111.c: fix sparse warnings

divisor and other problems fixed.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: adl_pci9118.c: fix unsigned problem with divisors
Greg Kroah-Hartman [Tue, 4 May 2010 23:08:51 +0000 (16:08 -0700)]
Staging: comedi: adl_pci9118.c: fix unsigned problem with divisors

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: adv_pci1710.c: fix unsigned problem with divisors
Greg Kroah-Hartman [Tue, 4 May 2010 23:08:24 +0000 (16:08 -0700)]
Staging: comedi: adv_pci1710.c: fix unsigned problem with divisors

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: amplc_dio200.c: fix NULL sparse warnings
Greg Kroah-Hartman [Tue, 4 May 2010 23:07:43 +0000 (16:07 -0700)]
Staging: comedi: amplc_dio200.c: fix NULL sparse warnings

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: cb_pcidas64.c: fix sparse warnings
Greg Kroah-Hartman [Tue, 4 May 2010 23:07:03 +0000 (16:07 -0700)]
Staging: comedi: cb_pcidas64.c: fix sparse warnings

This fixes a bunch of iomem and other sparse warnings.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: das1800.c: fix a locking error on the error path.
Greg Kroah-Hartman [Tue, 4 May 2010 23:06:33 +0000 (16:06 -0700)]
Staging: comedi: das1800.c: fix a locking error on the error path.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: plx9080.h: properly mark iomem variables
Greg Kroah-Hartman [Tue, 4 May 2010 23:05:52 +0000 (16:05 -0700)]
Staging: comedi: plx9080.h: properly mark iomem variables

It's a __iomem *, so mark it as such.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: quatech_daqp_cs.c: fix up the irq
Greg Kroah-Hartman [Tue, 4 May 2010 23:05:09 +0000 (16:05 -0700)]
Staging: comedi: quatech_daqp_cs.c: fix up the irq

The irq needs to return the correct type.  Also fix up some other
sparse warnings that were found.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: addi_amcc_s5933.h: sparse cleanup
Greg Kroah-Hartman [Tue, 4 May 2010 23:04:19 +0000 (16:04 -0700)]
Staging: comedi: addi_amcc_s5933.h: sparse cleanup

Mark a variable static that does not need to be global.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 years agoStaging: comedi: addi_common.c: sparse cleanups
Greg Kroah-Hartman [Tue, 4 May 2010 23:03:35 +0000 (16:03 -0700)]
Staging: comedi: addi_common.c: sparse cleanups

Now that we are properly marking the variable as __iomem, don't
cast it.

Also fix up some other sparse warnings.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>