summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorQuentin Schulz <[email protected]>2025-05-28 15:06:18 +0200
committerTom Rini <[email protected]>2025-06-11 13:31:35 -0600
commitb4b66e2f298e10d53749ce8d3ca086acbfc62f6a (patch)
tree65a1d6974b791029b6ef1a6649f8121bc292f338 /include
parente04d137231f2e9e14708a32448c879125b8e308f (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 'include')
-rw-r--r--include/dm/read.h6
1 files changed, 3 insertions, 3 deletions
diff --git a/include/dm/read.h b/include/dm/read.h
index 894bc698bb4..87017f07621 100644
--- a/include/dm/read.h
+++ b/include/dm/read.h
@@ -90,8 +90,8 @@ int dev_read_u32(const struct udevice *dev, const char *propname, u32 *outp);
* @def: default value to return if the property has no value
* Return: property value, or @def if not found
*/
-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);
/**
* dev_read_u32_index() - read an indexed 32-bit integer from a device's DT
@@ -896,7 +896,7 @@ static inline int dev_read_u32(const struct udevice *dev,
}
static inline int dev_read_u32_default(const struct udevice *dev,
- const char *propname, int def)
+ const char *propname, u32 def)
{
return ofnode_read_u32_default(dev_ofnode(dev), propname, def);
}