22 May
2019
22 May
'19
3:21 p.m.
On Fri, 10 May 2019 at 11:48, Trent Piepho tpiepho@impinj.com wrote:
It was returning an int, which doesn't work if the u32 it is reading, or the default value, will overflow a signed int.
While it could be made to work, when using a C standard/compiler where casting negative signed values to unsigned has a defined behavior, combined with careful casting, it seems obvious one is meant to use ofnode_read_s32_default() with signed values.
Cc: Simon Glass sjg@chromium.org Signed-off-by: Trent Piepho tpiepho@impinj.com
drivers/core/ofnode.c | 2 +- include/dm/ofnode.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
Applied to u-boot-dm, thanks!