diff options
| author | Marek Vasut <[email protected]> | 2025-11-21 17:17:37 +0100 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-12-27 09:01:05 -0600 |
| commit | 3c43ca3025ef424a484101e265b8427e3a364303 (patch) | |
| tree | 3a4215d132f6a4316392f7444636d3d79f175040 /scripts/basic | |
| parent | f24a2124d26eb6b1aab3b20923359ede80105a8b (diff) | |
clk: fixed_rate: Avoid calling dev_read_*() if CONFIG_OF_PLATDATA=y
If CONFIG_OF_PLATDATA=y , then the udevice has no valid OF node associated
with it and ofnode_valid(node) evaluates to 0. The dev_read_u32_default()
call ultimately reaches ofnode_read_u32_index() which invokes fdt_getprop()
and passes result of ofnode_to_offset(node) as an offset parameter into it.
The ofnode_to_offset(node) returns -1 for invalid node, which leads to an
fdt_getprop(..., -1, ...) invocation, which will crash sandbox with SIGSEGV
because libfdt can not handle negative node offsets without full tree check,
which U-Boot inhibits to keep size lower.
Add dev_has_ofnode(dev) check and do not assign clock rate in case the
device has no valid node associated with it, and do not call any of the
dev_read_*() functions for devices without valid nodes.
Signed-off-by: Marek Vasut <[email protected]>
Diffstat (limited to 'scripts/basic')
0 files changed, 0 insertions, 0 deletions
