Re: [edk2] [PATCH 2/4] MdeModulePkg: Add AtaAtapiPolicy protocol definition

2018-06-04 Thread Ni, Ruiyu
Ok. I will update the comments. Thanks/Ray > -Original Message- > From: Zeng, Star > Sent: Monday, June 4, 2018 2:30 PM > To: Ni, Ruiyu ; edk2-devel@lists.01.org > Cc: Zeng, Star > Subject: RE: [PATCH 2/4] MdeModulePkg: Add AtaAtapiPolicy protocol > definition > > I see below

Re: [edk2] [PATCH 2/4] MdeModulePkg: Add AtaAtapiPolicy protocol definition

2018-06-04 Thread Zeng, Star
I see below definition for PuisEnable. + /// others: Since PUIS setting is non-volatile, platform can use other value than 0/1 to keep hardware PUIS setting. + /// + UINT8 PuisEnable; What will the code do for DeviceSleepEnable and AggressiveDeviceSleepEnable when their value is other

[edk2] [PATCH 2/4] MdeModulePkg: Add AtaAtapiPolicy protocol definition

2018-05-31 Thread Ruiyu Ni
The patch adds AtaAtapiPolicy protocol which is produced by platform and consumed by AtaAtapiPassThruDxe driver. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni Cc: Star Zeng --- MdeModulePkg/Include/Protocol/AtaAtapiPolicy.h | 59 ++