Message ID | 20171207102047.21635-1-colin.king@canonical.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 18907f20ea71b17b3cac91049b415b95cde5fa9c |
Delegated to: | Kalle Valo |
Headers | show |
On 12/7/2017 11:20 AM, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > Use the ARRAY_SIZE macro on rfseq_updategainu_events to determine > size of the array. Improvement suggested by coccinelle. Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com> > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-)
Colin Ian King <colin.king@canonical.com> wrote: > From: Colin Ian King <colin.king@canonical.com> > > Use the ARRAY_SIZE macro on rfseq_updategainu_events to determine > size of the array. Improvement suggested by coccinelle. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com> Patch applied to wireless-drivers-next.git, thanks. 18907f20ea71 brcmsmac: use ARRAY_SIZE on rfseq_updategainu_events
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c index 763e8ba6b178..7e01981bc5c8 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c @@ -16049,8 +16049,7 @@ static void wlc_phy_workarounds_nphy_gainctrl(struct brcms_phy *pi) wlc_phy_set_rfseq_nphy(pi, NPHY_RFSEQ_UPDATEGAINU, rfseq_updategainu_events, rfseq_updategainu_dlys, - sizeof(rfseq_updategainu_events) / - sizeof(rfseq_updategainu_events[0])); + ARRAY_SIZE(rfseq_updategainu_events)); mod_phy_reg(pi, 0x153, (0xff << 8), (90 << 8));