Stephen,
On Wed, Feb 13, 2013 at 5:17 PM, Stephen Warren swarren@wwwdotorg.org wrote:
On 02/13/2013 02:44 PM, 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/board/compulab/trimslice/trimslice.c b/board/compulab/trimslice/trimslice.c
+#ifdef CONFIG_TEGRA_MMC
It seems a little odd to add this ifdef when it's know that config option is enabled in the trimslice config file. I can somewhat understand leaving the ifdef in any other board files that already had it, but actively adding it here seems strange.
Well, I was trying to be consistent when check all the board files for a pin_mux_mmc() function, and thought that adding the ifdef to the only board file that didn't have it would be the correct thing to do. In the original code (before moving to DT MMC), if you disabled SD/MMC in the trimslice config file, you'd get a build error. That's no longer true w/these patches, but I'm a better-safe-than-sorry guy.
I'll remove it in v4
diff --git a/board/nvidia/common/board.c b/board/nvidia/common/board.c
@@ -184,7 +187,6 @@ int board_init(void) /* prepare the WB code to LP0 location */ warmboot_prepare_code(TEGRA_LP0_ADDR, TEGRA_LP0_SIZE); #endif
return 0;}
Unrelated change?
Yep, due to fast-and-furious cut&pasting. I'll clean it up in v4.
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)
/* Mark position as used */node_list[i] = -1;Is this actually necessary?
As Simon says in his response, it's used when there're multiple compatible IDs, like the I2C code I pulled it from. I'll remove it for MMC in v4.
I'll try and test these patches on all the Tegra20 boards I have tomorrow.
Thanks
Tom