[U-Boot] [PATCH v3 0/6] arm: tegra: apalis-tk1, mmc and ext clock loopback

[Masahiro Yamada]

2017-01-04 4:40 GMT+09:00 Tom Warren <TWarren at nvidia.com>:
> Marcel,
>
>> -----Original Message-----
>> From: Marcel Ziswiler [mailto:marcel at ziswiler.com]
>> Sent: Wednesday, December 21, 2016 9:26 PM
>> To: u-boot at lists.denx.de
>> Cc: Max Krummenacher <max.krummenacher at toradex.com>; Stefan Agner
>> <stefan.agner at toradex.com>; Marcel Ziswiler
>> <marcel.ziswiler at toradex.com>; York Sun <york.sun at nxp.com>;
>> Peter.Chubb at data61.csiro.au; Lokesh Vutla <lokeshvutla at ti.com>; Jaehoon
>> Chung <jh80.chung at samsung.com>; Hans de Goede <hdegoede at redhat.com>;
>> Heiko Schocher <hs at denx.de>; Albert Aribaud <albert.u.boot at aribaud.net>;
>> Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; Stephen Warren
>> <swarren at nvidia.com>; Jagan Teki <jteki at openedev.com>; Alexander Graf
>> <agraf at suse.de>; Tom Warren <TWarren at nvidia.com>; Joe Hershberger
>> <joe.hershberger at ni.com>; Lucas Stach <dev at lynxeye.de>; Allen Martin
>> <AMartin at nvidia.com>; Stefan Roese <sr at denx.de>; Alban Bedel
>> <alban.bedel at avionic-design.de>; Simon Glass <sjg at chromium.org>;
>> Masahiro Yamada <yamada.masahiro at socionext.com>; Ian Campbell
>> <ijc at hellion.org.uk>
>> Subject: [PATCH v3 0/6] arm: tegra: apalis-tk1, mmc and ext clock loopback
>>
>> From: Marcel Ziswiler <marcel.ziswiler at toradex.com>
>>
>>
>> This series adds support for the Toradex Apalis TK1 as well as moves the
>> CONFIG_TEGRA_MMC to Kconfig and introduces a new option to disable the
>> external clock loopback on SDMMC3.
>>
>> The whole series is also available here:
>>
>> http://git.toradex.com/cgit/u-boot-toradex.git/log/?h=for-next
>>
>> Changes in v3:
>> - My recent fresh install of Fedora 25 was missing an aarch64 toolchain
>>   which made moveconfig.py not properly handle the 6 newer Tegras. I now
>>   re-run moveconfig.py after installing an aarch64 toolchain and it all
>>   looks sane. Sorry about that, Tom.
> With Mashiro's recent 'mmc: move more driver config options to Kconfig' patchset (which is in TOT U-Boot now), your v3 patchset no longer applies cleanly.
>
> I'll rebase u-boot-tegra/master against u-boot/master and push a new repo to Denx so you can get your Apalis-TK1 patchset to work w/all of the recent changes.
>
> Thanks, and sorry for the extra work!
>

(Sorry, the previous mail had been sent before I completed it)

Marcel,

Sorry, I did not notice you were moving CONFIG_TEGRA_MMC.


Moreover, I am moving CONFIG_TEGRA_MMC and renaming in the following patch:

http://patchwork.ozlabs.org/patch/710054/


My goal is all MMC driver options prefixed with CONFIG_MMC_.
(and SDHCI variants with CONFIG_MMC_SDHCI_,
DesignWare variants with CONFIG_MMC_DW_ )


Here is a question about the Tegra MMC driver.

>From the compatible strings in drivers/mmc/tegra-mmc.c,
I suspect this driver is based on SDHCI
and in Linux, the option name is CONFIG_MMC_SDHCI_TEGRA.

Why doesn't the driver re-use driver/mmc/sdhci.c ?



-- 
Best Regards
Masahiro Yamada