On Tue, Mar 10, 2026 at 12:48:26PM +0100, Philipp Hahn wrote: > While doing some static code analysis I stumbled over a common pattern, > where IS_ERR() is combined with a NULL check. For that there is > IS_ERR_OR_NULL().
One thing you need to check for each of these cases is whether the tests are actually correct. There are certainly cases amongst those that you have identified where the check for NULL is redundant. For example, get_phy_device() never returns NULL, yet in your netdev patch, you have at least one instance where the return value of get_phy_device() is checked for NULL and IS_ERR() which you then turn into IS_ERR_OR_NULL(). Instead, the NULL check should be dropped (as a separate patch.) -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!
