Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Nick Kossifidis
* Get rid of calibration timer, instead use a software interrupt to schedule the calibration tasklet. a) We don't need a timer for this, there is no need for accuracy even with round_jiffies i think this is a waste of resources. Also we don't need to run calibration if we are idle

Re: [ath5k-devel] [PATCH 3/4] ath5k: Wakeup fixes

2009-07-31 Thread Ben Greear
On 07/31/2009 11:08 AM, Nick Kossifidis wrote: > * Don't put chip to full sleep because there are problems during > wakeup. Instead hold MAC/Baseband on warm reset state via a new > function ath5k_hw_on_hold. > > * Durring attach preserve pcicfg bits when enabling pci core > sw retr

Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Nick Kossifidis
2009/7/31 Bob Copeland : > On Fri, Jul 31, 2009 at 2:25 PM, Nick Kossifidis wrote: >>    Also we don't need to run calibration if we are idle (no interrupts). > > I think this is the big win right now... > >> c) Having calibration on a tasklet is better since during calibration >>    we can't trans

Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Pavel Roskin
On Fri, 2009-07-31 at 22:35 +0300, Nick Kossifidis wrote: > Nope calibration deals mostly with noise immunity, it calculates the > noise floor and fixes QAM constellation. It's not related to the synthesizer > so it's not related to the channel frequency. OK then, good to know. -- Regards, Pave

Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Nick Kossifidis
2009/7/31 Pavel Roskin : > On Fri, 2009-07-31 at 21:25 +0300, Nick Kossifidis wrote: > >> a) We don't need a timer for this, there is no need for accuracy >>     even with round_jiffies i think this is a waste of resources. >>     Also we don't need to run calibration if we are idle (no interrupts)

Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Bob Copeland
On Fri, Jul 31, 2009 at 2:25 PM, Nick Kossifidis wrote: >    Also we don't need to run calibration if we are idle (no interrupts). I think this is the big win right now... > c) Having calibration on a tasklet is better since during calibration >    we can't transmit or receive (antennas are detac

Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Pavel Roskin
On Fri, 2009-07-31 at 21:25 +0300, Nick Kossifidis wrote: > a) We don't need a timer for this, there is no need for accuracy > even with round_jiffies i think this is a waste of resources. > Also we don't need to run calibration if we are idle (no interrupts). It doesn't sound right to me

Re: [ath5k-devel] [PATCH 3/4] ath5k: Wakeup fixes

2009-07-31 Thread Luis R. Rodriguez
On Fri, Jul 31, 2009 at 12:03 PM, Ben Greear wrote: > On 07/31/2009 11:08 AM, Nick Kossifidis wrote: >> >>  * Don't put chip to full sleep because there are problems during >>    wakeup. Instead hold MAC/Baseband on warm reset state via a new >>    function ath5k_hw_on_hold. >> >>  * Durring attach

Re: [ath5k-devel] [PATCH 3/4] ath5k: Wakeup fixes

2009-07-31 Thread Luis R. Rodriguez
On Fri, Jul 31, 2009 at 11:39 AM, Nick Kossifidis wrote: > 2009/7/31 Luis R. Rodriguez : >> On Fri, Jul 31, 2009 at 11:08 AM, Nick >> Kossifidis wrote: >>>  * Don't put chip to full sleep because there are problems during >>>   wakeup. Instead hold MAC/Baseband on warm reset state via a new >>>   f

Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Luis R. Rodriguez
On Fri, Jul 31, 2009 at 11:25 AM, Nick Kossifidis wrote: > 2009/7/31 Luis R. Rodriguez : >> On Fri, Jul 31, 2009 at 11:10 AM, Nick >> Kossifidis wrote: >>>  * Get rid of calibration timer, instead use a software interrupt >>>   to schedule the calibration tasklet. >> >> An example of itemizing one

Re: [ath5k-devel] [PATCH 1/4]: Check EEPROM before tweaking SERDES

2009-07-31 Thread Luis R. Rodriguez
On Fri, Jul 31, 2009 at 11:02 AM, Nick Kossifidis wrote: >  * Read PCI-E infos offset from EEPROM and if it points to >   serdes section (0x40), enable serdes programming (further >   tweaking of serdes values during attach). This follows >   Legacy and Sam's HAL sources. > > --- >  drivers/net/wir

Re: [ath5k-devel] [PATCH 3/4] ath5k: Wakeup fixes

2009-07-31 Thread Nick Kossifidis
2009/7/31 Luis R. Rodriguez : > On Fri, Jul 31, 2009 at 11:08 AM, Nick > Kossifidis wrote: >>  * Don't put chip to full sleep because there are problems during >>   wakeup. Instead hold MAC/Baseband on warm reset state via a new >>   function ath5k_hw_on_hold. >> >>  * Durring attach preserve pcicf

Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Nick Kossifidis
* Get rid of calibration timer, instead use a software interrupt to schedule the calibration tasklet. Signed-off-by: Nick Kossifidis --- drivers/net/wireless/ath/ath5k/ath5k.h | 16 drivers/net/wireless/ath/ath5k/base.c | 28 ++-- drivers/net/w

Re: [ath5k-devel] [PATCH 3/4] ath5k: Wakeup fixes

2009-07-31 Thread Nick Kossifidis
* Don't put chip to full sleep because there are problems during wakeup. Instead hold MAC/Baseband on warm reset state via a new function ath5k_hw_on_hold. * Durring attach preserve pcicfg bits when enabling pci core sw retry fix. * Minor cleanups Signed-off-by: Nick Kossifidis --

Re: [ath5k-devel] [PATCH 2/4] ath5k: Linear PCDAC code fixes

2009-07-31 Thread Nick Kossifidis
* Set correct xpd curve indices for high/low gain curves during rfbuffer setup on RF5112B with both calibration curves available. * Don't return zero min power when we have the same pcdac value twice because it breaks interpolation. Instead return the right x barrier as we do when we ha

Re: [ath5k-devel] [PATCH 1/4] ath5k: Check EEPROM before tweaking SERDES

2009-07-31 Thread Nick Kossifidis
* Read PCI-E infos offset from EEPROM and if it points to serdes section (0x40), enable serdes programming (further tweaking of serdes values during attach). This follows Legacy and Sam's HAL sources. Signed-off-by: Nick Kossifidis --- drivers/net/wireless/ath/ath5k/attach.c | 56 +

Re: [ath5k-devel] [PATCH 1/4]: Check EEPROM before tweaking SERDES

2009-07-31 Thread Nick Kossifidis
2009/7/31 Luis R. Rodriguez : > On Fri, Jul 31, 2009 at 11:18 AM, Nick Kossifidis wrote: >> 2009/7/31 Luis R. Rodriguez : >>> >>> Can the changes for the code be done before the move, or after so that >>> way the actual code changes to the section can be easily readable? >>> >> >> Since we use EEPR

Re: [ath5k-devel] [PATCH 1/4]: Check EEPROM before tweaking SERDES

2009-07-31 Thread Luis R. Rodriguez
On Fri, Jul 31, 2009 at 11:18 AM, Nick Kossifidis wrote: > 2009/7/31 Luis R. Rodriguez : >> >> Can the changes for the code be done before the move, or after so that >> way the actual code changes to the section can be easily readable? >> > > Since we use EEPROM infos we must move this after eeprom

Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Nick Kossifidis
2009/7/31 Luis R. Rodriguez : > On Fri, Jul 31, 2009 at 11:10 AM, Nick > Kossifidis wrote: >>  * Get rid of calibration timer, instead use a software interrupt >>   to schedule the calibration tasklet. > > An example of itemizing one change. Explain the why, not the how. > We 've discussed this wi

Re: [ath5k-devel] [PATCH 1/4]: Check EEPROM before tweaking SERDES

2009-07-31 Thread Nick Kossifidis
2009/7/31 Luis R. Rodriguez : > > Can the changes for the code be done before the move, or after so that > way the actual code changes to the section can be easily readable? > Since we use EEPROM infos we must move this after eeprom initialization. > > This is just comment-picky mind using this s

Re: [ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Luis R. Rodriguez
On Fri, Jul 31, 2009 at 11:10 AM, Nick Kossifidis wrote: >  * Get rid of calibration timer, instead use a software interrupt >   to schedule the calibration tasklet. An example of itemizing one change. Explain the why, not the how. Luis ___ ath5k-deve

Re: [ath5k-devel] [PATCH 3/4] ath5k: Wakeup fixes

2009-07-31 Thread Luis R. Rodriguez
On Fri, Jul 31, 2009 at 11:08 AM, Nick Kossifidis wrote: >  * Don't put chip to full sleep because there are problems during >   wakeup. Instead hold MAC/Baseband on warm reset state via a new >   function ath5k_hw_on_hold. > >  * Durring attach preserve pcicfg bits when enabling pci core >   sw re

Re: [ath5k-devel] [PATCH 2/4] ath5k: Linear PCDAC code fixes

2009-07-31 Thread Nick Kossifidis
2009/7/31 Luis R. Rodriguez : > On Fri, Jul 31, 2009 at 11:05 AM, Nick > Kossifidis wrote: >>  * Set correct xpd curve indices for high/low gain curves during >>   rfbuffer setup on RF5112B with both calibration curves available. >> >>  * Don't return zero min power when we have the same pcdac valu

[ath5k-devel] [PATCH 4/4] ath5k: Use SWI to trigger calibration

2009-07-31 Thread Nick Kossifidis
* Get rid of calibration timer, instead use a software interrupt to schedule the calibration tasklet. --- drivers/net/wireless/ath/ath5k/ath5k.h | 16 drivers/net/wireless/ath/ath5k/base.c | 28 ++-- drivers/net/wireless/ath/ath5k/base.h |3 +

[ath5k-devel] [PATCH 3/4] ath5k: Wakeup fixes

2009-07-31 Thread Nick Kossifidis
* Don't put chip to full sleep because there are problems during wakeup. Instead hold MAC/Baseband on warm reset state via a new function ath5k_hw_on_hold. * Durring attach preserve pcicfg bits when enabling pci core sw retry fix. * Minor cleanups --- drivers/net/wireless/ath/ath5k/

Re: [ath5k-devel] [PATCH 2/4] ath5k: Linear PCDAC code fixes

2009-07-31 Thread Luis R. Rodriguez
On Fri, Jul 31, 2009 at 11:05 AM, Nick Kossifidis wrote: >  * Set correct xpd curve indices for high/low gain curves during >   rfbuffer setup on RF5112B with both calibration curves available. > >  * Don't return zero min power when we have the same pcdac value >   twice because it breaks interpol

[ath5k-devel] [PATCH 2/4] ath5k: Linear PCDAC code fixes

2009-07-31 Thread Nick Kossifidis
* Set correct xpd curve indices for high/low gain curves during rfbuffer setup on RF5112B with both calibration curves available. * Don't return zero min power when we have the same pcdac value twice because it breaks interpolation. Instead return the right x barrier as we do when we ha

[ath5k-devel] [PATCH 1/4] ath5k: Check EEPROM before tweaking SERDES

2009-07-31 Thread Nick Kossifidis
* Read PCI-E infos offset from EEPROM and if it points to serdes section (0x40), enable serdes programming (further tweaking of serdes values during attach). This follows Legacy and Sam's HAL sources. --- drivers/net/wireless/ath/ath5k/attach.c | 56 +++ dr

[ath5k-devel] [PATCH 1/4]: Check EEPROM before tweaking SERDES

2009-07-31 Thread Nick Kossifidis
* Read PCI-E infos offset from EEPROM and if it points to serdes section (0x40), enable serdes programming (further tweaking of serdes values during attach). This follows Legacy and Sam's HAL sources. --- drivers/net/wireless/ath/ath5k/attach.c | 56 +++ dr