On Mon, 2013-07-15 at 08:15 -0400, John Greene wrote: > On 07/12/2013 02:00 PM, Joe Perches wrote: > > On Fri, 2013-07-12 at 10:55 -0400, John Greene wrote: [] > > Just trivia: > >> diff --git a/drivers/net/wireless/brcm80211/brcmsmac/ampdu.c b/drivers/net/wireless/brcm80211/brcmsmac/ampdu.c > > [] > >> @@ -928,7 +928,7 @@ brcms_c_ampdu_dotxstatus_complete(struct ampdu_info *ampdu, struct scb *scb, > > [] > >> - brcms_err(wlc->hw->d11core, > >> + brcms_dbg_ht(wlc->hw->d11core, > >> "%s: ampdu tx phy error (0x%x)\n", > >> __func__, txs->phyerr); > > > > Please keep the indentation of the arguments of > > the same statement aligned to the open parenthesis. [] > Thanks Joe. My oversight. I wish the checkpatch flagged this stuff. It does if you use the --strict option. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
