[PATCH] PCI: Drop ARM dependency from PCI_RCAR_GEN2

[PATCH] PCI: Drop ARM dependency from PCI_RCAR_GEN2

[Geert Uytterhoeven]

Since the reliance on ARM-specific struct pci_sys_data was removed, this
driver can be compile-tested on other architectures.

While at it, make the help text a bit more generic, as some members of
the R-Car Gen2 family have a different number of internal PCI
controllers.

Suggested-by: Ilpo Jarvinen <ilpo.jarvinen@linux.intel.com>
Fixes: 4a957563fe0231e0 ("PCI: rcar-gen2: Convert to use modern host bridge probe functions")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/pci/controller/Kconfig | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/pci/controller/Kconfig b/drivers/pci/controller/Kconfig
index 9f9d0c93cb461831..4c15caa1f1798ebb 100644
--- a/drivers/pci/controller/Kconfig
+++ b/drivers/pci/controller/Kconfig
@@ -259,12 +259,11 @@ config PCIE_RCAR_EP
 
 config PCI_RCAR_GEN2
 	bool "Renesas R-Car Gen2 Internal PCI controller"
-	depends on ARCH_RENESAS || COMPILE_TEST
-	depends on ARM
+	depends on (ARCH_RENESAS && ARM) || COMPILE_TEST
 	help
 	  Say Y here if you want internal PCI support on R-Car Gen2 SoC.
-	  There are 3 internal PCI controllers available with a single
-	  built-in EHCI/OHCI host controller present on each one.
+	  Each internal PCI controller contains a single built-in EHCI/OHCI
+	  host controller.
 
 config PCIE_ROCKCHIP
 	bool
-- 
2.43.0

Re: [PATCH] PCI: Drop ARM dependency from PCI_RCAR_GEN2

[Ilpo Järvinen]

[-- Attachment #1: Type: text/plain, Size: 1634 bytes --]

On Fri, 3 Oct 2025, Geert Uytterhoeven wrote:

> Since the reliance on ARM-specific struct pci_sys_data was removed, this
> driver can be compile-tested on other architectures.
> 
> While at it, make the help text a bit more generic, as some members of
> the R-Car Gen2 family have a different number of internal PCI
> controllers.
> 
> Suggested-by: Ilpo Jarvinen <ilpo.jarvinen@linux.intel.com>
> Fixes: 4a957563fe0231e0 ("PCI: rcar-gen2: Convert to use modern host bridge probe functions")

I'm not sure if fixes tag is warranted for "fixing" COMPILE_TEST.

Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  drivers/pci/controller/Kconfig | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/pci/controller/Kconfig b/drivers/pci/controller/Kconfig
> index 9f9d0c93cb461831..4c15caa1f1798ebb 100644
> --- a/drivers/pci/controller/Kconfig
> +++ b/drivers/pci/controller/Kconfig
> @@ -259,12 +259,11 @@ config PCIE_RCAR_EP
>  
>  config PCI_RCAR_GEN2
>  	bool "Renesas R-Car Gen2 Internal PCI controller"
> -	depends on ARCH_RENESAS || COMPILE_TEST
> -	depends on ARM
> +	depends on (ARCH_RENESAS && ARM) || COMPILE_TEST
>  	help
>  	  Say Y here if you want internal PCI support on R-Car Gen2 SoC.
> -	  There are 3 internal PCI controllers available with a single
> -	  built-in EHCI/OHCI host controller present on each one.
> +	  Each internal PCI controller contains a single built-in EHCI/OHCI
> +	  host controller.
>  
>  config PCIE_ROCKCHIP
>  	bool
> 

-- 
 i.

Re: [PATCH] PCI: Drop ARM dependency from PCI_RCAR_GEN2

[Manivannan Sadhasivam]

On Fri, 03 Oct 2025 10:35:34 +0200, Geert Uytterhoeven wrote:
> Since the reliance on ARM-specific struct pci_sys_data was removed, this
> driver can be compile-tested on other architectures.
> 
> While at it, make the help text a bit more generic, as some members of
> the R-Car Gen2 family have a different number of internal PCI
> controllers.
> 
> [...]

Applied, thanks!

[1/1] PCI: Drop ARM dependency from PCI_RCAR_GEN2
      commit: bd4e28f711a0c218641f72bb1602a2c8b9d36274

Best regards,
-- 
Manivannan Sadhasivam <mani@kernel.org>