On 04/29/20 18:36, Leif Lindholm wrote: > The EmbeddedPkg MmcDxe duplicates definitions an functionality
s/ an / and / > also available in MdeModulePkg. New platforms should not be > making use of this, and old platforms should consider migrating - > indicate this properly by creating a separate section in > Maintainers.txt and marking it "Obsolete". > > Cc: Andrew Fish <[email protected]> > Cc: Ard Biesheuvel <[email protected]> > Cc: Laszlo Ersek <[email protected]> > Cc: Michael D Kinney <[email protected]> > Signed-off-by: Leif Lindholm <[email protected]> > --- > Maintainers.txt | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Maintainers.txt b/Maintainers.txt > index 1733225722b6..4d62e0b334f3 100644 > --- a/Maintainers.txt > +++ b/Maintainers.txt > @@ -172,10 +172,17 @@ M: Alexei Fedorov <[email protected]> > > EmbeddedPkg > F: EmbeddedPkg/ > +X: EmbeddedPkg/Include/Protocol/MmcHost.h > +X: EmbeddedPkg/Universal/MmcDxe/ > W: https://github.com/tianocore/tianocore.github.io/wiki/EmbeddedPkg > M: Leif Lindholm <[email protected]> > M: Ard Biesheuvel <[email protected]> > > +EmbeddedPkg obsolete code > +F: EmbeddedPkg/Include/Protocol/MmcHost.h > +F: EmbeddedPkg/Universal/MmcDxe/ > +S: Obsolete > + > EmulatorPkg > F: EmulatorPkg/ > W: https://github.com/tianocore/tianocore.github.io/wiki/EmulatorPkg > Can't comment on EmbeddedPkg MmcDxe, but formally, the patch is pleasing. Reviewed-by: Laszlo Ersek <[email protected]> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#58333): https://edk2.groups.io/g/devel/message/58333 Mute This Topic: https://groups.io/mt/73356719/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
