[edk2-devel] [PATCH edk2-platforms v1 0/6] Fix platform support for dynamic tables framework

Sami Mujawar posted 6 patches 3 years, 2 months ago
Failed in applying to current master (apply log)
Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl                                                      |  16 +-
Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/Dsdt.asl          |  16 +-
Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtUart.asl      |  42 --
Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c      | 455 ++++++++++++++------
Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h      | 107 +----
Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf |   3 +-
Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc                                          |  11 +-
Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc                    |  19 -
Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/Dsdt.asl      |  22 +-
Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c  | 347 ++++++++++-----
Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h  |  73 +---
11 files changed, 615 insertions(+), 496 deletions(-)
delete mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtUart.asl
delete mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc
[edk2-devel] [PATCH edk2-platforms v1 0/6] Fix platform support for dynamic tables framework
Posted by Sami Mujawar 3 years, 2 months ago
This patch series fixes the following issues for FVP & Juno platform
when Dynamic Tables Framework is enabled.
 - Remove the usage of function macros as this does not comply with
   the edk2 coding standard.
 - Fix serial port usage on FVP.
 - Remove serial port description from SSDT and DSDT as this is now
   generated by Dynamic AML.

This series also includes a patch to remove PL050 KMI support on
Juno.

The changes can be seen at:
https://github.com/samimujawar/edk2-platforms/tree/1608_fix_config_mgr_handlers_v1

Sami Mujawar (6):
  Platform/ARM: FVP: Remove usage of function macros
  Platform/ARM: Juno: Remove usage of function macros
  Platform/ARM: FVP: Fix serial port usage
  Platform/ARM: Juno: Remove PL050 KMI description from DSDT
  Platform/ARM: Juno: Remove Serial port SSDT
  Platform/ARM: FVP: Remove Serial port description from DSDT

 Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl                                                      |  16 +-
 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/Dsdt.asl          |  16 +-
 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtUart.asl      |  42 --
 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c      | 455 ++++++++++++++------
 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h      | 107 +----
 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf |   3 +-
 Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc                                          |  11 +-
 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc                    |  19 -
 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/Dsdt.asl      |  22 +-
 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c  | 347 ++++++++++-----
 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h  |  73 +---
 11 files changed, 615 insertions(+), 496 deletions(-)
 delete mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtUart.asl
 delete mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc

-- 
'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#70684): https://edk2.groups.io/g/devel/message/70684
Mute This Topic: https://groups.io/mt/80029718/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/6] Fix platform support for dynamic tables framework
Posted by Leif Lindholm 3 years, 2 months ago
Hi Sami,

On Fri, Jan 22, 2021 at 13:29:29 +0000, Sami Mujawar wrote:
> This patch series fixes the following issues for FVP & Juno platform
> when Dynamic Tables Framework is enabled.
>  - Remove the usage of function macros as this does not comply with
>    the edk2 coding standard.
>  - Fix serial port usage on FVP.
>  - Remove serial port description from SSDT and DSDT as this is now
>    generated by Dynamic AML.
> 
> This series also includes a patch to remove PL050 KMI support on
> Juno.
> 
> The changes can be seen at:
> https://github.com/samimujawar/edk2-platforms/tree/1608_fix_config_mgr_handlers_v1
> 
> Sami Mujawar (6):
>   Platform/ARM: FVP: Remove usage of function macros
>   Platform/ARM: Juno: Remove usage of function macros
>   Platform/ARM: FVP: Fix serial port usage
>   Platform/ARM: Juno: Remove PL050 KMI description from DSDT
>   Platform/ARM: Juno: Remove Serial port SSDT
>   Platform/ARM: FVP: Remove Serial port description from DSDT

For the series:
Reviewed-by: Leif Lindholm <leif@nuviainc.com>
Pushed as c6d565eddf6f..fcf693c9e8d6.

Many thanks for quick response time.

> 
>  Platform/ARM/JunoPkg/AcpiTables/Dsdt.asl                                                      |  16 +-
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/Dsdt.asl          |  16 +-
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtUart.asl      |  42 --
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c      | 455 ++++++++++++++------
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h      | 107 +----
>  Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf |   3 +-
>  Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc                                          |  11 +-
>  Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc                    |  19 -
>  Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/Dsdt.asl      |  22 +-
>  Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c  | 347 ++++++++++-----
>  Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h  |  73 +---
>  11 files changed, 615 insertions(+), 496 deletions(-)
>  delete mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/AslTables/SsdtUart.asl
>  delete mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc
> 
> -- 
> 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'
> 


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