* [PATCH 1/2] [RESEND] wifi: ath12k: fix QCOM_RPROC_COMMON dependency
@ 2026-11-12 9:26 Arnd Bergmann
2026-11-12 9:26 ` [PATCH 2/2] wifi: ath12k: add GENERIC_MSI_IRQ dependency Arnd Bergmann
0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2026-11-12 9:26 UTC (permalink / raw)
To: Jeff Johnson
Cc: Arnd Bergmann, Eric Biggers, Sumit Garg, Konrad Dybcio,
linux-wireless, ath12k, linux-kernel
From: Arnd Bergmann <arnd@arndb.de>
The AHB driver depends on the remoteproc interface, which could
be in a loadable module when ath12k itself is built-in:
arm-linux-gnueabi-ld: drivers/net/wireless/ath/ath12k/ahb.o: in function `ath12k_ahb_probe':
ahb.c:(.text+0xb2c): undefined reference to `qcom_register_ssr_notifier'
arm-linux-gnueabi-ld: ahb.c:(.text+0x1000): undefined reference to `qcom_unregister_ssr_notifier'
arm-linux-gnueabi-ld: drivers/net/wireless/ath/ath12k/ahb.o: in function `ath12k_ahb_remove':
ahb.c:(.text+0x1146): undefined reference to `qcom_unregister_ssr_notifier'
Add a dependency to only allow the AHB portion to be enabled if either
remoteproc support is built-in or ath12k is itself a loadable module.
A better way to fix this would be to make the driver more modular and
ensure the AHB and PCI components are built into separate loadable modules
like it is done for the ATH10K driver, but that is a much larger rework
that someone else can do in the future.
Link: https://lore.kernel.org/all/20260313094852.3716715-1-arnd@kernel.org/
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/net/wireless/ath/ath12k/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/wireless/ath/ath12k/Kconfig b/drivers/net/wireless/ath/ath12k/Kconfig
index 0d5d1c55bfc1..4e8a301f2081 100644
--- a/drivers/net/wireless/ath/ath12k/Kconfig
+++ b/drivers/net/wireless/ath/ath12k/Kconfig
@@ -17,6 +17,7 @@ config ATH12K
config ATH12K_AHB
bool "Qualcomm ath12k AHB support"
depends on ATH12K && REMOTEPROC
+ depends on QCOM_RPROC_COMMON=y || ATH12K=m
select QCOM_MDT_LOADER
select QCOM_PAS
help
--
2.53.0
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 2/2] wifi: ath12k: add GENERIC_MSI_IRQ dependency
2026-11-12 9:26 [PATCH 1/2] [RESEND] wifi: ath12k: fix QCOM_RPROC_COMMON dependency Arnd Bergmann
@ 2026-11-12 9:26 ` Arnd Bergmann
2026-11-12 16:52 ` Jeff Johnson
0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2026-11-12 9:26 UTC (permalink / raw)
To: Jeff Johnson, Baochen Qiang, Aaradhana Sahu,
Vasanthakumar Thiagarajan, Sowmiya Sree Elavalagan
Cc: Arnd Bergmann, Eric Biggers, Sumit Garg, Konrad Dybcio,
linux-wireless, ath12k, linux-kernel
From: Arnd Bergmann <arnd@arndb.de>
The platform_device_msi_init_and_alloc_irqs() interface can be disabled
based on the configuration, though it is usually enabled. Without it,
the driver now fails to build:
drivers/net/wireless/ath/ath12k/pcic.c: In function 'ath12k_pcic_config_irq':
drivers/net/wireless/ath/ath12k/pcic.c:387:15: error: implicit declaration of function 'platform_device_msi_init_and_alloc_irqs' [-Wimplicit-function-declaration]
387 | ret = platform_device_msi_init_and_alloc_irqs(&pdev->dev,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ath/ath12k/pcic.c:424:9: error: implicit declaration of function 'platform_device_msi_free_irqs_all' [-Wimplicit-function-declaration]
424 | platform_device_msi_free_irqs_all(&pdev->dev);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Add a Kconfig dependency for it.
Fixes: b9bc9ef941bd ("wifi: ath12k: Add platform MSI support for CE")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
Note that this patch applied cleanly on top of another Kconfig
fix that I sent a while ago that is also still required,
see
https://lore.kernel.org/all/20260313094852.3716715-1-arnd@kernel.org/
---
drivers/net/wireless/ath/ath12k/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/wireless/ath/ath12k/Kconfig b/drivers/net/wireless/ath/ath12k/Kconfig
index 4e8a301f2081..ed93ff36fbe2 100644
--- a/drivers/net/wireless/ath/ath12k/Kconfig
+++ b/drivers/net/wireless/ath/ath12k/Kconfig
@@ -18,6 +18,7 @@ config ATH12K_AHB
bool "Qualcomm ath12k AHB support"
depends on ATH12K && REMOTEPROC
depends on QCOM_RPROC_COMMON=y || ATH12K=m
+ depends on GENERIC_MSI_IRQ
select QCOM_MDT_LOADER
select QCOM_PAS
help
--
2.53.0
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH 2/2] wifi: ath12k: add GENERIC_MSI_IRQ dependency
2026-11-12 9:26 ` [PATCH 2/2] wifi: ath12k: add GENERIC_MSI_IRQ dependency Arnd Bergmann
@ 2026-11-12 16:52 ` Jeff Johnson
0 siblings, 0 replies; 3+ messages in thread
From: Jeff Johnson @ 2026-11-12 16:52 UTC (permalink / raw)
To: Arnd Bergmann, Jeff Johnson, Baochen Qiang, Aaradhana Sahu,
Vasanthakumar Thiagarajan, Sowmiya Sree Elavalagan
Cc: Arnd Bergmann, Eric Biggers, Sumit Garg, Konrad Dybcio,
linux-wireless, ath12k, linux-kernel
On 11/12/2026 1:26 AM, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> The platform_device_msi_init_and_alloc_irqs() interface can be disabled
> based on the configuration, though it is usually enabled. Without it,
> the driver now fails to build:
>
> drivers/net/wireless/ath/ath12k/pcic.c: In function 'ath12k_pcic_config_irq':
> drivers/net/wireless/ath/ath12k/pcic.c:387:15: error: implicit declaration of function 'platform_device_msi_init_and_alloc_irqs' [-Wimplicit-function-declaration]
> 387 | ret = platform_device_msi_init_and_alloc_irqs(&pdev->dev,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/net/wireless/ath/ath12k/pcic.c:424:9: error: implicit declaration of function 'platform_device_msi_free_irqs_all' [-Wimplicit-function-declaration]
> 424 | platform_device_msi_free_irqs_all(&pdev->dev);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> Add a Kconfig dependency for it.
>
> Fixes: b9bc9ef941bd ("wifi: ath12k: Add platform MSI support for CE")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> Note that this patch applied cleanly on top of another Kconfig
> fix that I sent a while ago that is also still required,
> see
> https://lore.kernel.org/all/20260313094852.3716715-1-arnd@kernel.org/
> ---
> drivers/net/wireless/ath/ath12k/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/wireless/ath/ath12k/Kconfig b/drivers/net/wireless/ath/ath12k/Kconfig
> index 4e8a301f2081..ed93ff36fbe2 100644
> --- a/drivers/net/wireless/ath/ath12k/Kconfig
> +++ b/drivers/net/wireless/ath/ath12k/Kconfig
> @@ -18,6 +18,7 @@ config ATH12K_AHB
> bool "Qualcomm ath12k AHB support"
> depends on ATH12K && REMOTEPROC
> depends on QCOM_RPROC_COMMON=y || ATH12K=m
> + depends on GENERIC_MSI_IRQ
> select QCOM_MDT_LOADER
> select QCOM_PAS
> help
Perhaps because I picked up the original of the 1st patch 'b4 ty' didn't thank
for this patch, but it has been picked up in my ath-next branch
/jeff
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-11-12 16:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-11-12 9:26 [PATCH 1/2] [RESEND] wifi: ath12k: fix QCOM_RPROC_COMMON dependency Arnd Bergmann
2026-11-12 9:26 ` [PATCH 2/2] wifi: ath12k: add GENERIC_MSI_IRQ dependency Arnd Bergmann
2026-11-12 16:52 ` Jeff Johnson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®