[0/6] Deduplicate driver initialization code

Message ID	20241105042240.2994849-1-samuel.holland@sifive.com
Headers	show Return-Path: <opensbi-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org> From: Samuel Holland <samuel.holland@sifive.com> To: opensbi@lists.infradead.org Subject: [PATCH 0/6] Deduplicate driver initialization code Date: Mon, 4 Nov 2024 20:20:02 -0800 Message-ID: <20241105042240.2994849-1-samuel.holland@sifive.com> MIME-Version: 1.0 preview: Upstreaming support for some of SiFive's security and power management features will require adding some new driver subsystems (e.g. caches, WorldGuard gadgets, power management domains, etc.). To avo [...] Content analysis details: (-2.1 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at https://www.dnswl.org/, no trust [2607:f8b0:4864:20:0:0:0:529 listed in] [list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record -0.1 DKIM_VALID_EF Message has a valid DKIM or DK signature from envelope-from domain -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Precedence: list Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "opensbi" <opensbi-bounces@lists.infradead.org> Errors-To: opensbi-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org
Series	Deduplicate driver initialization code \| expand [0/6] Deduplicate driver initialization code [1/6] lib: utils/gpio: Remove fdt_gpio_driver() function [2/6] treewide: Make carray arrays const and NULL-terminated [3/6] lib: utils/fdt: Add helpers for generic driver initialization [4/6] lib: utils/i2c: Use fdt_driver for initialization [5/6] lib: utils/reset: Use fdt_driver for initialization [6/6] lib: utils/serial: Use fdt_driver for initialization

Message ID

20241105042240.2994849-1-samuel.holland@sifive.com

Headers

From: Samuel Holland <samuel.holland@sifive.com>
To: opensbi@lists.infradead.org
Subject: [PATCH 0/6] Deduplicate driver initialization code
Date: Mon,  4 Nov 2024 20:20:02 -0800
Message-ID: <20241105042240.2994849-1-samuel.holland@sifive.com>
MIME-Version: 1.0
Precedence: list
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: "opensbi" <opensbi-bounces@lists.infradead.org>
Errors-To: opensbi-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org

Series

Deduplicate driver initialization code | expand

Message

Samuel Holland Nov. 5, 2024, 4:20 a.m. UTC

Upstreaming support for some of SiFive's security and power management
features will require adding some new driver subsystems (e.g. caches,
WorldGuard gadgets, power management domains, etc.). To avoid copy-and-
pasting the FDT-based driver matching code to each new subsystem, this
series refactors it into some efficient and easily-reusable helper
functions.

This series only converts three existing subsystems to use the new
helpers, but the remaining subsystems are trivial to convert once the
corresponding patch series I have already sent is applied:
 - gpio: http://lists.infradead.org/pipermail/opensbi/2024-September/007340.html
 - ipi: http://lists.infradead.org/pipermail/opensbi/2024-October/007546.html
 - irqchip: http://lists.infradead.org/pipermail/opensbi/2024-November/007555.html
 - regmap: http://lists.infradead.org/pipermail/opensbi/2024-September/007340.html
 - timer: http://lists.infradead.org/pipermail/opensbi/2024-September/007332.html


Samuel Holland (6):
  lib: utils/gpio: Remove fdt_gpio_driver() function
  treewide: Make carray arrays const and NULL-terminated
  lib: utils/fdt: Add helpers for generic driver initialization
  lib: utils/i2c: Use fdt_driver for initialization
  lib: utils/reset: Use fdt_driver for initialization
  lib: utils/serial: Use fdt_driver for initialization

 include/sbi_utils/fdt/fdt_driver.h           | 59 +++++++++++++++
 include/sbi_utils/i2c/fdt_i2c.h              |  8 +-
 include/sbi_utils/reset/fdt_reset.h          | 15 +---
 include/sbi_utils/serial/fdt_serial.h        |  6 +-
 lib/sbi/sbi_ecall.c                          |  5 +-
 lib/sbi/tests/sbi_unit_test.c                |  5 +-
 lib/utils/fdt/fdt_driver.c                   | 80 ++++++++++++++++++++
 lib/utils/fdt/objects.mk                     |  1 +
 lib/utils/gpio/fdt_gpio.c                    | 22 +-----
 lib/utils/i2c/fdt_i2c.c                      | 31 +-------
 lib/utils/i2c/fdt_i2c_adapter_drivers.carray |  2 +-
 lib/utils/i2c/fdt_i2c_dw.c                   |  2 +-
 lib/utils/i2c/fdt_i2c_sifive.c               |  2 +-
 lib/utils/ipi/fdt_ipi.c                      |  5 +-
 lib/utils/irqchip/fdt_irqchip.c              |  5 +-
 lib/utils/regmap/fdt_regmap.c                |  5 +-
 lib/utils/reset/fdt_reset.c                  | 37 +--------
 lib/utils/reset/fdt_reset_atcwdt200.c        |  2 +-
 lib/utils/reset/fdt_reset_drivers.carray     |  2 +-
 lib/utils/reset/fdt_reset_gpio.c             |  4 +-
 lib/utils/reset/fdt_reset_htif.c             |  2 +-
 lib/utils/reset/fdt_reset_sg2042_hwmon_mcu.c |  2 +-
 lib/utils/reset/fdt_reset_sunxi_wdt.c        |  2 +-
 lib/utils/reset/fdt_reset_syscon.c           |  4 +-
 lib/utils/serial/fdt_serial.c                | 53 ++-----------
 lib/utils/serial/fdt_serial_cadence.c        |  2 +-
 lib/utils/serial/fdt_serial_drivers.carray   |  2 +-
 lib/utils/serial/fdt_serial_gaisler.c        |  2 +-
 lib/utils/serial/fdt_serial_htif.c           |  2 +-
 lib/utils/serial/fdt_serial_litex.c          |  2 +-
 lib/utils/serial/fdt_serial_renesas_scif.c   |  2 +-
 lib/utils/serial/fdt_serial_shakti.c         |  2 +-
 lib/utils/serial/fdt_serial_sifive.c         |  2 +-
 lib/utils/serial/fdt_serial_uart8250.c       |  2 +-
 lib/utils/serial/fdt_serial_xlnx_uartlite.c  |  2 +-
 lib/utils/timer/fdt_timer.c                  |  5 +-
 platform/generic/platform.c                  |  5 +-
 platform/generic/sifive/fu740.c              |  9 ++-
 platform/generic/starfive/jh7110.c           |  9 ++-
 scripts/carray.sh                            |  7 +-
 40 files changed, 211 insertions(+), 205 deletions(-)
 create mode 100644 include/sbi_utils/fdt/fdt_driver.h
 create mode 100644 lib/utils/fdt/fdt_driver.c