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. SamReceived on Fri May 01 2009 - 13:57:25 UTC
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:47 UTC