* [PATCH] rt2800: fix TX_PIN_CFG setting for non MT7620 chips
@ 2017-08-25 11:38 Stanislaw Gruszka
2017-08-25 12:15 ` Daniel Golle
0 siblings, 1 reply; 3+ messages in thread
From: Stanislaw Gruszka @ 2017-08-25 11:38 UTC (permalink / raw)
To: linux-wireless; +Cc: Daniel Golle
Since commit 41977e86c984 ("rt2x00: add support for MT7620") we do not
initialize TX_PIN_CFG setting. This cause breakage at least on some
RT3573 devices. To fix the problem patch restores previous behaviour
for non MT7620 chips.
Fixes: 41977e86c984 ("rt2x00: add support for MT7620")
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1480829
Reported-and-tested-by: Jussi Eloranta <jussi.eloranta@csun.edu>
Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
---
drivers/net/wireless/ralink/rt2x00/rt2800lib.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
index d11c7b2..5672aec 100644
--- a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
+++ b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
@@ -3699,7 +3699,10 @@ static void rt2800_config_channel(struct rt2x00_dev *rt2x00dev,
if (rt2x00_rt(rt2x00dev, RT3572))
rt2800_rfcsr_write(rt2x00dev, 8, 0);
- rt2800_register_read(rt2x00dev, TX_PIN_CFG, &tx_pin);
+ if (rt2x00_rt(rt2x00dev, RT6352))
+ rt2800_register_read(rt2x00dev, TX_PIN_CFG, &tx_pin);
+ else
+ tx_pin = 0;
switch (rt2x00dev->default_ant.tx_chain_num) {
case 3:
--
2.7.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] rt2800: fix TX_PIN_CFG setting for non MT7620 chips
2017-08-25 11:38 [PATCH] rt2800: fix TX_PIN_CFG setting for non MT7620 chips Stanislaw Gruszka
@ 2017-08-25 12:15 ` Daniel Golle
2017-08-25 14:50 ` Stanislaw Gruszka
0 siblings, 1 reply; 3+ messages in thread
From: Daniel Golle @ 2017-08-25 12:15 UTC (permalink / raw)
To: Stanislaw Gruszka; +Cc: linux-wireless
Hi Stanislaw,
On Fri, Aug 25, 2017 at 01:38:29PM +0200, Stanislaw Gruszka wrote:
> Since commit 41977e86c984 ("rt2x00: add support for MT7620") we do not
> initialize TX_PIN_CFG setting. This cause breakage at least on some
> RT3573 devices. To fix the problem patch restores previous behaviour
> for non MT7620 chips.
>
> Fixes: 41977e86c984 ("rt2x00: add support for MT7620")
> Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1480829
> Reported-and-tested-by: Jussi Eloranta <jussi.eloranta@csun.edu>
> Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
> ---
> drivers/net/wireless/ralink/rt2x00/rt2800lib.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> index d11c7b2..5672aec 100644
> --- a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> +++ b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> @@ -3699,7 +3699,10 @@ static void rt2800_config_channel(struct rt2x00_dev *rt2x00dev,
> if (rt2x00_rt(rt2x00dev, RT3572))
> rt2800_rfcsr_write(rt2x00dev, 8, 0);
>
> - rt2800_register_read(rt2x00dev, TX_PIN_CFG, &tx_pin);
> + if (rt2x00_rt(rt2x00dev, RT6352))
> + rt2800_register_read(rt2x00dev, TX_PIN_CFG, &tx_pin);
should be
tx_pin = rt2800_register_read(rt2x00dev, TX_PIN_CFG);
since the calling convention of rt2800_register_read was changed in
commit eebd68e782 rt2x00: convert rt2800_register_read return type.
Probably you based the patch on an outdated tree...?
> + else
> + tx_pin = 0;
>
> switch (rt2x00dev->default_ant.tx_chain_num) {
> case 3:
> --
> 2.7.5
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] rt2800: fix TX_PIN_CFG setting for non MT7620 chips
2017-08-25 12:15 ` Daniel Golle
@ 2017-08-25 14:50 ` Stanislaw Gruszka
0 siblings, 0 replies; 3+ messages in thread
From: Stanislaw Gruszka @ 2017-08-25 14:50 UTC (permalink / raw)
To: Daniel Golle; +Cc: linux-wireless
On Fri, Aug 25, 2017 at 02:15:08PM +0200, Daniel Golle wrote:
> Hi Stanislaw,
>
> On Fri, Aug 25, 2017 at 01:38:29PM +0200, Stanislaw Gruszka wrote:
> > Since commit 41977e86c984 ("rt2x00: add support for MT7620") we do not
> > initialize TX_PIN_CFG setting. This cause breakage at least on some
> > RT3573 devices. To fix the problem patch restores previous behaviour
> > for non MT7620 chips.
> >
> > Fixes: 41977e86c984 ("rt2x00: add support for MT7620")
> > Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1480829
> > Reported-and-tested-by: Jussi Eloranta <jussi.eloranta@csun.edu>
> > Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
> > ---
> > drivers/net/wireless/ralink/rt2x00/rt2800lib.c | 5 ++++-
> > 1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> > index d11c7b2..5672aec 100644
> > --- a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> > +++ b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> > @@ -3699,7 +3699,10 @@ static void rt2800_config_channel(struct rt2x00_dev *rt2x00dev,
> > if (rt2x00_rt(rt2x00dev, RT3572))
> > rt2800_rfcsr_write(rt2x00dev, 8, 0);
> >
> > - rt2800_register_read(rt2x00dev, TX_PIN_CFG, &tx_pin);
> > + if (rt2x00_rt(rt2x00dev, RT6352))
> > + rt2800_register_read(rt2x00dev, TX_PIN_CFG, &tx_pin);
>
> should be
>
> tx_pin = rt2800_register_read(rt2x00dev, TX_PIN_CFG);
>
> since the calling convention of rt2800_register_read was changed in
> commit eebd68e782 rt2x00: convert rt2800_register_read return type.
> Probably you based the patch on an outdated tree...?
Yes, is was on 4.12. Will post v2.
Thanks
Stanislaw
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-08-25 14:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-25 11:38 [PATCH] rt2800: fix TX_PIN_CFG setting for non MT7620 chips Stanislaw Gruszka
2017-08-25 12:15 ` Daniel Golle
2017-08-25 14:50 ` Stanislaw Gruszka
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.