kernel compile fails without AH_SUPPORT_AR5416

Sam Leffler sam at freebsd.org
Fri May 1 15:57:26 UTC 2009


Sam Leffler wrote:
> Bruce Simpson wrote:
>> Hi,
>>
>> Can you please try this patch?
>> I can't commit it until STABLE is unfrozen after 7.2-RELEASE is cut.
>>
>> Sam Leffler wrote:
>>> Bruce Simpson wrote:
>>>  
>>>> Hi,
>>>>
>>>> Looks like I'm late to the party. I was responsible for committing 
>>>> these
>>>> ath(4) changes to RELENG_7.
>>>> I can't remember if I tested the kernel compile without the
>>>> AH_SUPPORT_AR5416 option or not, I have been so incredibly busy.
>>>> ...
>>> ru had a change to fix this but decided not to; can't say why.
>>> Otherwise there is a better way to fix this which I alluded to in
>>> previous mail--use the config-generated #define that is generated for
>>> the "ath_hal" device.
> Do not modify ah_desc.h like you've done.  Add this to conf/options
>
> ATH_HAL   opt_ah.h
>
> and use that to enable AH_SUPPORT_AR5416.
>
> Also changes must go in head first.

To clarify the first comment: you've made it impossible to build code 
w/o the extended format descriptor; this is what I find unacceptable.

    Sam



More information about the freebsd-current mailing list