Message ID | 1432918442-2043-1-git-send-email-tony@atomide.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Fri, May 29, 2015 at 09:54:02AM -0700, Tony Lindgren wrote: > Looks like I only built test the dev_pm_set_wake_irq and not the > dev_pm_set_dedicated_wake_irq case on x86. > > Turns out there's a typo for the dev_pm_set_dedicated_wake_irq > prototype that causes a build error if CONFIG_COMPILE_TEST > and CONFIG_MMC_OMAP_HS are selected. > > Cc: Stephen Rothwell <sfr@canb.auug.org.au> > Cc: Ulf Hansson <ulf.hansson@linaro.org> > Reported-by: Jim Davis <jim.epost@gmail.com> > Signed-off-by: Tony Lindgren <tony@atomide.com> cool: Reviewed-by: Felipe Balbi <balbi@ti.com> > --- > include/linux/pm_wakeirq.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/include/linux/pm_wakeirq.h b/include/linux/pm_wakeirq.h > index 4046fa1..cd5b62d 100644 > --- a/include/linux/pm_wakeirq.h > +++ b/include/linux/pm_wakeirq.h > @@ -30,8 +30,7 @@ static inline int dev_pm_set_wake_irq(struct device *dev, int irq) > return 0; > } > > -static inline int dev_pm_set_dedicated__wake_irq(struct device *dev, > - int irq) > +static inline int dev_pm_set_dedicated_wake_irq(struct device *dev, int irq) > { > return 0; > } > -- > 2.1.4 >
On Friday, May 29, 2015 12:18:37 PM Felipe Balbi wrote: > > --2fjX3cMESU3XgGmZ > Content-Type: text/plain; charset=us-ascii > Content-Disposition: inline > Content-Transfer-Encoding: quoted-printable > > On Fri, May 29, 2015 at 09:54:02AM -0700, Tony Lindgren wrote: > > Looks like I only built test the dev_pm_set_wake_irq and not the > > dev_pm_set_dedicated_wake_irq case on x86. > >=20 > > Turns out there's a typo for the dev_pm_set_dedicated_wake_irq > > prototype that causes a build error if CONFIG_COMPILE_TEST > > and CONFIG_MMC_OMAP_HS are selected. > >=20 > > Cc: Stephen Rothwell <sfr@canb.auug.org.au> > > Cc: Ulf Hansson <ulf.hansson@linaro.org> > > Reported-by: Jim Davis <jim.epost@gmail.com> > > Signed-off-by: Tony Lindgren <tony@atomide.com> > > cool: > > Reviewed-by: Felipe Balbi <balbi@ti.com> Applied, thanks!
diff --git a/include/linux/pm_wakeirq.h b/include/linux/pm_wakeirq.h index 4046fa1..cd5b62d 100644 --- a/include/linux/pm_wakeirq.h +++ b/include/linux/pm_wakeirq.h @@ -30,8 +30,7 @@ static inline int dev_pm_set_wake_irq(struct device *dev, int irq) return 0; } -static inline int dev_pm_set_dedicated__wake_irq(struct device *dev, - int irq) +static inline int dev_pm_set_dedicated_wake_irq(struct device *dev, int irq) { return 0; }
Looks like I only built test the dev_pm_set_wake_irq and not the dev_pm_set_dedicated_wake_irq case on x86. Turns out there's a typo for the dev_pm_set_dedicated_wake_irq prototype that causes a build error if CONFIG_COMPILE_TEST and CONFIG_MMC_OMAP_HS are selected. Cc: Stephen Rothwell <sfr@canb.auug.org.au> Cc: Ulf Hansson <ulf.hansson@linaro.org> Reported-by: Jim Davis <jim.epost@gmail.com> Signed-off-by: Tony Lindgren <tony@atomide.com> --- include/linux/pm_wakeirq.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)