Messages in this thread | | | Date | Tue, 21 Sep 2021 09:02:23 +0200 | From | Jiri Pirko <> | Subject | Re: [PATCH net-next] devlink: Make devlink_register to be void |
| |
Mon, Sep 20, 2021 at 10:39:15PM CEST, kuba@kernel.org wrote: >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.
Well, that was the original reason why I made it to return int, so the drivers are prepared. But truth is that given the time this is on and the need to return int never really materialized, I tend to ack with the cleanup.
| |