public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Mudusuru, Giri P" <giri.p.mudusuru@intel.com>
To: "Yao, Jiewen" <jiewen.yao@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	"Gao, Liming" <liming.gao@intel.com>
Subject: Re: [PATCH] MdePkg: Add ACPI Low Power Idle Table (LPIT) definitions
Date: Tue, 18 Oct 2016 13:55:49 +0000	[thread overview]
Message-ID: <4666AEFED60F8E4198B42BB01DCEABDF76F17149@ORSMSX113.amr.corp.intel.com> (raw)
In-Reply-To: <74D8A39837DF1E4DA445A8C0B3885C50386B3161@shsmsx102.ccr.corp.intel.com>

Agree Jiewen will update it before pushing the patch.

Thanks, 
-Giri

> -----Original Message-----
> From: Yao, Jiewen
> Sent: Tuesday, October 18, 2016 6:30 AM
> To: Mudusuru, Giri P <giri.p.mudusuru@intel.com>; edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>
> Subject: RE: [edk2] [PATCH] MdePkg: Add ACPI Low Power Idle Table (LPIT)
> definitions
> 
> HI
> I recommend we just use #include <IndustryStandard/Acpi.h>.
> Acpi.h includes all ACPI version. There is no need to include Acpi61.h.
> 
> With this change, reviewed-by: Jiewen.yao@intel.com
> 
> 
> > -----Original Message-----
> > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> > Giri P Mudusuru
> > Sent: Tuesday, October 18, 2016 7:25 PM
> > To: edk2-devel@lists.01.org
> > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Yao, Jiewen
> > <jiewen.yao@intel.com>; Gao, Liming <liming.gao@intel.com>
> > Subject: [edk2] [PATCH] MdePkg: Add ACPI Low Power Idle Table (LPIT)
> > definitions
> >
> > ACPI Low Power Idle Table (LPIT) Revision 001, dated July 2014
> > http://www.uefi.org/sites/default/files/resources/ACPI_Low_Power_Idle_T
> > able.pdf
> >
> > Cc: Michael Kinney <michael.d.kinney@intel.com>
> > Cc: Liming Gao <liming.gao@intel.com>
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Contributed-under: TianoCore Contribution Agreement 1.0
> > Signed-off-by: Giri P Mudusuru <giri.p.mudusuru@intel.com>
> > ---
> >  .../Include/IndustryStandard/LowPowerIdleTable.h   | 82
> > ++++++++++++++++++++++
> >  1 file changed, 82 insertions(+)
> >  create mode 100644
> > MdePkg/Include/IndustryStandard/LowPowerIdleTable.h
> >
> > diff --git a/MdePkg/Include/IndustryStandard/LowPowerIdleTable.h
> > b/MdePkg/Include/IndustryStandard/LowPowerIdleTable.h
> > new file mode 100644
> > index 0000000..b29db04
> > --- /dev/null
> > +++ b/MdePkg/Include/IndustryStandard/LowPowerIdleTable.h
> > @@ -0,0 +1,82 @@
> > +/** @file
> > +  ACPI Low Power Idle Table (LPIT) definitions
> > +
> > +  Copyright (c) 2016, 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
> > +  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.
> > +
> > +  @par Revision Reference:
> > +    - ACPI Low Power Idle Table (LPIT) Revision 001, dated July 2014
> > +
> > http://www.uefi.org/sites/default/files/resources/ACPI_Low_Power_Idle_T
> > able.pdf
> > +
> > +  @par Glossary:
> > +    - GAS - Generic Address Structure
> > +    - LPI - Low Power Idle
> > +**/
> > +#ifndef _LOW_POWER_IDLE_TABLE_H_
> > +#define _LOW_POWER_IDLE_TABLE_H_
> > +
> > +#include <IndustryStandard/Acpi61.h>
> > +
> > +#pragma pack(1)
> > +
> > +///
> > +/// LPI Structure Types
> > +///
> > +#define ACPI_LPI_STRUCTURE_TYPE_NATIVE_CSTATE     0x00
> > +
> > +///
> > +/// Low Power Idle (LPI) State Flags
> > +///
> > +typedef union {
> > +  struct {
> > +    UINT32 Disabled           : 1;  ///< If set, LPI state is not used
> > +    /**
> > +      If set, Residency counter is not available for this LPI state and
> > +      Residency Counter Frequency is invalid
> > +    **/
> > +    UINT32 CounterUnavailable : 1;
> > +    UINT32 Reserved           : 30; ///< Reserved for future use. Must
> > be zero
> > +  } Bits;
> > +  UINT32 Data32;
> > +} ACPI_LPI_STATE_FLAGS;
> > +
> > +///
> > +/// Low Power Idle (LPI) structure with Native C-state instruction entry
> > trigger descriptor
> > +///
> > +typedef struct {
> > +  UINT32                                  Type;   ///< LPI State
> > descriptor Type 0
> > +  UINT32                                  Length; ///< Length of
> > LPI state Descriptor Structure
> > +  ///
> > +  /// Unique LPI state identifier: zero based, monotonically increasing
> > identifier
> > +  ///
> > +  UINT16                                  UniqueId;
> > +  UINT8                                   Reserved[2];  ///<
> > Must be Zero
> > +  ACPI_LPI_STATE_FLAGS                    Flags;        ///< LPI
> > state flags
> > +  /**
> > +    The LPI entry trigger, matching an existing _CST.Register object,
> > represented as a
> > +    Generic Address Structure. All processors must request this state or
> > deeper to trigger.
> > +  **/
> > +  EFI_ACPI_6_1_GENERIC_ADDRESS_STRUCTURE  EntryTrigger;
> > +  UINT32                                  Residency;  ///<
> > Minimum residency or break-even in uSec
> > +  UINT32                                  Latency;    ///<
> > Worst case exit latency in uSec
> > +  /**
> > +    [optional] Residency counter, represented as a Generic Address
> > Structure.
> > +    If not present, Flags[1] bit should be set.
> > +  **/
> > +  EFI_ACPI_6_1_GENERIC_ADDRESS_STRUCTURE  ResidencyCounter;
> > +  /**
> > +    [optional] Residency counter frequency in cycles per second. Value 0
> > indicates that
> > +    counter runs at TSC frequency. Valid only if Residency Counter is
> > present.
> > +  **/
> > +  UINT64
> > ResidencyCounterFrequency;
> > +} ACPI_LPI_NATIVE_CSTATE_DESCRIPTOR;
> > +
> > +#pragma pack()
> > +
> > +#endif
> > --
> > 2.9.0.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel


  reply	other threads:[~2016-10-18 13:55 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-18 11:24 [PATCH] MdePkg: Add ACPI Low Power Idle Table (LPIT) definitions Giri P Mudusuru
2016-10-18 13:30 ` Yao, Jiewen
2016-10-18 13:55   ` Mudusuru, Giri P [this message]
2016-10-20  5:10     ` Gao, Liming

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4666AEFED60F8E4198B42BB01DCEABDF76F17149@ORSMSX113.amr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox