- 06 Sep, 2011 40 commits
-
-
Alexey Khoroshilov authored
mei_open() acquires dev->device_lock mutex and try to allocate mei_cl, but if the allocation fails it goes to return statement. As a result dev->device_lock left locked. The patch fixes goto to unlock dev->device_lock mutex on this path. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by:
Alexey Khoroshilov <khoroshilov@ispras.ru> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Roland Vossen authored
Substituted macro. Reported-by:
Johannes Berg <johannes@sipsolutions.net> Reported-by:
Joe Perches <joe@perches.com> Reviewed-by:
Pieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by:
Arend van Spriel <arend@broadcom.com> Signed-off-by:
Roland Vossen <rvossen@broadcom.com> Signed-off-by:
-
Roland Vossen authored
Substituted/removed macro's. Reported-by:
-
Roland Vossen authored
Substituted/removed macro's. Reported-by:
-
Roland Vossen authored
AP support was only partially implemented and never invoked. AP related code sections removed. Reported-by:
-
Roland Vossen authored
Substituted/removed macro's. Reported-by:
-
Roland Vossen authored
Substituted macro's. Reported-by:
-
Roland Vossen authored
Substituted macro's. Reported-by:
-
Arend van Spriel authored
Kconfig options are by default not selected so adding the statement 'default n' to the option is not needed. Reported-by:
-
Arend van Spriel authored
The source file main.c contained several sparse warnings which have been cleaned up. Reported-by:
Aaro Koskinen <aaro.koskinen@iki.fi> Reviewed-by:
Franky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by:
-
Arend van Spriel authored
Several sparse warning were issued due to missing __user qualifier for brcmfmac variables. This patch adds those. Reported-by:
-
Arend van Spriel authored
Running sparse showed following warning: Using plain integer as NULL pointer. This has been fixed. Reported-by:
-
Arend van Spriel authored
Sparse warning was given when using the extern keyword in the function definition. This patch removes those instances. Reported-by:
-
Roland Vossen authored
Moved, deleted or substituted macro's. Reported-by:
-
Roland Vossen authored
Moved, deleted or substituted macro's. Reported-by:
-
Roland Vossen authored
Moved, deleted or substituted macro's. Reported-by:
-
Roland Vossen authored
Moved, deleted or substituted macro's. Reported-by:
-
Roland Vossen authored
Moved, deleted or substituted macro's. Reported-by:
-
Roland Vossen authored
Reported-by:
-
Roland Vossen authored
Reported-by:
-
Arend van Spriel authored
Cleanup resulted in merging several files and made several symbols local to a source file. These were found by running sparse as it gave the warning: symbol 'foo' was not declared. Should it be static? Reported-by:
-
Arend van Spriel authored
Some changes have been made to get rid of sparse warnings in the source file mac80211_if.c. Reported-by:
-
Arend van Spriel authored
Running sparse over the driver sources showed 1356 sparse messages and quite a number are due to missing __iomem attribute in some places. This patch reduces the number significantly. Reported-by:
-
Arend van Spriel authored
Cleanup resulted in merging several files and made several symbols local to a source file. These were found by running sparse as it gave the warning: symbol 'foo' was not declared. Should it be static? Reported-by:
-
Franky Lin authored
Reshuffle function order in dhd_linux of fullmac to get rid of static function declaration Reported-by:
-
Franky Lin authored
Reshuffle function order in bcmsdh_sdmmc of fullmac to get rid of static function declaration Reported-by:
-
Franky Lin authored
sdioh command type is not used in brcmf_sdioh_request_word. This patch removes the related code. Reviewed-by:
-
Franky Lin authored
sdioh data mode is not used in brcmf_sdioh_request_buffer. The patch removes the related code. Reviewed-by:
-
Franky Lin authored
The address stored in sdiodev can be used directly instead of using the wrapper function brcmf_sdcard_cur_sbwad. Reviewed-by:
-
Franky Lin authored
In fullmac, brcmf_sdbrcm_set_siaddr_window and brcmf_sdcard_set_sbaddr_window have identical fucntionality. Remove the one in dhd_sdio.c Reported-by:
Rafal Milecki <zajec5@gmail.com> Reviewed-by:
-
Franky Lin authored
Use unify config marco for debug info. Related event debug print out can be enabled/disabled by the dedicated event message level. Reported-by:
-
Roland Vossen authored
Code cleanup. Advantage is more readable code and better type checking. Reported-by:
-
Roland Vossen authored
Macros impacting the code flow are sort of frowned upon. No other macro's in the source code impact the code flow. Replaced macro by function. Reported-by:
-
Arend van Spriel authored
Function prototypes for static functions are not strictly needed and considered unwanted by linux community. This patch reorders the functions in brcmsmac/main.c and gets rid of the prototypes. Reported-by:
-
Henry Ptasinski authored
Also eliminate a cast to int that could cause issues with very large values of di->rxbufsize (as suggested by Dan Carpenter). Signed-off-by:
Henry Ptasinski <henryp@broadcom.com> Signed-off-by:
-
Roland Vossen authored
nicpci.c, srom.c and stf.c Reported-by:
-
Roland Vossen authored
Reported-by:
-
Roland Vossen authored
Reported-by:
-
Roland Vossen authored
Reported-by:
-
Roland Vossen authored
Reported-by:
-
