public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 2.6.23-rc1] PM: fix compiler error of PPC dart_iommu
@ 2007-07-24 11:20 Ryusuke Konishi
  2007-07-24 21:19 ` Rafael J. Wysocki
  0 siblings, 1 reply; 2+ messages in thread
From: Ryusuke Konishi @ 2007-07-24 11:20 UTC (permalink / raw)
  To: Pavel Machek, Rafael J. Wysocki
  Cc: moriai.satoshi, Andrew Morton, Johannes Berg, linux-pm

A dummy inline function of register_nosave_region_late was accidentally
removed by the recent PM patch that introduced suspend notifiers.
This elimination causes the following compiler error on PPC machines.

  CC      arch/powerpc/sysdev/dart_iommu.o
arch/powerpc/sysdev/dart_iommu.c: In function 'iommu_init_late_dart':
arch/powerpc/sysdev/dart_iommu.c:376: error: implicit declaration of function 
'register_nosave_region_late'
make[1]: *** [arch/powerpc/sysdev/dart_iommu.o] Error 1
make: *** [arch/powerpc/sysdev] Error 2

This patch fixes the problem.

Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
---
 include/linux/suspend.h |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/include/linux/suspend.h b/include/linux/suspend.h
index e8e6da3..618f93c 100644
--- a/include/linux/suspend.h
+++ b/include/linux/suspend.h
@@ -125,6 +125,9 @@ static inline int unregister_pm_notifier(struct notifier_block *nb)
 static inline void register_nosave_region(unsigned long b, unsigned long e)
 {
 }
+static inline void register_nosave_region_late(unsigned long b, unsigned long e)
+{
+}
 #endif
 
 #endif /* _LINUX_SWSUSP_H */

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 2.6.23-rc1] PM: fix compiler error of PPC dart_iommu
  2007-07-24 11:20 [PATCH 2.6.23-rc1] PM: fix compiler error of PPC dart_iommu Ryusuke Konishi
@ 2007-07-24 21:19 ` Rafael J. Wysocki
  0 siblings, 0 replies; 2+ messages in thread
From: Rafael J. Wysocki @ 2007-07-24 21:19 UTC (permalink / raw)
  To: ryusuke; +Cc: moriai.satoshi, linux-pm, Johannes Berg, Andrew Morton

On Tuesday, 24 July 2007 13:20, Ryusuke Konishi wrote:
> A dummy inline function of register_nosave_region_late was accidentally
> removed by the recent PM patch that introduced suspend notifiers.
> This elimination causes the following compiler error on PPC machines.
> 
>   CC      arch/powerpc/sysdev/dart_iommu.o
> arch/powerpc/sysdev/dart_iommu.c: In function 'iommu_init_late_dart':
> arch/powerpc/sysdev/dart_iommu.c:376: error: implicit declaration of function 
> 'register_nosave_region_late'
> make[1]: *** [arch/powerpc/sysdev/dart_iommu.o] Error 1
> make: *** [arch/powerpc/sysdev] Error 2
> 
> This patch fixes the problem.
> 
> Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
> ---

Acked-by: Rafael J. Wysocki <rjw@sisk.pl>

Thanks!

>  include/linux/suspend.h |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/include/linux/suspend.h b/include/linux/suspend.h
> index e8e6da3..618f93c 100644
> --- a/include/linux/suspend.h
> +++ b/include/linux/suspend.h
> @@ -125,6 +125,9 @@ static inline int unregister_pm_notifier(struct notifier_block *nb)
>  static inline void register_nosave_region(unsigned long b, unsigned long e)
>  {
>  }
> +static inline void register_nosave_region_late(unsigned long b, unsigned long e)
> +{
> +}
>  #endif
>  
>  #endif /* _LINUX_SWSUSP_H */
> 
> 

-- 
"Premature optimization is the root of all evil." - Donald Knuth

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-07-24 21:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-07-24 11:20 [PATCH 2.6.23-rc1] PM: fix compiler error of PPC dart_iommu Ryusuke Konishi
2007-07-24 21:19 ` Rafael J. Wysocki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox