On Wed, Nov 27, 2013 at 4:22 AM, Wei Liu <wei.l...@citrix.com> wrote: > On Tue, Nov 26, 2013 at 02:47:13PM -0500, Konrad Rzeszutek Wilk wrote: >> On Tue, Nov 26, 2013 at 07:25:58PM +0000, Wei Liu wrote: >> > E820 definitions copied from IntelFrameworkModulePkg/Csm/ >> > LegacyBiosDxe/LegacyBiosInterface.h. >> > >> > Contributed-under: TianoCore Contribution Agreement 1.0 >> > Signed-off-by: Wei Liu <wei.l...@citrix.com> >> > Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com> >> > --- >> > OvmfPkg/Include/IndustryStandard/E820.h | 46 >> > +++++++++++++++++++++++++++++++ >> > 1 file changed, 46 insertions(+) >> > create mode 100644 OvmfPkg/Include/IndustryStandard/E820.h >> > >> > diff --git a/OvmfPkg/Include/IndustryStandard/E820.h >> > b/OvmfPkg/Include/IndustryStandard/E820.h >> > new file mode 100644 >> > index 0000000..e7e0c25 >> > --- /dev/null >> > +++ b/OvmfPkg/Include/IndustryStandard/E820.h >> > @@ -0,0 +1,46 @@ >> > +/** @file >> > + >> > +Copyright (c) 2006 - 2013, Intel Corporation. All rights reserved.<BR> >> >> <BR> ? >> > > Copy-n-paste error, phew. > > <BR> stands for new line in HTML. I have no idea how it came up in the > original header.
Actually Intel wants that appended to all Intel copyright lines in EDK II files. I'm not sure why, but I suspected something related to forcing a newline in doxygen. Hot seems to keep an eye on it, so maybe he knows. -Jordan >> > +Copyright (c) 2013, Citrix Systems UK Ltd. >> > + >> > +This program and the accompanying materials >> > +are licensed and made available under the terms and conditions >> > +of the BSD License which accompanies this distribution. The >> > +full text of the license may be found at >> > +http://opensource.org/licenses/bsd-license.php >> > + >> > +THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, >> > +WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR >> > IMPLIED. >> > + >> > +**/ >> > + >> > + >> > +#ifndef __E820_H__ >> > +#define __E820_H__ >> > + >> > +#pragma pack(1) >> > + >> > +typedef enum { >> > + EfiAcpiAddressRangeMemory = 1, >> > + EfiAcpiAddressRangeReserved = 2, >> > + EfiAcpiAddressRangeACPI = 3, >> > + EfiAcpiAddressRangeNVS = 4 >> >> Odd, Linux also has E820_UNUSABLE >> >> should that be in there as well? > > The original header from where I copied these definitions doens't have > that and nobody is using that at the moment. I think we can add that in > later. > > Wei. > > _______________________________________________ > Xen-devel mailing list > xen-de...@lists.xen.org > http://lists.xen.org/xen-devel ------------------------------------------------------------------------------ Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel