[PATCH net-next] net: ti: icssm-prueth: unwind cleanly in probe()
Jakub Kicinski
kuba at kernel.org
Fri Sep 19 07:04:56 PDT 2025
On Thu, 18 Sep 2025 12:48:26 +0300 Dan Carpenter wrote:
> This error handling triggers a Smatch warning:
>
> drivers/net/ethernet/ti/icssm/icssm_prueth.c:1574 icssm_prueth_probe()
> warn: 'prueth->pru1' is an error pointer or valid
>
> The warning is harmless because the pru_rproc_put() function has an
> IS_ERR_OR_NULL() check built in. However, there is a small bug if
> syscon_regmap_lookup_by_phandle() fails. In that case we should call
> of_node_put() on eth0_node and eth1_node.
>
> It's a little bit easier to re-write this code to only free things which
> we know have been allocated successfully.
icssm maintainers - please review
More information about the linux-arm-kernel
mailing list