[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20100212161951.GB2906@sortiz.org>
Date: Fri, 12 Feb 2010 17:19:59 +0100
From: Samuel Ortiz <sameo@...ux.intel.com>
To: Christoph Egger <siccegge@...d.informatik.uni-erlangen.de>
Cc: Balaji T K <balajitk@...com>,
Santosh Shilimkar <santosh.shilimkar@...com>,
Rajendra Nayak <rnayak@...com>,
Kevin Hilman <khilman@...prootsystems.com>,
linux-kernel@...r.kernel.org, vamos@...informatik.uni-erlangen.de
Subject: Re: [PATCH] typo in kernel source (INPUT_TWL4030_PWBUTTON) (resend)
Hi Christoph,
On Fri, Feb 12, 2010 at 04:19:46PM +0100, Christoph Egger wrote:
> Hi all!
>
> Accoring to my mail-log and a quick look at the mfd tree this
> patch seems to have been lost. I guess it just has been missed among
> other stuff or is anything wrong with it?
I applied this patch yesterday, to my for-next branch.
Cheers,
Samuel.
> ----
> As part of the VAMOS[0] research project at the University of
> Erlangen we're checking referential integrity between kernel KConfig
> options and in-code Conditional blocks.
>
> While working on this we detected a spelling error in
> drivers/mfd/twl-core.c which can be corrected using the attached
> patch.
>
> Please keep me informed of this patch getting confirmed /
> merged so we can keep track of it.
>
> Regards
>
> Christoph Egger
>
> [0] http://vamos1.informatik.uni-erlangen.de/
>
> ----
> From 83cadb82b627f35333742d1585c2f63e40e76ef4 Mon Sep 17 00:00:00 2001
> From: Christoph Egger <siccegge@...d.informatik.uni-erlangen.de>
> Date: Wed, 13 Jan 2010 10:56:27 +0100
> Subject: [PATCH] Correct spelling for config INPUT_TWL4030_PWRBUTTON
>
> While the non-module variant of the TWL4030 Powerbutton is querried
> with INPUT_TWL4030_PWRBUTTON, the _MODULE variant misses the R and is
> checking therefor INPUT_TWL4030_PWBUTTON which won't work as there's
> nosuch option in kconfig
>
> Signed-off-by: Christoph Egger <siccegge@...d.informatik.uni-erlangen.de>
> ---
> drivers/mfd/twl-core.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c
> index 2a76065..aeaeac4 100644
> --- a/drivers/mfd/twl-core.c
> +++ b/drivers/mfd/twl-core.c
> @@ -129,7 +129,7 @@
> #define TWL_NUM_SLAVES 4
>
> #if defined(CONFIG_INPUT_TWL4030_PWRBUTTON) \
> - || defined(CONFIG_INPUT_TWL4030_PWBUTTON_MODULE)
> + || defined(CONFIG_INPUT_TWL4030_PWRBUTTON_MODULE)
> #define twl_has_pwrbutton() true
> #else
> #define twl_has_pwrbutton() false
> --
> 1.6.3.3
>
>
--
Intel Open Source Technology Centre
http://oss.intel.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists