Re: [edk2] [PATCH] SecurityPkg: Remove superfluous return statement

2017-02-16 Thread Thomas Huth
On 25.01.2017 10:58, Thomas Huth wrote: > 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-of

Re: [edk2] [PATCH 1/2] MdeModulePkg/AcpiTableDxe: condense whitespace around FADT.{DSDT, X_DSDT}

2017-02-02 Thread Thomas Huth
m...@redhat.com> > Cc: Leo Duran <leo.du...@amd.com> > Cc: Michael Tsirkin <mtsir...@redhat.com> > Cc: Phil Dennis-Jordan <p...@philjordan.eu> > Cc: Star Zeng <star.z...@intel.com> > Contributed-under: TianoCo

[edk2] [PATCH] OptionRomPkg: Remove superfluous return statement

2017-01-25 Thread Thomas Huth
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 <th...@redhat.com> --- OptionRomPkg/Bus/Usb/FtdiUsbSerialDxe

[edk2] [PATCH] SecurityPkg: Remove superfluous return statement

2017-01-25 Thread Thomas Huth
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 <th...@redhat.com> --- SecurityPkg/Tcg/Opal/OpalPasswordSmm/

Re: [edk2] [PATCH] MdeModulePkg: Remove superfluous return statements

2017-01-19 Thread Thomas Huth
ed-by: Star Zeng <star.z...@intel.com> > > Thanks, > Star > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Thomas > Huth > Sent: Thursday, January 19, 2017 6:16 PM > To: edk2-de...@ml01.01.org > Cc: Tian, Feng <f

[edk2] [PATCH] MdeModulePkg: Remove superfluous return statements

2017-01-19 Thread Thomas Huth
If the code eventually returns "Status" anyway, it does not make sense to explicitely return "Status" in case of an error, too. (this patch has been created with Coccinelle) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth <th...@redhat.com

[edk2] [PATCH] NetworkPkg: Remove superfluous return statement.

2017-01-19 Thread Thomas Huth
If the code eventually returns "Status" anyway, it does not make sense to explicitely return "Status" in case of an error, too. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth <th...@redhat.com> --- NetworkPkg/HttpDxe/HttpsSupport.c |

[edk2] [PATCH] OvmfPkg: Remove superfluous return statements.

2017-01-19 Thread Thomas Huth
If the code eventually returns "Status" anyway, it does not make sense to explicitely return "Status" in case of an error, too. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth <th...@redhat.com> --- OvmfPkg/Library/NvVarsFileLib/F

[edk2] [PATCH] PcAtChipsetPkg: Fix typing errors

2017-01-18 Thread Thomas Huth
Correct the reported by the codespell utility in some files of PcAtChipsetPkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth <th...@redhat.com> --- PcAtChipsetPkg/8254TimerDxe/Timer.c | 6 +++--- PcAtChipsetPkg/8254TimerDxe/T

[edk2] [PATCH] MdePkg: Fix some typing errors in the header files

2016-09-12 Thread Thomas Huth
Correct the typos in some header files of MdePkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth <th...@redhat.com> --- MdePkg/Include/AArch64/ProcessorBind.h| 2 +- MdePkg/Include/Arm/Processor

Re: [edk2] [PATCH 2/3] OvmfPkg: convert C files with LF line terminators to CRLF

2016-09-12 Thread Thomas Huth
rdan Justen <jordan.l.jus...@intel.com> > Cc: Thomas Huth <th...@redhat.com> > --- > OvmfPkg/Library/NvVarsFileLib/NvVarsFileLib.h | 122 +-- > OvmfPkg/VirtioRngDxe/VirtioRng.h | 92 +- > OvmfPkg/Library/NvVarsFileLib/FsAccess.c | 1034 +

Re: [edk2] [PATCH 1/3] OvmfPkg/IndustryStandard: make "Xen/grant_table.h" pure ASCII

2016-09-12 Thread Thomas Huth
.0 > Signed-off-by: Laszlo Ersek <ler...@redhat.com> > Cc: Jordan Justen <jordan.l.jus...@intel.com> > Cc: Thomas Huth <th...@redhat.com> > --- > OvmfPkg/Include/IndustryStandard/Xen/grant_table.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

[edk2] [PATCH] MdePkg: Fix some typing errors

2016-09-09 Thread Thomas Huth
Correct the typos in some files of MdePkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth <th...@redhat.com> --- .../BaseExtractGuidedSectionLib.c| 6 +++--- MdePkg/Library/BaseLib/String.c | 2 +- MdePkg/L

[edk2] [PATCH] OvmfPkg: Fix typing errors

2016-09-09 Thread Thomas Huth
Correct some typos (discovered with the codespell utility) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth <th...@redhat.com> --- OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c | 2 +- OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.c | 4 ++-- O