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

Vasily Khoruzhick anarsoul at gmail.com
Sun Feb 26 10:35:10 EST 2012


2012/2/26 Russell King - ARM Linux <linux at arm.linux.org.uk>:
> 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.

Ok, thanks.

Regards
Vasily



More information about the linux-arm-kernel mailing list