The CHIPTYPE macro simply expands to the macro argument so it is
redundant and as such removed.

Reviewed-by: Roland Vossen <[email protected]>
Signed-off-by: Arend van Spriel <[email protected]>
---
 drivers/staging/brcm80211/include/bcmdefs.h |    1 -
 drivers/staging/brcm80211/util/siutils.c    |   26 +++++++++++++-------------
 2 files changed, 13 insertions(+), 14 deletions(-)

diff --git a/drivers/staging/brcm80211/include/bcmdefs.h 
b/drivers/staging/brcm80211/include/bcmdefs.h
index 4ce4486..350bd6d 100644
--- a/drivers/staging/brcm80211/include/bcmdefs.h
+++ b/drivers/staging/brcm80211/include/bcmdefs.h
@@ -51,7 +51,6 @@
 #define SPI_BUS                        6       /* gSPI target */
 #define RPC_BUS                        7       /* RPC target */
 
-#define CHIPTYPE(bus)  (bus)
 #define CHIPID(chip)   (chip)
 #define CHIPREV(rev)   (rev)
 
diff --git a/drivers/staging/brcm80211/util/siutils.c 
b/drivers/staging/brcm80211/util/siutils.c
index 1a9ff1a..7697e1c 100644
--- a/drivers/staging/brcm80211/util/siutils.c
+++ b/drivers/staging/brcm80211/util/siutils.c
@@ -567,7 +567,7 @@ static si_info_t *si_doattach(si_info_t *sii, uint devid, 
struct osl_info *osh,
        sih->issim = IS_SIM(sih->chippkg);
 
        /* scan for cores */
-       if (CHIPTYPE(sii->pub.socitype) == SOCI_AI) {
+       if (sii->pub.socitype == SOCI_AI) {
                SI_MSG(("Found chip type AI (0x%08x)\n", w));
                /* pass chipc address instead of original core base */
                ai_scan(&sii->pub, (void *)cc, devid);
@@ -764,7 +764,7 @@ void si_deregister_intr_callback(si_t *sih)
 
 uint si_flag(si_t *sih)
 {
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                return ai_flag(sih);
        else {
                ASSERT(0);
@@ -774,7 +774,7 @@ uint si_flag(si_t *sih)
 
 void si_setint(si_t *sih, int siflag)
 {
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                ai_setint(sih, siflag);
        else
                ASSERT(0);
@@ -806,7 +806,7 @@ bool si_backplane64(si_t *sih)
 #ifndef BCMSDIO
 uint si_corerev(si_t *sih)
 {
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                return ai_corerev(sih);
        else {
                ASSERT(0);
@@ -849,7 +849,7 @@ void *si_setcore(si_t *sih, uint coreid, uint coreunit)
        if (!GOODIDX(idx))
                return NULL;
 
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                return ai_setcoreidx(sih, idx);
        else {
 #ifdef BCMSDIO
@@ -864,7 +864,7 @@ void *si_setcore(si_t *sih, uint coreid, uint coreunit)
 #ifndef BCMSDIO
 void *si_setcoreidx(si_t *sih, uint coreidx)
 {
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                return ai_setcoreidx(sih, coreidx);
        else {
                ASSERT(0);
@@ -916,7 +916,7 @@ void si_restore_core(si_t *sih, uint coreid, uint intr_val)
 
 u32 si_core_cflags(si_t *sih, u32 mask, u32 val)
 {
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                return ai_core_cflags(sih, mask, val);
        else {
                ASSERT(0);
@@ -926,7 +926,7 @@ u32 si_core_cflags(si_t *sih, u32 mask, u32 val)
 
 u32 si_core_sflags(si_t *sih, u32 mask, u32 val)
 {
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                return ai_core_sflags(sih, mask, val);
        else {
                ASSERT(0);
@@ -936,7 +936,7 @@ u32 si_core_sflags(si_t *sih, u32 mask, u32 val)
 
 bool si_iscoreup(si_t *sih)
 {
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                return ai_iscoreup(sih);
        else {
 #ifdef BCMSDIO
@@ -951,7 +951,7 @@ bool si_iscoreup(si_t *sih)
 void si_write_wrapperreg(si_t *sih, u32 offset, u32 val)
 {
        /* only for 4319, no requirement for SOCI_SB */
-       if (CHIPTYPE(sih->socitype) == SOCI_AI) {
+       if (sih->socitype == SOCI_AI) {
                ai_write_wrap_reg(sih, offset, val);
        }
 }
@@ -959,7 +959,7 @@ void si_write_wrapperreg(si_t *sih, u32 offset, u32 val)
 uint si_corereg(si_t *sih, uint coreidx, uint regoff, uint mask, uint val)
 {
 
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                return ai_corereg(sih, coreidx, regoff, mask, val);
        else {
 #ifdef BCMSDIO
@@ -974,7 +974,7 @@ uint si_corereg(si_t *sih, uint coreidx, uint regoff, uint 
mask, uint val)
 void si_core_disable(si_t *sih, u32 bits)
 {
 
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                ai_core_disable(sih, bits);
 #ifdef BCMSDIO
        else
@@ -984,7 +984,7 @@ void si_core_disable(si_t *sih, u32 bits)
 
 void si_core_reset(si_t *sih, u32 bits, u32 resetbits)
 {
-       if (CHIPTYPE(sih->socitype) == SOCI_AI)
+       if (sih->socitype == SOCI_AI)
                ai_core_reset(sih, bits, resetbits);
 #ifdef BCMSDIO
        else
-- 
1.7.1


_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to