PM / Wakeirq: Fix typo in prototype for dev_pm_set_dedicated_wake_irq
diff mbox

Message ID 1432918442-2043-1-git-send-email-tony@atomide.com
State New
Headers show

Commit Message

Tony Lindgren May 29, 2015, 4:54 p.m. UTC
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(-)

Comments

Felipe Balbi May 29, 2015, 5:18 p.m. UTC | #1
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
>
Rafael J. Wysocki May 30, 2015, 1:16 a.m. UTC | #2
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!

Patch
diff mbox

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;
 }