if_ath - should it be compiled without ath_hal support?

[Garrett Cooper]

On Sun, Aug 28, 2011 at 5:21 PM, Adrian Chadd <adrian at freebsd.org> wrote:
> that looks a bit special.
>
> On 29 August 2011 07:38, Garrett Cooper <yanegomi at gmail.com> wrote:
>
>>> I'd like to eventually split out the ath_hal again into a separate
>>> module and re-instate the separation between HAL and driver code
>>> (primarily to enforce cleaner code and more stable APIs.)
>>
>> Good to know. I was just a bit confused with the manpages that say
>> they're optional after I discovered looking at the driver that they're
>> hardwired in.
>
> I plan on fleshing out the driver documentation in the Wiki a little more.
> Like everything, all I need is time. :)
>
>>> If you're seeing issues like the above, chances are that you're seeing
>>> some corruption. Can you please print out the whole value of *rs (ie,
>>> all the fields of the ath_rx_status struct) so I can see how valid it
>>> is?
>>
>> Sure:
>>
>> (kgdb) p *rs
>> $1 = {rs_datalen = 244, rs_status = 0 '\0', rs_phyerr = 0 '\0',
>> rs_rssi = 0 '\0', rs_keyix = 127 '\177', rs_rate = 27 '\033', rs_more
>> = 0 '\0', rs_tstamp = 2097280, rs_antenna = 8399232, rs_rssi_ctl =
>> "0\200)", rs_rssi_ext = "\200\000\037", rs_isaggr = 1 '\001',
>> rs_moreaggr = 1 '\001', rs_num_delims = 0 '\0', rs_flags = 252 'ü',
>> rs_evm0 = 0, rs_evm1 = 0, rs_evm2 = 0, rs_evm3 = 0, rs_evm4 = 0}
>
> It doesn't look like garbage to begin with .. and then it does.
>
> How have you compiled it? Some of those fields are currently optional
> (ie, require the AR5416 (11n) support to be enabled) - have you
> somehow managed to compile part of the code with it, part of the code
> without it?

Just

options ATH_DEBUG

in my $KERNCONF and I built the ath and ath_pci modules.

> Which NIC is this for again? AR9285?

Yup:

ath0 at pci0:2:0:0:        class=0x028000 card=0x10891a3b chip=0x002b168c
rev=0x01 hdr=0x00
    vendor     = 'Atheros Communications Inc.'
    device     = 'AR9285 Wireless Network Adapter (PCI-Express)'
    class      = network

Thanks!
-Garrett