Commit f3a616e2 authored by Raag Jadav's avatar Raag Jadav Committed by Uwe Kleine-König
Browse files

pwm: lpss: drop redundant runtime PM handles 



We no longer need empty runtime PM handles for PCI devices after
commits [1] and [2]. Drop them and let PCI core take care of power
state transitions.

[1] c5eb1190 ("PCI / PM: Allow runtime PM without callback functions")
[2] fa885b06 ("PCI/PM: Allow runtime PM with no PM callbacks at all")

Reviewed-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarRaag Jadav <raag.jadav@intel.com>
Link: https://lore.kernel.org/r/20240605131533.20037-3-raag.jadav@intel.com


Signed-off-by: default avatarUwe Kleine-König <ukleinek@kernel.org>
parent 07612a76

drivers/pwm/pwm-lpss-pci.c

+0 −22
Original line number Diff line number Diff line
@@ -46,25 +46,6 @@ static void pwm_lpss_remove_pci(struct pci_dev *pdev) 
	pm_runtime_get_sync(&pdev->dev);

}



static int pwm_lpss_runtime_suspend_pci(struct device *dev)

{

	/*

	 * The PCI core will handle transition to D3 automatically. We only

	 * need to provide runtime PM hooks for that to happen.

	 */

	return 0;

}



static int pwm_lpss_runtime_resume_pci(struct device *dev)

{

	return 0;

}



static DEFINE_RUNTIME_DEV_PM_OPS(pwm_lpss_pci_pm,

				 pwm_lpss_runtime_suspend_pci,

				 pwm_lpss_runtime_resume_pci,

				 NULL);



static const struct pci_device_id pwm_lpss_pci_ids[] = {

	{ PCI_VDEVICE(INTEL, 0x0ac8), (unsigned long)&pwm_lpss_bxt_info},

	{ PCI_VDEVICE(INTEL, 0x0f08), (unsigned long)&pwm_lpss_byt_info},
@@ -84,9 +65,6 @@ static struct pci_driver pwm_lpss_driver_pci = { 
	.id_table = pwm_lpss_pci_ids,

	.probe = pwm_lpss_probe_pci,

	.remove = pwm_lpss_remove_pci,

	.driver = {

		.pm = pm_ptr(&pwm_lpss_pci_pm),

	},

};

module_pci_driver(pwm_lpss_driver_pci);