[PATCH] pci: Fix device_find_first_child() return value handling

Tom Rini trini at konsulko.com
Tue Aug 15 16:42:59 CEST 2023


On Sun, Jul 16, 2023 at 05:53:24PM +0200, Marek Vasut wrote:

> This function only ever returns 0, but may not assign the second
> parameter. Same thing for device_find_next_child(). Do not assign
> ret to stop proliferation of this misuse.
> 
> Reported-by: Jonas Karlman <jonas at kwiboo.se>
> Signed-off-by: Marek Vasut <marex at denx.de>
> Reviewed-by: Simon Glass <sjg at chromium.org>

Applied to u-boot/next, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230815/18660bf7/attachment.sig>


More information about the U-Boot mailing list