[PATCH v2 0/4] DynamicTablesPkg: Pcie generation updates


Jeff Brasen
 

Add fixes/features to dynamic PCIe support

- Correct issue with translation in generated ACPI tables.
- Allow for more than 16 controllers to be generated.
- Allow optional use of segment number as UID for cases where ACPI path is needed in other places.
- Add support for override protocol that allows platform specific modification of node prior to creation.

Update:
v2 - Add IsPosDecode logic to translation patch

Jeff Brasen (4):
DynamicTablesPkg: AcpiSsdtPcieLibArm: Correct translation value
DynamicTablesPkg: AcpiSsdtPcieLibArm: Allow use of segment number as
UID
DynamicTablesPkg: AcpiSsdtPcieLibArm: Support UID > 0xF
DynamicTablesPkg: AcpiSsdtPcieLibArm: Add support for override
protocol

DynamicTablesPkg/DynamicTablesPkg.dec | 6 ++
.../Protocol/SsdtPcieOverrideProtocol.h | 63 ++++++++++++++++
.../AcpiSsdtPcieLibArm/SsdtPcieGenerator.c | 73 ++++++++++++++++---
.../AcpiSsdtPcieLibArm/SsdtPcieGenerator.h | 2 +-
.../Arm/AcpiSsdtPcieLibArm/SsdtPcieLibArm.inf | 7 ++
5 files changed, 139 insertions(+), 12 deletions(-)
create mode 100644 DynamicTablesPkg/Include/Protocol/SsdtPcieOverrideProtocol.h

--
2.25.1


Jeff Brasen
 

Pierre,

Any thoughts on this v2 version?

-Jeff

-----Original Message-----
From: Jeff Brasen <jbrasen@...>
Sent: Friday, July 1, 2022 10:33 AM
To: devel@edk2.groups.io
Cc: Pierre.Gondois@...; Sami.Mujawar@...;
Alexei.Fedorov@...; Jeff Brasen <jbrasen@...>
Subject: [PATCH v2 0/4] DynamicTablesPkg: Pcie generation updates

Add fixes/features to dynamic PCIe support

- Correct issue with translation in generated ACPI tables.
- Allow for more than 16 controllers to be generated.
- Allow optional use of segment number as UID for cases where ACPI path is
needed in other places.
- Add support for override protocol that allows platform specific modification
of node prior to creation.

Update:
v2 - Add IsPosDecode logic to translation patch

Jeff Brasen (4):
DynamicTablesPkg: AcpiSsdtPcieLibArm: Correct translation value
DynamicTablesPkg: AcpiSsdtPcieLibArm: Allow use of segment number as
UID
DynamicTablesPkg: AcpiSsdtPcieLibArm: Support UID > 0xF
DynamicTablesPkg: AcpiSsdtPcieLibArm: Add support for override
protocol

DynamicTablesPkg/DynamicTablesPkg.dec | 6 ++
.../Protocol/SsdtPcieOverrideProtocol.h | 63 ++++++++++++++++
.../AcpiSsdtPcieLibArm/SsdtPcieGenerator.c | 73 ++++++++++++++++---
.../AcpiSsdtPcieLibArm/SsdtPcieGenerator.h | 2 +-
.../Arm/AcpiSsdtPcieLibArm/SsdtPcieLibArm.inf | 7 ++
5 files changed, 139 insertions(+), 12 deletions(-) create mode 100644
DynamicTablesPkg/Include/Protocol/SsdtPcieOverrideProtocol.h

--
2.25.1


PierreGondois
 

On 7/7/22 18:59, Jeff Brasen wrote:
Pierre,
Any thoughts on this v2 version?
-Jeff

-----Original Message-----
From: Jeff Brasen <jbrasen@...>
Sent: Friday, July 1, 2022 10:33 AM
To: devel@edk2.groups.io
Cc: Pierre.Gondois@...; Sami.Mujawar@...;
Alexei.Fedorov@...; Jeff Brasen <jbrasen@...>
Subject: [PATCH v2 0/4] DynamicTablesPkg: Pcie generation updates

Add fixes/features to dynamic PCIe support

- Correct issue with translation in generated ACPI tables.
- Allow for more than 16 controllers to be generated.
- Allow optional use of segment number as UID for cases where ACPI path is
needed in other places.
- Add support for override protocol that allows platform specific modification
of node prior to creation.

Update:
v2 - Add IsPosDecode logic to translation patch

Jeff Brasen (4):
DynamicTablesPkg: AcpiSsdtPcieLibArm: Correct translation value
DynamicTablesPkg: AcpiSsdtPcieLibArm: Allow use of segment number as
UID
DynamicTablesPkg: AcpiSsdtPcieLibArm: Support UID > 0xF
DynamicTablesPkg: AcpiSsdtPcieLibArm: Add support for override
protocol

DynamicTablesPkg/DynamicTablesPkg.dec | 6 ++
.../Protocol/SsdtPcieOverrideProtocol.h | 63 ++++++++++++++++
.../AcpiSsdtPcieLibArm/SsdtPcieGenerator.c | 73 ++++++++++++++++---
.../AcpiSsdtPcieLibArm/SsdtPcieGenerator.h | 2 +-
.../Arm/AcpiSsdtPcieLibArm/SsdtPcieLibArm.inf | 7 ++
5 files changed, 139 insertions(+), 12 deletions(-) create mode 100644
DynamicTablesPkg/Include/Protocol/SsdtPcieOverrideProtocol.h

--
2.25.1


Jeff Brasen
 

Ok, I never got those responses, will have to check what happened with my email. Will respond to those.

-----Original Message-----
From: Pierre Gondois <pierre.gondois@...>
Sent: Friday, July 8, 2022 1:31 AM
To: Jeff Brasen <jbrasen@...>; devel@edk2.groups.io
Cc: Sami.Mujawar@...; Alexei.Fedorov@...
Subject: Re: [PATCH v2 0/4] DynamicTablesPkg: Pcie generation updates

External email: Use caution opening links or attachments


Hello Jeff,
I had some questions about the V1 at:
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk
2.groups.io%2Fg%2Fdevel%2Ftopic%2F92089320%2390932&amp;data=05%7
C01%7Cjbrasen%40nvidia.com%7C8a911ded72fd403690f208da60b3e0a9%7C
43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637928622942274065%7C
Unknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJB
TiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=p3jCMX%
2Fi0wf60FA2DgsGV7b3Vq6qw8jP6Kjp82YOoKg%3D&amp;reserved=0
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk
2.groups.io%2Fg%2Fdevel%2Ftopic%2F92089321%2390933&amp;data=05%7
C01%7Cjbrasen%40nvidia.com%7C8a911ded72fd403690f208da60b3e0a9%7C
43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637928622942274065%7C
Unknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJB
TiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=IBjcEWb5
VY7iXXkwOuP5LZN8c3aFdB25ophGXAVfHjQ%3D&amp;reserved=0

Regards,
Pierre

On 7/7/22 18:59, Jeff Brasen wrote:
Pierre,

Any thoughts on this v2 version?

-Jeff


-----Original Message-----
From: Jeff Brasen <jbrasen@...>
Sent: Friday, July 1, 2022 10:33 AM
To: devel@edk2.groups.io
Cc: Pierre.Gondois@...; Sami.Mujawar@...;
Alexei.Fedorov@...; Jeff Brasen <jbrasen@...>
Subject: [PATCH v2 0/4] DynamicTablesPkg: Pcie generation updates

Add fixes/features to dynamic PCIe support

- Correct issue with translation in generated ACPI tables.
- Allow for more than 16 controllers to be generated.
- Allow optional use of segment number as UID for cases where ACPI
path is needed in other places.
- Add support for override protocol that allows platform specific
modification of node prior to creation.

Update:
v2 - Add IsPosDecode logic to translation patch

Jeff Brasen (4):
DynamicTablesPkg: AcpiSsdtPcieLibArm: Correct translation value
DynamicTablesPkg: AcpiSsdtPcieLibArm: Allow use of segment number
as
UID
DynamicTablesPkg: AcpiSsdtPcieLibArm: Support UID > 0xF
DynamicTablesPkg: AcpiSsdtPcieLibArm: Add support for override
protocol

DynamicTablesPkg/DynamicTablesPkg.dec | 6 ++
.../Protocol/SsdtPcieOverrideProtocol.h | 63 ++++++++++++++++
.../AcpiSsdtPcieLibArm/SsdtPcieGenerator.c | 73 ++++++++++++++++-
--
.../AcpiSsdtPcieLibArm/SsdtPcieGenerator.h | 2 +-
.../Arm/AcpiSsdtPcieLibArm/SsdtPcieLibArm.inf | 7 ++
5 files changed, 139 insertions(+), 12 deletions(-) create mode
100644 DynamicTablesPkg/Include/Protocol/SsdtPcieOverrideProtocol.h

--
2.25.1