Re: [edk2] [PATCH 3/3] MdeModulePkg/ResetSystem: Implement ResetNotification protocol

2017-06-29 Thread Ni, Ruiyu
I fully agree. Sorry I missed that part. I will update all of them before checking the code in. Regards, Ray >-Original Message- >From: Zeng, Star >Sent: Thursday, June 29, 2017 10:30 PM >To: Ni, Ruiyu ; edk2-devel@lists.01.org >Cc: Zeng, Star >Subject: RE: [PATCH 3/3] MdeModulePkg/Reset

Re: [edk2] [PATCH 3/3] MdeModulePkg/ResetSystem: Implement ResetNotification protocol

2017-06-29 Thread Zeng, Star
How about to also update the description in ResetSystemRuntimeDxe.inf/ResetSystem.c/ResetSystemRuntimeDxe.uni? :) For example in *.inf: # This driver implements Reset Architectural Protocol. -> # This driver implements Reset Architectural and Notification Protocol. With that update, Reviewed-by:

[edk2] [PATCH 3/3] MdeModulePkg/ResetSystem: Implement ResetNotification protocol

2017-06-29 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Star Zeng --- .../Universal/ResetSystemRuntimeDxe/ResetSystem.c | 137 - .../Universal/ResetSystemRuntimeDxe/ResetSystem.h | 21 +++- .../ResetSystemRuntimeDxe.inf |