[edk2-devel] [PATCH v6] MdeModulePkg: Add EDK2 Platform Boot Manager Protocol

[Ni, Ray]

Sunny had one comment regarding the API name and I replied.
Let's wait for Sunny's further comments as well.

Thanks,
Ray

From: Ashish Singhal <ashishsingha at nvidia.com>
Sent: Tuesday, December 24, 2019 11:00 AM
To: Ni, Ray <ray.ni at intel.com>; devel at edk2.groups.io; Wang, Jian J <jian.j.wang at intel.com>; Wu, Hao A <hao.a.wu at intel.com>; Gao, Zhichao <zhichao.gao at intel.com>; Kinney, Michael D <michael.d.kinney at intel.com>; afish at apple.com; Wang, Sunny (HPS SW) <sunnywang at hpe.com>
Subject: Re: [edk2-devel] [PATCH v6] MdeModulePkg: Add EDK2 Platform Boot Manager Protocol

Hi Ray,

Patch 7 has been submitted. Please let me know if there are any other changes needed. I would wait for your patch acceptance email.

Thanks
Ashish
________________________________
From: Ni, Ray <ray.ni at intel.com<mailto:ray.ni at intel.com>>
Sent: Monday, December 23, 2019 7:38 PM
To: devel at edk2.groups.io<mailto:devel at edk2.groups.io> <devel at edk2.groups.io<mailto:devel at edk2.groups.io>>; Ashish Singhal <ashishsingha at nvidia.com<mailto:ashishsingha at nvidia.com>>; Wang, Jian J <jian.j.wang at intel.com<mailto:jian.j.wang at intel.com>>; Wu, Hao A <hao.a.wu at intel.com<mailto:hao.a.wu at intel.com>>; Gao, Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>; Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>; afish at apple.com<mailto:afish at apple.com> <afish at apple.com<mailto:afish at apple.com>>; Wang, Sunny (HPS SW) <sunnywang at hpe.com<mailto:sunnywang at hpe.com>>
Subject: RE: [edk2-devel] [PATCH v6] MdeModulePkg: Add EDK2 Platform Boot Manager Protocol

External email: Use caution opening links or attachments


> +(EFIAPI *REFRESH_ALL_BOOT_OPTIONS) (

I recommend to rename it to "PLATFORM_BOOT_MANAGER_REFRESH_ALL_BOOT_OPTIONS" to avoid
future name confliction with other protocols.
With that, Reviewed-by: Ray Ni <ray.ni at intel.com<mailto:ray.ni at intel.com>>
________________________________
This email message is for the sole use of the intended recipient(s) and may contain confidential information.  Any unauthorized review, use, disclosure or distribution is prohibited.  If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message.
________________________________

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

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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20191224/866199ba/attachment.htm>