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 <af...@apple.com>
> Cc: Ard Biesheuvel <ard.biesheu...@arm.com>
> Cc: Laszlo Ersek <ler...@redhat.com>
> Cc: Michael D Kinney <michael.d.kin...@intel.com>
> Signed-off-by: Leif Lindholm <l...@nuviainc.com>
> ---
>  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 <alexei.fedo...@arm.com>
>  
>  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 <l...@nuviainc.com>
>  M: Ard Biesheuvel <ard.biesheu...@arm.com>
>  
> +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 <ler...@redhat.com>


-=-=-=-=-=-=-=-=-=-=-=-
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: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to