[PATCH 0/4] staging: brcm80211: cleanup and small bugfix on error codes

Roland Vossen rvossen at broadcom.com
Mon Apr 11 13:16:22 UTC 2011


Code cleanup on error codes, to make them more in accordance with kernel coding
style. The next step is to replace broadcom specific error codes by linux error
codes, however to facilitate the review process, this is not part of this patch.
There was a small bug in the fullmac that caused a positive error value to be
returned instead of a negative one, that has been fixed as well.

  staging: brcm80211: removed unused error codes
  staging: brcm80211: replaced BCME_OK by 0
  staging: brcm80211: made error codes in bcmutils.h positive
  staging: brcm80211: bugfix for fullmac return codes

 drivers/staging/brcm80211/brcmfmac/bcmsdh.c        |   32 ++--
 drivers/staging/brcm80211/brcmfmac/bcmsdh_sdmmc.c  |   18 +-
 .../brcm80211/brcmfmac/bcmsdh_sdmmc_linux.c        |    4 +-
 drivers/staging/brcm80211/brcmfmac/dhd_cdc.c       |   10 +-
 drivers/staging/brcm80211/brcmfmac/dhd_common.c    |   28 ++--
 drivers/staging/brcm80211/brcmfmac/dhd_linux.c     |   96 +++++-----
 drivers/staging/brcm80211/brcmfmac/dhd_sdio.c      |  158 ++++++++--------
 drivers/staging/brcm80211/brcmsmac/phy/wlc_phy_n.c |   46 ++--
 drivers/staging/brcm80211/brcmsmac/wl_mac80211.c   |    2 +-
 drivers/staging/brcm80211/brcmsmac/wlc_ampdu.c     |   10 +-
 drivers/staging/brcm80211/brcmsmac/wlc_bmac.c      |    4 +-
 drivers/staging/brcm80211/brcmsmac/wlc_channel.c   |    2 +-
 drivers/staging/brcm80211/brcmsmac/wlc_key.h       |   10 +-
 drivers/staging/brcm80211/brcmsmac/wlc_main.c      |  212 ++++++++++----------
 drivers/staging/brcm80211/brcmsmac/wlc_stf.c       |   22 +-
 drivers/staging/brcm80211/include/bcmutils.h       |   72 +++----
 drivers/staging/brcm80211/util/bcmotp.c            |   30 ++--
 drivers/staging/brcm80211/util/bcmsrom.c           |   12 +-
 drivers/staging/brcm80211/util/bcmutils.c          |   14 +-
 drivers/staging/brcm80211/util/nvram/nvram_ro.c    |    8 +-
 20 files changed, 388 insertions(+), 402 deletions(-)





More information about the devel mailing list