Re: [U-Boot] [PATCH 3/9] mmc: Add Marvell Xenon SDHCI controller driver

[Stefan Roese]

Hi Masahiro,

On 20.01.2017 00:16, Masahiro Yamada wrote:

2017-01-19 16:39 GMT+09:00 Stefan Roese :

diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig
index 9ed8da39ef..87cc9439f3 100644
--- a/drivers/mmc/Kconfig
+++ b/drivers/mmc/Kconfig
@@ -287,6 +287,17 @@ config MMC_SDHCI_SPEAR

  If unsure, say N.

+config MMC_XENON_SDHCI
+   bool "SDHCI support for the Xenon SDHCI controller"
+   depends on MMC_SDHCI && DM_MMC && OF_CONTROL
+   help
+ Support for Xenon SDHCI host controller on Marvell Armada 3700
+ 7k/8k ARM SoCs platforms
+
+ If you have a controller with this interface, say Y here.
+
+ If unsure, say N.
+
 config MMC_SDHCI_TEGRA
bool "SDHCI platform support for the Tegra SD/MMC Controller"
depends on TEGRA



Can you rename the CONFIG symbol to MMC_SDHCI_XENON, please?


Sure.


The renaming is underway, but I want to make this consistent
in the MMC subsystem in the end.


Thanks for working on this. I'll queue this change in v2 and will send
it in a few days.

Thanks,
Stefan
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/9] mmc: Add Marvell Xenon SDHCI controller driver

[Masahiro Yamada]

Hi Stefan,


2017-01-19 16:39 GMT+09:00 Stefan Roese :
> diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig
> index 9ed8da39ef..87cc9439f3 100644
> --- a/drivers/mmc/Kconfig
> +++ b/drivers/mmc/Kconfig
> @@ -287,6 +287,17 @@ config MMC_SDHCI_SPEAR
>
>   If unsure, say N.
>
> +config MMC_XENON_SDHCI
> +   bool "SDHCI support for the Xenon SDHCI controller"
> +   depends on MMC_SDHCI && DM_MMC && OF_CONTROL
> +   help
> + Support for Xenon SDHCI host controller on Marvell Armada 3700
> + 7k/8k ARM SoCs platforms
> +
> + If you have a controller with this interface, say Y here.
> +
> + If unsure, say N.
> +
>  config MMC_SDHCI_TEGRA
> bool "SDHCI platform support for the Tegra SD/MMC Controller"
> depends on TEGRA


Can you rename the CONFIG symbol to MMC_SDHCI_XENON, please?

The renaming is underway, but I want to make this consistent
in the MMC subsystem in the end.




-- 
Best Regards
Masahiro Yamada
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot