SecurityPkg OpalPasswordDxe: Suppress option for special device.

According to current Pyrite SSC Spec 1.00, there is no parameter
for RevertSP method. So suppress KeepUserData option for the
Pyrite Ssc type device.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Eric Dong <eric.dong@intel.com>
Reviewed-by: Feng Tian <feng.tian@intel.com>

diff --git a/SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalHii.c b/SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalHii.c
index eb7d59b52c2a0da7e89aa5750d5a46079fc6b085..94dce6c5058c8edd80a4243a0c359fd99c6700af 100644 (file)
--- a/SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalHii.c
+++ b/SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalHii.c
@@ -374,7 +374,13 @@ HiiSelectDiskAction (
 \r
     case HII_KEY_ID_GOTO_REVERT:\r
       gHiiConfiguration.AvailableFields |= HII_FIELD_PASSWORD;\r
-      gHiiConfiguration.AvailableFields |= HII_FIELD_KEEP_USER_DATA;\r
+      if (OpalDisk->SupportedAttributes.PyriteSsc != 1) {\r
+        //\r
+        // According to current Pyrite SSC Spec 1.00, there is no parameter for RevertSP method.\r
+        // So issue RevertSP method without any parameter by suppress KeepUserData option.\r
+        //\r
+        gHiiConfiguration.AvailableFields |= HII_FIELD_KEEP_USER_DATA;\r
+      }\r
       if (AvailActions.RevertKeepDataForced) {\r
         gHiiConfiguration.AvailableFields |= HII_FIELD_KEEP_USER_DATA_FORCED;\r
       }\r
@@ -611,6 +617,14 @@ HiiPopulateDiskInfoForm(
         // Default initialize keep user Data to be true\r
         //\r
         gHiiConfiguration.KeepUserData = 1;\r
+        if (OpalDisk->SupportedAttributes.PyriteSsc == 1) {\r
+          //\r
+          // According to current Pyrite SSC Spec 1.00, there is no parameter for RevertSP method.\r
+          // So issue RevertSP method without any parameter by set default value to FALSE.\r
+          //\r
+          gHiiConfiguration.KeepUserData = 0;\r
+        }\r
+\r
       }\r
     }\r
   }\r