Re: [PATCH] OptionRomPkg: Remove superfluous return statement

["Ni, Ruiyu" <ruiyu.ni@intel.com>]

Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

Thanks/Ray

> -----Original Message-----
> From: Thomas Huth [mailto:thuth@redhat.com]
> Sent: Wednesday, January 25, 2017 6:02 PM
> To: edk2-devel@ml01.01.org
> Cc: Ni, Ruiyu <ruiyu.ni@intel.com>
> Subject: [PATCH] OptionRomPkg: Remove superfluous return statement
> 
> If the code eventually returns "Status" anyway, it does not make sense to
> explicitly return "Status" in case of an error, too.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  OptionRomPkg/Bus/Usb/FtdiUsbSerialDxe/FtdiUsbSerialDriver.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/OptionRomPkg/Bus/Usb/FtdiUsbSerialDxe/FtdiUsbSerialDriver.c
> b/OptionRomPkg/Bus/Usb/FtdiUsbSerialDxe/FtdiUsbSerialDriver.c
> index c03606b..1eceb23 100644
> --- a/OptionRomPkg/Bus/Usb/FtdiUsbSerialDxe/FtdiUsbSerialDriver.c
> +++ b/OptionRomPkg/Bus/Usb/FtdiUsbSerialDxe/FtdiUsbSerialDriver.c
> @@ -2404,9 +2404,6 @@ SetAttributes (
>               DataBits,
>               StopBits
>               );
> -  if (EFI_ERROR (Status)) {
> -    return Status;
> -  }
> 
>    return Status;
>  }
> --
> 1.8.3.1