Stephen, On Wed, Feb 13, 2013 at 5:17 PM, Stephen Warren <swar...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot