On Friday, October 27, 2017 9:26:10 AM CET Jeffy Chen wrote: > Allow platforms not to provide some of the pci_platform_pm_ops's > callbacks. So? What exactly is wrong with having empty ops in there? Is it really better to have everyone do extra checks every time an op is invoked even when all of the ops are present? > Also change the return value from -ENOSYS to -ENODEV for: > warning: drivers/pci/pci.c,594: ENOSYS means 'invalid syscall nr' and nothing else Moving stuff around and changing it at the same time is a bad idea. Change it in one patch and move it around in another one and you'll be less likely to make a mistake. Moreover, reviewing it will be easier too, IMO. Thanks, Rafael
