Re: [PATCH edk2-platforms 3/3] Platform/Secure96Dxe: add ACPI description of the GPIO LEDs

["Leif Lindholm" <leif.lindholm@linaro.org>]

On Fri, Apr 26, 2019 at 01:56:54PM +0200, Ard Biesheuvel wrote:
> On Fri, 26 Apr 2019 at 13:16, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> >
> > On Thu, Apr 25, 2019 at 02:32:54PM +0200, Ard Biesheuvel wrote:
> > > Wire up the new 96boards mezzanine SSDT loading support, and use it
> > > to describe the four GPIO LEDs on the Secure96 mezzanine board.
> > >
> > > Contributed-under: TianoCore Contribution Agreement 1.1
> > > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > > ---
> > >  Platform/96Boards/Secure96Dxe/Secure96.asl    | 103 ++++++++++++++++++++
> > >  Platform/96Boards/Secure96Dxe/Secure96Dxe.c   |  59 ++++++++++-
> > >  Platform/96Boards/Secure96Dxe/Secure96Dxe.inf |   1 +
> > >  3 files changed, 158 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/Platform/96Boards/Secure96Dxe/Secure96.asl b/Platform/96Boards/Secure96Dxe/Secure96.asl
> > > new file mode 100644
> > > index 000000000000..bb9dac462a33
> > > --- /dev/null
> > > +++ b/Platform/96Boards/Secure96Dxe/Secure96.asl
> > > @@ -0,0 +1,103 @@
> > > +/** @file
> > > + * Copyright (c) 2019, Linaro Limited. All rights reserved.
> > > + *
> > > + * 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.
> > > + */
> > > +
> > > +#include "Secure96.h"
> > > +
> > > +DefinitionBlock ("Secure96.aml", "SSDT", 2, "LINARO", "SECURE96", 1)
> > > +{
> > > +    External (\_SB.LS96.GPIO)
> > > +
> > > +    Scope (_SB)
> > > +    {
> > > +        Device (LD96)
> > > +        {
> > > +            Name (_HID, "PRP0001")  // _HID: Hardware ID
> > > +            Name (_UID, 0x00)       // _UID: Unique ID
> > > +            Name (_DSD, Package () {
> > > +                ToUUID("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"),
> >
> > Surely the above stanza is the most repeated bit of boiler plate in
> > the entire tree? I count 207 instances before this goes in.
> > Can we stick a #define in some common header for this and reuse?
> > ACPI_DSD_UUID?
> >
> > As I said, not required for this patch, but would be worth addressing.
> >
> 
> We should add the GUID to MdePkg, given that it occurs in the ACPI spec.

Yeah. What form would be practically consumable by .asl?

/
    Leif