From: Simon Glass Date: Sat, 7 Dec 2019 04:41:36 +0000 (-0700) Subject: dm: core: Fix offset_to_ofnode() with invalid offset X-Git-Url: http://git.dujemihanovic.xyz/?a=commitdiff_plain;h=b14c53398683952723540f82027fb0cbfba4f7de;p=u-boot.git dm: core: Fix offset_to_ofnode() with invalid offset If the offset is -1 this function correctly sets up a null ofnode. But if the offset is any other negative number (e.g. -FDT_ERR_BADPATH) then it does the wrong thing. An offset of -1 in ofnode indicates that the ofnode is not valid. Any other negative value is not handled by ofnode_valid(). We could of course change that function, but it seems much better to always use the same value for an invalid node. Fix it by setting the offset to -1 if it is invalid for any reason. Signed-off-by: Simon Glass Reviewed-by: Bin Meng --- diff --git a/include/dm/ofnode.h b/include/dm/ofnode.h index 5c4cbf0998..4282169706 100644 --- a/include/dm/ofnode.h +++ b/include/dm/ofnode.h @@ -118,7 +118,7 @@ static inline ofnode offset_to_ofnode(int of_offset) if (of_live_active()) node.np = NULL; else - node.of_offset = of_offset; + node.of_offset = of_offset >= 0 ? of_offset : -1; return node; }