[edk2-devel] [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components

Jeremy Linton posted 3 patches 4 years, 10 months ago
[edk2-devel] [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components
Posted by Jeremy Linton 4 years, 10 months ago
Add some additional IORT nodes for the USB & EMMC devices, realistically
we probably only need to have a single node with the lowest AddressSizeLimit
but this is conceptually "cleaner" should anyone actually try and use these
values rather than the _DMA provided ones.

Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
---
 Platform/RaspberryPi/AcpiTables/Iort.aslc | 44 ++++++++++++++++++++++++++++++-
 1 file changed, 43 insertions(+), 1 deletion(-)

diff --git a/Platform/RaspberryPi/AcpiTables/Iort.aslc b/Platform/RaspberryPi/AcpiTables/Iort.aslc
index 00720194bb..810307ae37 100644
--- a/Platform/RaspberryPi/AcpiTables/Iort.aslc
+++ b/Platform/RaspberryPi/AcpiTables/Iort.aslc
@@ -20,6 +20,8 @@ typedef struct {
 typedef struct {
   EFI_ACPI_6_0_IO_REMAPPING_TABLE      Iort;
   RPI4_NC_NODE                         NamedCompNode;
+  RPI4_NC_NODE                         NamedCompNode2;
+  RPI4_NC_NODE                         NamedCompNode3;
 } RPI4_IO_REMAPPING_STRUCTURE;
 
 STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
@@ -27,7 +29,7 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
     ACPI_HEADER (EFI_ACPI_6_0_IO_REMAPPING_TABLE_SIGNATURE,
                  RPI4_IO_REMAPPING_STRUCTURE,
                  EFI_ACPI_IO_REMAPPING_TABLE_REVISION),
-    1,                                              // NumNodes
+    3,                                              // NumNodes
     sizeof (EFI_ACPI_6_0_IO_REMAPPING_TABLE),       // NodeOffset
     0                                               // Reserved
   }, {
@@ -50,6 +52,46 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
     }, {
       "\\_SB_.SCB0.XHC0"                            // ObjectName
     }
+  }, {
+    // gpu/dwc usb named component node
+    {
+      {
+        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type
+        sizeof (RPI4_NC_NODE),                      // Length
+        0x0,                                        // Revision
+        0x0,                                        // Reserved
+        0x0,                                        // NumIdMappings
+        0x0,                                        // IdReference
+      },
+      0x0,                                          // Flags
+      0x0,                                          // CacheCoherent
+      0x0,                                          // AllocationHints
+      0x0,                                          // Reserved
+      0x0,                                          // MemoryAccessFlags
+      30,                                           // AddressSizeLimit
+    }, {
+      "\\_SB_.GDV0.USB0"                            // ObjectName
+    }
+  }, {
+    // emmc2 named component node
+    {
+      {
+        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type
+        sizeof (RPI4_NC_NODE),                      // Length
+        0x0,                                        // Revision
+        0x0,                                        // Reserved
+        0x0,                                        // NumIdMappings
+        0x0,                                        // IdReference
+      },
+      0x0,                                          // Flags
+      0x0,                                          // CacheCoherent
+      0x0,                                          // AllocationHints
+      0x0,                                          // Reserved
+      0x0,                                          // MemoryAccessFlags
+      30,                                           // AddressSizeLimit
+    }, {
+      "\\_SB_.GDV1.SDC3"                            // ObjectName
+    }
   }
 };
 
-- 
2.13.7



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#73814): https://edk2.groups.io/g/devel/message/73814
Mute This Topic: https://groups.io/mt/81935644/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components
Posted by Pete Batard 4 years, 10 months ago
On 2021.04.08 06:58, Jeremy Linton wrote:
> Add some additional IORT nodes for the USB & EMMC devices, realistically
> we probably only need to have a single node with the lowest AddressSizeLimit
> but this is conceptually "cleaner" should anyone actually try and use these
> values rather than the _DMA provided ones.
> 
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
>   Platform/RaspberryPi/AcpiTables/Iort.aslc | 44 ++++++++++++++++++++++++++++++-
>   1 file changed, 43 insertions(+), 1 deletion(-)
> 
> diff --git a/Platform/RaspberryPi/AcpiTables/Iort.aslc b/Platform/RaspberryPi/AcpiTables/Iort.aslc
> index 00720194bb..810307ae37 100644
> --- a/Platform/RaspberryPi/AcpiTables/Iort.aslc
> +++ b/Platform/RaspberryPi/AcpiTables/Iort.aslc
> @@ -20,6 +20,8 @@ typedef struct {
>   typedef struct {
> 
>     EFI_ACPI_6_0_IO_REMAPPING_TABLE      Iort;
> 
>     RPI4_NC_NODE                         NamedCompNode;
> 
> +  RPI4_NC_NODE                         NamedCompNode2;
> 
> +  RPI4_NC_NODE                         NamedCompNode3;
> 
>   } RPI4_IO_REMAPPING_STRUCTURE;
> 
>   
> 
>   STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
> 
> @@ -27,7 +29,7 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
>       ACPI_HEADER (EFI_ACPI_6_0_IO_REMAPPING_TABLE_SIGNATURE,
> 
>                    RPI4_IO_REMAPPING_STRUCTURE,
> 
>                    EFI_ACPI_IO_REMAPPING_TABLE_REVISION),
> 
> -    1,                                              // NumNodes
> 
> +    3,                                              // NumNodes
> 
>       sizeof (EFI_ACPI_6_0_IO_REMAPPING_TABLE),       // NodeOffset
> 
>       0                                               // Reserved
> 
>     }, {
> 
> @@ -50,6 +52,46 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
>       }, {
> 
>         "\\_SB_.SCB0.XHC0"                            // ObjectName
> 
>       }
> 
> +  }, {
> 
> +    // gpu/dwc usb named component node
> 
> +    {
> 
> +      {
> 
> +        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type
> 
> +        sizeof (RPI4_NC_NODE),                      // Length
> 
> +        0x0,                                        // Revision
> 
> +        0x0,                                        // Reserved
> 
> +        0x0,                                        // NumIdMappings
> 
> +        0x0,                                        // IdReference
> 
> +      },
> 
> +      0x0,                                          // Flags
> 
> +      0x0,                                          // CacheCoherent
> 
> +      0x0,                                          // AllocationHints
> 
> +      0x0,                                          // Reserved
> 
> +      0x0,                                          // MemoryAccessFlags
> 
> +      30,                                           // AddressSizeLimit
> 
> +    }, {
> 
> +      "\\_SB_.GDV0.USB0"                            // ObjectName
> 
> +    }
> 
> +  }, {
> 
> +    // emmc2 named component node
> 
> +    {
> 
> +      {
> 
> +        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type
> 
> +        sizeof (RPI4_NC_NODE),                      // Length
> 
> +        0x0,                                        // Revision
> 
> +        0x0,                                        // Reserved
> 
> +        0x0,                                        // NumIdMappings
> 
> +        0x0,                                        // IdReference
> 
> +      },
> 
> +      0x0,                                          // Flags
> 
> +      0x0,                                          // CacheCoherent
> 
> +      0x0,                                          // AllocationHints
> 
> +      0x0,                                          // Reserved
> 
> +      0x0,                                          // MemoryAccessFlags
> 
> +      30,                                           // AddressSizeLimit
> 
> +    }, {
> 
> +      "\\_SB_.GDV1.SDC3"                            // ObjectName
> 
> +    }
> 
>     }
> 
>   };
> 
>   
> 

Reviewed-by: Pete Batard <pete@akeo.ie>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#73835): https://edk2.groups.io/g/devel/message/73835
Mute This Topic: https://groups.io/mt/81935644/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components
Posted by Samer El-Haj-Mahmoud 4 years, 9 months ago
+ Ard’s new e-mail address

> -----Original Message-----
> From: Pete Batard <pete@akeo.ie>
> Sent: Thursday, April 8, 2021 5:48 AM
> To: Jeremy Linton <Jeremy.Linton@arm.com>; devel@edk2.groups.io
> Cc: Ard Biesheuvel <Ard.Biesheuvel@arm.com>; leif@nuviainc.com; Samer
> El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Andrei Warkentin
> (awarkentin@vmware.com) <awarkentin@vmware.com>
> Subject: Re: [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further
> named components
>
> On 2021.04.08 06:58, Jeremy Linton wrote:
> > Add some additional IORT nodes for the USB & EMMC devices,
> > realistically we probably only need to have a single node with the
> > lowest AddressSizeLimit but this is conceptually "cleaner" should
> > anyone actually try and use these values rather than the _DMA provided
> ones.
> >
> > Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> > ---
> >   Platform/RaspberryPi/AcpiTables/Iort.aslc | 44
> ++++++++++++++++++++++++++++++-
> >   1 file changed, 43 insertions(+), 1 deletion(-)
> >
> > diff --git a/Platform/RaspberryPi/AcpiTables/Iort.aslc
> > b/Platform/RaspberryPi/AcpiTables/Iort.aslc
> > index 00720194bb..810307ae37 100644
> > --- a/Platform/RaspberryPi/AcpiTables/Iort.aslc
> > +++ b/Platform/RaspberryPi/AcpiTables/Iort.aslc
> > @@ -20,6 +20,8 @@ typedef struct {
> >   typedef struct {
> >
> >     EFI_ACPI_6_0_IO_REMAPPING_TABLE      Iort;
> >
> >     RPI4_NC_NODE                         NamedCompNode;
> >
> > +  RPI4_NC_NODE                         NamedCompNode2;
> >
> > +  RPI4_NC_NODE                         NamedCompNode3;
> >
> >   } RPI4_IO_REMAPPING_STRUCTURE;
> >
> >
> >
> >   STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
> >
> > @@ -27,7 +29,7 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
> >       ACPI_HEADER (EFI_ACPI_6_0_IO_REMAPPING_TABLE_SIGNATURE,
> >
> >                    RPI4_IO_REMAPPING_STRUCTURE,
> >
> >                    EFI_ACPI_IO_REMAPPING_TABLE_REVISION),
> >
> > -    1,                                              // NumNodes
> >
> > +    3,                                              // NumNodes
> >
> >       sizeof (EFI_ACPI_6_0_IO_REMAPPING_TABLE),       // NodeOffset
> >
> >       0                                               // Reserved
> >
> >     }, {
> >
> > @@ -50,6 +52,46 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
> >       }, {
> >
> >         "\\_SB_.SCB0.XHC0"                            // ObjectName
> >
> >       }
> >
> > +  }, {
> >
> > +    // gpu/dwc usb named component node
> >
> > +    {
> >
> > +      {
> >
> > +        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type
> >
> > +        sizeof (RPI4_NC_NODE),                      // Length
> >
> > +        0x0,                                        // Revision
> >
> > +        0x0,                                        // Reserved
> >
> > +        0x0,                                        // NumIdMappings
> >
> > +        0x0,                                        // IdReference
> >
> > +      },
> >
> > +      0x0,                                          // Flags
> >
> > +      0x0,                                          // CacheCoherent
> >
> > +      0x0,                                          // AllocationHints
> >
> > +      0x0,                                          // Reserved
> >
> > +      0x0,                                          // MemoryAccessFlags
> >
> > +      30,                                           // AddressSizeLimit
> >
> > +    }, {
> >
> > +      "\\_SB_.GDV0.USB0"                            // ObjectName
> >
> > +    }
> >
> > +  }, {
> >
> > +    // emmc2 named component node
> >
> > +    {
> >
> > +      {
> >
> > +        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type
> >
> > +        sizeof (RPI4_NC_NODE),                      // Length
> >
> > +        0x0,                                        // Revision
> >
> > +        0x0,                                        // Reserved
> >
> > +        0x0,                                        // NumIdMappings
> >
> > +        0x0,                                        // IdReference
> >
> > +      },
> >
> > +      0x0,                                          // Flags
> >
> > +      0x0,                                          // CacheCoherent
> >
> > +      0x0,                                          // AllocationHints
> >
> > +      0x0,                                          // Reserved
> >
> > +      0x0,                                          // MemoryAccessFlags
> >
> > +      30,                                           // AddressSizeLimit
> >
> > +    }, {
> >
> > +      "\\_SB_.GDV1.SDC3"                            // ObjectName
> >
> > +    }
> >
> >     }
> >
> >   };
> >
> >
> >
>
> Reviewed-by: Pete Batard <pete@akeo.ie>
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#74892): https://edk2.groups.io/g/devel/message/74892
Mute This Topic: https://groups.io/mt/81935644/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components
Posted by Andrei Warkentin 4 years, 10 months ago
Reviewed-by: Andrei Warkentin <awarkentin@vmware.com>
________________________________
From: Jeremy Linton <jeremy.linton@arm.com>
Sent: Thursday, April 8, 2021 12:58 AM
To: devel@edk2.groups.io <devel@edk2.groups.io>
Cc: ard.biesheuvel@arm.com <ard.biesheuvel@arm.com>; leif@nuviainc.com <leif@nuviainc.com>; pete@akeo.ie <pete@akeo.ie>; samer.el-haj-mahmoud@arm.com <samer.el-haj-mahmoud@arm.com>; Andrei Warkentin <awarkentin@vmware.com>; Jeremy Linton <jeremy.linton@arm.com>
Subject: [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components

Add some additional IORT nodes for the USB & EMMC devices, realistically
we probably only need to have a single node with the lowest AddressSizeLimit
but this is conceptually "cleaner" should anyone actually try and use these
values rather than the _DMA provided ones.

Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
---
 Platform/RaspberryPi/AcpiTables/Iort.aslc | 44 ++++++++++++++++++++++++++++++-
 1 file changed, 43 insertions(+), 1 deletion(-)

diff --git a/Platform/RaspberryPi/AcpiTables/Iort.aslc b/Platform/RaspberryPi/AcpiTables/Iort.aslc
index 00720194bb..810307ae37 100644
--- a/Platform/RaspberryPi/AcpiTables/Iort.aslc
+++ b/Platform/RaspberryPi/AcpiTables/Iort.aslc
@@ -20,6 +20,8 @@ typedef struct {
 typedef struct {

   EFI_ACPI_6_0_IO_REMAPPING_TABLE      Iort;

   RPI4_NC_NODE                         NamedCompNode;

+  RPI4_NC_NODE                         NamedCompNode2;

+  RPI4_NC_NODE                         NamedCompNode3;

 } RPI4_IO_REMAPPING_STRUCTURE;



 STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {

@@ -27,7 +29,7 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
     ACPI_HEADER (EFI_ACPI_6_0_IO_REMAPPING_TABLE_SIGNATURE,

                  RPI4_IO_REMAPPING_STRUCTURE,

                  EFI_ACPI_IO_REMAPPING_TABLE_REVISION),

-    1,                                              // NumNodes

+    3,                                              // NumNodes

     sizeof (EFI_ACPI_6_0_IO_REMAPPING_TABLE),       // NodeOffset

     0                                               // Reserved

   }, {

@@ -50,6 +52,46 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
     }, {

       "\\_SB_.SCB0.XHC0"                            // ObjectName

     }

+  }, {

+    // gpu/dwc usb named component node

+    {

+      {

+        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type

+        sizeof (RPI4_NC_NODE),                      // Length

+        0x0,                                        // Revision

+        0x0,                                        // Reserved

+        0x0,                                        // NumIdMappings

+        0x0,                                        // IdReference

+      },

+      0x0,                                          // Flags

+      0x0,                                          // CacheCoherent

+      0x0,                                          // AllocationHints

+      0x0,                                          // Reserved

+      0x0,                                          // MemoryAccessFlags

+      30,                                           // AddressSizeLimit

+    }, {

+      "\\_SB_.GDV0.USB0"                            // ObjectName

+    }

+  }, {

+    // emmc2 named component node

+    {

+      {

+        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type

+        sizeof (RPI4_NC_NODE),                      // Length

+        0x0,                                        // Revision

+        0x0,                                        // Reserved

+        0x0,                                        // NumIdMappings

+        0x0,                                        // IdReference

+      },

+      0x0,                                          // Flags

+      0x0,                                          // CacheCoherent

+      0x0,                                          // AllocationHints

+      0x0,                                          // Reserved

+      0x0,                                          // MemoryAccessFlags

+      30,                                           // AddressSizeLimit

+    }, {

+      "\\_SB_.GDV1.SDC3"                            // ObjectName

+    }

   }

 };



--
2.13.7



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#73856): https://edk2.groups.io/g/devel/message/73856
Mute This Topic: https://groups.io/mt/81935644/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components
Posted by Samer El-Haj-Mahmoud 4 years, 9 months ago
+ Ard's new e-mail address


From: Andrei Warkentin <awarkentin@vmware.com>
Sent: Thursday, April 8, 2021 10:17 AM
To: Jeremy Linton <Jeremy.Linton@arm.com>; devel@edk2.groups.io
Cc: Ard Biesheuvel <Ard.Biesheuvel@arm.com>; leif@nuviainc.com; pete@akeo.ie; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
Subject: Re: [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components

Reviewed-by: Andrei Warkentin <awarkentin@vmware.com<mailto:awarkentin@vmware.com>>
________________________________
From: Jeremy Linton <jeremy.linton@arm.com<mailto:jeremy.linton@arm.com>>
Sent: Thursday, April 8, 2021 12:58 AM
To: devel@edk2.groups.io<mailto:devel@edk2.groups.io> <devel@edk2.groups.io<mailto:devel@edk2.groups.io>>
Cc: ard.biesheuvel@arm.com<mailto:ard.biesheuvel@arm.com> <ard.biesheuvel@arm.com<mailto:ard.biesheuvel@arm.com>>; leif@nuviainc.com<mailto:leif@nuviainc.com> <leif@nuviainc.com<mailto:leif@nuviainc.com>>; pete@akeo.ie<mailto:pete@akeo.ie> <pete@akeo.ie<mailto:pete@akeo.ie>>; samer.el-haj-mahmoud@arm.com<mailto:samer.el-haj-mahmoud@arm.com> <samer.el-haj-mahmoud@arm.com<mailto:samer.el-haj-mahmoud@arm.com>>; Andrei Warkentin <awarkentin@vmware.com<mailto:awarkentin@vmware.com>>; Jeremy Linton <jeremy.linton@arm.com<mailto:jeremy.linton@arm.com>>
Subject: [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components

Add some additional IORT nodes for the USB & EMMC devices, realistically
we probably only need to have a single node with the lowest AddressSizeLimit
but this is conceptually "cleaner" should anyone actually try and use these
values rather than the _DMA provided ones.

Signed-off-by: Jeremy Linton <jeremy.linton@arm.com<mailto:jeremy.linton@arm.com>>
---
 Platform/RaspberryPi/AcpiTables/Iort.aslc | 44 ++++++++++++++++++++++++++++++-
 1 file changed, 43 insertions(+), 1 deletion(-)

diff --git a/Platform/RaspberryPi/AcpiTables/Iort.aslc b/Platform/RaspberryPi/AcpiTables/Iort.aslc
index 00720194bb..810307ae37 100644
--- a/Platform/RaspberryPi/AcpiTables/Iort.aslc
+++ b/Platform/RaspberryPi/AcpiTables/Iort.aslc
@@ -20,6 +20,8 @@ typedef struct {
 typedef struct {

   EFI_ACPI_6_0_IO_REMAPPING_TABLE      Iort;

   RPI4_NC_NODE                         NamedCompNode;

+  RPI4_NC_NODE                         NamedCompNode2;

+  RPI4_NC_NODE                         NamedCompNode3;

 } RPI4_IO_REMAPPING_STRUCTURE;



 STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {

@@ -27,7 +29,7 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
     ACPI_HEADER (EFI_ACPI_6_0_IO_REMAPPING_TABLE_SIGNATURE,

                  RPI4_IO_REMAPPING_STRUCTURE,

                  EFI_ACPI_IO_REMAPPING_TABLE_REVISION),

-    1,                                              // NumNodes

+    3,                                              // NumNodes

     sizeof (EFI_ACPI_6_0_IO_REMAPPING_TABLE),       // NodeOffset

     0                                               // Reserved

   }, {

@@ -50,6 +52,46 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
     }, {

       "\\_SB_.SCB0.XHC0<file://_SB_.SCB0.XHC0>"                            // ObjectName

     }

+  }, {

+    // gpu/dwc usb named component node

+    {

+      {

+        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type

+        sizeof (RPI4_NC_NODE),                      // Length

+        0x0,                                        // Revision

+        0x0,                                        // Reserved

+        0x0,                                        // NumIdMappings

+        0x0,                                        // IdReference

+      },

+      0x0,                                          // Flags

+      0x0,                                          // CacheCoherent

+      0x0,                                          // AllocationHints

+      0x0,                                          // Reserved

+      0x0,                                          // MemoryAccessFlags

+      30,                                           // AddressSizeLimit

+    }, {

+      "\\_SB_.GDV0.USB0<file://_SB_.GDV0.USB0>"                            // ObjectName

+    }

+  }, {

+    // emmc2 named component node

+    {

+      {

+        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type

+        sizeof (RPI4_NC_NODE),                      // Length

+        0x0,                                        // Revision

+        0x0,                                        // Reserved

+        0x0,                                        // NumIdMappings

+        0x0,                                        // IdReference

+      },

+      0x0,                                          // Flags

+      0x0,                                          // CacheCoherent

+      0x0,                                          // AllocationHints

+      0x0,                                          // Reserved

+      0x0,                                          // MemoryAccessFlags

+      30,                                           // AddressSizeLimit

+    }, {

+      "\\_SB_.GDV1.SDC3<file://_SB_.GDV1.SDC3>"                            // ObjectName

+    }

   }

 };



--
2.13.7
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#74893): https://edk2.groups.io/g/devel/message/74893
Mute This Topic: https://groups.io/mt/81935644/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named components
Posted by Samer El-Haj-Mahmoud 4 years, 9 months ago
+ Ard's new e-mail address

> -----Original Message-----
> From: Jeremy Linton <jeremy.linton@arm.com>
> Sent: Thursday, April 8, 2021 1:59 AM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel <Ard.Biesheuvel@arm.com>; leif@nuviainc.com;
> pete@akeo.ie; Samer El-Haj-Mahmoud <Samer.El-Haj-
> Mahmoud@arm.com>; Andrei Warkentin (awarkentin@vmware.com)
> <awarkentin@vmware.com>; Jeremy Linton <Jeremy.Linton@arm.com>
> Subject: [PATCH 2/3] Platform/RaspberryPi/AcpiTables: Add further named
> components
>
> Add some additional IORT nodes for the USB & EMMC devices, realistically
> we probably only need to have a single node with the lowest
> AddressSizeLimit
> but this is conceptually "cleaner" should anyone actually try and use these
> values rather than the _DMA provided ones.
>
> Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
> ---
>  Platform/RaspberryPi/AcpiTables/Iort.aslc | 44
> ++++++++++++++++++++++++++++++-
>  1 file changed, 43 insertions(+), 1 deletion(-)
>
> diff --git a/Platform/RaspberryPi/AcpiTables/Iort.aslc
> b/Platform/RaspberryPi/AcpiTables/Iort.aslc
> index 00720194bb..810307ae37 100644
> --- a/Platform/RaspberryPi/AcpiTables/Iort.aslc
> +++ b/Platform/RaspberryPi/AcpiTables/Iort.aslc
> @@ -20,6 +20,8 @@ typedef struct {
>  typedef struct {
>
>    EFI_ACPI_6_0_IO_REMAPPING_TABLE      Iort;
>
>    RPI4_NC_NODE                         NamedCompNode;
>
> +  RPI4_NC_NODE                         NamedCompNode2;
>
> +  RPI4_NC_NODE                         NamedCompNode3;
>
>  } RPI4_IO_REMAPPING_STRUCTURE;
>
>
>
>  STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
>
> @@ -27,7 +29,7 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
>      ACPI_HEADER (EFI_ACPI_6_0_IO_REMAPPING_TABLE_SIGNATURE,
>
>                   RPI4_IO_REMAPPING_STRUCTURE,
>
>                   EFI_ACPI_IO_REMAPPING_TABLE_REVISION),
>
> -    1,                                              // NumNodes
>
> +    3,                                              // NumNodes
>
>      sizeof (EFI_ACPI_6_0_IO_REMAPPING_TABLE),       // NodeOffset
>
>      0                                               // Reserved
>
>    }, {
>
> @@ -50,6 +52,46 @@ STATIC RPI4_IO_REMAPPING_STRUCTURE Iort = {
>      }, {
>
>        "\\_SB_.SCB0.XHC0"                            // ObjectName
>
>      }
>
> +  }, {
>
> +    // gpu/dwc usb named component node
>
> +    {
>
> +      {
>
> +        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type
>
> +        sizeof (RPI4_NC_NODE),                      // Length
>
> +        0x0,                                        // Revision
>
> +        0x0,                                        // Reserved
>
> +        0x0,                                        // NumIdMappings
>
> +        0x0,                                        // IdReference
>
> +      },
>
> +      0x0,                                          // Flags
>
> +      0x0,                                          // CacheCoherent
>
> +      0x0,                                          // AllocationHints
>
> +      0x0,                                          // Reserved
>
> +      0x0,                                          // MemoryAccessFlags
>
> +      30,                                           // AddressSizeLimit
>
> +    }, {
>
> +      "\\_SB_.GDV0.USB0"                            // ObjectName
>
> +    }
>
> +  }, {
>
> +    // emmc2 named component node
>
> +    {
>
> +      {
>
> +        EFI_ACPI_IORT_TYPE_NAMED_COMP,              // Type
>
> +        sizeof (RPI4_NC_NODE),                      // Length
>
> +        0x0,                                        // Revision
>
> +        0x0,                                        // Reserved
>
> +        0x0,                                        // NumIdMappings
>
> +        0x0,                                        // IdReference
>
> +      },
>
> +      0x0,                                          // Flags
>
> +      0x0,                                          // CacheCoherent
>
> +      0x0,                                          // AllocationHints
>
> +      0x0,                                          // Reserved
>
> +      0x0,                                          // MemoryAccessFlags
>
> +      30,                                           // AddressSizeLimit
>
> +    }, {
>
> +      "\\_SB_.GDV1.SDC3"                            // ObjectName
>
> +    }
>
>    }
>
>  };
>
>
>
> --
> 2.13.7

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#74891): https://edk2.groups.io/g/devel/message/74891
Mute This Topic: https://groups.io/mt/81935644/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-