[PATCH 2/7] Celleb: Move pause, kexec_cpu_down to beat.c

Arnd Bergmann arnd at arndb.de
Wed Sep 26 21:40:50 EST 2007


On Wednesday 26 September 2007, Ishizaki Kou wrote:
> This patch is an update for "Beat on Celleb"
>   - Move beat_pause(), beat_kexec_cpu_down() from setup.c to beat.c
> 
> Signed-off-by: <Kou.Ishizaki at toshiba.co.jp>

Acked-by: Arnd Bergmann <arnd.bergmann at de.ibm.com>

The patch looks good, once you fix this one line:

> @@ -36,5 +36,9 @@ ssize_t beat_nvram_get_size(void);
>  ssize_t beat_nvram_read(char *, size_t, loff_t *);
>  ssize_t beat_nvram_write(char *, size_t, loff_t *);
>  int beat_set_xdabr(unsigned long);
> +void beat_power_save(void);
> +#ifdef CONFIG_KEXEC
> +void beat_kexec_cpu_down(int, int);
> +#endif

Function declarations in headers files should not be enclosed in #ifdef,
it only causes unnecessary rebuilds when the configuration changes, but
does not help.

	Arnd <><



More information about the Linuxppc-dev mailing list