PHY: where to put forcing gater clocks function

2010-01-08 Thread Rafał Miłecki
I need to implement
http://bcm-v4.sipsolutions.net/802.11/PHY/ClkFgc

Where I can put my
void b43_phy_clock_fgc(struct b43_wldev *dev, bool clock) { ... }
? Is end of phy_common.c file OK for this?

-- 
Rafał
___
Bcm43xx-dev mailing list
Bcm43xx-dev@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev


Re: PHY: where to put forcing gater clocks function

2010-01-08 Thread Michael Buesch
On Friday 08 January 2010 09:30:27 Rafał Miłecki wrote:
 I need to implement
 http://bcm-v4.sipsolutions.net/802.11/PHY/ClkFgc
 
 Where I can put my
 void b43_phy_clock_fgc(struct b43_wldev *dev, bool clock) { ... }
 ? Is end of phy_common.c file OK for this?

yes. Note that the 0x2 bit has defines in ssb_regs.h. So you should use 
them.
Also use ssb_read/write to access that register. It's essentially the same
as b43_read/write, but this way it's obvious to humans that we are accessing a 
backplane register.
Also maybe don't call the parameter clock but force. Would make more sense 
to me...

-- 
Greetings, Michael.
___
Bcm43xx-dev mailing list
Bcm43xx-dev@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev


Re: PHY: where to put forcing gater clocks function

2010-01-08 Thread Rafał Miłecki
W dniu 8 stycznia 2010 11:17 użytkownik Michael Buesch m...@bu3sch.de napisał:
 On Friday 08 January 2010 09:30:27 Rafał Miłecki wrote:
 I need to implement
 http://bcm-v4.sipsolutions.net/802.11/PHY/ClkFgc

 Where I can put my
 void b43_phy_clock_fgc(struct b43_wldev *dev, bool clock) { ... }
 ? Is end of phy_common.c file OK for this?

 yes. Note that the 0x2 bit has defines in ssb_regs.h. So you should use 
 them.
 Also use ssb_read/write to access that register. It's essentially the same
 as b43_read/write, but this way it's obvious to humans that we are accessing 
 a backplane register.
 Also maybe don't call the parameter clock but force. Would make more 
 sense to me...

Thanks Michael. I use CONSTANTS wherever I can and I did in my
implementation. Of course I used ssb_* as well :) And even after
posting patch I renamed clock to force :D

-- 
Rafał
___
Bcm43xx-dev mailing list
Bcm43xx-dev@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev


Re: PHY: where to put forcing gater clocks function

2010-01-08 Thread Rafał Miłecki
W dniu 8 stycznia 2010 09:30 użytkownik Rafał Miłecki
zaj...@gmail.com napisał:
 I need to implement
 http://bcm-v4.sipsolutions.net/802.11/PHY/ClkFgc

 Where I can put my
 void b43_phy_clock_fgc(struct b43_wldev *dev, bool clock) { ... }
 ? Is end of phy_common.c file OK for this?

Larry changed docs and now this is:
http://bcm-v4.sipsolutions.net/802.11/PHY/N/BmacPhyClkFgc
so it finally seems to be N-only function with some mysterious check
for PHY to be N...

I guess I'll just put this in phy_n.c and eventually in future we will
move it somewhere else if this will be needed.

-- 
Rafał
___
Bcm43xx-dev mailing list
Bcm43xx-dev@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev