On Sun, May 19, 2019 at 11:16:27PM +0300, Kamal Heib wrote: > The return value from ib_device_check_mandatory() is always 0 - change > it to be void and rename the function to emphasize the use of it, which > is setting the kverbs_provider flag. Actually check_mandatory is the real usage of that function and kverbs_provider is extra granularity. So yes to "void", no to function name change. Thanks