[PATCH 3/3] ARM: PXA27x: Zipit Z2: disable wake on GPIO0

Russell King - ARM Linux linux at arm.linux.org.uk
Sun Feb 26 10:29:18 EST 2012


On Sun, Feb 26, 2012 at 06:11:17PM +0300, Vasily Khoruzhick wrote:
> 2012/2/26 Russell King - ARM Linux <linux at arm.linux.org.uk>:
> > On Sun, Feb 26, 2012 at 04:47:42PM +0300, Vasily Khoruzhick wrote:
> >> We don't want machine to wake up on AC state change
> >
> > NAK.  Stop polluting the machine callback methods.  Use the right one.
> 
> Ok, what's right one?

        void                    (*fixup)(struct tag *, char **,
                                         struct meminfo *);
        void                    (*reserve)(void);/* reserve mem blocks  */
        void                    (*map_io)(void);/* IO mapping function  */
        void                    (*init_early)(void);
        void                    (*init_irq)(void);
        struct sys_timer        *timer;         /* system tick timer    */
        void                    (*init_machine)(void);

and chose a more appropriate one from that?  Maybe init_early() if it
has to be done early, or preferably the init_machine() if it can wait
until arch_initcall() time.

But stuffing it into fixup, reserve, map_io, init_irq or the timer
initialization is pure abuse of those callbacks.



More information about the linux-arm-kernel mailing list