public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Leif Lindholm <leif.lindholm@linaro.org>
To: Sami Mujawar <sami.mujawar@arm.com>
Cc: edk2-devel@lists.01.org, ard.biesheuvel@linaro.org,
	michael.d.kinney@intel.com, alexei.fedorov@arm.com,
	Matteo.Carlini@arm.com, Stephanie.Hughes-Fitt@arm.com,
	nd@arm.com
Subject: Re: [PATCH edk2-platforms v4 0/4] Platform/ARM: Platform support for Dynamic Tables Framework
Date: Tue, 26 Mar 2019 19:17:05 +0000	[thread overview]
Message-ID: <20190326191705.uasrrylwjyw2ia6a@bivouac.eciton.net> (raw)
In-Reply-To: <20190326152319.15036-1-sami.mujawar@arm.com>

On Tue, Mar 26, 2019 at 03:23:15PM +0000, Sami Mujawar wrote:
> Dynamic Tables Framework aims to reduce the amount of effort
> required for porting firmware to new platforms by simplifying
> the generation of firmware tables based on hardware description
> provided by a platform specific component.
> 
> The Dynamic Tables Framework core queries the platform specific
> component to retrieve the required hardware information for
> generating standardised firmware tables at run-time.
> 
> The platform specific component responsible for collating the
> hardware information is called the Configuration Manager.
> 
> This patch series introduce the Configuration Manager that
> provides the hardware description to Dynamic Tables Framework.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Sami Mujawar <sami.mujawar@arm.com>
> 
> The changes can be seen at:
> https://github.com/samimujawar/edk2-platforms/tree/365_dynamic_tables_framework_v4

For the series:
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

Pushed as 89f6901df0..8cb8080438.

/
    Leif

> This v4 patch series incorporates:
>   * Updates based on review comments to sort #include files and .inf sections
>     in alphabetical order.
> 
> Patches updated in this series are:
>   - Platform/ARM: Configuration Manager for Juno
>   - Platform/ARM: Configuration Manager for FVP
> 
> The corresponding edk2 code changes can be seen at:
> https://github.com/samimujawar/edk2/tree/365_dynamic_tables_framework_v2
> 
> Sami Mujawar (4):
>   Platform/ARM: Configuration Manager for Juno
>   Platform/ARM: Dynamic Tables support for Juno
>   Platform/ARM: Configuration Manager for FVP
>   Platform/ARM: Dynamic Tables support for FVP
> 
>  Platform/ARM/JunoPkg/ArmJuno.dsc                                                                  |  12 +-
>  Platform/ARM/JunoPkg/ArmJuno.fdf                                                                  |  12 +
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManager.dsc.inc                            |  29 +
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c          | 750 ++++++++++++++++++++
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h          | 179 +++++
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf     |  87 +++
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/Platform.h                      |  99 +++
>  Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl                           | 276 +++++++
>  Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/PlatformASLTablesLib.inf           |  45 ++
>  Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtJunoUsb.asl                    | 123 ++++
>  Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtPci.asl                        | 201 ++++++
>  Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtUart.asl                       |  48 ++
>  Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c                                              |   9 +-
>  Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc                                              |  15 +
>  Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.fdf                                              |  16 +-
>  Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc                        |  31 +
>  Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c      | 682 ++++++++++++++++++
>  Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h      | 181 +++++
>  Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf |  80 +++
>  Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/Platform.h                  |  99 +++
>  Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl                       |  73 ++
>  Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/PlatformASLTablesLib.inf       |  35 +
>  22 files changed, 3077 insertions(+), 5 deletions(-)
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManager.dsc.inc
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/Platform.h
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/PlatformASLTablesLib.inf
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtJunoUsb.asl
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtPci.asl
>  create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtUart.asl
>  create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc
>  create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c
>  create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h
>  create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf
>  create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/Platform.h
>  create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl
>  create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/PlatformASLTablesLib.inf
> 
> -- 
> 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'
> 
> 


      parent reply	other threads:[~2019-03-26 19:17 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-26 15:23 [PATCH edk2-platforms v4 0/4] Platform/ARM: Platform support for Dynamic Tables Framework Sami Mujawar
2019-03-26 15:23 ` [PATCH edk2-platforms v4 1/4] Platform/ARM: Configuration Manager for Juno Sami Mujawar
2019-03-26 15:23 ` [PATCH edk2-platforms v4 2/4] Platform/ARM: Dynamic Tables support " Sami Mujawar
2019-03-26 15:23 ` [PATCH edk2-platforms v4 3/4] Platform/ARM: Configuration Manager for FVP Sami Mujawar
2019-03-26 15:23 ` [PATCH edk2-platforms v4 4/4] Platform/ARM: Dynamic Tables support " Sami Mujawar
2019-03-26 19:17 ` Leif Lindholm [this message]

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=20190326191705.uasrrylwjyw2ia6a@bivouac.eciton.net \
    --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