From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web11.74149.1584379809425602430 for ; Mon, 16 Mar 2020 10:30:09 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: thomas.abraham@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 1BEB41045 for ; Mon, 16 Mar 2020 10:30:09 -0700 (PDT) Received: from mail-il1-f176.google.com (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 054C73F9E6 for ; Mon, 16 Mar 2020 10:30:09 -0700 (PDT) Received: by mail-il1-f176.google.com with SMTP id a6so17331376ilc.4 for ; Mon, 16 Mar 2020 10:30:08 -0700 (PDT) X-Gm-Message-State: ANhLgQ0HtlVTnQrPiLdaHIxNOeHEr6EybTR4x/wUb6CTTXvQTxstjcr7 1Qt/E3qakEmB121B9IpuAwrVFkAQeBpFiS5ErYU= X-Google-Smtp-Source: ADFU+vsGgfR+C3KISCSnkZBtdvhs1ZSIu/+wTtNi2VcFdMLanJxHL0bd7Mr2k2NLiLMkjMQqbp+GnNySG2/jbxG/2Vg= X-Received: by 2002:a92:d4d0:: with SMTP id o16mr942216ilm.40.1584379808421; Mon, 16 Mar 2020 10:30:08 -0700 (PDT) MIME-Version: 1.0 References: <20200312143520.16918-1-aditya.angadi@arm.com> <20200312143520.16918-7-aditya.angadi@arm.com> In-Reply-To: <20200312143520.16918-7-aditya.angadi@arm.com> From: "Thomas Abraham" Date: Mon, 16 Mar 2020 22:59:57 +0530 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [edk2-platforms][PATCH 6/8] Platform/ARM/Sgi: Add initial support for RD-Daniel Config-M platform To: devel@edk2.groups.io, aditya.angadi@arm.com Cc: leif@nuviainc.com, michael.d.kinney@intel.com, vijayenthiran.subramaniam@arm.com Content-Type: text/plain; charset="UTF-8" On Thu, Mar 12, 2020 at 8:06 PM Aditya Angadi wrote: > > Add information in the SGI platform descriptor HOB to pick the correct > ACPI table to install for RD-Daniel Config-M > > Signed-off-by: Aditya Angadi Reviewed-by: Thomas Abraham > --- > .../SgiPkg/Drivers/PlatformDxe/PlatformDxe.c | 5 +++ > .../Drivers/PlatformDxe/PlatformDxe.inf | 1 + > Platform/ARM/SgiPkg/Include/SgiPlatform.h | 4 ++ > Platform/ARM/SgiPkg/RdDaniel.dsc | 38 +++++++++++++++++++ > 4 files changed, 48 insertions(+) > create mode 100644 Platform/ARM/SgiPkg/RdDaniel.dsc > > diff --git a/Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.c b/Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.c > index 387397d745..7e0de765f7 100644 > --- a/Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.c > +++ b/Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.c > @@ -46,6 +46,11 @@ STATIC SGI_PLATFORM_ACPI_TABLE_GUID_LOOKUP AcpiTableGuidLookup[] = { > RD_E1_EDGE_CONF_ID, > MULTI_CHIP_MODE_DISABLED, > &gRdE1EdgeAcpiTablesFileGuid), > + ACPI_GUID_LOOKUP ( > + RD_DANIEL_PART_NUM, > + RD_DANIEL_CFGM_CONF_ID, > + MULTI_CHIP_MODE_DISABLED, > + &gRdDanielCfgMAcpiTablesFileGuid), > }; > > VOID > diff --git a/Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.inf b/Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.inf > index 741dcc75ed..82569820b7 100644 > --- a/Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.inf > +++ b/Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.inf > @@ -34,6 +34,7 @@ > gRdN1EdgeAcpiTablesFileGuid > gRdN1EdgeX2AcpiTablesFileGuid > gRdE1EdgeAcpiTablesFileGuid > + gRdDanielCfgMAcpiTablesFileGuid > > [FeaturePcd] > gArmSgiTokenSpaceGuid.PcdVirtioBlkSupported > diff --git a/Platform/ARM/SgiPkg/Include/SgiPlatform.h b/Platform/ARM/SgiPkg/Include/SgiPlatform.h > index d87fb2b540..b6a427b8b6 100644 > --- a/Platform/ARM/SgiPkg/Include/SgiPlatform.h > +++ b/Platform/ARM/SgiPkg/Include/SgiPlatform.h > @@ -70,6 +70,10 @@ > #define RD_N1_EDGE_CONF_ID 0x1 > #define RD_E1_EDGE_CONF_ID 0x2 > > +//RDDANIEL Platform Identification values > +#define RD_DANIEL_PART_NUM 0x78A > +#define RD_DANIEL_CFGM_CONF_ID 0x1 > + > #define SGI_CONFIG_MASK 0x0F > #define SGI_CONFIG_SHIFT 0x1C > #define SGI_PART_NUM_MASK 0xFFF > diff --git a/Platform/ARM/SgiPkg/RdDaniel.dsc b/Platform/ARM/SgiPkg/RdDaniel.dsc > new file mode 100644 > index 0000000000..09607004ed > --- /dev/null > +++ b/Platform/ARM/SgiPkg/RdDaniel.dsc > @@ -0,0 +1,38 @@ > +# > +# Copyright (c) 2020, ARM Limited. All rights reserved. > +# > +# SPDX-License-Identifier: BSD-2-Clause-Patent > +# > + > +################################################################################ > +# > +# Defines Section - statements that will be processed to create a Makefile. > +# > +################################################################################ > +[Defines] > + PLATFORM_NAME = ArmSgi > + PLATFORM_GUID = d301ac4e-0828-4cef-b754-34ca9b6781b5 > + PLATFORM_VERSION = 0.1 > + DSC_SPECIFICATION = 0x0001001B > + OUTPUT_DIRECTORY = Build/$(PLATFORM_NAME) > + SUPPORTED_ARCHITECTURES = AARCH64|ARM > + BUILD_TARGETS = NOOPT|DEBUG|RELEASE > + SKUID_IDENTIFIER = DEFAULT > + FLASH_DEFINITION = Platform/ARM/SgiPkg/SgiPlatform.fdf > + BUILD_NUMBER = 1 > + > +# include common definitions from SgiPlatform.dsc > +!include Platform/ARM/SgiPkg/SgiPlatform.dsc > + > +################################################################################ > +# > +# Pcd Section - list of all EDK II PCD Entries defined by this Platform > +# > +################################################################################ > + > +[PcdsFixedAtBuild.common] > + # GIC Base Addresses > + gArmTokenSpaceGuid.PcdGicDistributorBase|0x30000000 > + gArmTokenSpaceGuid.PcdGicRedistributorsBase|0x30140000 > + gArmSgiTokenSpaceGuid.PcdGicSize|0x200000 > + > -- > 2.17.1 > > > >