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.138155.1598028902187185643 for ; Fri, 21 Aug 2020 09:55:02 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: vijayenthiran.subramaniam@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 25990113E for ; Fri, 21 Aug 2020 09:55:01 -0700 (PDT) Received: from mail-io1-f51.google.com (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 109D43F802 for ; Fri, 21 Aug 2020 09:55:01 -0700 (PDT) Received: by mail-io1-f51.google.com with SMTP id m23so1780317iol.8 for ; Fri, 21 Aug 2020 09:55:01 -0700 (PDT) X-Gm-Message-State: AOAM530bq4FXSZsrjSqmHQc6MwIyU6jMWKJxnggcv8+wNQKjbnZtFsP9 vJq/RPgLAYzNDJPkIZs23awUZthZpQLQ3JzCB98= X-Google-Smtp-Source: ABdhPJzx9sTDWtUvSV2AAMhemvEAJIewrCc3LKw0KcxoQVUEyuqc9nxYPCgApG56QOmpNfq9jCkmxSLg9bcd+94ct7o= X-Received: by 2002:a02:a915:: with SMTP id n21mr3341732jam.78.1598028900625; Fri, 21 Aug 2020 09:55:00 -0700 (PDT) MIME-Version: 1.0 References: <1597848823-27171-1-git-send-email-vijayenthiran.subramaniam@arm.com> <20200820095956.GA1191@vanye> In-Reply-To: <20200820095956.GA1191@vanye> From: "Vijayenthiran Subramaniam" Date: Fri, 21 Aug 2020 16:54:49 +0000 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [PATCH] SgiPkg: Add chip count PCD to build information file for rdn1edgex2 To: devel@edk2.groups.io, leif@nuviainc.com Cc: Ard.Biesheuvel@arm.com, thomas.abraham@arm.com, Aditya.Angadi@arm.com, richard.storer@arm.com, sami.mujawar@arm.com Content-Type: text/plain; charset="UTF-8" Hi Leif, On Thu, Aug 20, 2020 at 10:00 AM Leif Lindholm wrote: > > On Wed, Aug 19, 2020 at 20:23:43 +0530, Vijayenthiran Subramaniam wrote: > > PcdChipCount constant is used to define the maximum number chips > > included in the multi-chip configuration. Add this constant to the > > RDN1EdgeX2 ACPI tables build information (INF) file to allow this > > constant to be used in the RDN1EdgeX2 ACPI tables. > > This patch fixes a build failure, the commit message should say so, > and refer to which commit it fixes. > Okay, I will do that. > Also, this fixes a build failure introduced 3.5 months ago, and no one > noticed until now. Are we sure this platform belongs upstream? > This was noticed after the patch was merged upstream. This probably slipped through during the patch rework and rebase. This was fixed outside of the upstream code but missed to get this merged upstream. Regards, Vijayenthiran > / > Leif > > > Signed-off-by: Vijayenthiran Subramaniam > > --- > > Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2AcpiTables.inf | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2AcpiTables.inf b/Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2AcpiTables.inf > > index 974d9db543..d44f02ab0c 100644 > > --- a/Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2AcpiTables.inf > > +++ b/Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2AcpiTables.inf > > @@ -45,6 +45,8 @@ > > gArmSgiTokenSpaceGuid.PcdDramBlock2Base > > gArmSgiTokenSpaceGuid.PcdDramBlock2Size > > > > + gArmSgiTokenSpaceGuid.PcdChipCount > > + > > gArmTokenSpaceGuid.PcdArmArchTimerSecIntrNum > > gArmTokenSpaceGuid.PcdArmArchTimerIntrNum > > gArmTokenSpaceGuid.PcdArmArchTimerHypIntrNum > > -- > > 2.17.1 > >