* [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
@ 2018-05-15 19:34 Marvin Häuser
[not found] ` <CB6E33457884FA40993F35157061515CA3CFA2BB@FMSMSX103.amr.corp.intel.com>
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Marvin Häuser @ 2018-05-15 19:34 UTC (permalink / raw)
To: edk2-devel@lists.01.org; +Cc: michael.d.kinney@intel.com, liming.gao@intel.com
This patch adds the HPET Event Timer Block ID definition that can be
found in the IA-PC HPET Specification, section 3.2.4.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
---
MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h | 18 +++++++++++++++++-
1 file changed, 17 insertions(+), 1 deletion(-)
diff --git a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
index 0d83cd5335de..926445233944 100644
--- a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
+++ b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
@@ -2,7 +2,7 @@
ACPI high precision event timer table definition, at www.intel.com
Specification name is IA-PC HPET (High Precision Event Timers) Specification.
- Copyright (c) 2007 - 2008, Intel Corporation. All rights reserved.<BR>
+ Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>
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
@@ -22,6 +22,22 @@
//
#pragma pack(1)
+///
+/// HPET Event Timer Block ID described in IA-PC HPET Specification, 3.2.4.
+///
+typedef union {
+ struct {
+ UINT32 Revision : 8;
+ UINT32 NumberOfTimers : 5;
+ UINT32 CounterSize : 1;
+ UINT32 Reserved : 1;
+ UINT32 LegacyRoute : 1;
+ UINT32 VendorId : 16;
+ } Bits;
+ UINT32 Uint32;
+} EFI_ACPI_HIGH_PRECISION_EVENT_TIMER_BLOCK_ID;
+
+
///
/// High Precision Event Timer Table header definition.
///
--
2.17.0.windows.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
[not found] ` <CB6E33457884FA40993F35157061515CA3CFA2BB@FMSMSX103.amr.corp.intel.com>
@ 2018-05-15 21:35 ` Marvin Häuser
2018-05-15 21:55 ` Carsey, Jaben
0 siblings, 1 reply; 8+ messages in thread
From: Marvin Häuser @ 2018-05-15 21:35 UTC (permalink / raw)
To: edk2-devel@lists.01.org; +Cc: Carsey, Jaben
Hey Jaben,
I'm used to keeping the copyright date updated and because both the specification where the definition originates from and the present header are published by Intel, this effectively is all Intel IP.
Was this an incorrect change nevertheless? I'm not an expert in OSS contribution law.
Thanks,
Marvin
> -----Original Message-----
> From: Carsey, Jaben <jaben.carsey@intel.com>
> Sent: Tuesday, May 15, 2018 11:26 PM
> To: Marvin Häuser <Marvin.Haeuser@outlook.com>
> Subject: RE: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
>
> Marvin,
>
> Non-technical question here, but why do you update intel copyright from a
> non-intel email address. This seems odd to me...
>
> Thanks
> -Jaben
>
> > -----Original Message-----
> > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> > Marvin Häuser
> > Sent: Tuesday, May 15, 2018 12:35 PM
> > To: edk2-devel@lists.01.org
> > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> > <liming.gao@intel.com>
> > Subject: [edk2] [PATCH] MdePkg/Hpet: Add Event Timer Block ID
> definition.
> >
> > This patch adds the HPET Event Timer Block ID definition that can be
> > found in the IA-PC HPET Specification, section 3.2.4.
> >
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
> > ---
> > MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h | 18
> > +++++++++++++++++-
> > 1 file changed, 17 insertions(+), 1 deletion(-)
> >
> > diff --git
> > a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > index 0d83cd5335de..926445233944 100644
> > --- a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > +++ b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > @@ -2,7 +2,7 @@
> > ACPI high precision event timer table definition, at www.intel.com
> > Specification name is IA-PC HPET (High Precision Event Timers)
> > Specification.
> >
> > - Copyright (c) 2007 - 2008, Intel Corporation. All rights
> > reserved.<BR>
> > + Copyright (c) 2007 - 2018, Intel Corporation. All rights
> > + reserved.<BR>
> > 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 @@ -22,6 +22,22 @@ // #pragma pack(1)
> >
> > +///
> > +/// HPET Event Timer Block ID described in IA-PC HPET Specification, 3.2.4.
> > +///
> > +typedef union {
> > + struct {
> > + UINT32 Revision : 8;
> > + UINT32 NumberOfTimers : 5;
> > + UINT32 CounterSize : 1;
> > + UINT32 Reserved : 1;
> > + UINT32 LegacyRoute : 1;
> > + UINT32 VendorId : 16;
> > + } Bits;
> > + UINT32 Uint32;
> > +} EFI_ACPI_HIGH_PRECISION_EVENT_TIMER_BLOCK_ID;
> > +
> > +
> > ///
> > /// High Precision Event Timer Table header definition.
> > ///
> > --
> > 2.17.0.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
2018-05-15 21:35 ` Marvin Häuser
@ 2018-05-15 21:55 ` Carsey, Jaben
0 siblings, 0 replies; 8+ messages in thread
From: Carsey, Jaben @ 2018-05-15 21:55 UTC (permalink / raw)
To: Marvin Häuser, edk2-devel@lists.01.org
I have no clue on OSS laws/rules, I was just confused. I think that you do no need to update any copyright (except your own).
The code change looks good and that is about all I can really say. :)
-Jaben
> -----Original Message-----
> From: Marvin Häuser [mailto:Marvin.Haeuser@outlook.com]
> Sent: Tuesday, May 15, 2018 2:35 PM
> To: edk2-devel@lists.01.org
> Cc: Carsey, Jaben <jaben.carsey@intel.com>
> Subject: RE: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
> Importance: High
>
> Hey Jaben,
>
> I'm used to keeping the copyright date updated and because both the
> specification where the definition originates from and the present header
> are published by Intel, this effectively is all Intel IP.
> Was this an incorrect change nevertheless? I'm not an expert in OSS
> contribution law.
>
> Thanks,
> Marvin
>
> > -----Original Message-----
> > From: Carsey, Jaben <jaben.carsey@intel.com>
> > Sent: Tuesday, May 15, 2018 11:26 PM
> > To: Marvin Häuser <Marvin.Haeuser@outlook.com>
> > Subject: RE: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
> >
> > Marvin,
> >
> > Non-technical question here, but why do you update intel copyright from a
> > non-intel email address. This seems odd to me...
> >
> > Thanks
> > -Jaben
> >
> > > -----Original Message-----
> > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> > > Marvin Häuser
> > > Sent: Tuesday, May 15, 2018 12:35 PM
> > > To: edk2-devel@lists.01.org
> > > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> > > <liming.gao@intel.com>
> > > Subject: [edk2] [PATCH] MdePkg/Hpet: Add Event Timer Block ID
> > definition.
> > >
> > > This patch adds the HPET Event Timer Block ID definition that can be
> > > found in the IA-PC HPET Specification, section 3.2.4.
> > >
> > > Contributed-under: TianoCore Contribution Agreement 1.1
> > > Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
> > > ---
> > > MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h | 18
> > > +++++++++++++++++-
> > > 1 file changed, 17 insertions(+), 1 deletion(-)
> > >
> > > diff --git
> > > a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > > b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > > index 0d83cd5335de..926445233944 100644
> > > --- a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > > +++
> b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > > @@ -2,7 +2,7 @@
> > > ACPI high precision event timer table definition, at www.intel.com
> > > Specification name is IA-PC HPET (High Precision Event Timers)
> > > Specification.
> > >
> > > - Copyright (c) 2007 - 2008, Intel Corporation. All rights
> > > reserved.<BR>
> > > + Copyright (c) 2007 - 2018, Intel Corporation. All rights
> > > + reserved.<BR>
> > > 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 @@ -22,6 +22,22 @@ // #pragma pack(1)
> > >
> > > +///
> > > +/// HPET Event Timer Block ID described in IA-PC HPET Specification,
> 3.2.4.
> > > +///
> > > +typedef union {
> > > + struct {
> > > + UINT32 Revision : 8;
> > > + UINT32 NumberOfTimers : 5;
> > > + UINT32 CounterSize : 1;
> > > + UINT32 Reserved : 1;
> > > + UINT32 LegacyRoute : 1;
> > > + UINT32 VendorId : 16;
> > > + } Bits;
> > > + UINT32 Uint32;
> > > +} EFI_ACPI_HIGH_PRECISION_EVENT_TIMER_BLOCK_ID;
> > > +
> > > +
> > > ///
> > > /// High Precision Event Timer Table header definition.
> > > ///
> > > --
> > > 2.17.0.windows.1
> > >
> > > _______________________________________________
> > > edk2-devel mailing list
> > > edk2-devel@lists.01.org
> > > https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
2018-05-15 19:34 [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition Marvin Häuser
[not found] ` <CB6E33457884FA40993F35157061515CA3CFA2BB@FMSMSX103.amr.corp.intel.com>
@ 2018-05-16 22:32 ` Carsey, Jaben
2018-05-17 12:43 ` Marvin Häuser
2018-05-17 1:31 ` Zeng, Star
2 siblings, 1 reply; 8+ messages in thread
From: Carsey, Jaben @ 2018-05-16 22:32 UTC (permalink / raw)
To: Marvin Häuser, edk2-devel@lists.01.org
Cc: Kinney, Michael D, Gao, Liming
Marvin,
Can you resubmit your patch without the modification to the intel copyright?
Can you also do this for any other outstanding (not yet pushed) patches also?
-Jaben
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Marvin Häuser
> Sent: Tuesday, May 15, 2018 12:35 PM
> To: edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>
> Subject: [edk2] [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
>
> This patch adds the HPET Event Timer Block ID definition that can be
> found in the IA-PC HPET Specification, section 3.2.4.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
> ---
> MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h | 18
> +++++++++++++++++-
> 1 file changed, 17 insertions(+), 1 deletion(-)
>
> diff --git
> a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> index 0d83cd5335de..926445233944 100644
> --- a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> +++ b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> @@ -2,7 +2,7 @@
> ACPI high precision event timer table definition, at www.intel.com
> Specification name is IA-PC HPET (High Precision Event Timers)
> Specification.
>
> - Copyright (c) 2007 - 2008, Intel Corporation. All rights reserved.<BR>
> + Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>
> 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
> @@ -22,6 +22,22 @@
> //
> #pragma pack(1)
>
> +///
> +/// HPET Event Timer Block ID described in IA-PC HPET Specification, 3.2.4.
> +///
> +typedef union {
> + struct {
> + UINT32 Revision : 8;
> + UINT32 NumberOfTimers : 5;
> + UINT32 CounterSize : 1;
> + UINT32 Reserved : 1;
> + UINT32 LegacyRoute : 1;
> + UINT32 VendorId : 16;
> + } Bits;
> + UINT32 Uint32;
> +} EFI_ACPI_HIGH_PRECISION_EVENT_TIMER_BLOCK_ID;
> +
> +
> ///
> /// High Precision Event Timer Table header definition.
> ///
> --
> 2.17.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
2018-05-15 19:34 [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition Marvin Häuser
[not found] ` <CB6E33457884FA40993F35157061515CA3CFA2BB@FMSMSX103.amr.corp.intel.com>
2018-05-16 22:32 ` Carsey, Jaben
@ 2018-05-17 1:31 ` Zeng, Star
2018-05-17 11:44 ` Marvin H?user
2 siblings, 1 reply; 8+ messages in thread
From: Zeng, Star @ 2018-05-17 1:31 UTC (permalink / raw)
To: Marvin.Haeuser@outlook.com, edk2-devel@lists.01.org
Cc: Kinney, Michael D, Gao, Liming, Yao, Jiewen, Zeng, Star
Was HighPrecisionEventTimerTable.h just created for ACPI related, but not for HPET register related?
We also see AlertStandardFormatTable.h, DmaRemappingReportingTable.h, etc. They are all ACPI related.
What is the criteria about including ACPI related, and including register/command/message related?
Should they be included in same header file, or separated header files?
We also see HEPT register related defined in PcAtChipsetPkg\Include\Register\Hpet.h.
Thanks,
Star
-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Marvin H?user
Sent: Wednesday, May 16, 2018 3:35 AM
To: edk2-devel@lists.01.org
Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>
Subject: [edk2] [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
This patch adds the HPET Event Timer Block ID definition that can be found in the IA-PC HPET Specification, section 3.2.4.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
---
MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h | 18 +++++++++++++++++-
1 file changed, 17 insertions(+), 1 deletion(-)
diff --git a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
index 0d83cd5335de..926445233944 100644
--- a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
+++ b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
@@ -2,7 +2,7 @@
ACPI high precision event timer table definition, at www.intel.com
Specification name is IA-PC HPET (High Precision Event Timers) Specification.
- Copyright (c) 2007 - 2008, Intel Corporation. All rights reserved.<BR>
+ Copyright (c) 2007 - 2018, Intel Corporation. All rights
+ reserved.<BR>
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
@@ -22,6 +22,22 @@
//
#pragma pack(1)
+///
+/// HPET Event Timer Block ID described in IA-PC HPET Specification, 3.2.4.
+///
+typedef union {
+ struct {
+ UINT32 Revision : 8;
+ UINT32 NumberOfTimers : 5;
+ UINT32 CounterSize : 1;
+ UINT32 Reserved : 1;
+ UINT32 LegacyRoute : 1;
+ UINT32 VendorId : 16;
+ } Bits;
+ UINT32 Uint32;
+} EFI_ACPI_HIGH_PRECISION_EVENT_TIMER_BLOCK_ID;
+
+
///
/// High Precision Event Timer Table header definition.
///
--
2.17.0.windows.1
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
2018-05-17 1:31 ` Zeng, Star
@ 2018-05-17 11:44 ` Marvin H?user
2018-05-18 2:30 ` Zeng, Star
0 siblings, 1 reply; 8+ messages in thread
From: Marvin H?user @ 2018-05-17 11:44 UTC (permalink / raw)
To: edk2-devel@lists.01.org
Cc: Zeng, Star, michael.d.kinney@intel.com, liming.gao@intel.com,
jiewen.yao@intel.com
Hey Star,
Actually the definition added is part of the definition of the ACPI Table header.
I think there might be confusion because the first 32 bits match the first 32 bits of the Capabilities register.
However, they are defined separately and the ACPI field lacks the upper 32 bits.
Thanks,
Marvin.
> -----Original Message-----
> From: Zeng, Star <star.zeng@intel.com>
> Sent: Thursday, May 17, 2018 3:31 AM
> To: Marvin.Haeuser@outlook.com; edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>; Zeng, Star
> <star.zeng@intel.com>
> Subject: RE: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
>
> Was HighPrecisionEventTimerTable.h just created for ACPI related, but not
> for HPET register related?
>
> We also see AlertStandardFormatTable.h, DmaRemappingReportingTable.h,
> etc. They are all ACPI related.
> What is the criteria about including ACPI related, and including
> register/command/message related?
> Should they be included in same header file, or separated header files?
>
> We also see HEPT register related defined in
> PcAtChipsetPkg\Include\Register\Hpet.h.
>
>
> Thanks,
> Star
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Marvin H?user
> Sent: Wednesday, May 16, 2018 3:35 AM
> To: edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>
> Subject: [edk2] [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
>
> This patch adds the HPET Event Timer Block ID definition that can be found in
> the IA-PC HPET Specification, section 3.2.4.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
> ---
> MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h | 18
> +++++++++++++++++-
> 1 file changed, 17 insertions(+), 1 deletion(-)
>
> diff --git
> a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> index 0d83cd5335de..926445233944 100644
> --- a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> +++ b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> @@ -2,7 +2,7 @@
> ACPI high precision event timer table definition, at www.intel.com
> Specification name is IA-PC HPET (High Precision Event Timers)
> Specification.
>
> - Copyright (c) 2007 - 2008, Intel Corporation. All rights reserved.<BR>
> + Copyright (c) 2007 - 2018, Intel Corporation. All rights
> + reserved.<BR>
> 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
> @@ -22,6 +22,22 @@
> //
> #pragma pack(1)
>
> +///
> +/// HPET Event Timer Block ID described in IA-PC HPET Specification, 3.2.4.
> +///
> +typedef union {
> + struct {
> + UINT32 Revision : 8;
> + UINT32 NumberOfTimers : 5;
> + UINT32 CounterSize : 1;
> + UINT32 Reserved : 1;
> + UINT32 LegacyRoute : 1;
> + UINT32 VendorId : 16;
> + } Bits;
> + UINT32 Uint32;
> +} EFI_ACPI_HIGH_PRECISION_EVENT_TIMER_BLOCK_ID;
> +
> +
> ///
> /// High Precision Event Timer Table header definition.
> ///
> --
> 2.17.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
2018-05-16 22:32 ` Carsey, Jaben
@ 2018-05-17 12:43 ` Marvin Häuser
0 siblings, 0 replies; 8+ messages in thread
From: Marvin Häuser @ 2018-05-17 12:43 UTC (permalink / raw)
To: edk2-devel@lists.01.org; +Cc: Carsey, Jaben
Hey Jaben,
Done.
Regards,
Marvin.
> -----Original Message-----
> From: Carsey, Jaben <jaben.carsey@intel.com>
> Sent: Thursday, May 17, 2018 12:32 AM
> To: Marvin Häuser <Marvin.Haeuser@outlook.com>; edk2-
> devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>
> Subject: RE: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
>
> Marvin,
>
> Can you resubmit your patch without the modification to the intel copyright?
>
> Can you also do this for any other outstanding (not yet pushed) patches also?
>
> -Jaben
>
> > -----Original Message-----
> > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> > Marvin Häuser
> > Sent: Tuesday, May 15, 2018 12:35 PM
> > To: edk2-devel@lists.01.org
> > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> > <liming.gao@intel.com>
> > Subject: [edk2] [PATCH] MdePkg/Hpet: Add Event Timer Block ID
> definition.
> >
> > This patch adds the HPET Event Timer Block ID definition that can be
> > found in the IA-PC HPET Specification, section 3.2.4.
> >
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
> > ---
> > MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h | 18
> > +++++++++++++++++-
> > 1 file changed, 17 insertions(+), 1 deletion(-)
> >
> > diff --git
> > a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > index 0d83cd5335de..926445233944 100644
> > --- a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > +++ b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> > @@ -2,7 +2,7 @@
> > ACPI high precision event timer table definition, at www.intel.com
> > Specification name is IA-PC HPET (High Precision Event Timers)
> > Specification.
> >
> > - Copyright (c) 2007 - 2008, Intel Corporation. All rights
> > reserved.<BR>
> > + Copyright (c) 2007 - 2018, Intel Corporation. All rights
> > + reserved.<BR>
> > 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 @@ -22,6 +22,22 @@ // #pragma pack(1)
> >
> > +///
> > +/// HPET Event Timer Block ID described in IA-PC HPET Specification, 3.2.4.
> > +///
> > +typedef union {
> > + struct {
> > + UINT32 Revision : 8;
> > + UINT32 NumberOfTimers : 5;
> > + UINT32 CounterSize : 1;
> > + UINT32 Reserved : 1;
> > + UINT32 LegacyRoute : 1;
> > + UINT32 VendorId : 16;
> > + } Bits;
> > + UINT32 Uint32;
> > +} EFI_ACPI_HIGH_PRECISION_EVENT_TIMER_BLOCK_ID;
> > +
> > +
> > ///
> > /// High Precision Event Timer Table header definition.
> > ///
> > --
> > 2.17.0.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
2018-05-17 11:44 ` Marvin H?user
@ 2018-05-18 2:30 ` Zeng, Star
0 siblings, 0 replies; 8+ messages in thread
From: Zeng, Star @ 2018-05-18 2:30 UTC (permalink / raw)
To: Marvin H?user, edk2-devel@lists.01.org
Cc: Kinney, Michael D, Gao, Liming, Yao, Jiewen, Zeng, Star
Got it. Good information.
Thanks,
Star
-----Original Message-----
From: Marvin H?user [mailto:Marvin.Haeuser@outlook.com]
Sent: Thursday, May 17, 2018 7:45 PM
To: edk2-devel@lists.01.org
Cc: Zeng, Star <star.zeng@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>
Subject: RE: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
Hey Star,
Actually the definition added is part of the definition of the ACPI Table header.
I think there might be confusion because the first 32 bits match the first 32 bits of the Capabilities register.
However, they are defined separately and the ACPI field lacks the upper 32 bits.
Thanks,
Marvin.
> -----Original Message-----
> From: Zeng, Star <star.zeng@intel.com>
> Sent: Thursday, May 17, 2018 3:31 AM
> To: Marvin.Haeuser@outlook.com; edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>; Zeng, Star
> <star.zeng@intel.com>
> Subject: RE: [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
>
> Was HighPrecisionEventTimerTable.h just created for ACPI related, but
> not for HPET register related?
>
> We also see AlertStandardFormatTable.h, DmaRemappingReportingTable.h,
> etc. They are all ACPI related.
> What is the criteria about including ACPI related, and including
> register/command/message related?
> Should they be included in same header file, or separated header files?
>
> We also see HEPT register related defined in
> PcAtChipsetPkg\Include\Register\Hpet.h.
>
>
> Thanks,
> Star
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Marvin H?user
> Sent: Wednesday, May 16, 2018 3:35 AM
> To: edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>
> Subject: [edk2] [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition.
>
> This patch adds the HPET Event Timer Block ID definition that can be
> found in the IA-PC HPET Specification, section 3.2.4.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com>
> ---
> MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h | 18
> +++++++++++++++++-
> 1 file changed, 17 insertions(+), 1 deletion(-)
>
> diff --git
> a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> index 0d83cd5335de..926445233944 100644
> --- a/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> +++ b/MdePkg/Include/IndustryStandard/HighPrecisionEventTimerTable.h
> @@ -2,7 +2,7 @@
> ACPI high precision event timer table definition, at www.intel.com
> Specification name is IA-PC HPET (High Precision Event Timers)
> Specification.
>
> - Copyright (c) 2007 - 2008, Intel Corporation. All rights
> reserved.<BR>
> + Copyright (c) 2007 - 2018, Intel Corporation. All rights
> + reserved.<BR>
> 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 @@ -22,6 +22,22 @@ // #pragma pack(1)
>
> +///
> +/// HPET Event Timer Block ID described in IA-PC HPET Specification, 3.2.4.
> +///
> +typedef union {
> + struct {
> + UINT32 Revision : 8;
> + UINT32 NumberOfTimers : 5;
> + UINT32 CounterSize : 1;
> + UINT32 Reserved : 1;
> + UINT32 LegacyRoute : 1;
> + UINT32 VendorId : 16;
> + } Bits;
> + UINT32 Uint32;
> +} EFI_ACPI_HIGH_PRECISION_EVENT_TIMER_BLOCK_ID;
> +
> +
> ///
> /// High Precision Event Timer Table header definition.
> ///
> --
> 2.17.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-05-18 2:30 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-05-15 19:34 [PATCH] MdePkg/Hpet: Add Event Timer Block ID definition Marvin Häuser
[not found] ` <CB6E33457884FA40993F35157061515CA3CFA2BB@FMSMSX103.amr.corp.intel.com>
2018-05-15 21:35 ` Marvin Häuser
2018-05-15 21:55 ` Carsey, Jaben
2018-05-16 22:32 ` Carsey, Jaben
2018-05-17 12:43 ` Marvin Häuser
2018-05-17 1:31 ` Zeng, Star
2018-05-17 11:44 ` Marvin H?user
2018-05-18 2:30 ` Zeng, Star
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox