[edk2-devel] [PATCH 0/3] BaseTools, EmbeddedPkg, Maintainers.txt: Obsolete some drivers
Laszlo Ersek
lersek at redhat.com
Thu Apr 30 11:17:26 UTC 2020
On 04/29/20 23:47, Ard Biesheuvel wrote:
> 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
>
You could also (or alternatively) add a separate file "DEPRECATED.txt"
to the directory -- sometimes people don't read file-top comments,
before duplicating or editing code. Something that's visible with a
simple "ls -l" might stand out more.
Just a thought, I'm neutral on this.
Thanks
Laszlo
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#58430): https://edk2.groups.io/g/devel/message/58430
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]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list