On 8/20/24 01:51, Philipp Stanner wrote: > pcim_release_region() is the managed counterpart of > pci_release_region(). It can be useful in some cases where drivers want > to manually release a requested region before the driver's remove() > callback is invoked. > > Make pcim_release_region() a public function. > > Signed-off-by: Philipp Stanner <pstanner@xxxxxxxxxx> Looks fine to me. But I think this should be squashed with patch 2 (I do not see the point of having 2 patches to export 2 functions that are complementary). Either way: Reviewed-by: Damien Le Moal <dlemoal@xxxxxxxxxx> -- Damien Le Moal Western Digital Research