MdeModulePkg AtaAtapiPassThru: Always do S.M.A.R.T. check if device support

If S.M.A.R.T. is supported, it can be checked.

Signed-off-by: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Reviewed-by: Hao A Wu <hao.a.wu@intel.com>

diff --git a/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AhciMode.c b/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AhciMode.c
index 1053695b3b625f9cf81f8fbcff64501e47caf830..7626bac38d00b0f2799710dff8fa7186c8ab3545 100644 (file)
--- a/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AhciMode.c
+++ b/MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AhciMode.c
@@ -1833,18 +1833,17 @@ AhciAtaSmartSupport (
                    ATA_ATAPI_TIMEOUT,\r
                    NULL\r
                    );\r
-\r
-        if (!EFI_ERROR (Status)) {\r
-          Status = AhciAtaSmartReturnStatusCheck (\r
-                     PciIo,\r
-                     AhciRegisters,\r
-                     (UINT8)Port,\r
-                     (UINT8)PortMultiplier,\r
-                     AtaStatusBlock\r
-                     );\r
-        }\r
       }\r
     }\r
+\r
+    AhciAtaSmartReturnStatusCheck (\r
+      PciIo,\r
+      AhciRegisters,\r
+      (UINT8)Port,\r
+      (UINT8)PortMultiplier,\r
+      AtaStatusBlock\r
+      );\r
+\r
     DEBUG ((EFI_D_INFO, "Enabled S.M.A.R.T feature at port [%d] PortMultiplier [%d]!\n",\r
             Port, PortMultiplier));\r
   }\r