[v2] get rid of DeviceTree from SbsaQemu

[edk2-devel] [PATCH edk2-platforms v2 0/4] get rid of DeviceTree from SbsaQemu

Marcin Juszkiewicz posted 4 patches 3 months, 1 week ago

Diff against v3
Download series mbox

Failed in applying to current master (apply log)

There is a newer version of this series

Platform/Qemu/SbsaQemu/SbsaQemu.dsc                 |   4 +-
Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.inf    |   6 +-
.../Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.inf     |   4 +-
.../SbsaQemuPlatformDxe/SbsaQemuPlatformDxe.inf     |   4 +-
.../SbsaQemu/Library/FdtHelperLib/FdtHelperLib.inf  |  33 ----
.../SbsaQemuHardwareInfoLib.inf                     |  34 ++++
.../SbsaQemu/Include/IndustryStandard/SbsaQemuSmc.h |   2 +
.../Qemu/SbsaQemu/Include/Library/FdtHelperLib.h    |  36 ----
.../Include/Library/SbsaQemuHardwareInfoLib.h       |  45 +++++
Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.c      |  10 +-
.../Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c       |  16 +-
.../SbsaQemuPlatformDxe/SbsaQemuPlatformDxe.c       |   9 +-
.../SbsaQemu/Library/FdtHelperLib/FdtHelperLib.c    |  98 ----------
.../SbsaQemuHardwareInfoLib.c                       | 204 ++++++++++++++++++++
14 files changed, 308 insertions(+), 197 deletions(-)

Expand all Fold all

[edk2-devel] [PATCH edk2-platforms v2 0/4] get rid of DeviceTree from SbsaQemu

Posted by Marcin Juszkiewicz 3 months, 1 week ago

We want to stop parsing DeviceTree to gather hardware information.

Instead we ask TF-A for those details using SMC calls. On real hardware
platform it could be asking on-board Embedded Controller.

If TF-A answer to SMC call would be not success then we assume that it
is old version and go back to parsing DeviceTree data directly. Just now
it is present in one place (SbsaQemuHardwareInfoLib) together with new
code for handling SMC stuff.

I hope that we can drop FdtHandler part as part of 202411 release.

TF-A part: https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/25707

After this change the only place where DT data is parsed directly is
checking for memory nodes. Patches for that code will be sent once we
get SMC calls into TF-A.

Signed-off-by: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org>
---
Marcin Juszkiewicz (4):
      Platform/SbsaQemu: add SbsaQemuHardwareInfoLib
      Platform/SbsaQemu: read amount of cpus during init
      Platform/SbsaQemu: use PcdCoreCount directly
      Platform/SbsaQemu: move FdtHandlerLib to SbsaQemuHardwareInfoLib

 Platform/Qemu/SbsaQemu/SbsaQemu.dsc                 |   4 +-
 Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.inf    |   6 +-
 .../Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.inf     |   4 +-
 .../SbsaQemuPlatformDxe/SbsaQemuPlatformDxe.inf     |   4 +-
 .../SbsaQemu/Library/FdtHelperLib/FdtHelperLib.inf  |  33 ----
 .../SbsaQemuHardwareInfoLib.inf                     |  34 ++++
 .../SbsaQemu/Include/IndustryStandard/SbsaQemuSmc.h |   2 +
 .../Qemu/SbsaQemu/Include/Library/FdtHelperLib.h    |  36 ----
 .../Include/Library/SbsaQemuHardwareInfoLib.h       |  45 +++++
 Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.c      |  10 +-
 .../Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c       |  16 +-
 .../SbsaQemuPlatformDxe/SbsaQemuPlatformDxe.c       |   9 +-
 .../SbsaQemu/Library/FdtHelperLib/FdtHelperLib.c    |  98 ----------
 .../SbsaQemuHardwareInfoLib.c                       | 204 ++++++++++++++++++++
 14 files changed, 308 insertions(+), 197 deletions(-)
---
base-commit: a1598bbcd167f4a5cf61229156426ef6f0784ab3
change-id: 20240115-no-dt-for-cpu-2c511393df93

Best regards,
-- 
Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org>



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