lkml.org 
[lkml]   [2013]   [May]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH -v7 10/11] arm, change reboot_mode to use enum reboot_mode

* Robin Holt <holt@sgi.com> wrote:

> #ifndef __ASSEMBLY__
>
> #include <linux/types.h>
> +#include <linux/reboot.h>
>
> void iop3xx_map_io(void);
> void iop_init_cp6_handler(void);
> void iop_init_time(unsigned long tickrate);
> -void iop3xx_restart(char, const char *);
> +void iop3xx_restart(enum reboot_mode, const char *);

IMO the introduction of the enum is making things cleaner here all around.

As long as it's an equivalent transformation for ARM with no reduction in
functionality (i.e. please fix any bugs Russell found), this all looks
like a nice series.

Is there no parameter namespace clash between reboot modifier strings on
various architectures? I.e. 's' meaning something on x86 and meaning a
different thing on ARM or some other architecture.

Thanks,

Ingo


\
 
 \ /
  Last update: 2013-05-02 10:01    [W:0.133 / U:0.628 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site