Change sizeof(x) / sizeof(*x) to ARRAY_SIZE(x). Signed-off-by: Kulikov Vasiliy <segooon@xxxxxxxxx> --- drivers/staging/otus/hal/hpani.c | 16 ++++++---------- 1 files changed, 6 insertions(+), 10 deletions(-) diff --git a/drivers/staging/otus/hal/hpani.c b/drivers/staging/otus/hal/hpani.c index f53e483..9b9420c 100644 --- a/drivers/staging/otus/hal/hpani.c +++ b/drivers/staging/otus/hal/hpani.c @@ -72,7 +72,6 @@ s32_t BEACON_RSSI(zdev_t *dev) void zfHpAniAttach(zdev_t *dev) { -#define N(a) (sizeof(a) / sizeof(a[0])) u32_t i; struct zsHpPriv *HpPriv; @@ -125,7 +124,6 @@ void zfHpAniAttach(zdev_t *dev) HpPriv->stats.ast_nodestats.ns_avgbrssi = ZM_RSSI_DUMMY_MARKER; HpPriv->stats.ast_nodestats.ns_avgrssi = ZM_RSSI_DUMMY_MARKER; HpPriv->stats.ast_nodestats.ns_avgtxrssi = ZM_RSSI_DUMMY_MARKER; -#undef N } /* @@ -133,7 +131,6 @@ void zfHpAniAttach(zdev_t *dev) */ u8_t zfHpAniControl(zdev_t *dev, ZM_HAL_ANI_CMD cmd, int param) { -#define N(a) (sizeof(a)/sizeof(a[0])) typedef s32_t TABLE[]; struct zsHpPriv *HpPriv; struct zsAniState *aniState; @@ -148,9 +145,9 @@ u8_t zfHpAniControl(zdev_t *dev, ZM_HAL_ANI_CMD cmd, int param) { u32_t level = param; - if (level >= N(HpPriv->totalSizeDesired)) { + if (level >= ARRAY_SIZE(HpPriv->totalSizeDesired)) { zm_debug_msg1("level out of range, desired level : ", level); - zm_debug_msg1("max level : ", N(HpPriv->totalSizeDesired)); + zm_debug_msg1("max level : ", ARRAY_SIZE(HpPriv->totalSizeDesired)); return FALSE; } @@ -260,10 +257,10 @@ u8_t zfHpAniControl(zdev_t *dev, ZM_HAL_ANI_CMD cmd, int param) const TABLE firstep = { 0, 4, 8 }; u32_t level = param; - if (level >= N(firstep)) + if (level >= ARRAY_SIZE(firstep)) { zm_debug_msg1("level out of range, desired level : ", level); - zm_debug_msg1("max level : ", N(firstep)); + zm_debug_msg1("max level : ", ARRAY_SIZE(firstep)); return FALSE; } zfDelayWriteInternalReg(dev, AR_PHY_FIND_SIG, @@ -283,10 +280,10 @@ u8_t zfHpAniControl(zdev_t *dev, ZM_HAL_ANI_CMD cmd, int param) const TABLE cycpwrThr1 = { 2, 4, 6, 8, 10, 12, 14, 16 }; u32_t level = param; - if (level >= N(cycpwrThr1)) + if (level >= ARRAY_SIZE(cycpwrThr1)) { zm_debug_msg1("level out of range, desired level : ", level); - zm_debug_msg1("max level : ", N(cycpwrThr1)); + zm_debug_msg1("max level : ", ARRAY_SIZE(cycpwrThr1)); return FALSE; } zfDelayWriteInternalReg(dev, AR_PHY_TIMING5, @@ -335,7 +332,6 @@ u8_t zfHpAniControl(zdev_t *dev, ZM_HAL_ANI_CMD cmd, int param) return FALSE; } return TRUE; -#undef N } void zfHpAniRestart(zdev_t* dev) -- 1.7.0.4 -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html