On Tue, Jan 18, 2022 at 12:18:58AM +0300, Sergey Shtylyov wrote: > After following the call tree of phy_set_max_speed(), it became clear > that this function never returns anything but 0, so we can change its > result type to *void* and drop the result checks from the three drivers > that actually bothered to do it... > > Found by Linux Verification Center (linuxtesting.org) with the SVACE static > analysis tool. > > Signed-off-by: Sergey Shtylyov <s.shtylyov@xxxxxx> Seems reasonable. net-next is closed at the moment, so please repost once it opens. Reviewed-by: Andrew Lunn <andrew@xxxxxxx> Andrew