[PATCH 4/4] brcm80211: deinline brcmf_sdio_clrintr, save 8979 bytes

From: Denys Vlasenko
Date: Sun Mar 30 2014 - 17:32:15 EST


Automated script discovered that without forced inlining,
gcc-4.7 generates smaller code for this function.

There is no need to declare static functions inline anyway:
nowadays gcc detects single-callsite static functions
which benefit from inlining.

Signed-off-by: Denys Vlasenko <dvlasenk@xxxxxxxxxx>
---
drivers/net/wireless/brcm80211/brcmfmac/dhd_sdio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/brcm80211/brcmfmac/dhd_sdio.c b/drivers/net/wireless/brcm80211/brcmfmac/dhd_sdio.c
index 47a6f39..55a6d0d 100644
--- a/drivers/net/wireless/brcm80211/brcmfmac/dhd_sdio.c
+++ b/drivers/net/wireless/brcm80211/brcmfmac/dhd_sdio.c
@@ -2516,7 +2516,7 @@ static void brcmf_sdio_bus_stop(struct device *dev)
bus->tx_seq = bus->rx_seq = 0;
}

-static inline void brcmf_sdio_clrintr(struct brcmf_sdio *bus)
+static void brcmf_sdio_clrintr(struct brcmf_sdio *bus)
{
unsigned long flags;

--
1.8.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/