Skip to content

efi/arm: refactor DT EFI param parsing

Ard Biesheuvel linux-arm-kernel linux-efi 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3efi/arm: move FDT param discovery code out of efi.cdrivers/firmware/efi/Makefile drivers/firmware/efi/efi.c drivers/firmware/efi/fdtparams.c+143 -1350 replies
2/3efi/arm: move FDT specific definitions into fdtparams.cdrivers/firmware/efi/arm-init.c drivers/firmware/efi/fdtparams.c include/linux/efi.h+30 -270 replies
3/3efi/arm: rewrite FDT param discovery routinesdrivers/firmware/efi/fdtparams.c+85 -1182 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/3] efi/arm: refactor DT EFI param parsing

In preparation for another arrival (RISC-V), do some janitorial work on the FDT param parsing code for EFI as well. The code sits in an #ifdef block in efi.c, which is usually a good indicator that it should be moved into its own source file. Then, we simplify the DT handling, by switching to the FDT library, instead of going through the more high level early OF enumeration code, which in our case will be doing libfdt calls under the hood as well, as EFI on ARM specifically uses *flattened* DT. (note that the EFI stub for ARM uses libfdt as well to populate the DT properties that we read back here)

Ard Biesheuvel (3):
efi/arm: move FDT param discovery code out of efi.c efi/arm: move FDT specific definitions into fdtparams.c efi/arm: rewrite FDT param discovery routines

 drivers/firmware/efi/Makefile    |   1 +
 drivers/firmware/efi/arm-init.c  |  17 +--
 drivers/firmware/efi/efi.c       | 135 --------------------
 drivers/firmware/efi/fdtparams.c | 125 ++++++++++++++++++
 include/linux/efi.h              |  10 +-
 5 files changed, 133 insertions(+), 155 deletions(-)
 create mode 100644 drivers/firmware/efi/fdtparams.c
Signature · 1 line
2.17.1