* [PATCH] b43: HT-PHY: init: add missing small-tables writes
@ 2011-08-13 15:54 Rafał Miłecki
2011-08-23 12:45 ` Rafał Miłecki
0 siblings, 1 reply; 7+ messages in thread
From: Rafał Miłecki @ 2011-08-13 15:54 UTC (permalink / raw)
To: linux-wireless, John W. Linville; +Cc: b43-dev, Rafał Miłecki
Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
---
John: this one depends on
[RFC][PATCH] b43: HT-PHY: allow writing longer tables with a single call
, please apply the one above.
---
drivers/net/wireless/b43/phy_ht.c | 33 ++++++++++++++++++++++++++++++---
1 files changed, 30 insertions(+), 3 deletions(-)
diff --git a/drivers/net/wireless/b43/phy_ht.c b/drivers/net/wireless/b43/phy_ht.c
index 80c448d..81c8656 100644
--- a/drivers/net/wireless/b43/phy_ht.c
+++ b/drivers/net/wireless/b43/phy_ht.c
@@ -352,18 +352,45 @@ static int b43_phy_ht_op_init(struct b43_wldev *dev)
b43_phy_write(dev, 0x0b9, 0x0072);
- /* TODO: Some ops here */
+ b43_httab_write_few(dev, B43_HTTAB16(7, 0x14e), 2, 0x010f, 0x010f);
+ b43_httab_write_few(dev, B43_HTTAB16(7, 0x15e), 2, 0x010f, 0x010f);
+ b43_httab_write_few(dev, B43_HTTAB16(7, 0x16e), 2, 0x010f, 0x010f);
b43_phy_ht_afe_unk1(dev);
- /* TODO: Some ops here */
+ b43_httab_write_few(dev, B43_HTTAB16(7, 0x130), 9, 0x777, 0x111, 0x111,
+ 0x777, 0x111, 0x111, 0x777, 0x111, 0x111);
+
+ b43_httab_write(dev, B43_HTTAB16(7, 0x120), 0x0777);
+ b43_httab_write(dev, B43_HTTAB16(7, 0x124), 0x0777);
+
+ b43_httab_write(dev, B43_HTTAB16(8, 0x00), 0x02);
+ b43_httab_write(dev, B43_HTTAB16(8, 0x10), 0x02);
+ b43_httab_write(dev, B43_HTTAB16(8, 0x20), 0x02);
+
+ b43_httab_write_few(dev, B43_HTTAB16(8, 0x08), 4,
+ 0x8e, 0x96, 0x96, 0x96);
+ b43_httab_write_few(dev, B43_HTTAB16(8, 0x18), 4,
+ 0x8f, 0x9f, 0x9f, 0x9f);
+ b43_httab_write_few(dev, B43_HTTAB16(8, 0x28), 4,
+ 0x8f, 0x9f, 0x9f, 0x9f);
+
+ b43_httab_write_few(dev, B43_HTTAB16(8, 0x0c), 4, 0x2, 0x2, 0x2, 0x2);
+ b43_httab_write_few(dev, B43_HTTAB16(8, 0x1c), 4, 0x2, 0x2, 0x2, 0x2);
+ b43_httab_write_few(dev, B43_HTTAB16(8, 0x2c), 4, 0x2, 0x2, 0x2, 0x2);
b43_phy_maskset(dev, 0x0280, 0xff00, 0x3e);
b43_phy_maskset(dev, 0x0283, 0xff00, 0x3e);
b43_phy_maskset(dev, B43_PHY_OFDM(0x0141), 0xff00, 0x46);
b43_phy_maskset(dev, 0x0283, 0xff00, 0x40);
- /* TODO: Some ops here */
+ b43_httab_write_few(dev, B43_HTTAB16(00, 0x8), 4,
+ 0x09, 0x0e, 0x13, 0x18);
+ b43_httab_write_few(dev, B43_HTTAB16(01, 0x8), 4,
+ 0x09, 0x0e, 0x13, 0x18);
+ /* TODO: Did wl mean 2 instead of 40? */
+ b43_httab_write_few(dev, B43_HTTAB16(40, 0x8), 4,
+ 0x09, 0x0e, 0x13, 0x18);
b43_phy_maskset(dev, B43_PHY_OFDM(0x24), 0x3f, 0xd);
b43_phy_maskset(dev, B43_PHY_OFDM(0x64), 0x3f, 0xd);
--
1.7.3.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH] b43: HT-PHY: init: add missing small-tables writes
2011-08-13 15:54 [PATCH] b43: HT-PHY: init: add missing small-tables writes Rafał Miłecki
@ 2011-08-23 12:45 ` Rafał Miłecki
2011-08-23 13:49 ` John W. Linville
0 siblings, 1 reply; 7+ messages in thread
From: Rafał Miłecki @ 2011-08-23 12:45 UTC (permalink / raw)
To: linux-wireless, John W. Linville; +Cc: b43-dev, Rafał Miłecki
W dniu 13 sierpnia 2011 17:54 u?ytkownik Rafa? Mi?ecki
<zajec5@gmail.com> napisa?:
>
> Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
> ---
> John: this one depends on
> [RFC][PATCH] b43: HT-PHY: allow writing longer tables with a single call
> , please apply the one above.
John, I've asked you to apply the mentioned patch before this one. By
applying *just* this one you've broken build for ppl trying *BROKEN*
HT support.
--
Rafa?
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] b43: HT-PHY: init: add missing small-tables writes
2011-08-23 12:45 ` Rafał Miłecki
@ 2011-08-23 13:49 ` John W. Linville
2011-08-23 15:09 ` Rafał Miłecki
2011-08-23 16:52 ` David Woodhouse
0 siblings, 2 replies; 7+ messages in thread
From: John W. Linville @ 2011-08-23 13:49 UTC (permalink / raw)
To: Rafał Miłecki; +Cc: linux-wireless, b43-dev
On Tue, Aug 23, 2011 at 02:45:51PM +0200, Rafa? Mi?ecki wrote:
> W dniu 13 sierpnia 2011 17:54 u?ytkownik Rafa? Mi?ecki
> <zajec5@gmail.com> napisa?:
> >
> > Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
> > ---
> > John: this one depends on
> > [RFC][PATCH] b43: HT-PHY: allow writing longer tables with a single call
> > , please apply the one above.
>
> John, I've asked you to apply the mentioned patch before this one. By
> applying *just* this one you've broken build for ppl trying *BROKEN*
> HT support.
Please, _please_, _PLEASE_ stop doing this. I process a lot of
patches, and sometimes I will miss these directives.
If your patch is not RFC, then don't post it as RFC. If you post
a patch as "Request For Comment" and I don't have any comments,
I'm going to delete it. If you later decide it is no longer RFC,
then I've probably already deleted it. So you need to re-post the
patch as just "PATCH".
I no longer have "b43: HT-PHY: allow writing longer tables with a
single call". If it is needed, then you need to repost it.
John
P.S. In general, it is best to post patches with dependencies as a series.
--
John W. Linville Someday the world will need a hero, and you
linville at tuxdriver.com might be all we have. Be ready.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] b43: HT-PHY: init: add missing small-tables writes
2011-08-23 13:49 ` John W. Linville
@ 2011-08-23 15:09 ` Rafał Miłecki
2011-08-23 16:52 ` David Woodhouse
1 sibling, 0 replies; 7+ messages in thread
From: Rafał Miłecki @ 2011-08-23 15:09 UTC (permalink / raw)
To: John W. Linville; +Cc: linux-wireless, b43-dev
W dniu 23 sierpnia 2011 15:49 u?ytkownik John W. Linville
<linville@tuxdriver.com> napisa?:
> On Tue, Aug 23, 2011 at 02:45:51PM +0200, Rafa? Mi?ecki wrote:
>> W dniu 13 sierpnia 2011 17:54 u?ytkownik Rafa? Mi?ecki
>> <zajec5@gmail.com> napisa?:
>> >
>> > Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
>> > ---
>> > John: this one depends on
>> > [RFC][PATCH] b43: HT-PHY: allow writing longer tables with a single call
>> > , please apply the one above.
>>
>> John, I've asked you to apply the mentioned patch before this one. By
>> applying *just* this one you've broken build for ppl trying *BROKEN*
>> HT support.
>
> Please, _please_, _PLEASE_ stop doing this. ?I process a lot of
> patches, and sometimes I will miss these directives.
>
> If your patch is not RFC, then don't post it as RFC. ?If you post
> a patch as "Request For Comment" and I don't have any comments,
> I'm going to delete it. ?If you later decide it is no longer RFC,
> then I've probably already deleted it. ?So you need to re-post the
> patch as just "PATCH".
>
> I no longer have "b43: HT-PHY: allow writing longer tables with a
> single call". ?If it is needed, then you need to repost it.
Sorry, I didn't know about this "RFC" rule. I expected patch to be
applied after spending a few days without comments. Will re-send in
the future.
--
Rafa?
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] b43: HT-PHY: init: add missing small-tables writes
2011-08-23 13:49 ` John W. Linville
2011-08-23 15:09 ` Rafał Miłecki
@ 2011-08-23 16:52 ` David Woodhouse
2011-08-24 17:56 ` Kalle Valo
1 sibling, 1 reply; 7+ messages in thread
From: David Woodhouse @ 2011-08-23 16:52 UTC (permalink / raw)
To: John W. Linville; +Cc: Rafał Miłecki, linux-wireless, b43-dev
On Tue, 2011-08-23 at 09:49 -0400, John W. Linville wrote:
> Please, _please_, _PLEASE_ stop doing this. I process a lot of
> patches, and sometimes I will miss these directives.
Best just to publish a git tree which John can pull in to
wireless-testing.git, much as trees get pulled to linux-next.
That way it'll *always* have the patches you want, in the order you
want.
--
dwmw2
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 5818 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/b43-dev/attachments/20110823/2f749092/attachment.bin>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] b43: HT-PHY: init: add missing small-tables writes
2011-08-23 16:52 ` David Woodhouse
@ 2011-08-24 17:56 ` Kalle Valo
2011-08-24 19:57 ` David Woodhouse
0 siblings, 1 reply; 7+ messages in thread
From: Kalle Valo @ 2011-08-24 17:56 UTC (permalink / raw)
To: David Woodhouse
Cc: John W. Linville, Rafał Miłecki, linux-wireless,
b43-dev
David Woodhouse <dwmw2@infradead.org> writes:
> On Tue, 2011-08-23 at 09:49 -0400, John W. Linville wrote:
>> Please, _please_, _PLEASE_ stop doing this. I process a lot of
>> patches, and sometimes I will miss these directives.
>
> Best just to publish a git tree which John can pull in to
> wireless-testing.git, much as trees get pulled to linux-next.
>
> That way it'll *always* have the patches you want, in the order you
> want.
That's true, but setting up git tree for wireless is not trivial and
not for the faint of heart. Especially due to differences between
wireless-next and wireless-testing. Sending a patch series is a lot
easier and causes less tears.
--
Kalle Valo
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] b43: HT-PHY: init: add missing small-tables writes
2011-08-24 17:56 ` Kalle Valo
@ 2011-08-24 19:57 ` David Woodhouse
0 siblings, 0 replies; 7+ messages in thread
From: David Woodhouse @ 2011-08-24 19:57 UTC (permalink / raw)
To: Kalle Valo
Cc: John W. Linville, Rafał Miłecki, linux-wireless,
b43-dev
On Wed, 2011-08-24 at 20:56 +0300, Kalle Valo wrote:
> That's true, but setting up git tree for wireless is not trivial and
> not for the faint of heart. Especially due to differences between
> wireless-next and wireless-testing. Sending a patch series is a lot
> easier and causes less tears.
Most of the time it wouldn't need to be based on wireless-testing. It
could just be based on Linus's last release. If wireless-next is
actually a real stable git tree, it could perhaps be based on that too.
--
dwmw2
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 5818 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/b43-dev/attachments/20110824/655c74a8/attachment.bin>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-08-24 19:57 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-13 15:54 [PATCH] b43: HT-PHY: init: add missing small-tables writes Rafał Miłecki
2011-08-23 12:45 ` Rafał Miłecki
2011-08-23 13:49 ` John W. Linville
2011-08-23 15:09 ` Rafał Miłecki
2011-08-23 16:52 ` David Woodhouse
2011-08-24 17:56 ` Kalle Valo
2011-08-24 19:57 ` David Woodhouse
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).