From mboxrd@z Thu Jan 1 00:00:00 1970 From: marek.vasut@gmail.com (Marek Vasut) Date: Sun, 26 Feb 2012 23:21:49 +0100 Subject: [PATCH 3/3] ARM: PXA27x: Zipit Z2: disable wake on GPIO0 In-Reply-To: References: <1330264062-5750-1-git-send-email-anarsoul@gmail.com> <20120226152918.GP22562@n2100.arm.linux.org.uk> Message-ID: <201202262321.50101.marek.vasut@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org > 2012/2/26 Russell King - ARM Linux : > > On Sun, Feb 26, 2012 at 06:11:17PM +0300, Vasily Khoruzhick wrote: > >> 2012/2/26 Russell King - ARM Linux : > >> > 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. > Or this should be patched in the bootloader too. Also, please dont introduce ad- hoc constants like "1 << 0", ever! M