[PATCH] soc: qcom: PDR_HELPERS, PDR_MSG are independent of QCOM_SOC

[PATCH] soc: qcom: PDR_HELPERS, PDR_MSG are independent of QCOM_SOC

[Randy Dunlap]

Currently QCOM_PDR_MSG depends on QCOM_SOC (it's inside the
"if QCOM_SOC" block). It appears to be independent of QCOM_SOC, as
though it was entered in the wrong location in the Kconfig file.

Moving QCOM_PDR_MSG after the end of the QCOM_SOC block fixes the
kconfig unmet dependencies warning and survives both randconfig and
allmodconfig builds.

WARNING: unmet direct dependencies detected for QCOM_PDR_MSG
  Depends on [n]: QCOM_SOC [=n]
  Selected by [y]:
  - QCOM_PDR_HELPERS [=y] && NET [=y]

Caused by SLIM_QCOM_NGD_CTRL selecting QCOM_PDR_HELPERS.

Fixes: 0ac5c7d933de ("soc: qcom: pdr: extract PDR message marshalling data")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
---
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konradybcio@kernel.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: Dmitry Baryshkov <lumag@kernel.org>
Cc: Arnd Bergmann <arnd@arndb.de>

 drivers/soc/qcom/Kconfig |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

--- linux-next-20260726.orig/drivers/soc/qcom/Kconfig
+++ linux-next-20260726/drivers/soc/qcom/Kconfig
@@ -90,9 +90,6 @@ config QCOM_PD_MAPPER
 	  implementation of the service. It is a simpler alternative to the
 	  userspace daemon.
 
-config QCOM_PDR_MSG
-	tristate
-
 config QCOM_PMIC_PDCHARGER_ULOG
 	tristate "Qualcomm PMIC PDCharger ULOG driver"
 	depends on RPMSG
@@ -297,6 +294,9 @@ endif
 # Options selected by other drivers from different subsystems must be outside
 # of the menuconfig if-block:
 
+config QCOM_PDR_MSG
+	tristate
+
 config QCOM_INLINE_CRYPTO_ENGINE
 	tristate
 	select QCOM_SCM

Re: [PATCH] soc: qcom: PDR_HELPERS, PDR_MSG are independent of QCOM_SOC

[Julian Braha]

On 7/27/26 22:11, Randy Dunlap wrote:
> Currently QCOM_PDR_MSG depends on QCOM_SOC (it's inside the
> "if QCOM_SOC" block). It appears to be independent of QCOM_SOC, as
> though it was entered in the wrong location in the Kconfig file.
> 
> Moving QCOM_PDR_MSG after the end of the QCOM_SOC block fixes the
> kconfig unmet dependencies warning and survives both randconfig and
> allmodconfig builds.
> 
> WARNING: unmet direct dependencies detected for QCOM_PDR_MSG
>   Depends on [n]: QCOM_SOC [=n]
>   Selected by [y]:
>   - QCOM_PDR_HELPERS [=y] && NET [=y]
> 
> Caused by SLIM_QCOM_NGD_CTRL selecting QCOM_PDR_HELPERS.
> 
> Fixes: 0ac5c7d933de ("soc: qcom: pdr: extract PDR message marshalling data")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> ---
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Konrad Dybcio <konradybcio@kernel.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: Dmitry Baryshkov <lumag@kernel.org>
> Cc: Arnd Bergmann <arnd@arndb.de>
> 
>  drivers/soc/qcom/Kconfig |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> --- linux-next-20260726.orig/drivers/soc/qcom/Kconfig
> +++ linux-next-20260726/drivers/soc/qcom/Kconfig
> @@ -90,9 +90,6 @@ config QCOM_PD_MAPPER
>  	  implementation of the service. It is a simpler alternative to the
>  	  userspace daemon.
>  
> -config QCOM_PDR_MSG
> -	tristate
> -
>  config QCOM_PMIC_PDCHARGER_ULOG
>  	tristate "Qualcomm PMIC PDCharger ULOG driver"
>  	depends on RPMSG
> @@ -297,6 +294,9 @@ endif
>  # Options selected by other drivers from different subsystems must be outside
>  # of the menuconfig if-block:
>  
> +config QCOM_PDR_MSG
> +	tristate
> +
>  config QCOM_INLINE_CRYPTO_ENGINE
>  	tristate
>  	select QCOM_SCM

This is a better solution than my earlier attempt:
https://lore.kernel.org/all/20260719022242.139472-1-julianbraha@gmail.com/

- Julian Braha