diff options
| author | Quentin Schulz <[email protected]> | 2025-05-28 15:06:18 +0200 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-06-11 13:31:35 -0600 |
| commit | b4b66e2f298e10d53749ce8d3ca086acbfc62f6a (patch) | |
| tree | 65a1d6974b791029b6ef1a6649f8121bc292f338 /drivers/core | |
| parent | e04d137231f2e9e14708a32448c879125b8e308f (diff) | |
dm: core: use u32 instead of int for dev_read_u32_default
dev_read_u32_default is for getting a u32 from a Device Tree property
and allows to take a default value if that property is missing.
Considering it calls ofnode_read_u32_default which takes a u32 and
returns a u32, it should do the same instead of using an int, especially
considering that int size is typically architecture-specific, as opposed
to u32.
This incidentally matches all other dev_read_*_default functions (except
dev_read_s32_default which will be tackled in the next commit).
Fixes: 47a0fd3bad38 ("dm: core: Implement live tree 'read' functions")
Signed-off-by: Quentin Schulz <[email protected]>
Reviewed-by: Simon Glass <[email protected]>
Diffstat (limited to 'drivers/core')
| -rw-r--r-- | drivers/core/read.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/core/read.c b/drivers/core/read.c index 55c19f335ae..981204bffe4 100644 --- a/drivers/core/read.c +++ b/drivers/core/read.c @@ -38,8 +38,8 @@ int dev_read_u32(const struct udevice *dev, const char *propname, u32 *outp) return ofnode_read_u32(dev_ofnode(dev), propname, outp); } -int dev_read_u32_default(const struct udevice *dev, const char *propname, - int def) +u32 dev_read_u32_default(const struct udevice *dev, const char *propname, + u32 def) { return ofnode_read_u32_default(dev_ofnode(dev), propname, def); } |
