mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] irqchip: st: avoid compile error when PM is disabled
@ 2017-01-04  3:46 Chris Packham
  2017-01-09 14:16 ` Thomas Gleixner
  0 siblings, 1 reply; 2+ messages in thread
From: Chris Packham @ 2017-01-04  3:46 UTC (permalink / raw)
  To: tglx, jason, marc.zyngier; +Cc: lee.jones, linux-kernel, Chris Packham

When power management is disabled in the kernel configuration compiler
will complain that st_irq_syscfg_resume is defined but not used. Wrap
the function definition with CONFIG_PM_SLEEP as per other users of
SIMPLE_DEV_PM_OPS.

Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
 drivers/irqchip/irq-st.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/irqchip/irq-st.c b/drivers/irqchip/irq-st.c
index 9af48a85c16f..afec640b6d65 100644
--- a/drivers/irqchip/irq-st.c
+++ b/drivers/irqchip/irq-st.c
@@ -180,6 +180,7 @@ static int st_irq_syscfg_probe(struct platform_device *pdev)
 	return st_irq_syscfg_enable(pdev);
 }
 
+#ifdef CONFIG_PM_SLEEP
 static int st_irq_syscfg_resume(struct device *dev)
 {
 	struct st_irq_syscfg *ddata = dev_get_drvdata(dev);
@@ -187,6 +188,7 @@ static int st_irq_syscfg_resume(struct device *dev)
 	return regmap_update_bits(ddata->regmap, ddata->syscfg,
 				  ST_A9_IRQ_MASK, ddata->config);
 }
+#endif
 
 static SIMPLE_DEV_PM_OPS(st_irq_syscfg_pm_ops, NULL, st_irq_syscfg_resume);
 
-- 
2.11.0.24.ge6920cf

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] irqchip: st: avoid compile error when PM is disabled
  2017-01-04  3:46 [PATCH] irqchip: st: avoid compile error when PM is disabled Chris Packham
@ 2017-01-09 14:16 ` Thomas Gleixner
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Gleixner @ 2017-01-09 14:16 UTC (permalink / raw)
  To: Chris Packham; +Cc: jason, marc.zyngier, lee.jones, linux-kernel

On Wed, 4 Jan 2017, Chris Packham wrote:

> When power management is disabled in the kernel configuration compiler
> will complain that st_irq_syscfg_resume is defined but not used. Wrap
> the function definition with CONFIG_PM_SLEEP as per other users of
> SIMPLE_DEV_PM_OPS.

That's already fixed by:

f357563f958d: irqchip/st: Mark st_irq_syscfg_resume() __maybe_unused

Thanks,

	tglx

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-01-09 14:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-04  3:46 [PATCH] irqchip: st: avoid compile error when PM is disabled Chris Packham
2017-01-09 14:16 ` Thomas Gleixner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®