Could you also update the description for EFI_SERIAL_SET_ATTRIBUTES in MdePkg/Include/Protocol/SerialIo.h? That could be done in a separated patch.
Thanks, Star -----Original Message----- From: Julien Grall [mailto:julien.gr...@linaro.org] Sent: Saturday, November 25, 2017 12:20 AM To: Zeng, Star <star.z...@intel.com>; Dong, Eric <eric.d...@intel.com>; pankaj.ban...@nxp.com; ler...@redhat.com; leif.lindh...@linaro.org Cc: edk2-devel@lists.01.org; Julien Grall <julien.gr...@linaro.org> Subject: [PATCH v2 1/3] MdeModulePkg/SerialDxe: Describe correctly EFI_DEVICE_ERROR for SetAttributes Per the UEFIv2.7 spec, EFI_DEVICE_ERROR is returned when the serial device is not functioning correctly. Update the description to avoid confusion. Contributed-under: Tianocore Contribution Agreement 1.1 Signed-off-by: Julien Grall <julien.gr...@linaro.org> --- MdeModulePkg/Universal/SerialDxe/SerialIo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/SerialDxe/SerialIo.c b/MdeModulePkg/Universal/SerialDxe/SerialIo.c index 964d0329f4..5946a6e2f3 100644 --- a/MdeModulePkg/Universal/SerialDxe/SerialIo.c +++ b/MdeModulePkg/Universal/SerialDxe/SerialIo.c @@ -265,7 +265,7 @@ SerialReset ( stop bits. @retval EFI_SUCCESS The device was reset. - @retval EFI_DEVICE_ERROR The serial device could not be reset. + @retval EFI_DEVICE_ERROR The serial device is not functioning correctly. **/ EFI_STATUS -- 2.11.0 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel