Re: [PATCH] mmc: sdhci-brcmstb: Add ARCH_BCM2835 option

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Peter,

On 08:42 Thu 20 Jun     , Peter Robinson wrote:
> The Raspberry Pi devices have to date all used ARCH_BCM2835
> as their SoC arch dependency so configurations that use this
> and not BRCMSTB won't end up with this module in their config.

AFAIK the BCM2712 (RPi5) is the only SoC of the Raspberry family
that has an MMC host controller that is more or less like the one
present on the STB line silicon. Adding ARCH_BCM2835 to the dependency
will then allow RPi<=4 to use this driver for which they do not
have the relevant hw.
If I'm correct about the aforementioned point, I'd be inclined to
avoid this change, then.

Many thanks,
Andrea

> 
> Cc: Andrea della Porta <andrea.porta@xxxxxxxx>
> Cc: Stefan Wahren <wahrenst@xxxxxxx>
> Cc: Adrian Hunter <adrian.hunter@xxxxxxxxx>
> Cc: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
> Fixes: 40f22df5269e6 ("mmc: sdhci-brcmstb: Add BCM2712 support")
> Signed-off-by: Peter Robinson <pbrobinson@xxxxxxxxx>
> ---
>  drivers/mmc/host/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
> index bb0d4fb0892ae..eb3ecfe055910 100644
> --- a/drivers/mmc/host/Kconfig
> +++ b/drivers/mmc/host/Kconfig
> @@ -1016,7 +1016,7 @@ config MMC_SDHCI_MICROCHIP_PIC32
>  
>  config MMC_SDHCI_BRCMSTB
>  	tristate "Broadcom SDIO/SD/MMC support"
> -	depends on ARCH_BRCMSTB || BMIPS_GENERIC || COMPILE_TEST
> +	depends on ARCH_BRCMSTB || ARCH_BCM2835 || BMIPS_GENERIC || COMPILE_TEST
>  	depends on MMC_SDHCI_PLTFM
>  	select MMC_CQHCI
>  	default ARCH_BRCMSTB || BMIPS_GENERIC
> -- 
> 2.45.2
> 




[Index of Archives]     [Linux Memonry Technology]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux