public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
To: edk2-devel@lists.01.org, lersek@redhat.com
Cc: leif.lindholm@linaro.org, Ard Biesheuvel <ard.biesheuvel@linaro.org>
Subject: [PATCH v2 0/3] EmbeddedPkg: revert DTB loading to platform lib
Date: Fri, 31 Mar 2017 11:56:04 +0100	[thread overview]
Message-ID: <20170331105607.3477-1-ard.biesheuvel@linaro.org> (raw)

This updates the recently added DtPlatformDxe to indirect loading of the
actual DTB binary to a separate library. A base implementation is provided
that preserves the original behavior.

This way, we can allow platforms such as FVP or Juno (which use the same
firmware image but different DTB images) to override this library class
and insert its own logic to select the correct DTB, without resorting to
BEFORE depexes and dynamic PCDs, and the associated load order issues.

v2: - drop ArmPlatformPkg changes for now

Ard Biesheuvel (3):
  EmbeddedPkg: add DtPlatformDtbLoaderLib library class
  EmbeddedPkg: add base DtPlatformDtbLoaderLib implementation
  EmbeddedPkg/DtPlatformDxe: load platform DTB via new library

 EmbeddedPkg/Drivers/DtPlatformDxe/DtPlatformDxe.c                             | 26 ++++-----
 EmbeddedPkg/Drivers/DtPlatformDxe/DtPlatformDxe.inf                           |  2 +-
 EmbeddedPkg/EmbeddedPkg.dec                                                   |  1 +
 EmbeddedPkg/EmbeddedPkg.dsc                                                   |  6 ++
 EmbeddedPkg/Include/Library/DtPlatformDtbLoaderLib.h                          | 37 ++++++++++++
 EmbeddedPkg/Library/DtPlatformDtbLoaderBaseLib/DtPlatformDtbLoaderBaseLib.c   | 60 ++++++++++++++++++++
 EmbeddedPkg/Library/DtPlatformDtbLoaderBaseLib/DtPlatformDtbLoaderBaseLib.inf | 36 ++++++++++++
 7 files changed, 154 insertions(+), 14 deletions(-)
 create mode 100644 EmbeddedPkg/Include/Library/DtPlatformDtbLoaderLib.h
 create mode 100644 EmbeddedPkg/Library/DtPlatformDtbLoaderBaseLib/DtPlatformDtbLoaderBaseLib.c
 create mode 100644 EmbeddedPkg/Library/DtPlatformDtbLoaderBaseLib/DtPlatformDtbLoaderBaseLib.inf

-- 
2.9.3



             reply	other threads:[~2017-03-31 10:56 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-31 10:56 Ard Biesheuvel [this message]
2017-03-31 10:56 ` [PATCH v2 1/3] EmbeddedPkg: add DtPlatformDtbLoaderLib library class Ard Biesheuvel
2017-03-31 11:15   ` Laszlo Ersek
2017-03-31 10:56 ` [PATCH v2 2/3] EmbeddedPkg: add base DtPlatformDtbLoaderLib implementation Ard Biesheuvel
2017-03-31 11:29   ` Laszlo Ersek
2017-03-31 10:56 ` [PATCH v2 3/3] EmbeddedPkg/DtPlatformDxe: load platform DTB via new library Ard Biesheuvel
2017-03-31 11:39   ` Laszlo Ersek
2017-03-31 12:22     ` Ard Biesheuvel
2017-03-31 13:38       ` Laszlo Ersek
2017-03-31 12:26 ` [PATCH v2 0/3] EmbeddedPkg: revert DTB loading to platform lib Ard Biesheuvel

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=20170331105607.3477-1-ard.biesheuvel@linaro.org \
    --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