On 29 February 2016 at 15:36, Leif Lindholm <[email protected]> wrote: > On Mon, Feb 29, 2016 at 01:46:42PM +0000, Ryan Harkin wrote: >> >> This is a substantial contribution, and on inspection it looks fine. >> >> However, it does change bits of MmcDxe substantially - so I would >> >> really like to see some Tested-by:s for existing platforms. >> >> >> >> Ryan? >> >> I'd love to, but the only platform I have with an MMC card slot is >> TC2. And MMC support as been broken in EDK2 on TC2 for some time. >> >> The last time I saw it working was at this point: >> >> 203bead 2015-07-01 There needs to be a space between the output >> section name and the colon, i.e., [Ard Biesheuvel] >> >> But that point doesn't boot on TC2 unless you revert this patch: >> >> d340ef7 2014-08-26 ArmPkg/ArmArchTimerLib: Remove non required >> [depex] and IoLib [Olivier Martin] >> >> I'll add it to my list of other things someone else broke that I have to fix. > > Evan, you mentioned you still use TC2 from time to time? > > Ryan: Other than that - does this break MMC on the virtual platforms you > look after? >
Now that TC2 is working again, I'll test this series tomorrow. I'll check FVP while I'm at it. _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
