On Sun, Jan 3, 2021 at 7:17 PM Samuel Holland <sam...@sholland.org> wrote: > > stmmac_pltfr_remove does three things in one function, making it > inapproprate for unwinding the steps in the probe function. Currently, > a failure before the call to stmmac_dvr_probe would leak OF node > references due to missing a call to stmmac_remove_config_dt. And an > error in stmmac_dvr_probe would cause the driver to attempt to remove a > netdevice that was never added. Fix these by reordering the init and > splitting out the error handling steps. > > Fixes: 9f93ac8d4085 ("net-next: stmmac: Add dwmac-sun8i") > Fixes: 40a1dcee2d18 ("net: ethernet: dwmac-sun8i: Use the correct function in > exit path") > Signed-off-by: Samuel Holland <sam...@sholland.org>
Reviewed-by: Chen-Yu Tsai <w...@csie.org> -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. To view this discussion on the web, visit https://groups.google.com/d/msgid/linux-sunxi/CAGb2v67c-2ZHadh%3DEs6eKyRYOHWi%2BQ5Tf9dxX_dkbK-Z%3DmHs-g%40mail.gmail.com.