Message ID | 20230716155346.11241-1-marex@denx.de |
---|---|
State | New |
Delegated to: | Marek Vasut |
Headers | show |
Series | usb: mtu3: Fix device_find_first_child() return value handling | expand |
diff --git a/drivers/usb/mtu3/mtu3_plat.c b/drivers/usb/mtu3/mtu3_plat.c index b1b22b9357c..7eb8360eada 100644 --- a/drivers/usb/mtu3/mtu3_plat.c +++ b/drivers/usb/mtu3/mtu3_plat.c @@ -161,10 +161,10 @@ static int get_ssusb_rscs(struct udevice *dev, struct ssusb_mtk *ssusb) return -ENODEV; } - ret = device_find_first_child(dev, &child); - if (ret || !child) { - dev_err(dev, "failed to get child %d!\n", ret); - return ret; + device_find_first_child(dev, &child); + if (!child) { + dev_err(dev, "failed to get child!\n"); + return -ENODEV; } ssusb->mac_base = devfdt_remap_addr_name(child, "mac");
This function only ever returns 0, but may not assign the second parameter. Check the second parameter instead. Reported-by: Jonas Karlman <jonas@kwiboo.se> Signed-off-by: Marek Vasut <marex@denx.de> --- Cc: Chunfeng Yun <chunfeng.yun@mediatek.com> Cc: GSS_MTK_Uboot_upstream <GSS_MTK_Uboot_upstream@mediatek.com> Cc: Marek Vasut <marex@denx.de> Cc: Ryder Lee <ryder.lee@mediatek.com> Cc: Weijie Gao <weijie.gao@mediatek.com> --- drivers/usb/mtu3/mtu3_plat.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)