Re: [PATCH net-next] devlink: Make devlink_register to be void
From: Jakub Kicinski <kuba@kernel.org>
Date: 2021-09-20 20:41:21
Also in:
intel-wired-lan, linux-omap, linux-rdma, linux-staging, lkml
On Mon, 20 Sep 2021 17:41:44 +0300 Leon Romanovsky wrote:
From: Leon Romanovsky <leonro@nvidia.com> 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.