On 2010-12-14 3:42 AM, Sujith wrote: > From: Sujith Manoharan <Sujith.Manoharan@xxxxxxxxxxx> > > There is no need to configure PCI related registers for > USB devices. > > Signed-off-by: Sujith Manoharan <Sujith.Manoharan@xxxxxxxxxxx> > --- > diff --git a/drivers/net/wireless/ath/ath9k/hw.c b/drivers/net/wireless/ath/ath9k/hw.c > index 0f373be..2662553 100644 > --- a/drivers/net/wireless/ath/ath9k/hw.c > +++ b/drivers/net/wireless/ath/ath9k/hw.c > @@ -568,10 +568,12 @@ static int __ath9k_hw_init(struct ath_hw *ah) > ah->WARegVal |= (AR_WA_D3_L1_DISABLE | > AR_WA_ASPM_TIMER_BASED_DISABLE); > > - if (ah->is_pciexpress) > + if (ah->is_pciexpress) { > ath9k_hw_configpcipowersave(ah, 0, 0); > - else > - ath9k_hw_disablepcie(ah); > + } else { > + if (common->bus_ops->ath_bus_type != ATH_USB) > + ath9k_hw_disablepcie(ah); > + } I don't really see the point of this check, since (with my other patch) ath9k_hw_disablepcie already checks for AR_SREV_5416, which won't return true on any USB based device. - Felix -- 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