Commit 57f845d2 authored by Roland Vossen's avatar Roland Vossen Committed by Greg Kroah-Hartman

staging: brcm80211: W_REG macro cleanup in fullmac SDIO

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: default avatarRoland Vossen <rvossen@broadcom.com>
Reviewed-by: default avatarArend van Spriel <arend@broadcom.com>
Reviewed-by: default avatarFranky Lin <frankyl@broadcom.com>
Signed-off-by: default avatarArend van Spriel <arend@broadcom.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent bd5213eb
...@@ -51,22 +51,13 @@ ...@@ -51,22 +51,13 @@
}) })
#endif /* __mips__ */ #endif /* __mips__ */
#define W_REG(r, v, typ) do { \
brcmf_sdcard_reg_write(NULL, (r), sizeof(typ), (v)); \
} while (0)
#else /* __BIG_ENDIAN */ #else /* __BIG_ENDIAN */
#define R_REG(r, typ) \ #define R_REG(r, typ) \
brcmf_sdcard_reg_read(NULL, (r), sizeof(typ)) brcmf_sdcard_reg_read(NULL, (r), sizeof(typ))
#define W_REG(r, v, typ) do { \
brcmf_sdcard_reg_write(NULL, (r), sizeof(typ), (v)); \
} while (0)
#endif /* __BIG_ENDIAN */ #endif /* __BIG_ENDIAN */
#define AND_REG(r, v, typ) W_REG((r), R_REG(r, typ) & (v), typ) #define OR_REG(r, v, typ) \
#define OR_REG(r, v, typ) W_REG((r), R_REG(r, typ) | (v), typ) brcmf_sdcard_reg_write(NULL, (r), sizeof(typ), R_REG(r, typ) | (v))
#define SET_REG(r, mask, val, typ) \
W_REG((r), ((R_REG(r) & ~(mask)) | (val)), typ)
#ifdef BCMDBG #ifdef BCMDBG
...@@ -609,7 +600,7 @@ struct brcmf_bus { ...@@ -609,7 +600,7 @@ struct brcmf_bus {
uint console_addr; /* Console address from shared struct */ uint console_addr; /* Console address from shared struct */
#endif /* BCMDBG */ #endif /* BCMDBG */
uint regfails; /* Count of R_REG/W_REG failures */ uint regfails; /* Count of R_REG failures */
uint clkstate; /* State of sd and backplane clock(s) */ uint clkstate; /* State of sd and backplane clock(s) */
bool activity; /* Activity flag for clock down */ bool activity; /* Activity flag for clock down */
...@@ -850,7 +841,8 @@ w_sdreg32(struct brcmf_bus *bus, u32 regval, u32 reg_offset, u32 *retryvar) ...@@ -850,7 +841,8 @@ w_sdreg32(struct brcmf_bus *bus, u32 regval, u32 reg_offset, u32 *retryvar)
{ {
*retryvar = 0; *retryvar = 0;
do { do {
W_REG(bus->ci->buscorebase + reg_offset, regval, u32); brcmf_sdcard_reg_write(NULL, bus->ci->buscorebase + reg_offset,
sizeof(u32), regval);
} while (brcmf_sdcard_regfail(bus->card) && } while (brcmf_sdcard_regfail(bus->card) &&
(++(*retryvar) <= retry_limit)); (++(*retryvar) <= retry_limit));
if (*retryvar) { if (*retryvar) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment