Re: [PATCH] irq_work: unhide irq_work_queue_on declaration on non-SMP

2016-02-10 Thread Mark Brown
On Wed, Feb 10, 2016 at 04:27:42PM +0100, Rafael J. Wysocki wrote: > > +#else > > +static inline bool irq_work_queue_on(struct irq_work *work, int cpu) > > +{ > > + return irq_work_queue(work); > > +} > > #endif > I was thinking about this too, but then cpufreq will be the only user of

Re: [PATCH] irq_work: unhide irq_work_queue_on declaration on non-SMP

2016-02-10 Thread Rafael J. Wysocki
On Wed, Feb 10, 2016 at 4:07 PM, Arnd Bergmann wrote: > The cpufreq code uses 'if (IS_ENABLED(CONFIG_SMP))' to check > whether it should queue a task on the local CPU or a remote > one, however the irq_work_queue_on() function is not declared > when CONFIG_SMP is not set: > >

[PATCH] irq_work: unhide irq_work_queue_on declaration on non-SMP

2016-02-10 Thread Arnd Bergmann
The cpufreq code uses 'if (IS_ENABLED(CONFIG_SMP))' to check whether it should queue a task on the local CPU or a remote one, however the irq_work_queue_on() function is not declared when CONFIG_SMP is not set: drivers/cpufreq/cpufreq_governor.c: In function 'gov_queue_irq_work':

Re: [PATCH] irq_work: unhide irq_work_queue_on declaration on non-SMP

2016-02-10 Thread Rafael J. Wysocki
On Wed, Feb 10, 2016 at 4:07 PM, Arnd Bergmann wrote: > The cpufreq code uses 'if (IS_ENABLED(CONFIG_SMP))' to check > whether it should queue a task on the local CPU or a remote > one, however the irq_work_queue_on() function is not declared > when CONFIG_SMP is not set: > >

[PATCH] irq_work: unhide irq_work_queue_on declaration on non-SMP

2016-02-10 Thread Arnd Bergmann
The cpufreq code uses 'if (IS_ENABLED(CONFIG_SMP))' to check whether it should queue a task on the local CPU or a remote one, however the irq_work_queue_on() function is not declared when CONFIG_SMP is not set: drivers/cpufreq/cpufreq_governor.c: In function 'gov_queue_irq_work':

Re: [PATCH] irq_work: unhide irq_work_queue_on declaration on non-SMP

2016-02-10 Thread Mark Brown
On Wed, Feb 10, 2016 at 04:27:42PM +0100, Rafael J. Wysocki wrote: > > +#else > > +static inline bool irq_work_queue_on(struct irq_work *work, int cpu) > > +{ > > + return irq_work_queue(work); > > +} > > #endif > I was thinking about this too, but then cpufreq will be the only user of