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@nvidia.com>
Sent: Tuesday, December 24, 2019 11:00 AM
To: Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io; Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; afish@apple.com; Wang, Sunny (HPS SW) <sunnywang@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@intel.com>
Sent: Monday, December 23, 2019 7:38 PM
To: devel@edk2.groups.io <devel@edk2.groups.io>; Ashish Singhal <ashishsingha@nvidia.com>; Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; afish@apple.com <afish@apple.com>; Wang, Sunny (HPS SW) <sunnywang@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@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.