dm: core: use s32 instead of int for dev_read_s32_default
authorQuentin Schulz <quentin.schulz@cherry.de>
Wed, 28 May 2025 13:06:19 +0000 (15:06 +0200)
committerTom Rini <trini@konsulko.com>
Wed, 11 Jun 2025 19:31:35 +0000 (13:31 -0600)
commitad03050e26b1a61b3fe10a645a36da11652355e7
tree07000a40706477d357dc39d237f66ed6bc83053e
parentb4b66e2f298e10d53749ce8d3ca086acbfc62f6a
dm: core: use s32 instead of int for dev_read_s32_default

dev_read_s32_default is for getting an s32 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 s32/u32.

s32 and u32 being the same size, dev_read_s32* functions calling
ofnode_read_u32_default shouldn't be an issue (at the type level at
least) as the information will be stored appropriately in 4B regardless
of the sign.

This incidentally matches all other dev_read_*_default functions.

Fixes: a1b17e4f4c82 ("dm: core: Add a function to read into a unsigned int")
Signed-off-by: Quentin Schulz <quentin.schulz@cherry.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
drivers/core/read.c
include/dm/read.h