[U-Boot] [PATCH v2 2/2] Tegra: MMC: Add DT support to MMC driver for all T20 boards
Stephen Warren
swarren at wwwdotorg.org
Tue Feb 12 21:38:04 CET 2013
On 02/11/2013 10:17 AM, Tom Warren wrote:
> tegra_mmc_init() now parses the DT info for bus width, WP/CD GPIOs, etc.
> Tested on Seaboard, fully functional.
>
> Tamonten boards (medcom-wide, plutux, and tec) use a different/new
> dtsi file w/common settings.
> diff --git a/drivers/mmc/tegra_mmc.c b/drivers/mmc/tegra_mmc.c
> +static int process_nodes(const void *blob, int node_list[], int count)
...
> + /* build mmc_host[] for each controller */
> + for (i = 0; i < count; i++) {
...
> + /* Mark position as used */
> + node_list[i] = -1;
Is that needed? Does anything use that array after this function?
> diff --git a/include/configs/medcom-wide.h b/include/configs/medcom-wide.h
> diff --git a/include/configs/plutux.h b/include/configs/plutux.h
> diff --git a/include/configs/tec.h b/include/configs/tec.h
In all 3 of those files ...
> #define CONFIG_DEFAULT_DEVICE_TREE tegra20-medcom-wide
Why not change that define ...
> #define CONFIG_OF_CONTROL
> #define CONFIG_OF_SEPARATE
> +#undef CONFIG_ARCH_DEVICE_TREE
> +#define CONFIG_ARCH_DEVICE_TREE tegra20-tamonten
rather than adding that one? All the other Tegra boards only set
CONFIG_DEFAULT_DEVICE_TREE.
Aside from the few comments I and others have made, this series looks good.
More information about the U-Boot
mailing list