Fix bug in SerialRead returning error if no data was present.

No data should not be an error return. Causing lots of REPORT_STATUS_CODE messages out of the TerminalDxe driver.

Signed-off-by: andrewfish
Reviewed-by: andrewfish
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@11770 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/EmbeddedPkg/SerialDxe/SerialIo.c b/EmbeddedPkg/SerialDxe/SerialIo.c
index b9ea4b9c423c001d647eb054b695e620295b6d10..fe0b1711a8a7aa772dc8b88241202a7d34e0d5c3 100644 (file)
--- a/EmbeddedPkg/SerialDxe/SerialIo.c
+++ b/EmbeddedPkg/SerialDxe/SerialIo.c
@@ -193,8 +193,12 @@ SerialRead (
   if (SerialPortPoll()) {\r
     Count = SerialPortRead (Buffer, *BufferSize);\r
     *BufferSize = Count;\r
+    return (Count == 0) ? EFI_DEVICE_ERROR : EFI_SUCCESS;\r
   }\r
-  return (Count == 0) ? EFI_DEVICE_ERROR : EFI_SUCCESS;\r
+  \r
+  // No data to return\r
+  *BufferSize = 0;\r
+  return EFI_SUCCESS;\r
 }\r
 \r
 \r