[PATCH 2/2] dm: core: use s32 instead of int for dev_read_s32_default

Simon Glass sjg at chromium.org
Wed May 28 16:32:18 CEST 2025


On Wed, 28 May 2025 at 14:06, Quentin Schulz <foss+uboot at 0leil.net> wrote:
>
> From: Quentin Schulz <quentin.schulz at cherry.de>
>
> 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 at cherry.de>
> ---
>  drivers/core/read.c | 4 ++--
>  include/dm/read.h   | 8 ++++----
>  2 files changed, 6 insertions(+), 6 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list