[edk2-devel] [PATCH 0/3] BaseTools, EmbeddedPkg, Maintainers.txt: Obsolete some drivers

[Ard Biesheuvel]

On 4/29/20 11:45 PM, Leif Lindholm wrote:
> On Wed, Apr 29, 2020 at 22:04:08 +0200, Ard Biesheuvel wrote:
>>>> I am mostly concerned about the use of MmcDxe in new platforms. The other
>>>> bits I'm not too worried about, and I think it would be fine to move those
>>>> into Platform/ARM/VExpressPkg in edk2-platforms, instead of hoping that
>>>> someone will turn up and turn them into driver model drivers.
>>>
>>> We could, although I would prefer not adding code to edk2-platforms
>>> that would not be accepted was it submitted as a new contribution.
>>> The SATA controller, I would ideally re-review and merge properly.
>>>
>>> If we do include the other drivers in platform-specific directories, I
>>> want them to come with ... strongly worded readmes.
>>>
>>
>> Right.
>>
>> Should we have some format for that? A way to log shortcomings along with
>> the code?
> 
> Thinking a bit more on this, maybe what we should do is add a template
> to each file's top comment block. Draft proposal:
> 
>    *
>    * WARNING:
>    * This driver fails to follow the UEFI driver model without a good
>    * reason, and only remains in the tree because it is still used by
>    * a small number of platforms. It will removed when no longer used.
>    * New platforms should not use it, and no one should use this as
>    * reference code for developing new drivers.
>    *
> 

Works for me


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#58341): https://edk2.groups.io/g/devel/message/58341
Mute This Topic: https://groups.io/mt/73356717/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-