From 0d5df2aba72e7c8f2f172900fb7cfece27851a8f Mon Sep 17 00:00:00 2001
From: erictian <erictian@6f19259b-4bc3-4df7-8a09-765794883524>
Date: Wed, 31 Oct 2012 09:33:40 +0000
Subject: [PATCH] MdeModulePkg/AtaBusDxe: remove a redundant error check

Signed-off-by: Tian Feng <feng.tian@intel.com>
Reviewed-by: Li Elvin <elvin.li@intel.com>

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@13902 6f19259b-4bc3-4df7-8a09-765794883524
---
 MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c
index 5e5cab6575..d44359cfcf 100644
--- a/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c
+++ b/MdeModulePkg/Bus/Ata/AtaBusDxe/AtaPassThruExecute.c
@@ -433,9 +433,7 @@ DiscoverAtaDevice (
       // The command is issued successfully
       //
       Status = IdentifyAtaDevice (AtaDevice);
-      if (!EFI_ERROR (Status)) {
-        return Status;
-      }
+      return Status;
     }
   } while (Retry-- > 0);
 
-- 
2.39.2