Please re-review the attached NetworkPkg patch. Feedback from both of my open reviews has been incorporated.
Thanks, Randy Pawell _______________________________________________________________________ NetworkPkg: 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. - Delete excess local variables that are initialized but otherwise unused. - Add LibraryClasses.ARM & AARCH64 section in NetworkPkg.dsc file, containing a CompilerIntrinsicsLib null-library, required for successful standalone package builds (copied from MdeModulePkg.dsc). 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 8:51 AM To: Pawell, Randy; [email protected]; [email protected] Subject: RE: [edk2] NetworkPkg: Source fixes and cleanup for ARMGCC compile I would also suggest to add CompilerIntrinsicsLib.inf to LibraryClasses.AARCH64: [LibraryClasses.ARM, LibraryClasses.AARCH64] # # It is not possible to prevent ARM compiler calls to generic intrinsic functions. # This library provides the instrinsic functions generated by a given compiler. # [LibraryClasses.ARM] and NULL mean link this library into all ARM images. # NULL|ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf Reviewed-By: Olivier Martin <[email protected]> > -----Original Message----- > From: Pawell, Randy [mailto:[email protected]] > Sent: 25 November 2014 21:44 > To: [email protected]; [email protected]; Olivier > Martin > Subject: [edk2] NetworkPkg: Source fixes and cleanup for ARMGCC compile > > Please review the attached NetworkPkg patch. > > Thanks, > Randy Pawell > _______________________________________________________________________ > _____________________________ > > NetworkPkg: 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. > - Delete excess local variables that are initialized but otherwise > unused. > - Add a LibraryClasses.ARM section in NetworkPkg.dsc file, containing a > CompilerIntrinsicsLib null-library, required for successful > standalone > package builds (copied from MdeModulePkg.dsc). > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Randy Pawell <[email protected]>
HP_ARM_edk2_NetworkPkg_Patch01v2.patch
Description: HP_ARM_edk2_NetworkPkg_Patch01v2.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
