[U-Boot] [PATCH v2 2/3] tegra2: Add Avionic Design Plutux support.
Stephen Warren
swarren at nvidia.com
Wed Nov 16 17:03:35 CET 2011
Thierry Reding wrote at Wednesday, November 16, 2011 12:14 AM:
> * Thierry Reding wrote:
> > * Simon Glass wrote:
> [...]
> > >
> > > If you are not implementing MMC, just remove it and all the MMC code below.
> > >
> >
> > I don't understand. What makes you think I don't want MMC? Both the Plutux
> > and the Medcom boot from MMC. If you are referring to the CD being
> > hard-coded, then that is just because I copied from Harmony. You are right
> > though that I should probably fill the implementation. I'm not really sure
> > about the best way to do this, though. The struct mmc passed into
> > board_mmc_getcd() doesn't contain enough information to decide which GPIO to
> > check. I could assume it is always SD4 since that is the only controller that
> > is initialized, but would that be acceptable?
>
> I was going to implement proper card-detection (it's quite trivial) but then
> I saw that board_mmc_getcd() is never called in the Tegra2 MMC driver so I
> guess that should be implemented as well. Did anybody work on that already or
> would you like me to prepare a patch?
I noticed that too, but adding CD support to the Tegra driver itself didn't
seem right; surely it should be added to the MMC core?
Anyway, if you're looking at hacking on Tegra MMC, please do be aware of
a few patches I posted re: Tegra boards and MMC changes:
tegra2: Move board_mmc_init into board files
http://patchwork.ozlabs.org/patch/118143/
tegra2: Modify MMC driver to handle power and cd GPIOs
http://patchwork.ozlabs.org/patch/118142/
tegra2: Add support for Ventana
http://patchwork.ozlabs.org/patch/118144/
I hope these will be merged soon, but IIRC they haven't received any
feedback either way from Albert.
--
nvpublic
More information about the U-Boot
mailing list