- 06 Jul, 2011 40 commits
-
-
Arend van Spriel authored
The script checkpatch.pl flagged for some extern variables in dhd_linux.c source. Several have been fixed in this patch. Two remaining are in macro conditional code that needs cleanup so they are not resolved here. Reviewed-by:
Roland Vossen <rvossen@broadcom.com> Reviewed-by:
Pieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by:
Arend van Spriel <arend@broadcom.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Arend van Spriel authored
One of the checkpatch warning on this driver is about usage of the __attribute__ keyword. This patch changes this to using the __packed keyword. Reviewed-by:
-
Roland Vossen authored
SDIOH_API_ACCESS_RETRY_LIMIT is always defined. Signed-off-by:
-
Roland Vossen authored
And deleted unused definitions. Improves code since it makes clear that the definitions are only used by a specific .c file. Signed-off-by:
-
Roland Vossen authored
Signed-off-by:
-
Roland Vossen authored
Signed-off-by:
-
Roland Vossen authored
Signed-off-by:
-
Roland Vossen authored
Having both a 'generic' as a 'linux' specific variant of a file makes no sense in a Linux environment. Merged bcmsdh_sdmmc_linux.c into bcmsdh_sdmmc.c, and bcmsdh_linux.c into bcmsdh.c. Signed-off-by:
-
Roland Vossen authored
WLC (caps) was replaced, wlc (small caps) has not been replaced yet. Signed-off-by:
-
Arend van Spriel authored
The source file phy_n.c had couple of checkpatch warnings that have been fixed in this patch. Reviewed-by:
-
Arend van Spriel authored
The checkpatch script gave a hint that some character string arrays probably should have an additional const keyword. This patch changes the static const char * arrays to static const char * const arrays as suggested getting rid of two checkpatch warnings. Reviewed-by:
-
Arend van Spriel authored
The driver sources contained a couple of checkpatch warnings of the given class. These have been cleaned up. Reviewed-by:
Franky (Zhenhui) Lin <frankyl@broadcom.com> Signed-off-by:
-
Roland Vossen authored
Several structures contained a '_c_' in their name that does not add any information. Signed-off-by:
-
Roland Vossen authored
Signed-off-by:
-
Roland Vossen authored
Signed-off-by:
-
Roland Vossen authored
Reviewed-by:
-
Roland Vossen authored
typedefs are undesirable according to documentation/CodingStyle. Signed-off-by:
-
Roland Vossen authored
Reviewed-by:
-
Roland Vossen authored
Reviewed-by:
-
Roland Vossen authored
Reviewed-by:
-
Roland Vossen authored
Reviewed-by:
-
Roland Vossen authored
Reviewed-by:
-
Roland Vossen authored
ASSERTs are non desirable, replaced by error handling where applicable. Reviewed-by:
-
Arend van Spriel authored
The header file dhd_dbg.h contains several macros used throughout the code starting with DHD_. These have been renamed to get rid of this broadcom specific acronym. Reviewed-by:
-
Roland Vossen authored
CONFIG_MACH_SANDGATE2G, CONFIG_MMC_MSM7X00A, CONFIG_HAS_EARLYSUSPEND and CONFIG_WIFI_CONTROL_FUNC were never defined. Signed-off-by:
-
Roland Vossen authored
DHD is a Broadcom internal term and has been replaced by BRCMF. Signed-off-by:
-
Arend van Spriel authored
A checkpatch error was introduced in following commit: 0f22b8a staging: brcm80211: remove MMC_SDIO_ABORT macro definition The patch gets rid of the error. Reviewed-by:
-
Tomas Winkler authored
This reason for this feature was 'some versions of the videodev module overwrite the data which has been written by the call to usb_set_intfdata() in easycap_usb_probe(), replacing it with a pointer to the embedded v4l2_device structure. to detect this, the string in the easycap.telltale[] buffer is checked.' The upstream version of v4l2_device_register sets driver data only when it wasn't already set, therefore this is not needed Signed-off-by:
Tomas Winkler <tomas.winkler@intel.com> Signed-off-by:
-
Arend van Spriel authored
Renamed external function getting rid of wl_ prefix. Removed unused function from the API header file wl_cfg80211.h as it was not defined. Signed-off-by:
-
Arend van Spriel authored
The data structures in fullmac driver references are made easier by using provided macros in wl_cfg80211.h. The abbreviation 'wl' in these macros has been replaced by cfg or the macro has been removed if used only once. Signed-off-by:
-
Arend van Spriel authored
Making an effort getting rid of the wl_ prefix from the fullmac source files. Signed-off-by:
-
Roland Vossen authored
Signed-off-by:
-
Roland Vossen authored
Code cleanup. W_REG was defined the same for both big and little endian systems. Replaced W_REG on two spots with its expansion. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Warnings were due to pointer declarations being used for an address that does not reside on the CPU bus. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Decreases amd64 compiler warnings. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Leads to more readable code and better type checking. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Leads to more readable code and better type checking. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Fullmac contains macro's that access registers on the 32 bits backplane bus. These registers should not be declared as pointers since pointers can be 64 bits. The following patch series gets rid of these pointers and ultimately of all compiler warning when compiling for amd64. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Signed-off-by:
-
