* [PATCH 2/3] net: wireless: b43: Fixed Pointer issue
2015-10-17 23:01 [PATCH 1/3] net: wireless: rsi: Coding Spaces Paul McQuade
@ 2015-10-17 23:01 ` Paul McQuade
[not found] ` <1445122898-5810-2-git-send-email-paulmcquad-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-10-17 23:01 ` [PATCH 3/3] net: wireless: b43: statics Don't init to 0 Paul McQuade
2016-02-02 16:18 ` [1/3] net: wireless: rsi: Coding Spaces Sudip Mukherjee
2 siblings, 1 reply; 6+ messages in thread
From: Paul McQuade @ 2015-10-17 23:01 UTC (permalink / raw)
To: paulmcquad
Cc: kvalo, zajec5, johannes.berg, davem, joe, m, mike.looijmans,
cengelma, khoroshilov, linux-wireless, b43-dev, netdev,
linux-kernel
Moved around pointer to avoid ERROR
Signed-off-by: Paul McQuade <paulmcquad@gmail.com>
---
drivers/net/wireless/b43/dma.h | 14 +++++++-------
drivers/net/wireless/b43/main.c | 6 +++---
drivers/net/wireless/b43/main.h | 2 +-
drivers/net/wireless/b43/xmit.h | 2 +-
4 files changed, 12 insertions(+), 12 deletions(-)
enum b43_dmatype {
diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
index 2849070..040caa4 100644
--- a/drivers/net/wireless/b43/main.c
+++ b/drivers/net/wireless/b43/main.c
@@ -364,7 +364,7 @@ static struct ieee80211_supported_band b43_band_2ghz_limited = {
static void b43_wireless_core_exit(struct b43_wldev *dev);
static int b43_wireless_core_init(struct b43_wldev *dev);
-static struct b43_wldev * b43_wireless_core_stop(struct b43_wldev *dev);
+static struct b43_wldev *b43_wireless_core_stop(struct b43_wldev *dev);
static int b43_wireless_core_start(struct b43_wldev *dev);
static void b43_op_bss_info_changed(struct ieee80211_hw *hw,
struct ieee80211_vif *vif,
@@ -989,7 +989,7 @@ static void do_key_write(struct b43_wldev *dev,
* 0xffffffff and let's b43_op_update_tkip_key provide a
* correct pair.
*/
- rx_tkip_phase1_write(dev, index, 0xffffffff, (u16*)buf);
+ rx_tkip_phase1_write(dev, index, 0xffffffff, (u16 *)buf);
} else if (index >= pairwise_keys_start) /* clear it */
rx_tkip_phase1_write(dev, index, 0, NULL);
if (key)
@@ -4334,7 +4334,7 @@ out_unlock:
/* Locking: wl->mutex
* Returns the current dev. This might be different from the passed in dev,
* because the core might be gone away while we unlocked the mutex. */
-static struct b43_wldev * b43_wireless_core_stop(struct b43_wldev *dev)
+static struct b43_wldev *b43_wireless_core_stop(struct b43_wldev *dev)
{
struct b43_wl *wl;
struct b43_wldev *orig_dev;
diff --git a/drivers/net/wireless/b43/main.h b/drivers/net/wireless/b43/main.h
index c46430c..fa56b22 100644
--- a/drivers/net/wireless/b43/main.h
+++ b/drivers/net/wireless/b43/main.h
@@ -73,7 +73,7 @@ static inline int b43_is_ofdm_rate(int rate)
u8 b43_ieee80211_antenna_sanitize(struct b43_wldev *dev,
u8 antenna_nr);
-void b43_tsf_read(struct b43_wldev *dev, u64 * tsf);
+void b43_tsf_read(struct b43_wldev *dev, u64 *tsf);
void b43_tsf_write(struct b43_wldev *dev, u64 tsf);
u32 b43_shm_read32(struct b43_wldev *dev, u16 routing, u16 offset);
diff --git a/drivers/net/wireless/b43/xmit.h b/drivers/net/wireless/b43/xmit.h
index ba61153..fed8657 100644
--- a/drivers/net/wireless/b43/xmit.h
+++ b/drivers/net/wireless/b43/xmit.h
@@ -203,7 +203,7 @@ size_t b43_txhdr_size(struct b43_wldev *dev)
int b43_generate_txhdr(struct b43_wldev *dev,
- u8 * txhdr,
+ u8 *txhdr,
struct sk_buff *skb_frag,
struct ieee80211_tx_info *txctl, u16 cookie);
--
2.6.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 3/3] net: wireless: b43: statics Don't init to 0
2015-10-17 23:01 [PATCH 1/3] net: wireless: rsi: Coding Spaces Paul McQuade
2015-10-17 23:01 ` [PATCH 2/3] net: wireless: b43: Fixed Pointer issue Paul McQuade
@ 2015-10-17 23:01 ` Paul McQuade
2015-10-18 9:32 ` Rafał Miłecki
2016-02-02 16:18 ` [1/3] net: wireless: rsi: Coding Spaces Sudip Mukherjee
2 siblings, 1 reply; 6+ messages in thread
From: Paul McQuade @ 2015-10-17 23:01 UTC (permalink / raw)
To: paulmcquad
Cc: kvalo, zajec5, johannes.berg, davem, joe, m, mike.looijmans,
cengelma, khoroshilov, linux-wireless, b43-dev, netdev,
linux-kernel
Don't turn statics to 0 or NULL
Signed-off-by: Paul McQuade <paulmcquad@gmail.com>
---
drivers/net/wireless/b43/main.c | 2 +-
drivers/net/wireless/b43/phy_lp.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
index 040caa4..2b50650 100644
--- a/drivers/net/wireless/b43/main.c
+++ b/drivers/net/wireless/b43/main.c
@@ -109,7 +109,7 @@ int b43_modparam_verbose = B43_VERBOSITY_DEFAULT;
module_param_named(verbose, b43_modparam_verbose, int, 0644);
MODULE_PARM_DESC(verbose, "Log message verbosity: 0=error, 1=warn, 2=info(default), 3=debug");
-static int b43_modparam_pio = 0;
+static int b43_modparam_pio;
module_param_named(pio, b43_modparam_pio, int, 0644);
MODULE_PARM_DESC(pio, "Use PIO accesses by default: 0=DMA, 1=PIO");
diff --git a/drivers/net/wireless/b43/phy_lp.c b/drivers/net/wireless/b43/phy_lp.c
index 058a9f2..086f0ba 100644
--- a/drivers/net/wireless/b43/phy_lp.c
+++ b/drivers/net/wireless/b43/phy_lp.c
@@ -2502,7 +2502,7 @@ static int lpphy_b2063_tune(struct b43_wldev *dev,
{
struct ssb_bus *bus = dev->dev->sdev->bus;
- static const struct b206x_channel *chandata = NULL;
+ static const struct b206x_channel *chandata;
u32 crystal_freq = bus->chipco.pmu.crystalfreq * 1000;
u32 freqref, vco_freq, val1, val2, val3, timeout, timeoutref, count;
u16 old_comm15, scale;
--
2.6.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 3/3] net: wireless: b43: statics Don't init to 0
2015-10-17 23:01 ` [PATCH 3/3] net: wireless: b43: statics Don't init to 0 Paul McQuade
@ 2015-10-18 9:32 ` Rafał Miłecki
0 siblings, 0 replies; 6+ messages in thread
From: Rafał Miłecki @ 2015-10-18 9:32 UTC (permalink / raw)
To: Paul McQuade
Cc: Kalle Valo, Johannes Berg, David Miller, Joe Perches,
Michael Büsch, mike.looijmans, cengelma, khoroshilov,
linux-wireless@vger.kernel.org, b43-dev, Network Development,
Linux Kernel Mailing List
On 18 October 2015 at 01:01, Paul McQuade <paulmcquad@gmail.com> wrote:
> Don't turn statics to 0 or NULL
Same request as Michael's in 2/3. Your "statics Don't init to 0"
message sounds strange, statics *are* initialized to 0 by default (as
you probably know but described it in a unclear way).
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [1/3] net: wireless: rsi: Coding Spaces
2015-10-17 23:01 [PATCH 1/3] net: wireless: rsi: Coding Spaces Paul McQuade
2015-10-17 23:01 ` [PATCH 2/3] net: wireless: b43: Fixed Pointer issue Paul McQuade
2015-10-17 23:01 ` [PATCH 3/3] net: wireless: b43: statics Don't init to 0 Paul McQuade
@ 2016-02-02 16:18 ` Sudip Mukherjee
2 siblings, 0 replies; 6+ messages in thread
From: Sudip Mukherjee @ 2016-02-02 16:18 UTC (permalink / raw)
To: Paul Mcquade
Cc: kvalo, zajec5, johannes.berg, davem, joe, m, mike.looijmans,
cengelma, khoroshilov, linux-wireless, b43-dev, netdev,
linux-kernel
On Sun, Oct 18, 2015 at 12:01:36AM +0100, Paul Mcquade wrote:
> Removed space before that ','
>
> Signed-off-by: Paul McQuade <paulmcquad@gmail.com>
The From: name and Signed-off-by: name should be same else some of the
stats generating tools will break. It should be Paul McQuade, not
Paul Mcquade. Maybe you have modified your .gitconfig recently.
And in the subject net: wireless: is not required.
Can you please send v2..
regards
sudip
^ permalink raw reply [flat|nested] 6+ messages in thread