- 06 Sep, 2011 40 commits
-
-
Mark Einon authored
Use defines from include/linux/mii.h instead of et131x_phy.h and delete the latter defines. Signed-off-by:
Mark Einon <mark.einon@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Michal Marek authored
Remove dead and unused code in sbecom_inline_linux.h. This also fixes a build failure under Turkish locale, where include/config/modversions.h is incorrectly named as modversIons.h (an unrelated kbuild bug) and the driver tries to needlessly include the empty file. Reported-by:
Serdar KÖYLÜ <s.koylux@gmail.com> Cc: Bob Beers <bob.beers@gmail.com> Signed-off-by:
Michal Marek <mmarek@suse.cz> Signed-off-by:
-
Alexey Khoroshilov authored
If sep_ioctl() is called from a process that does not own current transaction, it unlocks unheld sep->ioctl_mutex and returns -EACCES leaving sep->sep_mutex acquired. The patch fixes the mutex lock-unlock mismatch. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by:
Alexey Khoroshilov <khoroshilov@ispras.ru> Signed-off-by:
-
Pelle Windestam authored
Fixed the checkpatch warning about sing simple_strtoul instead of kstrtoul() in panel.c. Signed-off-by:
Pelle Windestam <iceaway@gmail.com> Signed-off-by:
-
Mark Einon authored
Now we are using a phy_device, remove driver functionality that is now handled by the phydev. Signed-off-by:
-
Mark Einon authored
Use the phy_device equivalents instead. Signed-off-by:
-
Mark Einon authored
Handing over ioctls handled by the driver to the phydev. Signed-off-by:
-
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:
-
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:
-
