]>
git.openfabrics.org - ~shefty/rdma-dev.git/log
Arend van Spriel [Fri, 16 Sep 2011 19:28:06 +0000 (12:28 -0700)]
staging: brcm80211: remove global variable from bcmsdh_sdmmc.c
The source file contained some static global variables that were
needed per device handled by the driver. They have moved to the
struct brcmf_sdio_dev for that reason.
Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Arend van Spriel [Fri, 16 Sep 2011 19:28:05 +0000 (12:28 -0700)]
staging: brcm80211: remove for_each_bss() macro from brcmfmac driver
The macro was only used once in the code and has been removed for
clarity reading the code.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Arend van Spriel [Fri, 16 Sep 2011 19:28:04 +0000 (12:28 -0700)]
staging: brcm80211: remove brcmfmac interface name override
The brcmfmac had a module parameter which could be used to override
the name of the wireless net device name and it defaults to 'wlan'.
This is considered rather useless, because what is in a name.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Franky Lin [Fri, 16 Sep 2011 19:28:03 +0000 (12:28 -0700)]
staging: brcm80211: remove fullmac module_param for ARP offload feature
Use macro to replace global variable for ARP offload configuration
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Franky Lin [Fri, 16 Sep 2011 19:28:02 +0000 (12:28 -0700)]
staging: brcm80211: remove _brcmf_sysioc_thread in fullmac
_brcmf_sysioc_thread is handling deferred jobs from 4 callers. Use
work queue to handle set_mac_address/set_multicast_list requests.
brcmf_add_if/brcmf_del_if never run in atomic context and can call
brcmf_op_if directly.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Fri, 16 Sep 2011 19:28:01 +0000 (12:28 -0700)]
staging: brcm80211: fixed sparse endianness warnings on fullmac ratespec
Structure that is sent to the dongle has been annotated.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Fri, 16 Sep 2011 19:28:00 +0000 (12:28 -0700)]
staging: brcm80211: fixed sparse endianness warnings on fullmac scb pars
Structure that is sent to the dongle has been annotated.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Fri, 16 Sep 2011 19:27:59 +0000 (12:27 -0700)]
staging: brcm80211: fixed sparse endianness warnings on fullmac assoc pars
Structures that are sent to the dongle have been annotated.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Fri, 16 Sep 2011 19:27:58 +0000 (12:27 -0700)]
staging: brcm80211: fixed sparse endianness warnings on fullmac scan pars
Three structures that are sent to the dongle have been annotated for correct
endianness: struct brcmf_iscan_params, brcmf_scan_params,
brcmf_cfg80211_scan_req.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Fri, 16 Sep 2011 19:27:57 +0000 (12:27 -0700)]
staging: brcm80211: removed set_fs/get_fs/get_ds calls from brcmf_dev_ioctl()
brcmf_dev_ioctl() does not provide a user space buffer to
brcmf_netdev_ioctl_priv. Hence these calls are not necessary.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Fri, 16 Sep 2011 19:27:56 +0000 (12:27 -0700)]
staging: brcm80211: simplified brcmf_proto_ioctl()
Removed a function parameter since it was redundant.
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Franky Lin [Fri, 16 Sep 2011 19:27:55 +0000 (12:27 -0700)]
staging: brcm80211: remove global variable brcmf_radio_up from fullmac
Remove unnecessary global variable
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Franky Lin [Fri, 16 Sep 2011 19:27:54 +0000 (12:27 -0700)]
staging: brcm80211: remove global variable brcmf_roam from fullmac
Remove unnecessary global variable
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Franky Lin [Fri, 16 Sep 2011 19:27:53 +0000 (12:27 -0700)]
staging: brcm80211: remove fullmac module_param for packet filter feature
Use constant values for packet filter configuration
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Franky Lin [Fri, 16 Sep 2011 19:27:52 +0000 (12:27 -0700)]
staging: brcm80211: remove fullmac module_param brcmf_deferred_tx
Remove brcmf_deferred_tx as it should be always 1
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Franky Lin [Fri, 16 Sep 2011 19:27:51 +0000 (12:27 -0700)]
staging: brcm80211: remove fullmac module_param sd_f2_blocksize
Replaced the global variable with macro.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Franky Lin [Fri, 16 Sep 2011 19:27:50 +0000 (12:27 -0700)]
staging: brcm80211: remove SOFTAP code from fullmac
Dead code removal.
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Julia Lawall [Fri, 16 Sep 2011 06:57:33 +0000 (08:57 +0200)]
drivers/staging/mei/interface.c: take size of pointed value, not pointer
Sizeof a pointer-typed expression returns the size of the pointer, not that
of the pointed data.
The semantic patch that fixes this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@
expression *e;
type T;
identifier f;
@@
f(...,(T)e,...,
-sizeof(e)
+sizeof(*e)
,...)
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:17 +0000 (14:15 +0800)]
Staging: cx25821: cx25821-video.h: Fix long lines
Fix long function prototypes and line up arguments nicely.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:16 +0000 (14:15 +0800)]
Staging: cx25821: cx25821-video.h: Fix indent
Fix indent of function arguments.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:15 +0000 (14:15 +0800)]
Staging: cx25821: cx25821-video.h: Line up constant
Line up integer constant with others.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:14 +0000 (14:15 +0800)]
Staging: cx25821: Replace :? by if-else in cx25821-video.c
Replace :? operator by equivalent if-else statement where in improves
readability. Don't add else branch if it is not needed.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:13 +0000 (14:15 +0800)]
Staging: cx25821: Fix braces around statements in cx25821-video.c
Delete braces around single statements.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:12 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 35
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:11 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 34
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:10 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 33
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:09 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 32
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:08 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 31
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:07 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 30
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:06 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 29
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:05 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 28
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:04 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 27
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:03 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 26
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:02 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 25
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:01 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 24
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:15:00 +0000 (14:15 +0800)]
Staging: cx25821: Change indent with spaces to tabs 23
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:59 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 22
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:58 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 21
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:57 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 20
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:56 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 19
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:55 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 18
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:54 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 17
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:53 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 16
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:52 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 15
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:51 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 14
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:50 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 13
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:49 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 12
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:48 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 11
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:47 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 10
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:46 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 9
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:45 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 8
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:44 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 7
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:43 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 6
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:42 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 5
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:41 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 4
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:40 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 3
Change indent with spaces to tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:39 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 2
Change indent with spaces to use tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:38 +0000 (14:14 +0800)]
Staging: cx25821: Change indent with spaces to tabs 1
Changed indent with spaces to use tabs.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:37 +0000 (14:14 +0800)]
Staging: cx25821: Fix indent in comment
Change indent in comment to use tabs instead of spaces.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:36 +0000 (14:14 +0800)]
Staging: cx25821: Fix obscure line endings
Fix obscure line endings to less obscure ones. Rationale: readability.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:35 +0000 (14:14 +0800)]
Staging: cx25821: Fix function signatures
Fix long function signatures (over 80 characters) and indent them
properly.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Leonid V. Fedorenchik [Fri, 16 Sep 2011 06:14:34 +0000 (14:14 +0800)]
Staging: cx25821: Fix labels' placement in cx25821-video.c
Delete whitespace before labels.
Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:57 +0000 (09:49 +0200)]
staging: brcm80211: introduced fullmac function get/set u32 to/from dongle
Code cleanup. Avoids repetition of same code fragment.
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:56 +0000 (09:49 +0200)]
staging: brcm80211: fullmac sparse endianness encryption keys check
Added support by making a distinction between a struct and its little
endian relative.
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:55 +0000 (09:49 +0200)]
staging: brcm80211: cleaned up function brcmf_cfg80211_get_key()
Unnecessary endianess conversion and struct member setting was deleted.
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Thu, 15 Sep 2011 08:06:34 +0000 (10:06 +0200)]
staging: brcm80211: simplification of brcmf_netdev_ioctl_priv()
This function is only called from within the driver. Because of that
certain simplifications (not required to lock down user mem pages,
not required to check user permissions) could be performed.
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:53 +0000 (09:49 +0200)]
staging: brcm80211: separated public from private ioctl functions
net_device ioctl handler was called both by the OS as by the driver
itself. Split the ioctl handler into two functions to make code paths
more clear.
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:52 +0000 (09:49 +0200)]
staging: brcm80211: macro cleanup in softmac rate.h
Substituted/moved/deleted macro's.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Arend van Spriel [Tue, 13 Sep 2011 07:49:51 +0000 (09:49 +0200)]
staging: brcm80211: remove brcms_c_get_par and set_par functions
The source main.c provided an interface function that handled several
things demultiplexing on given identifier. This interface has been
replaced by separate functions to make it more straightforward.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Arend van Spriel [Tue, 13 Sep 2011 07:49:50 +0000 (09:49 +0200)]
staging: brcm80211: remove devpath related nvram variable lookup
The lookup function used for the devpath related functions did
not provide a variable table so the lookup would always fail. As
this makes the functions pretty useless these have been removed.
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:49 +0000 (09:49 +0200)]
staging: brcm80211: added clarification on softmac dma alignment
Descriptor ring can only start at 8KB alignment, this requirement
is dictated by DMA hardware.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:48 +0000 (09:49 +0200)]
staging: brcm80211: macro cleanup in softmac main.c
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:47 +0000 (09:49 +0200)]
staging: brcm80211: removed function brcms_c_wme_initparams_sta()
since its variable assignment can be done in a more straightforward
manner.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:46 +0000 (09:49 +0200)]
staging: brcm80211: cleanup of softmac macro EDCF_ENAB()
Since structure member pub->_wme is set to AUTO early in init, macro
EDCF_ENAB is always 'true' after that. All code that uses ECDF_ENAB()
appears in the code flow after the init.
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:45 +0000 (09:49 +0200)]
staging: brcm80211: macro cleanup in softmac dma
Substituted macro's by functions. Renamed several struct members in dma.c
to get rid of a set of macro's.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Roland Vossen [Tue, 13 Sep 2011 07:49:44 +0000 (09:49 +0200)]
staging: brcm80211: fix for fullmac build problem
Fullmac had a build error for CONFIG_PM_SLEEP=n
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Arend van Spriel [Tue, 13 Sep 2011 07:49:43 +0000 (09:49 +0200)]
staging: brcm80211: remove function ieee_set_channel()
The function ieee_set_channel() provides very little functionality
so it has been removed.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Arend van Spriel [Thu, 15 Sep 2011 08:06:21 +0000 (10:06 +0200)]
staging: brcm80211: remove ioctl layer from brcmsmac
The ioctl layer in brcmsmac only provided an interface layer that
was internal to the driver. This is considered pointless and has
been removed.
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Franky Lin [Tue, 13 Sep 2011 07:49:41 +0000 (09:49 +0200)]
staging: brcm80211: remove static function declaration in dhd_sdio
Reshuffle function order in dhd_sdio of fullmac to get rid of
static function declaration
Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:57 +0000 (00:20 +0100)]
staging: wlags49_h2: Remove continuous tx test
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:56 +0000 (00:20 +0100)]
staging: wlags49_h2: Reformat TX_CNTL comment
This tabular comment is useful, but the original tabbed layout
is for a tab-wdth of 4. Reformat with spaces.
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:55 +0000 (00:20 +0100)]
staging: wlags49_h2: Remove HCF_EXT_MB from code.
The HII and HII.5 configurations both use the mailboxes.
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:54 +0000 (00:20 +0100)]
staging: wlags49_h2: Remove CKIP support from hcf
CKIP is a Cisco proprietary alternative to TKIP. Support removed to
simplify code - the defines to interact with the firmware are left in
place.
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:53 +0000 (00:20 +0100)]
staging: wlags49_h2: Remove MUL/DIV_BY_2
Only used in one place, so just inline and get rid of the macro.
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:52 +0000 (00:20 +0100)]
staging: wlags49_h2: Remove references to SSN
SSN seems to be an old abbreviation for WPA. Remove all references.
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:51 +0000 (00:20 +0100)]
staging: wlags49_h2: Remove some unused defines
HCF_INT_OFF, _TCHAR_DEFINED, HCF_LITTLE_ENDIAN, HCF_TALLIES_EXTRA
HCF_ENTRY, HCF_EXIT, OUTPUTDEBUGMSG, ASSERTDEBUGMSG
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:50 +0000 (00:20 +0100)]
staging: wlags49_h2: Simplify HCF_ENCAP usage
Encapsulation should always be enabled. Support is either via firmware
or in the driver, so we need to keep the macro. The driver can stop
exporting hcf_encap
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:49 +0000 (00:20 +0100)]
staging: wlags49_h2: Remove HCF_DL_ONLY
This is supposed to profile the code so that you can build a driver
that will only update the firmware. We don't need to support this
configuration.
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:48 +0000 (00:20 +0100)]
staging: wlags49_h2: Reindent hcf.c
Untabify with tab-width set to 4 (to match VI header). Then reindent
with tab-width reset to 8.
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:47 +0000 (00:20 +0100)]
staging: wlags49_h2: Fix macro termination
Semi-colon terminate various macros (HCFASSERT, IF_PROT_TIME, IF_DMA,
PROT_CNT_INI, HCFLOGENTRY, HCFLOGEXIT, DAWA_ACK, DAWA_ZERO_FID) so
auto-indent will work correctly. Add appropriate do {} while
protection to the macros.
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:46 +0000 (00:20 +0100)]
staging: wlags49_h2: Reindent hcfdef.h
Untabified with tab-width set to 4. Reindented with linux style.
Also remove BITN macros.
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
David Kilroy [Thu, 15 Sep 2011 23:20:45 +0000 (00:20 +0100)]
staging: wlags49_h2: Remove non-linux configurations
For now just remove from hcfcfg.h, we will slowly remove the unused
configurations as we go
Signed-off-by: David Kilroy <kilroyd@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Jonathan Cameron [Wed, 14 Sep 2011 12:01:25 +0000 (13:01 +0100)]
staging:iio: move id and device name setting to iio_device_allocate.
The recent reorganization of the sysfs attribute registration had
the side effect of moving iio_device_register after registration of
triggers etc. The side effect of this is that the id hadn't been
allocated by the time of trigger registration. Thus all triggers
based on device got the name <dev_name>-dev0 instead of <dev_name>-devN
where N is the iio device id.
This should also fix the lack of device name for some error messages
that we have been seeing (and I'd been meaning to track down) as
that has now moved earlier as well.
Reported-by: Manuel Stahl <manuel.stahl@iis.fraunhofer.de>
Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk>
Tested-by: Manuel Stahl <manuel.stahl@iis.fraunhofer.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Jonathan Cameron [Wed, 14 Sep 2011 12:01:24 +0000 (13:01 +0100)]
staging:iio:adc:ad7606 add channel type specification to range attributes.
Cleaner and more consistent naming + makes one abi element we don't need
go away.
Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Jonathan Cameron [Wed, 14 Sep 2011 12:01:23 +0000 (13:01 +0100)]
staging:iio:tree wide. abi fixup for in_ out_ prefix introduction.
This patch set should bring all the attributes created outside
of chan_spec registration inline with the new abi.
Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk>
Acked-by: Michael Hennerich <michael.hennerich@analog.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Aaro Koskinen [Tue, 13 Sep 2011 19:49:35 +0000 (22:49 +0300)]
staging: xgifb: update TODO
There are no build warnings at the moment.
Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Aaro Koskinen [Tue, 13 Sep 2011 19:49:34 +0000 (22:49 +0300)]
staging: xgifb: add missing #include
Add a missing include to eliminate the following sparse warning:
drivers/staging/xgifb/vb_table.h:2572:28: warning: symbol 'XGI21_LCDCapList' was not declared. Should it be static?
Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Aaro Koskinen [Tue, 13 Sep 2011 19:49:33 +0000 (22:49 +0300)]
staging: xgifb: vb_init: use readl()/writel() to access iomapped memory
Use readl() and writel() in FB memory test instead of direct pointer
access, and also add iomem annotations for the FB memory.
The patch eliminates the following sparse warnings:
drivers/staging/xgifb/XGI_main_26.c:2113:69: warning: incorrect type in assignment (different address spaces)
drivers/staging/xgifb/XGI_main_26.c:2113:69: expected unsigned char *static [addressable] [toplevel] [assigned] pjVideoMemoryAddress
drivers/staging/xgifb/XGI_main_26.c:2113:69: got void [noderef] <asn:2>*
drivers/staging/xgifb/XGI_main_26.c:2399:30: warning: incorrect type in assignment (different address spaces)
drivers/staging/xgifb/XGI_main_26.c:2399:30: expected char [noderef] <asn:2>*screen_base
drivers/staging/xgifb/XGI_main_26.c:2399:30: got char *[addressable] [toplevel] [assigned] video_vbase
drivers/staging/xgifb/XGI_main_26.c:2430:31: warning: incorrect type in argument 1 (different address spaces)
drivers/staging/xgifb/XGI_main_26.c:2430:31: expected void volatile [noderef] <asn:2>*addr
drivers/staging/xgifb/XGI_main_26.c:2430:31: got char *[addressable] [toplevel] [assigned] video_vbase
drivers/staging/xgifb/XGI_main_26.c:2454:31: warning: incorrect type in argument 1 (different address spaces)
drivers/staging/xgifb/XGI_main_26.c:2454:31: expected void volatile [noderef] <asn:2>*addr
drivers/staging/xgifb/XGI_main_26.c:2454:31: got char *[addressable] [toplev
Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Aaro Koskinen [Tue, 13 Sep 2011 19:49:32 +0000 (22:49 +0300)]
staging: xgifb: unify #include order
Include Linux headers before driver's own headers.
Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Aaro Koskinen [Tue, 13 Sep 2011 19:49:31 +0000 (22:49 +0300)]
staging: xgifb: main: use readb()/readw() to access iomapped memory
Use readb()/readw() instead of direct pointer access to read I/O mapped
memory and also add __iomem annotation.
The patch eliminates the following sparse warnings:
drivers/staging/xgifb/XGI_main_26.c:2125:35: warning: incorrect type in assignment (different address spaces)
drivers/staging/xgifb/XGI_main_26.c:2125:35: expected char *[addressable] [toplevel] [assigned] mmio_vbase
drivers/staging/xgifb/XGI_main_26.c:2125:35: got void [noderef] <asn:2>*
drivers/staging/xgifb/XGI_main_26.c:2439:31: warning: incorrect type in argument 1 (different address spaces)
drivers/staging/xgifb/XGI_main_26.c:2439:31: expected void volatile [noderef] <asn:2>*addr
drivers/staging/xgifb/XGI_main_26.c:2439:31: got char *[addressable] [toplevel] [assigned] mmio_vbase
drivers/staging/xgifb/XGI_main_26.c:2463:31: warning: incorrect type in argument 1 (different address spaces)
drivers/staging/xgifb/XGI_main_26.c:2463:31: expected void volatile [noderef] <asn:2>*addr
drivers/staging/xgifb/XGI_main_26.c:2463:31: got char *[addressable] [toplevel] [assigned] mmio_vbase
Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>