History log of /drivers/staging/winbond/wbusb.c
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
bac2c126e452eb00f91305ba2c04a8b2bd95acf0 18-Nov-2011 Greg Kroah-Hartman <gregkh@suse.de> USB: convert drivers/staging/* to use module_usb_driver()

This converts the drivers in drivers/staging/* to use the
module_usb_driver() macro which makes the code smaller and a bit
simpler.

Added bonus is that it removes some unneeded kernel log messages about
drivers loading and/or unloading.

Cc: "David Täht" <d@teklibre.com>
Cc: Marek Belisko <marek.belisko@gmail.com>
Cc: Al Cho <acho@novell.com>
Cc: Forest Bond <forest@alittletooquiet.net>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Huajun Li <huajun.li.lee@gmail.com>
Cc: Zac Storer <zac.3.14159@gmail.com>
Cc: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: edwin_rong <edwin_rong@realsil.com.cn>
Cc: Ben Hutchings <ben@decadent.org.uk>
Cc: Julia Lawall <julia@diku.dk>
Cc: Ilia Mirkin <imirkin@alum.mit.edu>
Cc: Joe Perches <joe@perches.com>
Cc: Pekka Enberg <penberg@kernel.org>
Cc: "John W. Linville" <linville@tuxdriver.com>
Cc: Paul Gortmaker <paul.gortmaker@windriver.com>
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
452962366c11a9126fabac8cb28af49c27464408 30-Aug-2011 Paul Gortmaker <paul.gortmaker@windriver.com> staging: Add module.h to more drivers implicitly using it.

This 2nd batch of implicit module.h users only appeared when we
removed the unnecessary module.h from include/linux/miscdevice.h
[The 1st batch is already present in Greg's staging tree.]

Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
/drivers/staging/winbond/wbusb.c
37a41b4affa33bb237d3692bf51f1b5ebcaf29d8 21-Sep-2011 Eliad Peller <eliad@wizery.com> mac80211: add ieee80211_vif param to tsf functions

TSF can be kept per vif.
Add ieee80211_vif param to set/get/reset_tsf, and move
the debugfs entries to the per-vif directory.

Update all the drivers that implement these callbacks.

Signed-off-by: Eliad Peller <eliad@wizery.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
/drivers/staging/winbond/wbusb.c
7bb4568372856688bc070917265bce0b88bb7d4d 24-Feb-2011 Johannes Berg <johannes.berg@intel.com> mac80211: make tx() operation return void

The return value of the tx operation is commonly
misused by drivers, leading to errors. All drivers
will drop frames if they fail to TX the frame, and
they must also properly manage the queues (if they
didn't, mac80211 would already warn).

Removing the ability for drivers to return a BUSY
value also allows significant cleanups of the TX
TX handling code in mac80211.

Note that this also fixes a bug in ath9k_htc, the
old "return -1" there was wrong.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Tested-by: Sedat Dilek <sedat.dilek@googlemail.com> [ath5k]
Acked-by: Gertjan van Wingerde <gwingerde@gmail.com> [rt2x00]
Acked-by: Larry Finger <Larry.Finger@lwfinger.net> [b43, rtl8187, rtlwifi]
Acked-by: Luciano Coelho <coelho@ti.com> [wl12xx]
Signed-off-by: John W. Linville <linville@tuxdriver.com>
/drivers/staging/winbond/wbusb.c
2894c6cd0ea84c30fc028ba92f8e61fbe7971e65 28-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Kill struct wb_usb

This patch kills struct wb_usb which now only contains a pointer to struct
usb_device.

Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
9be98819d31c4e105ad028ee092d27a46a9e298a 28-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Remove unused fields from struct wb_usb

This patch removes two unused fields from struct wb_usb:

- DetectCount which is always zero

- IsUsb20 which is a write-only struct member

Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
4398954e5f91e340f5a2db53d219b6562e3e1bf0 28-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Kill wblinux_f.h header

The wblinux_f.h header file doesn't contain anything that's actually used. Kill
it.

Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
01b5ceeca5aa45735a987420c83228a9778a791f 28-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Merge mlmetxrx.c to mds.c

This patch merges mlmetxrx.c to mds.c because it's small and the functions are
only used in mto.c.

Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
9255650cbf997762bd667896b0b86661cd81c9b4 28-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Kill Vendor2 ifdef from hal_init_hardware

The Vendor2 macro is never defined so remove the ifdef'd block from
hal_init_hardware().

Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
2855bb79d75ad1419b1f54008f861a39517acfd5 28-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Use pr_debug() for debugging

Use pr_debug() for debugging printk's and kill the FULL_DEBUG macro. It would
be even better to use dev_dbg() but unfortunately looking up struct device in
the current code structure makes things very ugly.

Please note that I dropped the DataDmp() calls from
RFSynthesizer_SwitchingChannel() because that function doesn't exist.

Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
4d0d302257d3d0eccf371aab49c74b41797bebc3 01-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Kill struct hwdata ->NullPacketCount

This patch kills the NullPacketCount member of struct hwdata. It's not used for
anything so it's safe to remove it.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
6112063d804aa0afc8d4462b22e36fc259393b64 01-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Kill empty Mds_Destroy function

The Mds_Destroy() function doesn't do anything so kill it.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
b5ef076141acc223e55aa298abd2d2f5fb844874 01-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Rename wbhal_s.h to wbhal.h

This patch renames the wbhal_s.h header file to wbhal.h now that it contains
both structure and function definitions.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
72ca8819f7f4ac4ddf82ab72ee614075821a3d00 01-Nov-2010 Pekka Enberg <penberg@kernel.org> Staging: w35und: Merge wbhal_f.h to wbhal_s.h

This patch merges HAL struct and function definitions into one header file.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
11c9a55d08d3d79c7a690312966567f70758886a 19-Sep-2010 Pekka Enberg <penberg@kernel.org> staging: w35und: Inline MLMESendFrame() to wbsoft_tx()

The wbsoft_tx() function is a simple wrapper on top of MLMESendFrame() so
inline the latter to the former.

Cc: Lars Lindley <lindley@coyote.org>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
80a284bd154ee3a9d38ee9c264a3a2bf6114385e 19-Sep-2010 Pekka Enberg <penberg@kernel.org> staging: w35und: Use NETDEV_TX_BUSY if MLMESendFrame fails

This patch changes MLMESendFrame to return NETDEV_TX_BUSY if MLME frame is in
use so that wbsoft_tx() doesn't blindly return NETDEV_TX_OK in that case.

Cc: Sandro Bonazzola <sandro.bonazzola@gmail.com>
Cc: Lars Lindley <lindley@coyote.org>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
89b32d45fe88884d4780057dd18a098b402e358c 18-Sep-2010 Pekka Enberg <penberg@kernel.org> staging: w35und: Remove unused spinlocks

This patch removes unused spinlocks from "struct mlme_frame" and "struct
wbsoft_priv".

Cc: Lars Lindley <lindley@coyote.org>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Ruslan Pisarev <ruslan@rpisarev.org.ua>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
e9563355ac1175dd3440dc2ea5c28b27ed51a283 05-Aug-2010 Greg Kroah-Hartman <gregkh@suse.de> Staging: Merge staging-next into Linus's tree

Conflicts:
drivers/staging/Kconfig
drivers/staging/batman-adv/bat_sysfs.c
drivers/staging/batman-adv/device.c
drivers/staging/batman-adv/hard-interface.c
drivers/staging/cx25821/cx25821-audups11.c

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
f69b0d6451679f1466381a46ac7ab671a7b5668c 12-Jul-2010 Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Staging: fix typos concerning "address"

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
a2531293dbb7608fa672ff28efe3ab4027917a2f 18-Jul-2010 Pavel Machek <pavel@ucw.cz> update email address

pavel@suse.cz no longer works, replace it with working address.

Signed-off-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
/drivers/staging/winbond/wbusb.c
c8d1a126924fcbc1d61ceb830226e0c7afdcc841 21-May-2010 Greg Kroah-Hartman <gregkh@suse.de> Merge staging-next tree into Linus's latest version

Conflicts:
drivers/staging/arlan/arlan-main.c
drivers/staging/comedi/drivers/cb_das16_cs.c
drivers/staging/cx25821/cx25821-alsa.c
drivers/staging/dt3155/dt3155_drv.c
drivers/staging/hv/hv.c
drivers/staging/netwave/netwave_cs.c
drivers/staging/wavelan/wavelan.c
drivers/staging/wavelan/wavelan_cs.c
drivers/staging/wlags49_h2/wl_cs.c

This required a bit of hand merging due to the conflicts
that happened in the later .34-rc releases, as well as
some staging driver changing coming in through other trees
(v4l and pcmcia).

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
e5851c205d7d1920389ab4368f6f374d3594b93d 02-Apr-2010 Lars Lindley <lindley@coyote.org> Staging: winbond: wbusb.c Coding style fixes.

I fixed the reported checkpatch.pl problems except for a bunch of
long lines and some printk:s.
I also removed versioning comments.

Signed-off-by: Lars Lindley <lindley@coyote.org>
Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
22bedad3ce112d5ca1eaf043d4990fa2ed698c87 01-Apr-2010 Jiri Pirko <jpirko@redhat.com> net: convert multicast list to list_head

Converts the list and the core manipulating with it to be the same as uc_list.

+uses two functions for adding/removing mc address (normal and "global"
variant) instead of a function parameter.
+removes dev_mcast.c completely.
+exposes netdev_hw_addr_list_* macros along with __hw_addr_* functions for
manipulation with lists on a sandbox (used in bonding and 80211 drivers)

Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
/drivers/staging/winbond/wbusb.c
a71b676fa9b92119c0fb4e3fab34e45a17f15107 04-Mar-2010 Greg Kroah-Hartman <gregkh@suse.de> Staging: winbond: fix up wireless api errors

The wireless core made a number of api changes that caused
the winbond driver to break. This patch fixes those errors by
making the needed changes to the driver.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
a457732b836b970c82c7ba35b4cfc938c9c543f9 10-Jan-2010 Németh Márton <nm127@freemail.hu> staging: make USB device id constant

The id_table field of the struct usb_device_id is constant in <linux/usb.h>
so it is worth to make the initialization data also constant.

The semantic match that finds this kind of pattern is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@r@
disable decl_init,const_decl_init;
identifier I1, I2, x;
@@
struct I1 {
...
const struct I2 *x;
...
};
@s@
identifier r.I1, y;
identifier r.x, E;
@@
struct I1 y = {
.x = E,
};
@c@
identifier r.I2;
identifier s.E;
@@
const struct I2 E[] = ... ;
@depends on !c@
identifier r.I2;
identifier s.E;
@@
+ const
struct I2 E[] = ...;
// </smpl>

Signed-off-by: Németh Márton <nm127@freemail.hu>
Cc: Julia Lawall <julia@diku.dk>
Cc: cocci@diku.dk
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
440a233db4c117f34397abd3722d5802fe1b8c30 18-Dec-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: Convert typedef ChanInfo to struct chan_info

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
6d7fd647c236c5b4ac4a27b17180ea8d7d169c8d 08-Oct-2009 Jeff Mahoney <jeffm@suse.com> Staging: winbond: implement prepare_multicast and fix API usage

This patch adds a prepare_multicast callback for the winbond driver
to properly receive mc_count in ->configure_filter.

This also fixes incompatible pointer assignment build errors because
->configure_filter had changed.

This is build tested only, but that's more than the original code received.

Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
4074e77ca3c0c5047acb012a440022ca1f3a2dab 20-Sep-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: Fix ->beacon_int breakage

Commit f424afa17899408cbd267a4c4534ca6fc9d8f71c ("mac80211: remove
deprecated API") removed ->beacon_int from struct ieee80211_conf. Fix
breakage in w35und by setting beacon period in ->add_interface to
bss_conf.beacon_int.

Cc: Jiri Benc <jbenc@suse.cz>
Cc: Jiri Kosina <jkosina@suse.cz>
Cc: Johannes Berg <johannes@sipsolutions.net>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Sandro Bonazzola <sandro.bonazzola@gmail.com>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
9dccc61c83db762c2bef1e100643032b92548c87 12-Aug-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: Remove some write-only struct members

There's no point in keeping around struct members that are only written
to but never read.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
24b8a9dfc7746273bde5a2030e4f16391251e830 29-Apr-2009 Tim Gardner <tim.gardner@canonical.com> Staging: winbond: mac80211 - unify config_interface and bss_info_changed

The commit 'mac80211: unify config_interface and bss_info_changed' from
Johannes Berg <johannes@sipsolutions.net> removed the config_interface structure
tag from struct ieee80211_ops. The BSSID detection functionality migrated to
ieee80211_ops.bss_info_changed.

Since wbsoft_config_interface() was largely empty, there wasn't much to do
other then to remove the function itself. There is currently no support
for BSSID change detection.

Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
22a82bcd5bc99fb7411ea8eca6e926bb0a82a6d5 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: merge wbhal.c to wbusb.c

Impact: cleanup

This patch moves all the functions in wbhal.c that are used only in
wbusb.c to the latter file.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
833d0cd39ea0f1619e6ae9fa322d07761d59a8a3 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: make led lookup tables static

Impact: cleanup

No need to keep read-only data on the stack.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
7c49a0ac1294caa4ac791181c250f0b86d18a043 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: remove MODULE_AUTHOR

Impact: cleanup

The contact details in MODULE_AUTHOR do not reflect current state of
affairs so remove it.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
a32b9810d920e28072b955ed020f21672fa7eac5 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: reformat wbusb.c

Impact: cleanup

Use scripts/Lindent on the file and clean up the rest by hand.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
9ca748ce517a09068a897fddf9672f2a3286573f 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: inline hal_set_phy_type() to wb35_hw_init()

Impact: cleanup

The hal_set_phy_type() is called in wb35_hw_init() only so inline the
function there. Also remove a redundant assignment of ->phy_type to
RF_WB_242_1.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
bdbb8839567c38f15bfba95e7df8a73a45f49067 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: clean up comments in wbusb.c

Impact: cleanup

Remove some useless comments and clean up others.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
f592a859b2e9d17182bbef71c79851505e3592b8 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: simplify hal_init_hardware() and hal_halt()

Impact: cleanup

Now that hal_halt() is called from wb35_hw_halt() only where
->InitialResource is always set to 4, we can simplify
hal_init_hardware() and hal_halt().

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
a39ee671746b6483c6a259fef755a4a3e475e3e4 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: more simplify hal_init_hardware() error handling

Impact: fix, cleanup

If initialization in hal_init_hardware() fails, we call hal_halt() to
clean up and release resource. However, hal_halt() will attempt to
call del_timer_sync() on ->LEDTimer on "stage 3" although it's not
initialized at that point. Fix that up by simplifying error handling
logic in hal_init_hardware() with gotos.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
00e2e05dd08bc15b09514f5ce35f0b9a3534e9ba 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: simplify hal_init_hardware() error handling

Impact: cleanup

Use gotos to simplify the deep if-statement nesting in
hal_init_hardware().

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
cfe31f81e1ba7e0c590fc1f2fd688309b685654a 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: simplify error handling in wb35_hw_init()

Impact: cleanup

Change hal_init_hardware() to return an error code rather than a boolean
to simplify error handling in wb35_hw_init().

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
80767e6e1e545b5cd7e7c6962fe0e06589d48ae2 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: move hal_init_hardware() and hal_halt() int wbusb.c

Impact: cleanup

The hal_init_hardware() and hal_halt() functions are only used in
wbusb.c so move them there.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
2659851122f6bd822ff3a9a4a13453303c9b5d8e 08-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: replace switch error handling with gotos in wb35_hw_init()

Impact: cleanup

This patch replaces the switch-based error handling in wb35_hw_init()
with regular gotos.

Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
6ab32127205adf6c16942fdc2e3ee10f2b20bce5 27-Mar-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: remove unused code from wbsoft_configure_filter()

We don't initialize hardware multicast filter in the driver nor do we know how
to do that. Therefore, remove some code that isn't actually used from
wbsoft_configure_filter().

Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
82fbb01cf3c2bd150262269bd017431d7dd74063 16-Apr-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: unregister device from the ieee80211 stack upon ->disconnect()

Impact: fix module removal

This patch fixes an oops when the w35und module is removed from the
kernel and added back.

Reported-by: luoyi <luoyi.ly@gmail.com>
Tested-by: Sandro Bonazzola <sandro.bonazzola@gmail.com>
Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
8e41b4b65d20f1321bc969b5de6038d5be33c9bd 12-Jan-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: remove hw_data_t typedef

As this typedef is used everywhere in the driver, remove it in a separate
patch.

Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
eb62f3eaf7ff1e021a0f066a0f04ee45d0d1eadb 08-Jan-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: typedef removal

This patch removes some typedefs from the driver code. I also removed some
unused structs I spotted while removing the typedefs.

Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
0c59dbaadf1bebdccacd616f3ebf375d81027f02 08-Jan-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: kill WBDEBUG and remove common.h header file

The only remaining thing in common.h header file is the WBDEBUG() macro which
is unconditionally defined as printk(). Kill the macro and remove the header
file.

Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
64328c8711d72794446bbafe23890468786424cc 07-Jan-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: unify mto.h and mto_f.h header files

No need to keep function definitions separate from the rest so unify mto.h and
mto_f.h header files.

Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
acfa5110b83b171ec509eaf2d1a9e93a5f4709bd 30-Jan-2009 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: fix usb_control_msg() error handling in wb35_probe()

If successful, the usb_control_msg() function returns the number of
bytes transferred. Fix up wb35_probe() to only bail out if the function returns
a negative number. Also, fix up ieee80211_alloc_hw() error code to ENOMEM;
otherwise GCC complains that err might be undefined (and is right about that).

Acked-by: Pavel Machek <pavel@suse.cz>
Reported-and-tested-by: Sandro Bonazzola <sandro.bonazzola@gmail.com>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
05e361cae5e633c2b58967d1444cf6ae56662e5c 30-Jan-2009 Pavel Machek <pavel@suse.cz> Staging: w35und: fix registration with wlan stack

Initialize few more fields in wireless device structure so that
wireless core actually accepts our registration.

Signed-off-by: Pavel Machek <pavel@suse.cz>
Acked-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
f02466fc19f77fc47a0573d5c246b3333e076894 05-Jan-2009 Greg Kroah-Hartman <gregkh@suse.de> Staging: w35und: fix config build warnings

the wireless config function has changed with commit
e8975581f63870be42ff4662b293d1b0c8c21350 so fix up the wbusb driver to
work properly with that change.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
54369cc69c4625e48838b15151a39e643d9b6ad8 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: remove ->shutdown from struct wbsoft_priv

It's not actually used for anything, so remove it.

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
3cae503bf6dba576c13fd94c54fb6ddd9247799e 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: move packet_came() to wb35rx.c

The function no longer has dependencies to wbusb.c so we can move it to
wb35rx.c and make it static now.

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
c930e0c008a1b81429e5dec089970b50f97ef593 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: move global wbsoft_enabled to struct wbsoft_priv

This is a preparational step for moving packet_came() to wb35rx.c().

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
82f8c2cd469211e6ae7e0a3c51eb93ab78f2427c 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: remove unused ->ShutDowned member from struct LOCAL_PARA

It's not actually read by anyone so we might as well remove it.

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
912b209f50dea532c25f2178e20024700258949b 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: clean up wblinux.c a bit

This patch removes wrapper functions from wblinux.c and moves the hardware init
and halt functions to wbusb.c which has the only callers of them.

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
7b9a79bf4849fbf7389bd845052014b02d2b9bda 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: inline DRIVER_AUTHOR and DRIVER_DESC macros

They're used in one place so we can inline them.

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
7fff1316b9ecc415fb1d43f438c0668884a24881 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: remove global struct ieee80211_hw

Remove the my_dev global variable from wbusb.c by passing a pointer to struct
ieee80211_hw around so that packet_came() gets it.

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
1e8a2b60339a050f4fd383e332a5dd66b75f4f02 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: merge struct wb35_adapter to struct wbsoft_priv

This patch merges struct wb35_adapter to struct wbsoft_priv. Now we can finally
start passing a pointer to struct ieee80211_hw around where necessary.

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
cc18071068f518c3c01616d7900de3b77252ccdf 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: move struct wbsoft_priv to core.h and use it

This patch removes my_adapter global variable from wbusb.c by adding a
->adapter member to struct wbsoft_priv.

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c
9ce922fde7fb44a8690aa37d3c7f4f0cf5d921ca 30-Oct-2008 Pekka Enberg <penberg@cs.helsinki.fi> Staging: w35und: move source files to one directory

As we're trying to get rid of the "compatability layer" in the driver, move
everything under one directory. Keeping some of the files under
drivers/staging/winbond/linux is a major pain in the ass whenever you're
cleaning up the driver.

Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
/drivers/staging/winbond/wbusb.c