Stefano Brivio wrote:
On Wed, 11 Apr 2007 11:08:53 -0500
Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote:
On PPC architecture with phy->rev == 1, machine checks occur during
initialization of the "Extended G PHY registers". This problem was
also seen on bcm43xx-softmac, and was fixed by conditionally skipping
over certain reads/writes of these registers. The same solution has been
applied here with testing by David Woodhouse. Note: These modifications
are not found in the specifications, but are needed for PPC.
I don't think this patch has to be reverted, but I still think that it's
better to notice the reverse engineers team about failing operations
and understand the real problem rather than hiding it. Which works in some
cases, but sometimes just hide other bugs.
I'm almost done with rewriting the whole A/G/N setup and init routines,
following the new v4 specs. This issue could even be related.
I agree that it would be better if we could follow the specs and not have the machine check problem.
My impression is that early versions of the bcm43xx driver did not have this difficulty, but that it
appeared after a relatively recent change in the G init specs. BTW, these were the changes that
greatly improved performance, particularly on the phy->rev = 8 devices. Perhaps the Broadcom
software abandoned/ignored the phy->rev = 1 units running on PPC hardware, which seems to be the
only combination that shows the problem. I don't know about MIPS hardware, but the problem is not
seen on x86 architecture.
I look forward to your rewrite of the setup and init routines. Perhaps they will even let my 4311
work with mac80211, but I will be very surprised if the PPC machine check problem goes away.
Larry
-
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
