starfive-tech/opensbi.git/lib/utils/fdt, branch master

lib: utils/fdt: Allocate fdt_pmu_evt_select on the heap

2025-02-19T12:55:17+00:00

This reduces .bss size by 8 KiB, and should reduce overall memory usage since most platforms will have significantly fewer than 512 entries in this table. At the same time, it removes the fixed table size limit. Since the table is only used within fdt_pmu.c, instead of updating the extern declaration, make the table local to this file. Signed-off-by: Samuel Holland Reviewed-by: Anup Patel

lib: utils/fdt: Remove redundant PMU property length checks

2025-02-19T12:52:52+00:00

If a property value is too small, len will be zero after the division on the next line, so the property will be ignored. This is the same behavior as when the length check fails. Furthermore, the first two length checks were already ineffectual, because each item in those arrays is 12 bytes long, not 8. Signed-off-by: Samuel Holland Reviewed-by: Anup Patel

lib: utils: Initialize miscellaneous drivers in one pass

2025-02-12T16:09:25+00:00

For driver subsystems that are not tightly integrated into the OpenSBI init sequence, it is not important that the drivers are initialized in any particular order. By putting all of these drivers in one array, they can all be initialized with a single pass through the devicetree. This saves about 10 ms of boot time on HiFive Unmatched. Signed-off-by: Samuel Holland Reviewed-by: Anup Patel

lib: utils/fdt: Remove fdt_find_match()

2025-02-12T15:57:54+00:00

Now that all drivers are using the fdt_driver functions for initialization, this function is unused and can be removed. Signed-off-by: Samuel Holland Reviewed-by: Anup Patel

lib: utils:Check that hartid is valid

2025-02-12T03:54:09+00:00

It is possible that hartid may not be sequential and it should not be validated against SBI_HARTMASK_MAX_BITS. Instead we should check the index of the hartid, hart index, against SBI_HARTMASK_MAX_BITS. Signed-off-by: Raj Vishwanathan Reviewed-by: Anup Patel

lib: utils: Make the enforce permission bit configurable from DT

2025-02-11T12:26:48+00:00

The domain_support.md documentation states that the enforce permission bit (BIT[6]) could be set in the "regions" property of a domain instance DT node. However, this bit is masked in the current implementation. This patch unmasks the bit to make it configurable from DT. Signed-off-by: Chao Du Reviewed-by: Xiang W Reviewed-by: Anup Patel

lib: utils/fdt: update fdt_parse_aplic_node()

2025-02-11T11:28:24+00:00

1. Initialize struct imsic_data imsic to 0 at definition to prevent the use of uninitialized memory, ensuring the variable starts with known values. 2. Remove the redundant memset call on the "aplic" parameter since the memory for aplic is allocated using sbi_zalloc() by the caller irqchip_aplic_cold_init(), which guarantees it is already set to 0. Signed-off-by: Huang Borong Reviewed-by: Anup Patel

lib: utils: Mark RPMI drivers as experimental

2024-12-21T12:52:47+00:00

These drivers were merged on an experimental basis without the RPMI specification being frozen. As a result, they may not be compatible with the frozen version of the RPMI protocol. Additionally, their devicetree bindings have not been reviewed and are subject to change. Warn the user that these drivers make no compatibility guarantees, and that their behavior and devicetree bindings may change incompatibly in future versions of OpenSBI. Signed-off-by: Samuel Holland Reviewed-by: Anup Patel

lib: utils/fdt: Allow dynamic registration of FDT fixup callbacks

2024-12-06T03:56:38+00:00

It should possible to fixup FDT from any part of OpenSBI so add fdt_register_general_fixup() which allows dynamic registration of FDT fixup callbacks. Signed-off-by: Anup Patel

lib: utils/fdt: Add helpers for generic driver initialization

2024-11-28T11:43:05+00:00

Currently, each driver subsystem contains its own code for matching drivers against the platform's devicetree blob. This bloats firmware size because the several FDT scanning loops are almost exact copies of each other, and is confusing because the loops do have some subtle differences. Furthermore, the existing match algorithm is inefficient: it scans the FDT structure separately for each driver in the list. A faster algorithm scans the FDT blob only once, matching all drivers in the list for each `compatible` property seen. Add new helpers implementing this faster algorithm. Since they must iterate through the list of drivers, the driver structure cannot be opaque. However, since the driver list is an array of pointers, the `struct fdt_driver` can be embedded in a subsystem-specific driver structure if needed. These three helpers cover all existing use cases for driver initialization within OpenSBI. An additional benefit of centralized driver initialization is the consistent use of fdt_node_is_enabled(). Signed-off-by: Samuel Holland Reviewed-by: Anup Patel