On Tue, Apr 02, 2013 at 01:58:41PM -0600, Stephen Warren wrote: > On 04/02/2013 05:20 AM, Joseph Lo wrote: > > This patch adds "non-removable" property of MMC host where the eMMC device > > is for Tegra platform. > > > > And the "keep-power-in-suspend" property was used for the SDIO device that > > need this to go into suspend mode (e.g. BRCM43xx series). > > > arch/arm/boot/dts/tegra20-paz00.dts | 1 + > > arch/arm/boot/dts/tegra20-seaboard.dts | 2 ++ > > arch/arm/boot/dts/tegra20-ventana.dts | 2 ++ > > arch/arm/boot/dts/tegra20-whistler.dts | 1 + > > arch/arm/boot/dts/tegra30-beaver.dts | 1 + > > arch/arm/boot/dts/tegra30-cardhu-a02.dts | 1 + > > arch/arm/boot/dts/tegra30-cardhu-a04.dts | 1 + > > arch/arm/boot/dts/tegra30-cardhu.dtsi | 1 + > > I think there are a bunch of other boards that need a similar change, > right? There are certainly eMMC devices on other boards, and shouldn't > they be marked with non-removable? In particular, Beaver, Dalmore, and > Pluto, and likely some/all of the Colibri and Avionic Design boards. I > think it should be obvious which SDHCI devices need this, since there > won't be any cd-gpios property and the bus-width will likely be 8. For Tegra20 Tamonten we don't use eMMC. Ideally we'd be using NAND to boot from but that doesn't have mainline support. I have a semi-working patch and will probably spend some more time getting it ready. The MMC cards are all removable, though I guess since they are the boot device they still should remain powered in suspend? Thierry
Attachment:
pgpCDIhfTQkWe.pgp
Description: PGP signature