On Mon, 20 Sep 2021 17:41:44 +0300 Leon Romanovsky wrote: > From: Leon Romanovsky <leonro@xxxxxxxxxx> > > devlink_register() can't fail and always returns success, but all drivers > are obligated to check returned status anyway. This adds a lot of boilerplate > code to handle impossible flow. > > Make devlink_register() void and simplify the drivers that use that > API call. Unlike unused functions bringing back error handling may be non-trivial. I'd rather you deferred such cleanups until you're ready to post your full rework and therefore give us some confidence the revert will not be needed.
