Michal Kazior <michal.kazior@xxxxxxxxx> writes: > On 16 October 2013 08:57, Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> wrote: >> Michal Kazior <michal.kazior@xxxxxxxxx> writes: >> >>>> + /* disable pending irqs */ >>>> + ath10k_pci_core_write32(ar, PCIE_INTR_ENABLE_ADDRESS, 0); >>>> + ath10k_pci_core_write32(ar, PCIE_INTR_CLR_ADDRESS, ~0); >>> >>> These use SOC_CORE_BASE_ADDRESS as the suffix, not the RTC_SOC_BASE_ADDRESS. >> >> But I am using SOC_CORE_BASE_ADDRESS, right? > > Ah, right. Sorry. My eyesight failed me :) Those function names seem a > little confusing (_core_ for SOC_CORE and _soc_ for RTC_SOC) Yeah, I agree. I didn't research what RTC_SOC stands for, I just assumed it contains registers for controlling the whole SOC state. That's why I used "_soc_" here. And CORE registers control just one part of SOC. -- Kalle Valo -- 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
