Skip to content

OpenSBI compile-time C arrays

Anup Patel opensbi 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest11 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/11Makefile: Allow generated C source to be anywhere in build directoryMakefile+3 -31 reply
2/11Makefile: Add support for generating C array at compile timeMakefile scripts/carray.sh+88 -01 reply
3/11lib: utils/reset: Generate FDT reset driver list at compile-timelib/utils/reset/fdt_reset.c lib/utils/reset/fdt_reset_drivers.carray lib/utils/reset/objects.mk+20 -170 replies
4/11lib: utils/serial: Generate FDT serial driver list at compile-timelib/utils/serial/fdt_serial.c lib/utils/serial/fdt_serial_drivers.carray lib/utils/serial/objects.mk+26 -210 replies
5/11lib: utils/timer: Generate FDT timer driver list at compile-timelib/utils/timer/fdt_timer.c lib/utils/timer/fdt_timer_drivers.carray lib/utils/timer/objects.mk+12 -70 replies
6/11lib: utils/irqchip: Generate FDT irqchip driver list at compile-timelib/utils/irqchip/fdt_irqchip.c lib/utils/irqchip/fdt_irqchip_drivers.carray lib/utils/irqchip/objects.mk+16 -110 replies
7/11lib: utils/ipi: Generate FDT ipi driver list at compile-timelib/utils/ipi/fdt_ipi.c lib/utils/ipi/fdt_ipi_drivers.carray lib/utils/ipi/objects.mk+12 -70 replies
8/11lib: utils/i2c: Generate FDT i2c adapter driver list at compile-timelib/utils/i2c/fdt_i2c.c lib/utils/i2c/fdt_i2c_adapter_drivers.carray lib/utils/i2c/objects.mk+12 -90 replies
9/11lib: utils/gpio: Generate FDT gpio driver list at compile-timeinclude/sbi_utils/gpio/fdt_gpio.h lib/utils/gpio/fdt_gpio.c lib/utils/gpio/fdt_gpio_drivers.carray +1 more+17 -100 replies
10/11platform: generic: Generate platform override module list at compile-timeplatform/generic/objects.mk platform/generic/platform.c platform/generic/platform_override_modules.carray+13 -90 replies
11/11platform: generic: Move Sifive platform overrides into own directoryplatform/generic/objects.mk platform/generic/sifive/fu540.c platform/generic/sifive/fu740.c +1 more+9 -61 reply

Reviews trailers per patch

3 of 11 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchAtish Patra
SeriesR-b
1/11R-b
2/11R-b
3/11
4/11
5/11
6/11
7/11
8/11
9/11
10/11
11/11R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 00/11] OpenSBI compile-time C arrays

This series aims at removing hard-coded C arrays for drivers and modules in OpenSBI sources and replace it with dynamically generated arrays at compile-time. External firmwares which use OpenSBI as library will have to create these arrays separately because they typically don't use OpenSBI build system.

These patches can also be found in generated_carray_v1 branch at:
https://github.com/avpatel/opensbi.git

Anup Patel (11):
Makefile: Allow generated C source to be anywhere in build directory
Makefile: Add support for generating C array at compile time
lib: utils/reset: Generate FDT reset driver list at compile-time
lib: utils/serial: Generate FDT serial driver list at compile-time
lib: utils/timer: Generate FDT timer driver list at compile-time
lib: utils/irqchip: Generate FDT irqchip driver list at compile-time
lib: utils/ipi: Generate FDT ipi driver list at compile-time
lib: utils/i2c: Generate FDT i2c adapter driver list at compile-time
lib: utils/gpio: Generate FDT gpio driver list at compile-time
platform: generic: Generate platform override module list at
compile-time
platform: generic: Move Sifive platform overrides into own directory

 Makefile                                      | 17 +++-
 include/sbi_utils/gpio/fdt_gpio.h             |  2 +
 lib/utils/gpio/fdt_gpio.c                     | 18 ++---
 lib/utils/gpio/fdt_gpio_drivers.carray        |  3 +
 lib/utils/gpio/objects.mk                     |  4 +
 lib/utils/i2c/fdt_i2c.c                       | 14 ++--
 lib/utils/i2c/fdt_i2c_adapter_drivers.carray  |  3 +
 lib/utils/i2c/objects.mk                      |  4 +
 lib/utils/ipi/fdt_ipi.c                       | 12 ++-
 lib/utils/ipi/fdt_ipi_drivers.carray          |  3 +
 lib/utils/ipi/objects.mk                      |  4 +
 lib/utils/irqchip/fdt_irqchip.c               | 16 ++--
 lib/utils/irqchip/fdt_irqchip_drivers.carray  |  3 +
 lib/utils/irqchip/objects.mk                  |  8 ++
 lib/utils/reset/fdt_reset.c                   | 22 ++----
 lib/utils/reset/fdt_reset_drivers.carray      |  3 +
 lib/utils/reset/objects.mk                    | 12 +++
 lib/utils/serial/fdt_serial.c                 | 28 ++-----
 lib/utils/serial/fdt_serial_drivers.carray    |  3 +
 lib/utils/serial/objects.mk                   | 16 ++++
 lib/utils/timer/fdt_timer.c                   | 12 ++-
 lib/utils/timer/fdt_timer_drivers.carray      |  3 +
 lib/utils/timer/objects.mk                    |  4 +
 platform/generic/objects.mk                   |  3 +-
 platform/generic/platform.c                   | 14 ++--
 .../generic/platform_override_modules.carray  |  3 +
 .../{sifive_fu540.c => sifive/fu540.c}        |  0
 .../{sifive_fu740.c => sifive/fu740.c}        |  0
 platform/generic/sifive/objects.mk            |  9 +++
 scripts/carray.sh                             | 77 +++++++++++++++++++
 30 files changed, 224 insertions(+), 96 deletions(-)
 create mode 100644 lib/utils/gpio/fdt_gpio_drivers.carray
 create mode 100644 lib/utils/i2c/fdt_i2c_adapter_drivers.carray
 create mode 100644 lib/utils/ipi/fdt_ipi_drivers.carray
 create mode 100644 lib/utils/irqchip/fdt_irqchip_drivers.carray
 create mode 100644 lib/utils/reset/fdt_reset_drivers.carray
 create mode 100644 lib/utils/serial/fdt_serial_drivers.carray
 create mode 100644 lib/utils/timer/fdt_timer_drivers.carray
 create mode 100644 platform/generic/platform_override_modules.carray
 rename platform/generic/{sifive_fu540.c => sifive/fu540.c} (100%)
 rename platform/generic/{sifive_fu740.c => sifive/fu740.c} (100%)
 create mode 100644 platform/generic/sifive/objects.mk
 create mode 100755 scripts/carray.sh
Signature · 1 line
2.34.1