wifi: ath10k: fix QCOM_SMEM dependency
authorKalle Valo <quic_kvalo@quicinc.com>
Fri, 2 Dec 2022 10:30:27 +0000 (12:30 +0200)
committerKalle Valo <kvalo@kernel.org>
Fri, 2 Dec 2022 18:24:06 +0000 (20:24 +0200)
Nathan noticed that when HWSPINLOCK is disabled there's a Kconfig warning:

  WARNING: unmet direct dependencies detected for QCOM_SMEM
    Depends on [n]: (ARCH_QCOM [=y] || COMPILE_TEST [=n]) && HWSPINLOCK [=n]
    Selected by [m]:
    - ATH10K_SNOC [=m] && NETDEVICES [=y] && WLAN [=y] && WLAN_VENDOR_ATH [=y] && ATH10K [=m] && (ARCH_QCOM [=y] || COMPILE_TEST [=n])

The problem here is that QCOM_SMEM depends on HWSPINLOCK so we cannot select
QCOM_SMEM and instead we neeed to use 'depends on'.

Reported-by: Nathan Chancellor <nathan@kernel.org>
Link: https://lore.kernel.org/all/Y4YsyaIW+CPdHWv3@dev-arch.thelio-3990X/
Fixes: 4d79f6f34bbb ("wifi: ath10k: Store WLAN firmware version in SMEM image table")
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20221202103027.25974-1-kvalo@kernel.org
drivers/net/wireless/ath/ath10k/Kconfig

index e0a51dad8e4200cc64d5174d83f7353721be9307..e6ea884cafc190ac1bcbbf15d80823896e259572 100644 (file)
@@ -44,7 +44,7 @@ config ATH10K_SNOC
        tristate "Qualcomm ath10k SNOC support"
        depends on ATH10K
        depends on ARCH_QCOM || COMPILE_TEST
-       select QCOM_SMEM
+       depends on QCOM_SMEM
        select QCOM_SCM
        select QCOM_QMI_HELPERS
        help