Thread (58 messages) 58 messages, 3 authors, 2012-06-11

Re: [PATCH 14/18] brcmsmac: extend xmtfifo_sz array

From: Hauke Mehrtens <hauke@hauke-m.de>
Date: 2012-06-10 13:10:30

On 06/07/2012 09:21 PM, Arend van Spriel wrote:
On 06/07/2012 03:33 PM, Hauke Mehrtens wrote:
quoted
On 06/06/2012 12:24 PM, Arend van Spriel wrote:
quoted
On 06/06/2012 01:07 AM, Hauke Mehrtens wrote:
quoted
The xmtfifo_sz array contains the queue sizes for the different chip
revs. This array missed the sizes for the core rev 17 and 28. This
Should probably use core revs instead of chip revs and I think we are
talking about the 80211 core rev here.
Yes I will change this comment.

Are there any devices with the same phy type using different fifo sizes?
If this is not the case I think it is better to do this based on the phy
type.
I dived into this and basically there is no correlation to the phy type.
There are chips with same 80211 core rev and different phy type as there
are chips with different 80211 core rev and the same phy type.

Gr. AvS
Hi Arend,

Ok, then I will leave it based on the core rev like it is now. Are the
values I added there for core rev 17 and 28 correct? They are just
copied from the other phy-n device which has the same values set after
powering up and before they are explicitly set. The core rev 17 device
is working for me, but the core rev 28 is not, but I do not know why. I
was unable to find these values in b43 or the spec b43 is based on.

Hauke
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help