Please re-review the attached MdeModulePkg patch. Feedback from both of my open reviews has been incorporated.
Thanks, Randy Pawell _______________________________________________________________________ MdeModulePkg: Source fixes and cleanup for ARMGCC compiles - Fix EFI_IPv4_ADDRESS usages to use a macro to copy the structure instead of direct assignment, to avoid runtime alignment errors. - Fix a EFI_INPUT_KEY usage in TerminalDxe to use CopyMem() to copy the structure instead of direct assignment, to avoid runtime alignment error. - Delete excess local variables that are initialized but otherwise unused. - CompilerIntrinsicsLib library now imported for AARCH64, as well as ARM. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Randy Pawell <[email protected]> -----Original Message----- From: Olivier Martin [mailto:[email protected]] Sent: Monday, December 01, 2014 7:46 AM To: Pawell, Randy; [email protected]; [email protected] Subject: RE: [edk2] MdeModulePkg: Source fixes and cleanup for ARMGCC compiles I have not see anything controversial and I do not disagree with Feng's comments. These patches (this one and the one on 2014-11-26) look good to me. Reviewed-By: Olivier Martin <[email protected]> > -----Original Message----- > From: Pawell, Randy [mailto:[email protected]] > Sent: 25 November 2014 21:43 > To: [email protected]; [email protected]; Olivier > Martin > Subject: [edk2] MdeModulePkg: Source fixes and cleanup for ARMGCC > compiles > > Please review the attached MdeModulePkg patch. > > Thanks, > Randy Pawell > _______________________________________________________________________ > _____ > > MdeModulePkg: Source fixes and cleanup for ARMGCC compiles > - Fix EFI_IPv4_ADDRESS usages to use a macro or CopyMem() to copy the > structure instead of direct assignment, to avoid runtime alignment > errors. > - Fix a EFI_INPUT_KEY usage in TerminalDxe to use CopyMem() to copy > the > structure instead of direct assignment, to avoid runtime alignment > error. > - Delete excess local variables that are initialized but otherwise > unused. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Randy Pawell <[email protected]>
HP_ARM_edk2_MdeModulePkg_Patch01v3.patch
Description: HP_ARM_edk2_MdeModulePkg_Patch01v3.patch
------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=157005751&iu=/4140/ostg.clktrk
_______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel
