Subject: Re: [patch 01/11] powerpc/cell: add support for power button of future IBM cell blades

From: Christian Krafft <[email-masked]>
Date: Mon, 7 Jul 2008 11:23:22 +0200

On Mon, 07 Jul 2008 15:12:23 +1000
Benjamin Herrenschmidt <[email-masked]> wrote:

> On Fri, 2008-07-04 at 21:05 +0200, [email-masked] wrote:
> > plain text document attachment
> > (0001-powerpc-cell-add-support-for-power-button-of-future.patch)
> > This patch adds support for the power button on future IBM cell blades.
> > It actually doesn't shut down the machine. Instead it exposes an
> > input device /dev/input/event0 to userspace which sends KEY_POWER
> > if power button has been pressed.
> > haldaemon actually recognizes the button, so a plattform independent acpid
> > replacement should handle it correctly.
> > 
> > Signed-off-by: Christian Krafft <[email-masked]>
> > Signed-off-by: Arnd Bergmann <[email-masked]>
> > ---
> 
> And what if CONFIG_INPUT=n or m ?
> 
> Cheers,
> Ben.
> 

hm, good point.
Only wrapping with ifdefs will not fix the CONFIG_INPUT=m case.
Have to think about it again, will send an update ...

Cheers,
ck

-- 
Mit freundlichen Gruessen,
kind regards,

Christian Krafft
Linux Kernel Development
IBM Systems & Technology Group
Phone: +49-07031-16-2032

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats:	Martin Jetter
Geschaetsfuehung:		Herbert Kircher
Sitz der Gesellschaft:		Boelingen
Registergericht:		Amtsgericht Stuttgart, HRB 243294