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

["Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>]

[-- 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.