| Message ID | 20250320105721.10789-1-tiwai@suse.de (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | ALSA: oxygen: Fix dependency on CONFIG_PM_SLEEP | expand |
diff --git a/sound/pci/oxygen/oxygen.c b/sound/pci/oxygen/oxygen.c index 6d0deea47ee6..ff7439634d76 100644 --- a/sound/pci/oxygen/oxygen.c +++ b/sound/pci/oxygen/oxygen.c @@ -855,7 +855,7 @@ static struct pci_driver oxygen_driver = { .id_table = oxygen_ids, .probe = generic_oxygen_probe, .driver = { - .pm = pm_ptr(&oxygen_pci_pm), + .pm = pm_sleep_ptr(&oxygen_pci_pm), }, }; diff --git a/sound/pci/oxygen/se6x.c b/sound/pci/oxygen/se6x.c index c1bb9dd1cd54..9d009015d97e 100644 --- a/sound/pci/oxygen/se6x.c +++ b/sound/pci/oxygen/se6x.c @@ -138,7 +138,7 @@ static struct pci_driver se6x_driver = { .id_table = se6x_ids, .probe = se6x_probe, .driver = { - .pm = pm_ptr(&oxygen_pci_pm), + .pm = pm_sleep_ptr(&oxygen_pci_pm), }, .shutdown = oxygen_pci_shutdown, }; diff --git a/sound/pci/oxygen/virtuoso.c b/sound/pci/oxygen/virtuoso.c index 0bfe6d8dde3e..ded62199da7f 100644 --- a/sound/pci/oxygen/virtuoso.c +++ b/sound/pci/oxygen/virtuoso.c @@ -83,7 +83,7 @@ static struct pci_driver xonar_driver = { .id_table = xonar_ids, .probe = xonar_probe, .driver = { - .pm = pm_ptr(&oxygen_pci_pm), + .pm = pm_sleep_ptr(&oxygen_pci_pm), }, .shutdown = oxygen_pci_shutdown, };
The conversion to EXPORT_SIMPLE_DEV_PM_OPS() also replaced the pm ops assignment with pm_ptr() macro, but this made difference from the original code; it had conditional with ifdef CONFIG_PM_SLEEP, while we have now with CONFIG_PM, instead. This seems causing build errors with randomconfig. For fixing the inconsistency, replace pm_ptr() with pm_sleep_ptr(). Reported-by: kernel test robot <lkp@intel.com> Closes: https://lore.kernel.org/oe-kbuild-all/202503201853.7kB0BPRw-lkp@intel.com/ Signed-off-by: Takashi Iwai <tiwai@suse.de> --- sound/pci/oxygen/oxygen.c | 2 +- sound/pci/oxygen/se6x.c | 2 +- sound/pci/oxygen/virtuoso.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-)