From: Leif Lindholm <leif.lindholm@linaro.org>
To: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Marcin Wojtas <mw@semihalf.com>,
"edk2-devel (edk2-devel@lists.01.org)" <edk2-devel@lists.01.org>,
Nadav Haklai <nadavh@marvell.com>,
"jsd@semihalf.com" <jsd@semihalf.com>,
Grzegorz Jaszczyk <jaz@semihalf.com>
Subject: Re: [platforms: PATCH 7/9] Marvell/Armada70x0Db: Enable ACPI support
Date: Mon, 6 Aug 2018 18:44:04 +0100 [thread overview]
Message-ID: <CAF7UmSzdvMA1Xbk8YWWAgJY1qTzZn2OYKjr3wHXMZaQGpABWHQ@mail.gmail.com> (raw)
In-Reply-To: <CAKv+Gu8dDS3M1uD3iUzAh6nF+SJ9cSm+veSc4usuDWR-L2B4aw@mail.gmail.com>
On Mon, 6 Aug 2018, 17:09 Ard Biesheuvel, <ard.biesheuvel@linaro.org> wrote:
> On 6 August 2018 at 18:08, Marcin Wojtas <mw@semihalf.com> wrote:
> > HI Ard,
> >
> > pon., 6 sie 2018 o 13:54 Ard Biesheuvel <ard.biesheuvel@linaro.org>
> napisał(a):
> >>
> >> On 6 August 2018 at 01:28, Marcin Wojtas <mw@semihalf.com> wrote:
> >> > This patch introduces DSDT table and adds necessary
> >> > wiring in order to enable ACPI support on Armada 7040 DB.
> >> >
> >> > Contributed-under: TianoCore Contribution Agreement 1.1
> >> > Signed-off-by: Marcin Wojtas <mw@semihalf.com>
> >> > ---
> >> > Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc |
> 14 ++
> >> > Platform/Marvell/Armada70x0Db/Armada70x0Db.dsc |
> 3 +
> >> > Silicon/Marvell/Armada7k8k/Armada7k8k.fdf |
> 12 +
> >> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/AcpiTables.inf |
> 61 ++++++
> >> > Platform/Marvell/Armada70x0Db/Armada70x0Db.fdf.inc |
> 5 +
> >> > Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/Dsdt.asl |
> 229 ++++++++++++++++++++
> >> > Silicon/Marvell/Documentation/PortingGuide.txt |
> 22 ++
> >> > 7 files changed, 346 insertions(+)
> >> > create mode 100644
> Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/AcpiTables.inf
> >> > create mode 100644
> Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/Dsdt.asl
> >> >
> >> > diff --git a/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> b/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> >> > index f1ccda0..d4c67a2 100644
> >> > --- a/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> >> > +++ b/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> >> > @@ -593,6 +593,20 @@
> >> > ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf
> >> > !endif #$(INCLUDE_TFTP_COMMAND)
> >> >
> >> > +[Components.AARCH64]
> >> > + #
> >> > + # Generic ACPI modules
> >> > + #
> >> > + MdeModulePkg/Universal/Acpi/AcpiPlatformDxe/AcpiPlatformDxe.inf
> >> > + MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableDxe.inf {
> >> > + <LibraryClasses>
> >> > +
> PlatformHasAcpiLib|EmbeddedPkg/Library/PlatformHasAcpiLib/PlatformHasAcpiLib.inf
> >> > +
> >> > + <PcdsFixedAtBuild>
> >> > + # support ACPI v5.0 or later
> >> > + gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20
> >> > + }
> >> > +
> >> >
> [BuildOptions.common.EDKII.DXE_CORE,BuildOptions.common.EDKII.DXE_DRIVER,BuildOptions.common.EDKII.UEFI_DRIVER,BuildOptions.common.EDKII.UEFI_APPLICATION]
> >> > GCC:*_*_*_DLINK_FLAGS = -z common-page-size=0x1000
> >> >
> >> > diff --git a/Platform/Marvell/Armada70x0Db/Armada70x0Db.dsc
> b/Platform/Marvell/Armada70x0Db/Armada70x0Db.dsc
> >> > index d3dffb0..f6faff1 100644
> >> > --- a/Platform/Marvell/Armada70x0Db/Armada70x0Db.dsc
> >> > +++ b/Platform/Marvell/Armada70x0Db/Armada70x0Db.dsc
> >> > @@ -51,6 +51,9 @@
> >> > [Components.common]
> >> > Silicon/Marvell/Armada7k8k/DeviceTree/Armada70x0Db.inf
> >> >
> >> > +[Components.AARCH64]
> >> > + Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/AcpiTables.inf
> >> > +
> >> >
> ################################################################################
> >> > #
> >> > # Pcd Section - list of all EDK II PCD Entries defined by this
> Platform
> >> > diff --git a/Silicon/Marvell/Armada7k8k/Armada7k8k.fdf
> b/Silicon/Marvell/Armada7k8k/Armada7k8k.fdf
> >> > index 909ad3e..c064a43 100644
> >> > --- a/Silicon/Marvell/Armada7k8k/Armada7k8k.fdf
> >> > +++ b/Silicon/Marvell/Armada7k8k/Armada7k8k.fdf
> >> > @@ -215,6 +215,12 @@ FvNameGuid =
> 5eda4200-2c5f-43cb-9da3-0baf74b1b30c
> >> > # DTB
> >> > INF EmbeddedPkg/Drivers/DtPlatformDxe/DtPlatformDxe.inf
> >> >
> >> > +!if $(ARCH) == AARCH64
> >> > + # ACPI support
> >> > + INF MdeModulePkg/Universal/Acpi/AcpiPlatformDxe/AcpiPlatformDxe.inf
> >> > + INF MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableDxe.inf
> >> > +!endif
> >> > +
> >> > !include $(BOARD_DXE_FV_COMPONENTS)
> >> >
> >> > # PEI phase firmware volume
> >> > @@ -408,3 +414,9 @@ READ_LOCK_STATUS = TRUE
> >> > FILE FREEFORM = $(NAMED_GUID) {
> >> > RAW BIN |.dtb
> >> > }
> >> > +
> >> > +[Rule.Common.USER_DEFINED.ACPITABLE]
> >> > + FILE FREEFORM = $(NAMED_GUID) {
> >> > + RAW ASL |.aml
> >> > + RAW ACPI |.acpi
> >> > + }
> >> > diff --git
> a/Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/AcpiTables.inf
> b/Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/AcpiTables.inf
> >> > new file mode 100644
> >> > index 0000000..8732e10
> >> > --- /dev/null
> >> > +++
> b/Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/AcpiTables.inf
> >> > @@ -0,0 +1,61 @@
> >> > +## @file
> >> > +# Component description file for PlatformAcpiTables module.
> >> > +#
> >> > +# ACPI table data and ASL sources required to boot the platform.
> >> > +#
> >> > +# Copyright (c) 2018, Linaro, Ltd. All rights reserved.<BR>
> >> > +# Copyright (C) 2018, Marvell International Ltd. and its
> affiliates.<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.
> >> > +#
> >> > +##
> >> > +
> >> > +[Defines]
> >> > + INF_VERSION = 0x0001001A
> >> > + BASE_NAME = PlatformAcpiTables
> >> > + FILE_GUID =
> 7E374E25-8E01-4FEE-87F2-390C23C606CD
> >> > + MODULE_TYPE = USER_DEFINED
> >> > + VERSION_STRING = 1.0
> >> > +
> >> > +[Sources]
> >> > + Dsdt.asl
> >> > + ../Fadt.aslc
> >> > + ../Gtdt.aslc
> >> > + ../Madt.aslc
> >> > + ../Pptt.aslc
> >> > + ../Spcr.aslc
> >> > +
> >>
> >> Could we reshuffle these files so we nog longer rely relative includes?
> >> Possibly using a Common subdirectory?
> >
> > Would below directory tree be acceptable?
> >
> > Silicon/Marvell/Armada7k8k/AcpiTables/
> > ├── AcpiHeader.h
> > ├── Armada70x0Db
> > │ └── Dsdt.asl
> > ├── Armada70x0Db.inf
> > ├── Armada80x0Db
> > │ └── Dsdt.asl
> > ├── Armada80x0Db.inf
> > ├── Armada80x0McBin
> > │ └── Dsdt.asl
> > ├── Armada80x0McBin.inf
> > ├── Fadt.aslc
> > ├── Gtdt.aslc
> > ├── IcuInterrupts.h
> > ├── Madt.aslc
> > ├── Pptt.aslc
> > └── Spcr.aslc
> >
> > This way we will have following in the .inf
> >
> > Armada80x0McBin/Dsdt.asl
> > Fadt.aslc
> > Gtdt.aslc
> > Madt.aslc
> > Pptt.aslc
> > Spcr.aslc
> >
>
> Works for me. Leif?
>
Sure.
/
Leif
>
next prev parent reply other threads:[~2018-08-06 17:44 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-05 23:28 [platforms: PATCH 0/9] Armada7k8k DT/ACPI support Marcin Wojtas
2018-08-05 23:28 ` [platforms: PATCH 1/9] Marvell/Armada7k8k: Import device tree Marcin Wojtas
2018-08-05 23:28 ` [platforms: PATCH 2/9] Marvell/Armada7k8k: Enable including additional DXE FV components Marcin Wojtas
2018-08-05 23:28 ` [platforms: PATCH 3/9] Marvell/Armada70x0Db: Enable device tree support Marcin Wojtas
2018-08-05 23:28 ` [platforms: PATCH 4/9] Marvell/Armada80x0Db: " Marcin Wojtas
2018-08-05 23:28 ` [platforms: PATCH 5/9] Marvell/Armada80x0McBin: " Marcin Wojtas
2018-08-05 23:28 ` [platforms: PATCH 6/9] Marvell/Armada7k8k: Add common ACPI tables Marcin Wojtas
2018-08-05 23:28 ` [platforms: PATCH 7/9] Marvell/Armada70x0Db: Enable ACPI support Marcin Wojtas
2018-08-06 11:54 ` Ard Biesheuvel
2018-08-06 16:08 ` Marcin Wojtas
2018-08-06 16:09 ` Ard Biesheuvel
2018-08-06 17:44 ` Leif Lindholm [this message]
2018-08-06 17:55 ` Marcin Wojtas
2018-08-07 7:39 ` Ard Biesheuvel
2018-08-05 23:28 ` [platforms: PATCH 8/9] Marvell/Armada80x0Db: " Marcin Wojtas
2018-08-05 23:28 ` [platforms: PATCH 9/9] Marvell/Armada80x0McBin: " Marcin Wojtas
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=CAF7UmSzdvMA1Xbk8YWWAgJY1qTzZn2OYKjr3wHXMZaQGpABWHQ@mail.gmail.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