- 06 10月, 2011 29 次提交
-
-
由 Franky Lin 提交于
Remove some single line functions for bus register/unregister interface Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
threads_only is always true as we never use the tasklet implementation. So related code is removed. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
The driver uses variables which are stored in string format. Using strings as variable identifiers is disliked by the community. The driver has been cleaned up and the only module providing these variables is srom.c. The variable retrieval functions have been moved to srom.c in preparation of a more likable way to store and lookup these driver variables. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NAlwin Beukers <alwin@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
A bug was introduced by the following commit (Sep 13): staging: brcm80211: use endian annotated structures in brcmsmac Result was that 64 bits systems will not be able to acquire an IP address. Also the rmmod crashed. This has been fixed and retested on a Sparc64. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NAlwin Beukers <alwin@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Dpc and timer events can now be traced. Combined with Mac80211 driver callback tracing, all entry points into the driver can now be traced. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NAlwin Beukers <alwin@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
Removed the brcms_c_nmode_validate function as it only checks N-mode capability, which should always be true. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
Combined if statements in brcms_c_edcf_setparams and brcms_c_wme_setparams functions. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
- removed brcms_b_dotxstatus - changed 'fatal' argument of brcms_b_txstatus from INOUT to OUT. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
- removed copying field wlc->machwcap from brcms_hardware to brcms_c_info during module attach. - field wlc->machwcap was written but never read. Therefore it was removed. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
- removed call to brcms_b_tx_fifo_suspended on receipt of TX FIFO suspend/flush completion signal. - removed brcms_b_tx_fifo_suspended function as it is not being used elsewhere. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
- removed brcms_c_write_template_ram function. - removed brcms_c_mctrl function. - removed brcms_c_pllreq function. - removed brcms_c_mhf function. - removed brcms_c_rate_shm_offset function. - removed brcms_c_write_hw_bcntemplates function. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
Removed brcms_c_xmtfifo_sz_get function. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
- removed brcms_c_read_shm and brcms_c_write_shm functions. - removed redundant argument check from brcms_c_copyto_shm function. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
The comment says the rateset is being cleared when switching to GMODE_LEGACY_B, but the code that follows it does not do so. Besides, the rateset is already valid at this point. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
Empty functions are good candidates for removal. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
- removed unneeded return statements at end of functions. - changed return value of brcms_c_radio_monitor_start to void. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
Functions were trivial so not really adding value. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Alwin Beukers 提交于
Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Stop assigning priority to watchdog and dpc threads. Remove related code. Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
Use constant for interrupt/polling mode configuration. Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Franky Lin 提交于
rtnl lock is used improperly in fullmac. This patch intends to clean them up. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NArend van Spriel <arend@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Reviewed-by: NSukesh Srikakula <sukeshs@broadcom.com> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Fix double include as detected by checkincludes script. Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
The kernel provides bit operations to replace brcmu_bitcount so using those instead. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
The functions brcmu_bprintf and brcmu_binit could be replaced by snprintf kernel function. These are no longer used in both brcm80211 drivers so they have been removed. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
In the receive path of the brcmsmac driver probe request frames and frames with invalid mac address fields are dropped. This is only fine for STA mode. Anticipating coming AP and monitor mode functionality this is unwanted behavior. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Arend van Spriel 提交于
The ssn should be given a valid value by the driver or it should not override the value. Removed override to zero from the driver. Reported-by: NJohannes Berg <johannes@sipsolutions.net> Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NRoland Vossen <rvossen@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Roland Vossen 提交于
Several fields were unused or were only set once. They have been removed. Uncalled functions that relied on these constants have been removed as well. Reviewed-by: NPieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: NFranky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NArend van Spriel <arend@broadcom.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
- 05 10月, 2011 11 次提交
-
-
由 Mark Einon 提交于
The mechanism by which the device is put into low power sleep is broken in that the device can never come back out of low power mode afterwards. Temorary fix to bring the device back out of sleep almost immediately, until a suitable wake trigger can be found. Signed-off-by: NMark Einon <mark.einon@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Mark Einon 提交于
Adding copyright notices and adding myself as a module author. Signed-off-by: NMark Einon <mark.einon@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Mark Einon 提交于
set_mac is implemented, and there have been lots of new ethtool_ops added, so removing the TODO to add more kernel parameters. Signed-off-by: NMark Einon <mark.einon@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Mark Einon 提交于
The error_timer was only getting initialised and an initial jiffies value set following a probe. This could result in the timer needlessly expiring immediately after et131x_open is called. Now this is all done from the open call instead. Signed-off-by: NMark Einon <mark.einon@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Mark Einon 提交于
Added basic suspend & resume functionality. Tested on an et1310 device, and putting Fedora15 host in and out of hibernation successfully. Signed-off-by: NMark Einon <mark.einon@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Mark Einon 提交于
adapter->bmsr is no longer being updated, but is also used to check the link state in places. Remove bmsr from adapter, and replace link state checks with phydev->link check. Signed-off-by: NMark Einon <mark.einon@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Mark Einon 提交于
In several places in the code, the pm_csr register is read and the PHY_SW_COMA bit checked. Move this check into its own small function to make the code more readable. Signed-off-by: NMark Einon <mark.einon@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 Mark Einon 提交于
Removing this as I'm pretty sure its not true, and alloc_etherdev isn't provided by this driver anyway. Alternatively, its a badly written comment and I don't understand it. This drivers use of alloc_etherdev() is within keeping with other net devices, so I'm happy. Signed-off-by: NMark Einon <mark.einon@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 David Kilroy 提交于
The message is in the wrong place. We definitely don't want to return at this point, as we may need to turn off encryption. Reported-by: NHenk de Groot <pe1dnn@amsat.org> Signed-off-by: NDavid Kiroy <kilroyd@googlemail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 David Kilroy 提交于
The function no longer compiles, so we just remove it. Reported-by: NHenk de Groot <pe1dnn@amsat.org> Signed-off-by: NDavid Kilroy <kilroyd@googlemail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-
由 David Kilroy 提交于
in DBG machinery so you don't have to declare DBG_FUNC at the start of all functions. This just makes it easy to add DBG conforming to existing code. The patch reformats the changed #defines to satisfy checkstyle.pl Signed-off-by: NDavid Kilroy <kilroyd@googlemail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
-