Commits · d7e508880b8f674aa82d3d5d9b2de27c034329a3 · Kirill Smelkov / linux

05 Oct, 2010 40 commits

Staging: brcm80211: s/ushort/unsigned short/ · d7e50888

Greg Kroah-Hartman authored Oct 05, 2010

This changes the usage everywhere in the driver, and removes
the definition as it should no longer be used anywhere.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

d7e50888

Staging: brcm80211: s/uchar/unsigned char/ · 580a0bd9

Greg Kroah-Hartman authored Oct 05, 2010

This changes the usage everywhere in the driver, and removes
the definition as it should no longer be used anywhere.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

580a0bd9

Staging: brcm80211: s/int8/s8/ · 562c8850

Greg Kroah-Hartman authored Oct 05, 2010

This changes the usage everywhere in the driver, and removes
the definition as it should no longer be used anywhere.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

562c8850

Staging: brcm80211: typedefs.h: remove uint8 definition · 3bc4d492

Greg Kroah-Hartman authored Oct 05, 2010

It's no longer needed anywhere, so remove it.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

3bc4d492

Staging: brcm80211: include: s/uint8/u8/ · de9bca63

Greg Kroah-Hartman authored Oct 05, 2010

Replace uint8 with u8, the correct kernel type to be using here.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

de9bca63

Staging: brcm80211: phy: s/uint8/u8/ · e868ab03

Greg Kroah-Hartman authored Oct 05, 2010

Replace uint8 with u8, the correct kernel type to be using here.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

e868ab03

Staging: brcm80211: brcmfmac: s/uint8/u8/ · 3fd79f7c

Greg Kroah-Hartman authored Oct 05, 2010

Replace uint8 with u8, the correct kernel type to be using here.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

3fd79f7c

Staging: brcm80211: sys: s/uint8/u8/ · 41feb5ed

Greg Kroah-Hartman authored Oct 05, 2010

Replace uint8 with u8, the correct kernel type to be using here.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

41feb5ed

Staging: brcm80211: util: s/uint8/u8/ · 36ef9a1e

Greg Kroah-Hartman authored Oct 05, 2010

Replace uint8 with u8, the correct kernel type to be using here.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

36ef9a1e

Staging: brcm80211: typedefs.h: remove NULL definition · 42e5386d

Greg Kroah-Hartman authored Oct 05, 2010

No need for a driver to define NULL, the core kernel handles that.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

42e5386d

Staging: brcm80211: remove unit64 and int64 typedefs · 151da5a5

Greg Kroah-Hartman authored Oct 05, 2010

They are not used anywhere, nor should they be defined.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

151da5a5

Staging: brcm80211: typedefs.h move types.h to the top of the file · 7848789c

Greg Kroah-Hartman authored Oct 05, 2010

Include other files from the top, it's easier to unwind the logic that
way.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

7848789c

Staging: brcm80211: typedefs.h: remove version.h inclusion · cc79cad2

Greg Kroah-Hartman authored Oct 05, 2010

It isn't needed anymore.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

cc79cad2

Staging: brcm80211: typedefs.h: remove some unused #if logic · 5393ddf1

Greg Kroah-Hartman authored Oct 05, 2010

We never care about __STRICT_ANSI__ from within the kernel, so remove
this logic.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

5393ddf1

Staging: brcm80211: remove bool redefinition · a21142b2

Greg Kroah-Hartman authored Oct 05, 2010

The core kernel type code handles this properly, no driver
should ever do it.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

a21142b2

Staging: brcm80211: remove redefinition of size_t · 8fbfd3e6

Greg Kroah-Hartman authored Oct 05, 2010

No driver should ever do this.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

8fbfd3e6

Staging: brcm80211: remove PTRSZ definition · a725ad6c

Greg Kroah-Hartman authored Oct 05, 2010

It was never used.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

a725ad6c

Staging: brcm80211: remove floating point typedefs · a921fdc1

Greg Kroah-Hartman authored Oct 05, 2010

It's not ever used, so remove the typedef.  Floating point isn't used
in the kernel, so this could never be an issue here.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

a921fdc1

Staging: brcm80211: remove INLINE definition · 2d956e22

Greg Kroah-Hartman authored Oct 05, 2010

Use the "real" inline marking for functions.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

2d956e22

Staging: brcm80211: remove UNUSED_PARAMETER macro · 2a0b1392

Greg Kroah-Hartman authored Oct 05, 2010

It's not needed anywhere.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

2a0b1392

Staging: brcm80211: remove unneeded #ifdef checks · b7937582

Greg Kroah-Hartman authored Oct 05, 2010

For a .h file, you never need to check it, the .h file does it
itself.  For the typedef.h file, this is never needed.

Cc: Brett Rudley <brudley@broadcom.com>
Cc: Henry Ptasinski <henryp@broadcom.com>
Cc: Nohee Ko <noheek@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

b7937582

staging: brcm80211: Purge unused extern declarations · ea4381ff

Brett Rudley authored Oct 02, 2010

Purge unused extern declarations
Signed-off-by: Brett Rudley <brudley@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

ea4381ff

staging: brcm80211: Purge packets tags · 6d4e997b

Brett Rudley authored Oct 02, 2010

No longer need packet tags
Signed-off-by: Brett Rudley <brudley@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

6d4e997b

staging: brcm80211: Purge unused flags and macros from wlc_scb.h · 035f4c3b

Brett Rudley authored Oct 02, 2010

Purge unused flags and macros from wlc_scb.h
Signed-off-by: Brett Rudley <brudley@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

035f4c3b

staging: brcm80211: Purge unused #includes · be217b54

Brett Rudley authored Oct 01, 2010

Purging unused #includes.
Signed-off-by: Brett Rudley <brudley@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

be217b54

staging: brcm80211: Move #includes out of header · 3327989a

Brett Rudley authored Oct 01, 2010

Start the process of moving #includes out of headers and into individual C files.
For now, this patch addresses the softmac side of the driver, fullmac still to
be done.
Signed-off-by: Brett Rudley <brudley@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

3327989a

staging: brcm80211: Remove unneeded definitions and structures. · 34a5fa94

Henry Ptasinski authored Oct 01, 2010

Remove definitions and structures that are no longer needed.  Most of the
remaining ones can probably be replaced with the equivalent ones from
include/linux/ieee80211.h.
Signed-off-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

34a5fa94

staging: brcm80211: Remove unnecessary assertions. · 4594a934

Henry Ptasinski authored Oct 01, 2010

Remove assertions on the size of several structures.  These structures are
never used anywhere.
Signed-off-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

4594a934

staging: brcm80211: Remove unnecessary debug print routines. · 93ea8e6e

Henry Ptasinski authored Oct 01, 2010

Removed several unused and rarely used debug printout routines that look into
portions of the frame that are more properly left to the mac80211 stack.
Signed-off-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

93ea8e6e

staging: brcm80211: Remove unused field from wlc_info structure. · 453fb2cb

Henry Ptasinski authored Oct 01, 2010

Remove an unused field from the wlc_info structure, so that the underlying
structure can also be cleaned up.
Signed-off-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

453fb2cb

staging: brcm80211: Remove unnecessary assertions. · 8518a9e2

Henry Ptasinski authored Oct 01, 2010

Remove assertions on the size of several structures.  These structures are
never used anywhere.
Signed-off-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

8518a9e2

staging: brcm80211: Remove unused structure. · 47127f34

Henry Ptasinski authored Oct 01, 2010

Remove the wl_assoc_info_t structure. It's never used, and depends on
structures defined in other header files that can now also be cleaned up.
Signed-off-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

47127f34

staging: brcm80211: fix checkpatch error 'assignment in if condition' · 02266551

Jason Cooper authored Sep 30, 2010

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

02266551

staging: brcm80211: fix checkpatch error 'assignment in if condition' · 59334c2f

Jason Cooper authored Sep 30, 2010

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

59334c2f

staging: brcm80211: fix checkpatch error 'assignment in if condition' · 9b890325

Jason Cooper authored Sep 30, 2010

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

9b890325

staging: brcm80211: fix checkpatch error 'assignment in if condition' · a618cc28

Jason Cooper authored Sep 30, 2010

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

a618cc28

staging: brcm80211: fix checkpatch error 'assignment in if condition' · a1b04b0b

Jason Cooper authored Sep 30, 2010

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

a1b04b0b

staging: brcm80211: fix checkpatch error 'assignment in if condition' · 57c096b8

Jason Cooper authored Sep 30, 2010

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

57c096b8

staging: brcm80211: fix checkpatch errors 'assignment in if condition' · c74c95cd

Jason Cooper authored Sep 30, 2010

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

c74c95cd

staging: brcm80211: fix checkpatch error 'assignment in if condition' · eb7f37b3

Jason Cooper authored Sep 30, 2010

Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

eb7f37b3