Mon, Jan 15, 2024 at 03:46:20PM +0100, Philipp Stanner kirjoitti:
> Thanks to preceding cleanup steps, pcim_release() is now not needed
> anymore and can be replaced by pcim_disable_device(), which is the exact
> counterpart to pcim_enable_device().
> This permits removing further parts of the old devres API.
>
> Replace pcim_release() with pcim_disable_device().
> Remove the now surplus get_dr() function.
...
> + devm_add_action(&pdev->dev, pcim_disable_device, pdev);
No error check?
> + return pci_enable_device(pdev);
Maybe
ret = pci_enable_device(...);
if (ret)
return ret;
return devm_add_action_or_reset(...)?
I could think of side effects of this, so perhaps the commit message and/or
code needs a comment on why the above proposal can _not_ be used?
--
With Best Regards,
Andy Shevchenko
