profile/ivi/kernel-x86-ivi.git
13 years agostaging: fix rts_pstor build errors
Randy Dunlap [Mon, 31 Jan 2011 21:11:28 +0000 (13:11 -0800)]
staging: fix rts_pstor build errors

Fix multiple rts_pstor build errors.

When CONFIG_PCI is not enabled:

drivers/staging/rts_pstor/rtsx.c: In function 'rtsx_acquire_irq':
drivers/staging/rts_pstor/rtsx.c:324: error: implicit declaration of function 'pci_intx'
drivers/staging/rts_pstor/rtsx.c: In function 'rtsx_read_pci_cfg_byte':
drivers/staging/rts_pstor/rtsx.c:336: error: implicit declaration of function 'pci_get_domain_bus_and_slot'
drivers/staging/rts_pstor/rtsx.c:336: warning: assignment makes pointer from integer without a cast
drivers/staging/rts_pstor/rtsx.c: In function 'rtsx_shutdown':
drivers/staging/rts_pstor/rtsx.c:462: error: implicit declaration of function 'pci_disable_msi'
drivers/staging/rts_pstor/rtsx.c: In function 'rtsx_probe':
drivers/staging/rts_pstor/rtsx.c:981: error: implicit declaration of function 'pci_enable_msi'

When CONFIG_SCSI is not enabled:

In file included from drivers/staging/rts_pstor/rtsx.h:45,
                 from drivers/staging/rts_pstor/rtsx.c:28:
include/scsi/scsi_cmnd.h:27:25: warning: "BLK_MAX_CDB" is not defined
include/scsi/scsi_cmnd.h:28:3: error: #error MAX_COMMAND_SIZE can not be bigger than BLK_MAX_CDB
In file included from drivers/staging/rts_pstor/rtsx.h:45,
                 from drivers/staging/rts_pstor/rtsx.c:28:
include/scsi/scsi_cmnd.h: In function 'scsi_bidi_cmnd':
include/scsi/scsi_cmnd.h:184: error: implicit declaration of function 'blk_bidi_rq'
include/scsi/scsi_cmnd.h:185: error: dereferencing pointer to incomplete type
include/scsi/scsi_cmnd.h: In function 'scsi_in':
include/scsi/scsi_cmnd.h:191: error: dereferencing pointer to incomplete type
include/scsi/scsi_cmnd.h: In function 'scsi_get_lba':  CC      drivers/gpu/drm/nouveau/nv04_tv.o

include/scsi/scsi_cmnd.h:269: error: implicit declaration of function 'blk_rq_pos'
In file included from drivers/staging/rts_pstor/rtsx.h:48,
                 from drivers/staging/rts_pstor/rtsx.c:28:
include/scsi/scsi_eh.h: At top level:
include/scsi/scsi_eh.h:84: error: 'BLK_MAX_CDB' undeclared here (not in a function)
drivers/staging/rts_pstor/rtsx.c: In function 'slave_configure':
drivers/staging/rts_pstor/rtsx.c:107: error: implicit declaration of function 'blk_queue_dma_alignment'

Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Cc: wei_wang@realsil.com.cn
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: Westbridge: fix EXPORT_SYMBOL errors reported by checkpatch.pl
Sutharsan Ramamoorthy [Wed, 26 Jan 2011 05:11:50 +0000 (21:11 -0800)]
Staging: Westbridge: fix EXPORT_SYMBOL errors reported by checkpatch.pl

This patch fixes errors reported by checkpatch.pl
in westbridge device controller driver in the staging tree.
File containing EXPORT_SYMBOL() macros for all the APIs exported
by the westbridge software has been removed. EXPORT_SYMBOL()
macros are added after the corresponding function definitions.

Signed-off-by: Sutharsan Ramamoorthy <srmt@cypress.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoRevert "staging: brcm80211: assure common sources are truly common"
Greg Kroah-Hartman [Tue, 25 Jan 2011 11:07:55 +0000 (19:07 +0800)]
Revert "staging: brcm80211: assure common sources are truly common"

This reverts commit ead1256410cb5a79fd3615ba70ba56779c5d21e2 as it
broke the build when building with multiple threads at the same time.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Roland Vossen <rvossen@broadcom.com>
Cc: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: assure common sources are truly common
Arend van Spriel [Sun, 23 Jan 2011 07:30:38 +0000 (08:30 +0100)]
staging: brcm80211: assure common sources are truly common

Common code for brcm80211 drivers was resulting in different compiled
object files for the drivers due to compilation flags. This has been
aligned so that they are resulting in same object files. Kconfig now
allows both drivers to be build simultaneously.

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: zram: simplify zram_make_request
Nitin Gupta [Sat, 22 Jan 2011 12:36:15 +0000 (07:36 -0500)]
Staging: zram: simplify zram_make_request

zram_read() and zram_write() always return zero, so make them return
void to simplify the code.

Signed-off-by: Nitin Gupta <ngupta@vflare.org>
Signed-off-by: Jerome Marchand <jmarchan@redhat.com>
Acked-by: Jeff Moyer <jmoyer@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/ste_rmi4: Remove obsolete cleanup for clientdata
Wolfram Sang [Mon, 24 Jan 2011 10:44:29 +0000 (11:44 +0100)]
staging/ste_rmi4: Remove obsolete cleanup for clientdata

A few new i2c-drivers came into the kernel which clear the clientdata-pointer
on exit or error. This is obsolete meanwhile, the core will do it.

Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
Cc: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
Cc: Linus Walleij <linus.walleij@stericsson.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rt2860: rt_linux.c: Fix space after unary '*' operator.
L. Alberto Giménez [Sun, 23 Jan 2011 00:07:18 +0000 (01:07 +0100)]
Staging: rt2860: rt_linux.c: Fix space after unary '*' operator.

Fix checkpatch error raised by the use of spaces between the '*' operator and
the corresponding variable name.

Signed-off-by: L. Alberto Giménez <agimenez@sysvalve.es>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rt2860: rt_linux.h: Fix space after unary '*' operator.
L. Alberto Giménez [Sun, 23 Jan 2011 00:07:19 +0000 (01:07 +0100)]
Staging: rt2860: rt_linux.h: Fix space after unary '*' operator.

Fix checkpatch error raised by the use of spaces between the '*' operator and
the corresponding variable name.

Signed-off-by: L. Alberto Giménez <agimenez@sysvalve.es>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rt2860: wpa.h: Fix space after unary '*' operator.
L. Alberto Giménez [Sun, 23 Jan 2011 00:07:20 +0000 (01:07 +0100)]
Staging: rt2860: wpa.h: Fix space after unary '*' operator.

Fix checkpatch error raised by the use of spaces between the '*' operator and
the corresponding variable name.

Signed-off-by: L. Alberto Giménez <agimenez@sysvalve.es>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/easycap: make ALSA ops function static in sound.c
Tomas Winkler [Mon, 24 Jan 2011 15:08:22 +0000 (17:08 +0200)]
staging/easycap: make ALSA ops function static in sound.c

1. make also ops function static
2. move around code so to avid forward declarations

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/easycap: make oss ops function static in sound_oss.c
Tomas Winkler [Mon, 24 Jan 2011 15:08:21 +0000 (17:08 +0200)]
staging/easycap: make oss ops function static in sound_oss.c

1. make oss ops function static
2. move around code so to avid forward declarations
3. move OSS ioclts from ioctl.c to sound_oss.c

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/easycap: make OSS compilation optional instead of ALSA
Tomas Winkler [Mon, 24 Jan 2011 15:08:20 +0000 (17:08 +0200)]
staging/easycap: make OSS compilation optional instead of ALSA

OSS is deprecated yet currently it is reported to be more stable
therefore we keep it but make it optional
Revert the conditional compilation:
add  CONFIG_EASYCAP_OSS and kill EASYCAP_NEEDS_ALSA
move oss-only code from easycap_sound.c to easycap_sound_oss.c

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/easycap: implement strerror function
Tomas Winkler [Mon, 24 Jan 2011 11:27:02 +0000 (13:27 +0200)]
staging/easycap: implement strerror function

Replace long switch statements that just print out errno
with strerror function. It reduces around 700 lines from the code.

The function should be probably dropped at all but leave for now
to not break currently expected debug output.

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/easycap: easycap.h ident correctly signed_div_result
Tomas Winkler [Mon, 24 Jan 2011 11:27:01 +0000 (13:27 +0200)]
staging/easycap: easycap.h ident correctly signed_div_result

indent level 1 by tabs

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/easycap: fix missing backslash in ifdef statement
Tomas Winkler [Mon, 24 Jan 2011 11:27:00 +0000 (13:27 +0200)]
staging/easycap: fix missing backslash in ifdef statement

the backslash was removed by mistake in the patch
'staging:easycap: drop redundant backslashes from the code'
this breaks compilation only when EASYCAP_NEEDS_ALSA is not set

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: easycap.h use indentation for first level
Tomas Winkler [Sat, 22 Jan 2011 23:13:58 +0000 (01:13 +0200)]
staging: easycap: easycap.h use indentation for first level

replace
struct {
int a;
}

with  more readable

struct {
int a;
}

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: make functions local to easycap_main.c static
Tomas Winkler [Sat, 22 Jan 2011 23:13:57 +0000 (01:13 +0200)]
staging: easycap: make functions local to easycap_main.c static

1. remove declarations from the header file
2. rearange code in main.c to reduce number of forward declarations

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: group module parameters handling
Tomas Winkler [Sat, 22 Jan 2011 23:13:56 +0000 (01:13 +0200)]
staging: easycap: group module parameters handling

1. For readability group module parameters handling on one place
2. Introduce kernel config option EASY_DEBUG

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: use #ifndef __EASYCAP_H_ for header inclusion protection
Tomas Winkler [Sat, 22 Jan 2011 23:13:55 +0000 (01:13 +0200)]
staging: easycap: use #ifndef __EASYCAP_H_ for header inclusion protection

use common #ifndef __EASYCAP_H_ instead of if (!defined(EASYCAP_H))
for protecting header from double inclusion

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: remove redunant headers
Tomas Winkler [Sat, 22 Jan 2011 23:13:54 +0000 (01:13 +0200)]
staging: easycap: remove redunant headers

place all globals to easycap.h, which is included
by all c-files
easycap_standard: fix declaration vs. definiton conflict

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: fix sparse warnings :conversion of int to enum
Tomas Winkler [Sat, 22 Jan 2011 23:13:53 +0000 (01:13 +0200)]
staging: easycap: fix sparse warnings :conversion of int to enum

easycap_settings.c:587:58: warning: conversion of
easycap_settings.c:587:58:     unsigned int to
easycap_settings.c:587:58:     int enum v4l2_field
easycap_settings.c:593:63: warning: conversion of
easycap_settings.c:593:63:     unsigned int to
easycap_settings.c:593:63:     int enum v4l2_colorspace

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: fix sparse warnings 'Should it be static'
Tomas Winkler [Sat, 22 Jan 2011 23:13:52 +0000 (01:13 +0200)]
staging: easycap: fix sparse warnings 'Should it be static'

easycap_main.c:41:23: warning: symbol 'easycapdc60_dongle' was not declared. Should it be static?
easycap_main.c:49:22: warning: symbol 'easycap_usb_device_id_table' was not declared. Should it be static?
easycap_main.c:69:30: warning: symbol 'easycap_fops' was not declared. Should it be static?
easycap_main.c:82:29: warning: symbol 'easycap_vm_ops' was not declared. Should it be static?
easycap_main.c:87:25: warning: symbol 'easycap_class' was not declared. Should it be static?
easycap_main.c:95:35: warning: symbol 'v4l2_fops' was not declared. Should it be static?
easycap_main.c:5071:1: warning: symbol 'easycap_module_init' was not declared. Should it be static?
easycap_main.c:5101:1: warning: symbol 'easycap_module_exit' was not declared. Should it be static?
easycap_low.c:45:50: warning: symbol 'stk1160configPAL' was not declared. Should it be static?
easycap_low.c:87:28: warning: symbol 'stk1160configNTSC' was not declared. Should it be static?
easycap_low.c:129:50: warning: symbol 'saa7113configPAL' was not declared. Should it be static?
easycap_low.c:187:28: warning: symbol 'saa7113configNTSC' was not declared. Should it be static?
easycap_ioctl.c:915:5: warning: symbol 'adjust_mute' was not declared. Should it be static?
easycap_settings.c:42:31: warning: symbol 'easycap_standard' was not declared. Should it be static?
easycap_settings.c:312:23: warning: symbol 'easycap_format' was not declared. Should it be static?
easycap_settings.c:607:23: warning: symbol 'easycap_control' was not declared. Should it be static?

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Remove unused members from struct Stats
Mike McCormack [Mon, 24 Jan 2011 14:22:21 +0000 (23:22 +0900)]
staging: rtl8192e: Remove unused members from struct Stats

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Remove unused types and defines
Mike McCormack [Mon, 24 Jan 2011 14:21:42 +0000 (23:21 +0900)]
staging: rtl8192e: Remove unused types and defines

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Delete unused and write-only struct members
Mike McCormack [Mon, 24 Jan 2011 14:21:17 +0000 (23:21 +0900)]
staging: rtl8192e: Delete unused and write-only struct members

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Delete dead code
Mike McCormack [Mon, 24 Jan 2011 14:20:40 +0000 (23:20 +0900)]
staging: rtl8192e: Delete dead code

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Delete dead code from header
Mike McCormack [Mon, 24 Jan 2011 14:20:13 +0000 (23:20 +0900)]
staging: rtl8192e: Delete dead code from header

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Remove member that's always false
Mike McCormack [Mon, 24 Jan 2011 14:19:48 +0000 (23:19 +0900)]
staging: rtl8192e: Remove member that's always false

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Remove more unused struct members
Mike McCormack [Mon, 24 Jan 2011 14:19:25 +0000 (23:19 +0900)]
staging: rtl8192e: Remove more unused struct members

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Remove dead code
Mike McCormack [Mon, 24 Jan 2011 14:18:57 +0000 (23:18 +0900)]
staging: rtl8192e: Remove dead code

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Remove unused members from r8192_priv
Mike McCormack [Mon, 24 Jan 2011 14:18:22 +0000 (23:18 +0900)]
staging: rtl8192e: Remove unused members from r8192_priv

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Remove unused code to detect struct tx rings
Mike McCormack [Mon, 24 Jan 2011 14:16:30 +0000 (23:16 +0900)]
staging: rtl8192e: Remove unused code to detect struct tx rings

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Remove unused struct members
Mike McCormack [Mon, 24 Jan 2011 14:16:10 +0000 (23:16 +0900)]
staging: rtl8192e: Remove unused struct members

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Delete dead code in ieee80211 lib
Mike McCormack [Mon, 24 Jan 2011 14:15:40 +0000 (23:15 +0900)]
staging: rtl8192e: Delete dead code in ieee80211 lib

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Tidy up function header comments
Mike McCormack [Mon, 24 Jan 2011 14:15:03 +0000 (23:15 +0900)]
staging: rtl8192e: Tidy up function header comments

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Convert cck_txbbgain_ch14_table to a table
Mike McCormack [Mon, 24 Jan 2011 14:14:16 +0000 (23:14 +0900)]
staging: rtl8192e: Convert cck_txbbgain_ch14_table to a table

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Convert cck_txbbgain_table to a table
Mike McCormack [Mon, 24 Jan 2011 14:13:47 +0000 (23:13 +0900)]
staging: rtl8192e: Convert cck_txbbgain_table to a table

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Convert txbbgain_table to a table
Mike McCormack [Mon, 24 Jan 2011 14:13:23 +0000 (23:13 +0900)]
staging: rtl8192e: Convert txbbgain_table to a table

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Delete commented out code
Mike McCormack [Mon, 24 Jan 2011 14:12:48 +0000 (23:12 +0900)]
staging: rtl8192e: Delete commented out code

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: rtl8192e: Remove unused DMESGE/W macros
Mike McCormack [Mon, 24 Jan 2011 14:12:26 +0000 (23:12 +0900)]
staging: rtl8192e: Remove unused DMESGE/W macros

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: pohmelfs/dir.c: Remove unneeded mutex_unlock() from pohmelfs_rename()
Alexey Khoroshilov [Wed, 19 Jan 2011 21:13:44 +0000 (00:13 +0300)]
Staging: pohmelfs/dir.c: Remove unneeded mutex_unlock() from pohmelfs_rename()

I do not see any reason for the mutex_unlock(&inode->i_mutex); in
pohmelfs_rename().

Found by Linux Driver Verification project (linuxtesting.org).

Signed-off-by: Alexey Khoroshilov <khoroshilov@ispras.ru>
Acked-by: Evgeniy Polyakov <zbr@ioremap.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: synaptics: Update with the kernel object name of touch device
Naveen Kumar Gaddipati [Fri, 21 Jan 2011 10:20:16 +0000 (15:50 +0530)]
staging: synaptics: Update with the kernel object name of touch device

Update with the kernel object name of touch device for getting
the regulator of the synaptics rmi4 touch device.

Signed-off-by: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: Fix some incorrect use of positive error codes.
Ralph Loader [Fri, 21 Jan 2011 06:27:53 +0000 (19:27 +1300)]
staging: Fix some incorrect use of positive error codes.

Use -E... instead of just E... in a few places where negative error
codes are expected by a functions callers.  These were found by grepping
with coccinelle & then inspecting by hand to determine which were bugs.

The staging/cxt1e1 driver appears to intentionally use positive E...
error codes in some places, and negative -E... error codes in others,
making it hard to know which is intended where - very likely I missed
some problems in that driver.

Signed-off-by: Ralph Loader <suckfish@ihug.co.nz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: cs5535_gpio: check put_user() return code
Vasiliy Kulikov [Mon, 17 Jan 2011 10:08:49 +0000 (13:08 +0300)]
staging: cs5535_gpio: check put_user() return code

put_user() may fail, if so return -EFAULT.

Signed-off-by: Vasiliy Kulikov <segoon@openwall.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: comedi: Make INSN_BITS behavior consistent across drivers
Ian Abbott [Tue, 18 Jan 2011 17:44:33 +0000 (17:44 +0000)]
staging: comedi: Make INSN_BITS behavior consistent across drivers

Most comedi hardware drivers that support the INSN_BITS instruction
ignore the base channel (specified by insn->chanspec) and assume it is
0.  The base channel is supposed to affect how the mask (in data[0]) and
bits (in data[1]) are treated.  Bit 0 applies to the base channel, bit 1
applies to base channel plus 1, etc.

For subdevices with no more than 32 channels, this patch modifies the
chanspec and data before presenting it to the hardware driver, and
modifies the data bits read back by the hardware driver (into data[1]).
This makes it appear to the hardware driver that the base channel was
set to 0.

For subdevices with more than 32 channels, the instruction is left
unmodified, as it is assumed that the hardware driver takes note of the
base channel in this case in order to provide access beyond channel 31.

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: FT1000: remove duplicate inc of linux/slab.h
Jesper Juhl [Mon, 10 Jan 2011 22:23:10 +0000 (23:23 +0100)]
Staging: FT1000: remove duplicate inc of linux/slab.h

linux/slab.h is included twice in ft1000_dnld.c - remove duplicate.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/vt6655: fix sparse warning "obsolete struct initializer"
Peter Huewe [Mon, 10 Jan 2011 22:28:06 +0000 (23:28 +0100)]
staging/vt6655: fix sparse warning "obsolete struct initializer"

This patch fixes the sparse warnings
"obsolete struct initializer, use C99 syntax" in vt6655/device_main.c
by converting the struct to C99 syntax

KernelVersion: linux-next-20110110

Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/comedi/icp_multi: fix sparse warning "obsolete struct initializer"
Peter Huewe [Mon, 10 Jan 2011 22:18:33 +0000 (23:18 +0100)]
staging/comedi/icp_multi: fix sparse warning "obsolete struct initializer"

This patch fixes the sparse warnings
"obsolete struct initializer, use C99 syntax" in icp_multi.c
by converting the struct to C99 syntax

KernelVersion: linux-next-20110110

Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging/comedi/me4000: fix sparse warning "obsolete struct initializer"
Peter Huewe [Mon, 10 Jan 2011 22:07:16 +0000 (23:07 +0100)]
staging/comedi/me4000: fix sparse warning "obsolete struct initializer"

This patch fixes the sparse warnings in me4000.c:
me4000.c:122:1: warning: obsolete struct initializer, use C99 syntax
me4000.c:123:1: warning: obsolete struct initializer, use C99 syntax
me4000.c:124:1: warning: obsolete struct initializer, use C99 syntax
me4000.c:125:1: warning: obsolete struct initializer, use C99 syntax
by converting the struct to use C99 syntax

Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: replace STRINGIZE with __stringify()
Tomas Winkler [Tue, 18 Jan 2011 22:24:10 +0000 (00:24 +0200)]
staging: easycap: replace STRINGIZE with __stringify()

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: kill EASYCAP_NEEDS_CARD_CREATE
Tomas Winkler [Tue, 18 Jan 2011 22:24:09 +0000 (00:24 +0200)]
staging: easycap: kill EASYCAP_NEEDS_CARD_CREATE

for in-tree driver we can use snd_card_create
for backports to older kernels this can be easily wrapped

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: drop redunant backslashes from the code
Tomas Winkler [Tue, 18 Jan 2011 22:24:06 +0000 (00:24 +0200)]
staging: easycap: drop redunant backslashes from the code

remove \ from the code where C syntex doesnt require it

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: easycap: fix sparse warnings for module parameters
Tomas Winkler [Tue, 18 Jan 2011 12:03:23 +0000 (14:03 +0200)]
Staging: easycap: fix sparse warnings for module parameters

easycap_main.c:34:5: warning: symbol 'easycap_debug' was not declared. Should it be static?
easycap_main.c:36:5: warning: symbol 'easycap_gain' was not declared. Should it be static?

These two variables actually were declared in several places.
The variables are used in several files.
I've fixed "easycap_debug" so it gets declared in one place only and included properly.
For "easycap_gain" made it static and I created added a
->gain member to the easycap struct.  This seems cleaner than using a
global variable and later on we may make this controlable via sysfs.

Cc:Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Acked-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: easycap: add ALSA support
Mike Thomas [Mon, 10 Jan 2011 18:41:11 +0000 (18:41 +0000)]
staging: easycap: add ALSA support

This is necessary because some distributions are disabling OSS entirely.

Signed-off-by: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: r8712u: Firmware changes for driver
Larry Finger [Fri, 14 Jan 2011 21:02:18 +0000 (15:02 -0600)]
staging: r8712u: Firmware changes for driver

* select FW_LOADER in Kconfig - From: Stefan Lippers-Hollmann <s.l-h@gmx.de>
* declare MODULE_FIRMWARE for r8712u and change to correct directory
* delete 10K line farray.h containing internal firmware

Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: r8712u: Switch driver to use external firmware from linux-firmware
Larry Finger [Fri, 14 Jan 2011 20:54:18 +0000 (14:54 -0600)]
staging: r8712u: Switch driver to use external firmware from linux-firmware

Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: r8712u: Fix sparse message
Larry Finger [Fri, 14 Jan 2011 20:54:13 +0000 (14:54 -0600)]
staging: r8712u: Fix sparse message

Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: iio: add driver for MAX517/518/519
Roland Stigge [Wed, 12 Jan 2011 10:41:59 +0000 (11:41 +0100)]
Staging: iio: add driver for MAX517/518/519

IIO Driver for Maxim MAX517, MAX518 and MAX519 DAC

Signed-off-by: Roland Stigge <stigge@antcom.de>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: easycap: Make easycap_debug non-static
Mike Thomas [Mon, 10 Jan 2011 18:34:41 +0000 (18:34 +0000)]
Staging: easycap: Make easycap_debug non-static

The parameter easycap_debug appears in macros JOT and JOM and therefore
needs to be visible from all source files.  The easycap_ prefix should
be sufficient to avoid namespace clashes outside the module.

Signed-off-by: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agortl8192u_usb: Remove built-in firmware images
Ben Hutchings [Sun, 9 Jan 2011 04:20:04 +0000 (04:20 +0000)]
rtl8192u_usb: Remove built-in firmware images

These firmware images are already unused.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agovt6656: Use request_firmware() to load firmware
Ben Hutchings [Sun, 9 Jan 2011 04:16:48 +0000 (04:16 +0000)]
vt6656: Use request_firmware() to load firmware

The file added to linux-firmware is a copy of the current array which
does not have a recognisable header, so no validation is done.

Change the firmware version check to accept newer versions.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: comedi : Analog input trigerring modes for cb_pcidas
Brice Dubost [Fri, 7 Jan 2011 17:49:18 +0000 (18:49 +0100)]
staging: comedi : Analog input trigerring modes for cb_pcidas

This patch allows the possibility to choose between edgre triggering
and level trigerring, for the analog input, on the Measurement
Computing PCI-DAS* boards

Signed-off-by: Brice Dubost <braice@braice.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: keucr: Delete use kernel strcmp() & strcpy() from TODO file
Javier Martinez Canillas [Sun, 2 Jan 2011 22:14:29 +0000 (23:14 +0100)]
staging: keucr: Delete use kernel strcmp() & strcpy() from TODO file

staging: keucr: Delete use kernel strcmp() & strcpy() from TODO file

Signed-off-by: Javier Martinez Canillas <martinez.javier@gmail.com>
Acked-by: Dan Carpenter <error27@gmail.com>
Reviewed-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Cc: Al Cho <acho@novell.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: keucr: Delete StringCmp() and StringCopy custom functions
Javier Martinez Canillas [Sun, 2 Jan 2011 22:14:28 +0000 (23:14 +0100)]
staging: keucr: Delete StringCmp() and StringCopy custom functions

staging: keucr: Delete StringCmp() and StringCopy custom functions

Signed-off-by: Javier Martinez Canillas <martinez.javier@gmail.com>
Acked-by: Dan Carpenter <error27@gmail.com>
Reviewed-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Cc: Al Cho <acho@novell.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: keucr: Use memcpy() instead custom StringCopy() and some style cleanups
Javier Martinez Canillas [Sun, 2 Jan 2011 22:14:27 +0000 (23:14 +0100)]
staging: keucr: Use memcpy() instead custom StringCopy() and some style cleanups

staging: keucr: Use memcpy() instead custom StringCopy() and some style cleanups

Signed-off-by: Javier Martinez Canillas <martinez.javier@gmail.com>
Acked-by: Dan Carpenter <error27@gmail.com>
Reviewed-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Cc: Al Cho <acho@novell.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: keucr: Use memcmp() instead custom StringCmp() and some style cleanups
Javier Martinez Canillas [Sun, 2 Jan 2011 22:14:26 +0000 (23:14 +0100)]
staging: keucr: Use memcmp() instead custom StringCmp() and some style cleanups

staging: keucr: Use memcmp() instead custom StringCmp() and some style cleanups

Signed-off-by: Javier Martinez Canillas <martinez.javier@gmail.com>
Acked-by: Dan Carpenter <error27@gmail.com>
Reviewed-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Cc: Al Cho <acho@novell.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: tm6000: check usb_alloc_urb() return
Dan Carpenter [Mon, 3 Jan 2011 05:44:38 +0000 (08:44 +0300)]
Staging: tm6000: check usb_alloc_urb() return

usb_alloc_urb() can return NULL so check for that and return
-ENOMEM if it happens.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: tm6000: silence Sparse warning "dubious: !x | !y"
Dan Carpenter [Mon, 3 Jan 2011 05:47:40 +0000 (08:47 +0300)]
Staging: tm6000: silence Sparse warning "dubious: !x | !y"

Bitwise and logical or are the equivalent here, so this doesn't affect
runtime, but logical or was intended.  The original code causes a
warning in Sparse:  "warning: dubious: !x | !y"

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: spectra: don't read past array in Conv_Spare_Data_Log2Phy_Format()
roel kluin [Mon, 3 Jan 2011 19:59:14 +0000 (11:59 -0800)]
staging: spectra: don't read past array in Conv_Spare_Data_Log2Phy_Format()

It should decrement or we read past the array

Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: sep: update driver to SEP version 3.4.5
Mark Allyn [Tue, 4 Jan 2011 22:16:59 +0000 (14:16 -0800)]
staging: sep: update driver to SEP version 3.4.5

These changes enable the driver to work with SEP
version 3.4.5

Major change is to use non DMA access for any data
comming from a function that uses the external application
service on the SEP.

Signed-off-by: Mark Allyn <mark.a.allyn@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: sep: Remove un-needed debug prints
Mark Allyn [Tue, 4 Jan 2011 20:57:16 +0000 (12:57 -0800)]
staging: sep: Remove un-needed debug prints

Signed-off-by: Mark Allyn <mark.a.allyn@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: sep: Add comment to TODO to clean up un-needed debug prints
Mark Allyn [Tue, 4 Jan 2011 20:56:27 +0000 (12:56 -0800)]
staging: sep: Add comment to TODO to clean up un-needed debug prints

Signed-off-by: Mark Allyn <mark.a.allyn@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: add rts_pstor for Realtek PCIE cardreader
wwang [Fri, 21 Jan 2011 09:39:18 +0000 (17:39 +0800)]
staging: add rts_pstor for Realtek PCIE cardreader

rts_pstor is used to support Realtek PCI-E card readers,
including rts5209, rts5208, Barossa.

Signed-off-by: wwang <wei_wang@realsil.com.cn>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: hv: hv_utils depends on CONNECTOR
Randy Dunlap [Fri, 21 Jan 2011 17:18:56 +0000 (09:18 -0800)]
staging: hv: hv_utils depends on CONNECTOR

Don't build hv_utils when CONFIG_CONNECTOR is not enabled.
Fixes these build errors:

ERROR: "cn_add_callback" [drivers/staging/hv/hv_utils.ko] undefined!
ERROR: "cn_del_callback" [drivers/staging/hv/hv_utils.ko] undefined!
ERROR: "cn_netlink_send" [drivers/staging/hv/hv_utils.ko] undefined!

Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Greg Kroah-Hartman <gregkh@suse.de>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused type definitions from driver
Arend van Spriel [Fri, 21 Jan 2011 10:20:16 +0000 (11:20 +0100)]
staging: brcm80211: remove unused type definitions from driver

Quite some definitions are not referenced in the drivers sources and
clutter up the files so they are removed.

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused function from wlc_stf.c
Arend van Spriel [Fri, 21 Jan 2011 10:20:15 +0000 (11:20 +0100)]
staging: brcm80211: remove unused function from wlc_stf.c

Working through a list of unused functions in the driver tree.
This file has following redundant function(s):

wlc_stf_stbc_rx_get
wlc_stf_rxchain_set

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: removed unused inline function from wlc_ampdu.c
Arend van Spriel [Fri, 21 Jan 2011 10:20:14 +0000 (11:20 +0100)]
staging: brcm80211: removed unused inline function from wlc_ampdu.c

This file defined an inline function pkt_txh_seqnum() which was not
used and as such is removed.

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused function from wlc_event.c
Arend van Spriel [Fri, 21 Jan 2011 10:20:13 +0000 (11:20 +0100)]
staging: brcm80211: remove unused function from wlc_event.c

Working through a list of unused functions in the driver tree.
This file has following redundant function(s):

wlc_eventq_next
wlc_eventq_cnt

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused function from wlc_channel.c
Arend van Spriel [Fri, 21 Jan 2011 10:20:12 +0000 (11:20 +0100)]
staging: brcm80211: remove unused function from wlc_channel.c

Working through a list of unused functions in the driver tree.
This file has following redundant function(s):

wlc_channel_country_abbrev
wlc_channel_locale_flags
wlc_channel_get_chanvec
wlc_valid_40chanspec_in_band
wlc_channel_set_txpower_limit
wlc_valid_chanspec

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused function from wlc_bmac.c
Arend van Spriel [Fri, 21 Jan 2011 10:20:11 +0000 (11:20 +0100)]
staging: brcm80211: remove unused function from wlc_bmac.c

Working through a list of unused functions in the driver tree.
This file has following redundant function(s):

wlc_bmac_set_hw_etheraddr
wlc_cur_phy
wlc_bmac_revinfo_get
wlc_bmac_set_deaf
wlc_bmac_xmtfifo_sz_set
wlc_bmac_ifsctl_edcrs_set
wlc_bmac_set_ucode_loaded
wlc_bmac_set_clk
wlc_gpio_fast_deinit
wlc_bmac_radio_hw
wlc_bmac_set_txpwr_percent

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused functions from sbutils.c
Arend van Spriel [Fri, 21 Jan 2011 10:20:10 +0000 (11:20 +0100)]
staging: brcm80211: remove unused functions from sbutils.c

Cleaning up unused function from the driver sources. This
file contained the following unused functioin(s):

sb_base
sb_taclear
sb_serr_clear

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused inline funtion from siutils.h
Arend van Spriel [Fri, 21 Jan 2011 10:20:09 +0000 (11:20 +0100)]
staging: brcm80211: remove unused inline funtion from siutils.h

The header file contains a inline function, but it is not used by
the driver sources:

si_seci_init

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused function from dhd_common.c
Arend van Spriel [Fri, 21 Jan 2011 10:20:08 +0000 (11:20 +0100)]
staging: brcm80211: remove unused function from dhd_common.c

Working through a list of unused functions in the driver tree.
This file has following redundant function(s):

dhd_store_conn_status
print_buf
wl_event_to_host_order

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused function from dhd_cdc.c
Arend van Spriel [Fri, 21 Jan 2011 10:20:07 +0000 (11:20 +0100)]
staging: brcm80211: remove unused function from dhd_cdc.c

Working through a list of unused functions in the driver tree.
This file has following redundant function(s):

dhd_proto_fcinfo

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: remove unused function from bcmwifi.c
Arend van Spriel [Fri, 21 Jan 2011 10:20:06 +0000 (11:20 +0100)]
staging: brcm80211: remove unused function from bcmwifi.c

Working through a list of unused functions in the driver tree.
This file has following redundant function(s):

wf_chspec_ctlchspec

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: use KBUILD_MODNAME as driver name in registration
Arend van Spriel [Fri, 21 Jan 2011 09:54:52 +0000 (10:54 +0100)]
staging: brcm80211: use KBUILD_MODNAME as driver name in registration

The driver name was hardcoded and not same as the kernel module file
being build. Although there may be no strong requirement to this it
may provide increased consistency.

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: removed sys directory layer from brcmsmac driver
Arend van Spriel [Fri, 21 Jan 2011 09:54:50 +0000 (10:54 +0100)]
staging: brcm80211: removed sys directory layer from brcmsmac driver

Based on review comments moved sources from brcm80211/brcmsmac/sys
to its parent directory. The phy directory is kept for maintainance
logistics around phy source code.

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: cleanup on the brcm80211 include directory
Arend van Spriel [Fri, 21 Jan 2011 09:54:49 +0000 (10:54 +0100)]
staging: brcm80211: cleanup on the brcm80211 include directory

moved several files to specific source directory as these do not
need to be shared between drivers. Also removed some unused include
files from the include directory.

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agostaging: brcm80211: moved code around for cleanup
Arend van Spriel [Fri, 21 Jan 2011 09:54:48 +0000 (10:54 +0100)]
staging: brcm80211: moved code around for cleanup

Restructured code to have more consistent directory tree for the
two drivers.

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Remove pointless returns
Mike McCormack [Mon, 27 Dec 2010 12:53:24 +0000 (21:53 +0900)]
Staging: rtl8192e: Remove pointless returns

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Fix typo in enum name
Mike McCormack [Mon, 27 Dec 2010 12:53:05 +0000 (21:53 +0900)]
Staging: rtl8192e: Fix typo in enum name

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Make arrays const
Mike McCormack [Mon, 27 Dec 2010 12:52:52 +0000 (21:52 +0900)]
Staging: rtl8192e: Make arrays const

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Remove commented out printks
Mike McCormack [Mon, 27 Dec 2010 12:52:35 +0000 (21:52 +0900)]
Staging: rtl8192e: Remove commented out printks

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Remove cast in request_irq
Mike McCormack [Mon, 27 Dec 2010 12:51:09 +0000 (21:51 +0900)]
Staging: rtl8192e: Remove cast in request_irq

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Remove commented debugging code
Mike McCormack [Mon, 27 Dec 2010 12:50:47 +0000 (21:50 +0900)]
Staging: rtl8192e: Remove commented debugging code

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Remove unused RT_DEBUG_DATA macro
Mike McCormack [Mon, 27 Dec 2010 12:50:33 +0000 (21:50 +0900)]
Staging: rtl8192e: Remove unused RT_DEBUG_DATA macro

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Remove assert macro
Mike McCormack [Mon, 27 Dec 2010 12:50:14 +0000 (21:50 +0900)]
Staging: rtl8192e: Remove assert macro

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Convert cpMacAddr macro to inline function
Mike McCormack [Mon, 27 Dec 2010 12:49:49 +0000 (21:49 +0900)]
Staging: rtl8192e: Convert cpMacAddr macro to inline function

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Use compare_ether_addr instead of eqMacAddr
Mike McCormack [Mon, 27 Dec 2010 12:48:32 +0000 (21:48 +0900)]
Staging: rtl8192e: Use compare_ether_addr instead of eqMacAddr

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
13 years agoStaging: rtl8192e: Dump step we fail in init_firmware()
Mike McCormack [Mon, 27 Dec 2010 12:49:28 +0000 (21:49 +0900)]
Staging: rtl8192e: Dump step we fail in init_firmware()

Signed-off-by: Mike McCormack <mikem@ring3k.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>