Re: [PATCH] brcm80211: fix error report string

["Gustavo F. Padovan" <padovan@profusion.mobi>]

Hi Brett,

* Brett Rudley <brudley@broadcom.com> [2011-01-24 13:57:46 -0800]:

> > -----Original Message-----
> > From: Rafał Miłecki [mailto:zajec5@gmail.com]
> > Sent: Monday, January 24, 2011 1:17 PM
> > To: Gustavo F. Padovan
> > Cc: Brett Rudley; Henry Ptasinski; Dowan Kim; vossen@broadcom.com; Arend
> > Van Spriel; linux-wireless@vger.kernel.org
> > Subject: Re: [PATCH] brcm80211: fix error report string
> > 
> > 2011/1/24 Gustavo F. Padovan <padovan@profusion.mobi>:
> > > Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
> > > ---
> > >  drivers/staging/brcm80211/sys/wlc_mac80211.c |    2 +-
> > >  1 files changed, 1 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/drivers/staging/brcm80211/sys/wlc_mac80211.c
> > b/drivers/staging/brcm80211/sys/wlc_mac80211.c
> > > index 5eb41d6..ee1bcc5 100644
> > > --- a/drivers/staging/brcm80211/sys/wlc_mac80211.c
> > > +++ b/drivers/staging/brcm80211/sys/wlc_mac80211.c
> > > @@ -6910,7 +6910,7 @@ prep_mac80211_status(struct wlc_info *wlc,
> > d11rxhdr_t *rxh, struct sk_buff *p,
> > >                preamble = 0;
> > >                if (IS_CCK(rspec)) {
> > >                        if (rxh->PhyRxStatus_0 & PRXS0_SHORTH)
> > > -                               WL_ERROR("Short CCK\n");
> > > +                               WL_ERROR("%s: Short CCK\n, __func__");
> > 
> > I have no idea how WL_ERROR works, but are you sure of that?
> > 
> > You wrote:
> > "%s: Short CCK\n, __func__"
> > 
> > Didn't you mean:
> > "%s: Short CCK\n", __func__
> > ?
> > 
> 
> Rafal's suggestion is correct, the patch was incorrect.
> (Good eyes, Rafal)

I just sent a corrected one yesterday in this thread.

-- 
Gustavo F. Padovan
http://profusion.mobi