* PATCH Rtl8180 ch14 [3/3]
@ 2008-05-10 11:34 Andrea Merello
2008-05-14 20:08 ` [PATCH] Rtl8180 grf5101 ch14 John W. Linville
0 siblings, 1 reply; 2+ messages in thread
From: Andrea Merello @ 2008-05-10 11:34 UTC (permalink / raw)
To: John W. Linville; +Cc: linux-wireless, Alessandro Di Marco
Rtl8180: fix wrong parameter in RF code function invokation for gct radio
The grf5101 RF code needs to invoke grf5101_write_phy_antenna every time the
channel is being switch.
This should be done passing the channel number to that function.
Incorrectly we were passing the same value that is written on the
channel RF register.
This may cause problems when operating on ch 14.
This patch fixes it.
Thanks to Alessandro Di Marco who found this issue!
signed-off-by: Andrea Merello <andreamrl@tiscali.it>
---
--- a/drivers/net/wireless/rtl8180_grf5101.c 2008-02-02
14:48:52.000000000 +0100
+++ b/drivers/net/wireless/rtl8180_grf5101.c 2008-02-02
14:52:59.000000000 +0100
@@ -87,7 +87,7 @@ static void grf5101_rf_set_channel(struc
write_grf5101(dev, 0x0B, chan);
write_grf5101(dev, 0x07, 0x1000);
- grf5101_write_phy_antenna(dev, chan);
+ grf5101_write_phy_antenna(dev, conf->channel);
}
static void grf5101_rf_stop(struct ieee80211_hw *dev)
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Rtl8180 grf5101 ch14
2008-05-10 11:34 PATCH Rtl8180 ch14 [3/3] Andrea Merello
@ 2008-05-14 20:08 ` John W. Linville
0 siblings, 0 replies; 2+ messages in thread
From: John W. Linville @ 2008-05-14 20:08 UTC (permalink / raw)
To: Andrea Merello; +Cc: linux-wireless, Alessandro Di Marco
On Sat, May 10, 2008 at 01:34:16PM +0200, Andrea Merello wrote:
> Rtl8180: fix wrong parameter in RF code function invokation for gct r=
adio
>=20
> The grf5101 RF code needs to invoke grf5101_write_phy_antenna every t=
ime the
> channel is being switch.
>=20
> This should be done passing the channel number to that function.
> Incorrectly we were passing the same value that is written on the
> channel RF register.
> This may cause problems when operating on ch 14.
>=20
> This patch fixes it.
>=20
> Thanks to Alessandro Di Marco who found this issue!
>=20
>=20
> signed-off-by: Andrea Merello <andreamrl@tiscali.it>
> ---
>=20
> --- a/drivers/net/wireless/rtl8180_grf5101.c 2008-02-02
> 14:48:52.000000000 +0100
> +++ b/drivers/net/wireless/rtl8180_grf5101.c 2008-02-02
> 14:52:59.000000000 +0100
> @@ -87,7 +87,7 @@ static void grf5101_rf_set_channel(struc
> write_grf5101(dev, 0x0B, chan);
> write_grf5101(dev, 0x07, 0x1000);
>=20
> - grf5101_write_phy_antenna(dev, chan);
> + grf5101_write_phy_antenna(dev, conf->channel);
> }
>=20
> static void grf5101_rf_stop(struct ieee80211_hw *dev)
CC [M] drivers/net/wireless/rtl8180_grf5101.o
drivers/net/wireless/rtl8180_grf5101.c: In function =E2=80=98grf5101_rf=
_set_channel=E2=80=99:
drivers/net/wireless/rtl8180_grf5101.c:91: warning: passing argument 2 =
of =E2=80=98grf5101_write_phy_antenna=E2=80=99 makes integer from point=
er without a cast
This one is also whitespace-damaged...
John
--=20
John W. Linville
linville@tuxdriver.com
--
To unsubscribe from this list: send the line "unsubscribe linux-wireles=
s" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-05-14 20:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-05-10 11:34 PATCH Rtl8180 ch14 [3/3] Andrea Merello
2008-05-14 20:08 ` [PATCH] Rtl8180 grf5101 ch14 John W. Linville
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.