[edk2-devel] [PATCH edk2-platforms v1 0/3] Enable PCI/CPU topology generation for Juno

PierreGondois posted 3 patches 1 year, 11 months ago
Failed in applying to current master (apply log)
.../AslTables/Dsdt.asl                        | 198 --------
.../AslTables/SsdtPci.asl                     | 195 --------
.../ConfigurationManager.c                    | 425 +++++++++++++++++-
.../ConfigurationManager.h                    |  47 +-
.../ConfigurationManagerDxe.inf               |   1 -
5 files changed, 456 insertions(+), 410 deletions(-)
delete mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtPci.asl
[edk2-devel] [PATCH edk2-platforms v1 0/3] Enable PCI/CPU topology generation for Juno
Posted by PierreGondois 1 year, 11 months ago
From: Pierre Gondois <pierre.gondois@arm.com>

DEPENDENCY:
This patch set relies on a fix sent at:
https://edk2.groups.io/g/devel/message/89348

The DynamicTablesPkg can now generate the AML description of the
PCI/CPU topology information. Use these new features and remove
the static ASL description of the PCI/CPU topology when the
DynamicTables are used (i.e. when -D DYNAMIC_TABLES_FRAMEWORK is
provided).

Also provide a small fix for debug printing.

Pierre Gondois (3):
  Platform/ARM: Juno: Generate ASL description of CPU topology
  Platform/ARM: Juno: Print debug values in right order
  Platform/ARM: Juno: Generate ASL description of a PCIe bus

 .../AslTables/Dsdt.asl                        | 198 --------
 .../AslTables/SsdtPci.asl                     | 195 --------
 .../ConfigurationManager.c                    | 425 +++++++++++++++++-
 .../ConfigurationManager.h                    |  47 +-
 .../ConfigurationManagerDxe.inf               |   1 -
 5 files changed, 456 insertions(+), 410 deletions(-)
 delete mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtPci.asl

-- 
2.25.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89349): https://edk2.groups.io/g/devel/message/89349
Mute This Topic: https://groups.io/mt/90732523/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH edk2-platforms v1 0/3] Enable PCI/CPU topology generation for Juno
Posted by Sami Mujawar 1 year, 9 months ago
Pushed as bc93dea9b98c..31bd1678a1db

Regards,

Sami Mujawar

On 27/04/2022 03:59 pm, Pierre.Gondois@arm.com wrote:
> From: Pierre Gondois <pierre.gondois@arm.com>
>
> DEPENDENCY:
> This patch set relies on a fix sent at:
> https://edk2.groups.io/g/devel/message/89348
>
> The DynamicTablesPkg can now generate the AML description of the
> PCI/CPU topology information. Use these new features and remove
> the static ASL description of the PCI/CPU topology when the
> DynamicTables are used (i.e. when -D DYNAMIC_TABLES_FRAMEWORK is
> provided).
>
> Also provide a small fix for debug printing.
>
> Pierre Gondois (3):
>    Platform/ARM: Juno: Generate ASL description of CPU topology
>    Platform/ARM: Juno: Print debug values in right order
>    Platform/ARM: Juno: Generate ASL description of a PCIe bus
>
>   .../AslTables/Dsdt.asl                        | 198 --------
>   .../AslTables/SsdtPci.asl                     | 195 --------
>   .../ConfigurationManager.c                    | 425 +++++++++++++++++-
>   .../ConfigurationManager.h                    |  47 +-
>   .../ConfigurationManagerDxe.inf               |   1 -
>   5 files changed, 456 insertions(+), 410 deletions(-)
>   delete mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtPci.asl
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91470): https://edk2.groups.io/g/devel/message/91470
Mute This Topic: https://groups.io/mt/90732523/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH edk2-platforms v1 0/3] Enable PCI/CPU topology generation for Juno
Posted by Ard Biesheuvel 1 year, 11 months ago
On Wed, 27 Apr 2022 at 17:00, PierreGondois <pierre.gondois@arm.com> wrote:
>
> From: Pierre Gondois <pierre.gondois@arm.com>
>
> DEPENDENCY:
> This patch set relies on a fix sent at:
> https://edk2.groups.io/g/devel/message/89348
>
> The DynamicTablesPkg can now generate the AML description of the
> PCI/CPU topology information. Use these new features and remove
> the static ASL description of the PCI/CPU topology when the
> DynamicTables are used (i.e. when -D DYNAMIC_TABLES_FRAMEWORK is
> provided).
>
> Also provide a small fix for debug printing.
>
> Pierre Gondois (3):
>   Platform/ARM: Juno: Generate ASL description of CPU topology
>   Platform/ARM: Juno: Print debug values in right order
>   Platform/ARM: Juno: Generate ASL description of a PCIe bus
>

Sami, please review this and let me know when it can be merged.

>  .../AslTables/Dsdt.asl                        | 198 --------
>  .../AslTables/SsdtPci.asl                     | 195 --------
>  .../ConfigurationManager.c                    | 425 +++++++++++++++++-
>  .../ConfigurationManager.h                    |  47 +-
>  .../ConfigurationManagerDxe.inf               |   1 -
>  5 files changed, 456 insertions(+), 410 deletions(-)
>  delete mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtPci.asl
>
> --
> 2.25.1
>
>
>
> ------------
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#89349): https://edk2.groups.io/g/devel/message/89349
> Mute This Topic: https://groups.io/mt/90732523/5717338
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [ardb+tianocore@kernel.org]
> ------------
>
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89409): https://edk2.groups.io/g/devel/message/89409
Mute This Topic: https://groups.io/mt/90732523/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH edk2-platforms v1 0/3] Enable PCI/CPU topology generation for Juno
Posted by PierreGondois 1 year, 11 months ago
Hello Ard,
Sami is on leave for the next 2 weeks.
Also, just in case, this patch-set has a dependency on:
https://edk2.groups.io/g/devel/message/89348

Regards,
Pierre

On 4/29/22 15:35, Ard Biesheuvel wrote:
> On Wed, 27 Apr 2022 at 17:00, PierreGondois <pierre.gondois@arm.com> wrote:
>>
>> From: Pierre Gondois <pierre.gondois@arm.com>
>>
>> DEPENDENCY:
>> This patch set relies on a fix sent at:
>> https://edk2.groups.io/g/devel/message/89348
>>
>> The DynamicTablesPkg can now generate the AML description of the
>> PCI/CPU topology information. Use these new features and remove
>> the static ASL description of the PCI/CPU topology when the
>> DynamicTables are used (i.e. when -D DYNAMIC_TABLES_FRAMEWORK is
>> provided).
>>
>> Also provide a small fix for debug printing.
>>
>> Pierre Gondois (3):
>>    Platform/ARM: Juno: Generate ASL description of CPU topology
>>    Platform/ARM: Juno: Print debug values in right order
>>    Platform/ARM: Juno: Generate ASL description of a PCIe bus
>>
> 
> Sami, please review this and let me know when it can be merged.
> 
>>   .../AslTables/Dsdt.asl                        | 198 --------
>>   .../AslTables/SsdtPci.asl                     | 195 --------
>>   .../ConfigurationManager.c                    | 425 +++++++++++++++++-
>>   .../ConfigurationManager.h                    |  47 +-
>>   .../ConfigurationManagerDxe.inf               |   1 -
>>   5 files changed, 456 insertions(+), 410 deletions(-)
>>   delete mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtPci.asl
>>
>> --
>> 2.25.1
>>
>>
>>
>> ------------
>> Groups.io Links: You receive all messages sent to this group.
>> View/Reply Online (#89349): https://edk2.groups.io/g/devel/message/89349
>> Mute This Topic: https://groups.io/mt/90732523/5717338
>> Group Owner: devel+owner@edk2.groups.io
>> Unsubscribe: https://edk2.groups.io/g/devel/unsub [ardb+tianocore@kernel.org]
>> ------------
>>
>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89410): https://edk2.groups.io/g/devel/message/89410
Mute This Topic: https://groups.io/mt/90732523/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH edk2-platforms v1 0/3] Enable PCI/CPU topology generation for Juno
Posted by Ard Biesheuvel 1 year, 11 months ago
On Fri, 29 Apr 2022 at 15:39, Pierre Gondois <pierre.gondois@arm.com> wrote:
>
> Hello Ard,
> Sami is on leave for the next 2 weeks.
> Also, just in case, this patch-set has a dependency on:
> https://edk2.groups.io/g/devel/message/89348
>

Yes, that is why I asked him to review: he is the DynamicTablesPkg
maintainer, so he will need to merge the dependency first.


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