On Fri, Mar 18, 2011 at 12:16 AM, Luis R. Rodriguez <mcgrof@xxxxxxxxx> wrote: > On Thu, Mar 17, 2011 at 11:38 AM, Luis R. Rodriguez <mcgrof@xxxxxxxxx> wrote: >> On Wed, Mar 16, 2011 at 5:04 PM, Luis R. Rodriguez <mcgrof@xxxxxxxxx> wrote: >>> On Fri, Feb 25, 2011 at 4:01 AM, Vivek Natarajan <vnatarajan@xxxxxxxxxxx> wrote: >>>> Signed-off-by: Vivek Natarajan <vnatarajan@xxxxxxxxxxx> >>> >>> Was this a stable fix? If not why not? >> >> This certainly smells like a stable fix > > OK so AR9485 1.1 support did not make it into 2.6.38, and this fix is > applicable only to AR9485 1.1 so the patch in question is not > applicable to stable. The 2.6.38 release however does have AR9485 1.0 > support. Will we have any devices in the field with AR9485 1.0 ? Did > that revision not require a similar fix? On 2.6.38 and on linux-next I > see we have for 1.0: > > /* Awake Setting */ > > INIT_INI_ARRAY(&ah->iniPcieSerdes, > ar9485_1_0_pcie_phy_pll_on_clkreq_disable_L1, > > ARRAY_SIZE(ar9485_1_0_pcie_phy_pll_on_clkreq_disable_L1), > 2); > > /* Sleep Setting */ > > INIT_INI_ARRAY(&ah->iniPcieSerdesLowPower, > ar9485_1_0_pcie_phy_pll_on_clkreq_enable_L1, > > ARRAY_SIZE(ar9485_1_0_pcie_phy_pll_on_clkreq_enable_L1), > 2); We did not notice any issue with the older code atleast from my testing. There was an error in porting the internal codebase for AR9485 1.0 which is fixed by the above patch. Thanks Vivek. -- 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