[PATCH net-next 03/11] net: stmmac: dwmac-qcom-ethqos: Drop an if with an always false condition

[Simon Horman]

On Sun, Apr 02, 2023 at 04:30:17PM +0200, Uwe Kleine-König wrote:
> The remove callback is only ever called after .probe() returned
> successfully. After that get_stmmac_bsp_priv() always return non-NULL.
> 
> Side note: The early exit would also be a bug because the return value
> of qcom_ethqos_remove() is ignored by the device core and the device is
> unbound unconditionally. So exiting early resulted in a dangerous
> resource leak as all devm allocated resources (some memory and the
> register mappings) are freed but the network device stays around.  Using
> the network device afterwards probably oopses.
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>


Reviewed-by: Simon Horman <simon.horman at corigine.com>