mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 02:30:34 +02:00 
			
		
		
		
	rtc: brcmstb-waketimer: allow use as non-wake alarm
The wake interrupt only fires when the system is in a suspend state. Fortunately we have another interrupt that fires in a non-suspend state at the L2 controller UPG_AUX_AON. Add support for this interrupt line so we can use the alarm in a non-wake context. Signed-off-by: Doug Berger <opendmb@gmail.com> Acked-by: Florian Fainelli <f.fainelli@gmail.com> Link: https://lore.kernel.org/r/20230124201430.2502371-3-opendmb@gmail.com Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
This commit is contained in:
		
							parent
							
								
									6cc7a8262b
								
							
						
					
					
						commit
						24304a8715
					
				
					 1 changed files with 52 additions and 3 deletions
				
			
		| 
						 | 
					@ -28,6 +28,7 @@ struct brcmstb_waketmr {
 | 
				
			||||||
	struct device *dev;
 | 
						struct device *dev;
 | 
				
			||||||
	void __iomem *base;
 | 
						void __iomem *base;
 | 
				
			||||||
	unsigned int wake_irq;
 | 
						unsigned int wake_irq;
 | 
				
			||||||
 | 
						unsigned int alarm_irq;
 | 
				
			||||||
	struct notifier_block reboot_notifier;
 | 
						struct notifier_block reboot_notifier;
 | 
				
			||||||
	struct clk *clk;
 | 
						struct clk *clk;
 | 
				
			||||||
	u32 rate;
 | 
						u32 rate;
 | 
				
			||||||
| 
						 | 
					@ -56,6 +57,8 @@ static inline void brcmstb_waketmr_clear_alarm(struct brcmstb_waketmr *timer)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	u32 reg;
 | 
						u32 reg;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (timer->alarm_en && timer->alarm_irq)
 | 
				
			||||||
 | 
							disable_irq(timer->alarm_irq);
 | 
				
			||||||
	timer->alarm_en = false;
 | 
						timer->alarm_en = false;
 | 
				
			||||||
	reg = readl_relaxed(timer->base + BRCMSTB_WKTMR_COUNTER);
 | 
						reg = readl_relaxed(timer->base + BRCMSTB_WKTMR_COUNTER);
 | 
				
			||||||
	writel_relaxed(reg - 1, timer->base + BRCMSTB_WKTMR_ALARM);
 | 
						writel_relaxed(reg - 1, timer->base + BRCMSTB_WKTMR_ALARM);
 | 
				
			||||||
| 
						 | 
					@ -88,7 +91,25 @@ static irqreturn_t brcmstb_waketmr_irq(int irq, void *data)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	struct brcmstb_waketmr *timer = data;
 | 
						struct brcmstb_waketmr *timer = data;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	pm_wakeup_event(timer->dev, 0);
 | 
						if (!timer->alarm_irq)
 | 
				
			||||||
 | 
							pm_wakeup_event(timer->dev, 0);
 | 
				
			||||||
 | 
						return IRQ_HANDLED;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static irqreturn_t brcmstb_alarm_irq(int irq, void *data)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						struct brcmstb_waketmr *timer = data;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						/* Ignore spurious interrupts */
 | 
				
			||||||
 | 
						if (!brcmstb_waketmr_is_pending(timer))
 | 
				
			||||||
 | 
							return IRQ_HANDLED;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (timer->alarm_en) {
 | 
				
			||||||
 | 
							if (!device_may_wakeup(timer->dev))
 | 
				
			||||||
 | 
								writel_relaxed(WKTMR_ALARM_EVENT,
 | 
				
			||||||
 | 
									       timer->base + BRCMSTB_WKTMR_EVENT);
 | 
				
			||||||
 | 
							rtc_update_irq(timer->rtc, 1, RTC_IRQF | RTC_AF);
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	return IRQ_HANDLED;
 | 
						return IRQ_HANDLED;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
| 
						 | 
					@ -114,7 +135,7 @@ static void wktmr_read(struct brcmstb_waketmr *timer,
 | 
				
			||||||
static int brcmstb_waketmr_prepare_suspend(struct brcmstb_waketmr *timer)
 | 
					static int brcmstb_waketmr_prepare_suspend(struct brcmstb_waketmr *timer)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	struct device *dev = timer->dev;
 | 
						struct device *dev = timer->dev;
 | 
				
			||||||
	int ret = 0;
 | 
						int ret;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (device_may_wakeup(dev)) {
 | 
						if (device_may_wakeup(dev)) {
 | 
				
			||||||
		ret = enable_irq_wake(timer->wake_irq);
 | 
							ret = enable_irq_wake(timer->wake_irq);
 | 
				
			||||||
| 
						 | 
					@ -122,9 +143,17 @@ static int brcmstb_waketmr_prepare_suspend(struct brcmstb_waketmr *timer)
 | 
				
			||||||
			dev_err(dev, "failed to enable wake-up interrupt\n");
 | 
								dev_err(dev, "failed to enable wake-up interrupt\n");
 | 
				
			||||||
			return ret;
 | 
								return ret;
 | 
				
			||||||
		}
 | 
							}
 | 
				
			||||||
 | 
							if (timer->alarm_en && timer->alarm_irq) {
 | 
				
			||||||
 | 
								ret = enable_irq_wake(timer->alarm_irq);
 | 
				
			||||||
 | 
								if (ret) {
 | 
				
			||||||
 | 
									dev_err(dev, "failed to enable rtc interrupt\n");
 | 
				
			||||||
 | 
									disable_irq_wake(timer->wake_irq);
 | 
				
			||||||
 | 
									return ret;
 | 
				
			||||||
 | 
								}
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	return ret;
 | 
						return 0;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
/* If enabled as a wakeup-source, arm the timer when powering off */
 | 
					/* If enabled as a wakeup-source, arm the timer when powering off */
 | 
				
			||||||
| 
						 | 
					@ -192,7 +221,11 @@ static int brcmstb_waketmr_alarm_enable(struct device *dev,
 | 
				
			||||||
		    !brcmstb_waketmr_is_pending(timer))
 | 
							    !brcmstb_waketmr_is_pending(timer))
 | 
				
			||||||
			return -EINVAL;
 | 
								return -EINVAL;
 | 
				
			||||||
		timer->alarm_en = true;
 | 
							timer->alarm_en = true;
 | 
				
			||||||
 | 
							if (timer->alarm_irq)
 | 
				
			||||||
 | 
								enable_irq(timer->alarm_irq);
 | 
				
			||||||
	} else if (!enabled && timer->alarm_en) {
 | 
						} else if (!enabled && timer->alarm_en) {
 | 
				
			||||||
 | 
							if (timer->alarm_irq)
 | 
				
			||||||
 | 
								disable_irq(timer->alarm_irq);
 | 
				
			||||||
		timer->alarm_en = false;
 | 
							timer->alarm_en = false;
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -269,6 +302,19 @@ static int brcmstb_waketmr_probe(struct platform_device *pdev)
 | 
				
			||||||
	if (ret < 0)
 | 
						if (ret < 0)
 | 
				
			||||||
		goto err_clk;
 | 
							goto err_clk;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						brcmstb_waketmr_clear_alarm(timer);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						/* Attempt to initialize non-wake irq */
 | 
				
			||||||
 | 
						ret = platform_get_irq(pdev, 1);
 | 
				
			||||||
 | 
						if (ret > 0) {
 | 
				
			||||||
 | 
							timer->alarm_irq = (unsigned int)ret;
 | 
				
			||||||
 | 
							ret = devm_request_irq(dev, timer->alarm_irq, brcmstb_alarm_irq,
 | 
				
			||||||
 | 
									       IRQF_NO_AUTOEN, "brcmstb-waketimer-rtc",
 | 
				
			||||||
 | 
									       timer);
 | 
				
			||||||
 | 
							if (ret < 0)
 | 
				
			||||||
 | 
								timer->alarm_irq = 0;
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	timer->reboot_notifier.notifier_call = brcmstb_waketmr_reboot;
 | 
						timer->reboot_notifier.notifier_call = brcmstb_waketmr_reboot;
 | 
				
			||||||
	register_reboot_notifier(&timer->reboot_notifier);
 | 
						register_reboot_notifier(&timer->reboot_notifier);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -317,6 +363,8 @@ static int brcmstb_waketmr_resume(struct device *dev)
 | 
				
			||||||
		return 0;
 | 
							return 0;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	ret = disable_irq_wake(timer->wake_irq);
 | 
						ret = disable_irq_wake(timer->wake_irq);
 | 
				
			||||||
 | 
						if (timer->alarm_en && timer->alarm_irq)
 | 
				
			||||||
 | 
							disable_irq_wake(timer->alarm_irq);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	brcmstb_waketmr_clear_alarm(timer);
 | 
						brcmstb_waketmr_clear_alarm(timer);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -346,4 +394,5 @@ module_platform_driver(brcmstb_waketmr_driver);
 | 
				
			||||||
MODULE_LICENSE("GPL v2");
 | 
					MODULE_LICENSE("GPL v2");
 | 
				
			||||||
MODULE_AUTHOR("Brian Norris");
 | 
					MODULE_AUTHOR("Brian Norris");
 | 
				
			||||||
MODULE_AUTHOR("Markus Mayer");
 | 
					MODULE_AUTHOR("Markus Mayer");
 | 
				
			||||||
 | 
					MODULE_AUTHOR("Doug Berger");
 | 
				
			||||||
MODULE_DESCRIPTION("Wake-up timer driver for STB chips");
 | 
					MODULE_DESCRIPTION("Wake-up timer driver for STB chips");
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue