On Thu, Apr 8, 2010 at 2:18 PM, Pavel Roskin <proski@xxxxxxx> wrote: > On Thu, 2010-04-08 at 15:26 -0400, Luis R. Rodriguez wrote: >> AR9003 does not have a reset control for AHB. > ... >> + val = AR_RC_HOSTIF; >> + if (!AR_SREV_9300_20_OR_LATER(ah)) >> + val |= AR_RC_AHB; > > Perhaps AR_RC_AHB could be defined to 0 for AR9300 in the header? Just > an idea. Hm, I rather be explicit for now. A larger goal we have after all this will be to unify our code with the Atheros HAL. That itself should be easier for review if we are explicit about some things for review purposes. But that's just me thinking out loud. Luis -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html