From: Arend van Spriel Date: Mon, 8 Aug 2011 13:58:18 +0000 (+0200) Subject: staging: brcm80211: remove volatile keyword used in struct rte_console X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=85567f52f2ba6159b98b066c792e5eb7cdc9bf0c;p=openwrt%2Fstaging%2Fblogic.git staging: brcm80211: remove volatile keyword used in struct rte_console Two members vcons_in and vcons_out were declared using the volatile keyword. However, there is no reason for doing so. The member is used only to calculate the backplane address to access using offsetof. This address is passed to subsequent read function. Use of volatile is not warranted. Signed-off-by: Arend van Spriel Reviewed-by: Roland Vossen Reviewed-by: Franky Lin Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c b/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c index 3e7d5b273f1c..8cb34d673482 100644 --- a/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c +++ b/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c @@ -126,8 +126,8 @@ struct rte_console { * (at risk of conflicting with * the real UART). vcons_out is currently unused. */ - volatile uint vcons_in; - volatile uint vcons_out; + uint vcons_in; + uint vcons_out; /* Output (logging) buffer * Console output is written to a ring buffer log_buf at index log_idx.