Messages in this thread | | | From | "Rafael J. Wysocki" <> | Subject | Re: [PATCH] PM / sleep: Fix broken builds without CONFIG_PM_SLEEP_DEBUG | Date | Wed, 16 Sep 2015 04:48:37 +0200 |
| |
On Wednesday, September 16, 2015 07:30:23 AM Viresh Kumar wrote: > On 16-09-15, 03:43, Rafael J. Wysocki wrote: > > On Tuesday, September 15, 2015 01:42:21 PM Viresh Kumar wrote: > > > The variable 'wakeup_irq' is defined within #ifdef CONFIG_PM_SLEEP_DEBUG > > > and used outside of it. And that breaks kernel build: > > > > > > /home/viresh/linux/drivers/base/power/wakeup.c:871: undefined reference to `wakeup_irq' > > > /home/viresh/drivers/base/power/wakeup.c:871: undefined reference to `wakeup_irq' > > > > > > Fix it by defining the variable outside of the ifdef. > > > > > > Fixes: d1e59c235322 ("PM / sleep: Report interrupt that caused system wakeup") > > > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > > > > I've applied the v11 of the Alexandra's patch that doesn't have this problem AFAICS. > > For the record, as we have talked on IRC, even the v11 patch suffers > from this problem and you will be fixing it.
Yes, it was slightly messed up. Should be better now, though.
Thanks, Rafael
| |