Re: [PATCH] bcm43xx: Fix typo in B5PHY init specifications
David Woodhouse wrote: On Fri, 2007-03-02 at 11:30 -0600, Larry Finger wrote: There was an error in the B5PHY init specifications. This patch doesn't fix the machine check in bcm43xx_phy_initb5 which Pavel Roskin and I reported a couple of weeks ago. To get rid of that crash, I still have to revert an earlier 'spec update' patch (740ac4fb08866d702be90f167665d03759bd27d0). Yeah, the issue is that the Rev1 cards have to be init'd with g mode off. I'm not sure anyone has picked up this fix yet. The relevant spec fix is here: http://bcm-v4.sipsolutions.net/802.11/PHY/calinit -Joe ___ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
Re: [PATCH] bcm43xx: Fix typo in B5PHY init specifications
On Mon, Mar 12, 2007 at 09:23:30AM -0400, Joseph Jezak wrote: David Woodhouse wrote: On Fri, 2007-03-02 at 11:30 -0600, Larry Finger wrote: There was an error in the B5PHY init specifications. This patch doesn't fix the machine check in bcm43xx_phy_initb5 which Pavel Roskin and I reported a couple of weeks ago. To get rid of that crash, I still have to revert an earlier 'spec update' patch (740ac4fb08866d702be90f167665d03759bd27d0). Yeah, the issue is that the Rev1 cards have to be init'd with g mode off. I'm not sure anyone has picked up this fix yet. The relevant spec fix is here: http://bcm-v4.sipsolutions.net/802.11/PHY/calinit FWIW, by inspection it looks like the mac80211-based driver is (trying?) to implement this change. David, have you tried the mac80211 version? Does it still have the same crash? John -- John W. Linville [EMAIL PROTECTED] ___ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
Re: [PATCH] bcm43xx: Fix typo in B5PHY init specifications
On Mon, 2007-03-12 at 10:53 -0400, John W. Linville wrote: FWIW, by inspection it looks like the mac80211-based driver is (trying?) to implement this change. David, have you tried the mac80211 version? Does it still have the same crash? Should the one in 2.6.20-1.2982.fc7 be OK? I can try that relatively easily; anything else might need to wait till I get home. -- dwmw2 ___ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
Re: [PATCH] bcm43xx: Fix typo in B5PHY init specifications
David Woodhouse wrote: On Mon, 2007-03-12 at 10:53 -0400, John W. Linville wrote: FWIW, by inspection it looks like the mac80211-based driver is (trying?) to implement this change. David, have you tried the mac80211 version? Does it still have the same crash? Should the one in 2.6.20-1.2982.fc7 be OK? I can try that relatively easily; anything else might need to wait till I get home. I don't think it will improve anything. I don't currently have a copy of wireless-dev, but the code in the mb tree is identical with that of wireless-2.6. I'm currently trying to figure out the spec at http://bcm-v4.sipsolutions.net/802.11/PHY/calinit and what we need to change. Larry ___ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
Re: [PATCH] bcm43xx: Fix typo in B5PHY init specifications
On Mon, Mar 12, 2007 at 12:42:18PM -0500, Larry Finger wrote: David Woodhouse wrote: On Mon, 2007-03-12 at 10:53 -0400, John W. Linville wrote: FWIW, by inspection it looks like the mac80211-based driver is (trying?) to implement this change. David, have you tried the mac80211 version? Does it still have the same crash? Should the one in 2.6.20-1.2982.fc7 be OK? I can try that relatively easily; anything else might need to wait till I get home. I don't think it will improve anything. I don't currently have a copy of wireless-dev, but the code in the mb tree is identical with that of wireless-2.6. I'm currently trying to figure out the spec at http://bcm-v4.sipsolutions.net/802.11/PHY/calinit and what we need to change. Huh? Are you looking in drivers/net/wireless/mac80211/bcm43xx? It certainly seems to differ quite a bit from what is in drivers/net/wireless/bcm43xx (i.e. no the softmac-based version). John -- John W. Linville [EMAIL PROTECTED] ___ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
Re: [PATCH] bcm43xx: Fix typo in B5PHY init specifications
On Mon, Mar 12, 2007 at 04:16:34PM +0100, David Woodhouse wrote: On Mon, 2007-03-12 at 10:53 -0400, John W. Linville wrote: FWIW, by inspection it looks like the mac80211-based driver is (trying?) to implement this change. David, have you tried the mac80211 version? Does it still have the same crash? Should the one in 2.6.20-1.2982.fc7 be OK? I can try that relatively easily; anything else might need to wait till I get home. Yes, I think that will do. It certainly looks to be up-to-date. Thanks, John -- John W. Linville [EMAIL PROTECTED] ___ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
Re: [PATCH] bcm43xx: Fix typo in B5PHY init specifications
Joe, Joseph Jezak wrote: David Woodhouse wrote: On Fri, 2007-03-02 at 11:30 -0600, Larry Finger wrote: There was an error in the B5PHY init specifications. This patch doesn't fix the machine check in bcm43xx_phy_initb5 which Pavel Roskin and I reported a couple of weeks ago. To get rid of that crash, I still have to revert an earlier 'spec update' patch (740ac4fb08866d702be90f167665d03759bd27d0). Yeah, the issue is that the Rev1 cards have to be init'd with g mode off. I'm not sure anyone has picked up this fix yet. The relevant spec fix is here: http://bcm-v4.sipsolutions.net/802.11/PHY/calinit I think the bug may be even deeper. In bcm43xx_phy_connect, the G Mode Enable bit of TM State Low is only written for phy-rev = 5. I have not found any other place where that bit is set. From what I read in the specs, that bit should exist and be set for all G PHY's. Is that correct? I think I'm beginning to understand the confusion between phy-connected in the code and gmode in the specs. Larry ___ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
Re: Pull the patch sent under heading [PATCH] bcm43xx: Fix typo in B5PHY init specifications
John W. Linville wrote: On Sun, Mar 04, 2007 at 11:13:31PM -0600, Larry Finger wrote: Please pull the patch referenced in the subject. It completely kills my BCM4306 with phy-revision == 1. Joe Jezak and I will have to sort this one out. Just to be clear, I interpret this to mean do not apply that patch. Pull is often used to mean take the patch and apply it as in git pull. Let me know if I'm interpreting you incorrectly. Sorry to mess up the terminology. It should be discarded, ignored, and trashed!! Larry ___ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
Pull the patch sent under heading [PATCH] bcm43xx: Fix typo in B5PHY init specifications
John, Please pull the patch referenced in the subject. It completely kills my BCM4306 with phy-revision == 1. Joe Jezak and I will have to sort this one out. Thanks, Larry ___ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
[PATCH] bcm43xx: Fix typo in B5PHY init specifications
There was an error in the B5PHY init specifications.
Signed-off-by: Larry Finger [EMAIL PROTECTED]
---
John,
This patch is meant for wireless-2.6, 'upstream', and 'upstream-fixes'.
Larry
Index: wireless-2.6/drivers/net/wireless/bcm43xx/bcm43xx_phy.c
===
--- wireless-2.6.orig/drivers/net/wireless/bcm43xx/bcm43xx_phy.c
+++ wireless-2.6/drivers/net/wireless/bcm43xx/bcm43xx_phy.c
@@ -806,7 +806,7 @@ static void bcm43xx_phy_initb5(struct bc
/* Force to channel 7, even if not supported. */
bcm43xx_radio_selectchannel(bcm, 7, 0);
- if (radio-version != 0x2050) {
+ if (radio-version == 0x2050) {
bcm43xx_radio_write16(bcm, 0x0075, 0x0080);
bcm43xx_radio_write16(bcm, 0x0079, 0x0081);
}
___
Bcm43xx-dev mailing list
Bcm43xx-dev@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
