On Wed, May 21, 2014 at 01:59:04PM +0200, Jes Sorensen wrote:
> Dan Carpenter <dan.carpen...@oracle.com> writes:
> > On Wed, May 21, 2014 at 09:38:04AM +0200, jes.soren...@redhat.com wrote:
> >> From: Jes Sorensen <jes.soren...@redhat.com>
> >>
> >> Signed-off-by: Jes Sorensen <jes.soren...@redhat.com>
> >> ---
> >> drivers/staging/rtl8723au/hal/rtl8723a_cmd.c | 9 +++++----
> >> 1 file changed, 5 insertions(+), 4 deletions(-)
> >>
> >> diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> >> b/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> >> index dc044cf..eb747d6 100644
> >> --- a/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> >> +++ b/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> >> @@ -383,13 +383,14 @@ ConstructNullFunctionData(struct rtw_adapter
> >> *padapter, u8 *pframe,
> >> }
> >>
> >> if (bQoS == true) {
> >> - struct ieee80211_qos_hdr *pwlanqoshdr;
> >> + struct ieee80211_qos_hdr *qoshdr;
> >> + qoshdr = (struct ieee80211_qos_hdr *)pframe;
> >
> > Checkpatch complains about these nowadays.
> >
> > It's obviously a minor thing. No need to resend. No need to respond to
> > this email at all, in fact.
>
> Sorry you baffled me there - it complains about what here?
>
WARNING: Missing a blank line after declarations
#32: FILE: drivers/staging/rtl8723au/hal/rtl8723a_cmd.c:387:
+ struct ieee80211_qos_hdr *qoshdr;
+ qoshdr = (struct ieee80211_qos_hdr *)pframe;
total: 0 errors, 1 warnings, 18 lines checked
regards,
dan carpenter
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
