* [PATCH 01/03] [INPUT][KEYBOARD] Kconfig support for s3c matrix keypad
@ 2009-09-05 13:54 양진성
2009-09-05 15:56 ` Mark Brown
0 siblings, 1 reply; 2+ messages in thread
From: 양진성 @ 2009-09-05 13:54 UTC (permalink / raw)
To: linux-input
Cc: laforge, ben-linux,
김경일/AP개발팀(SYS.LSI)/E3(사원)/삼성전자,
'김국진/AP개발팀(SYS.LSI)/E5(책임)/삼성전자'
This patch supports to select configuration for s3c matrix keypad.
Signed-off-by: Jinsung Yang <jsgood.yang@samsung.com>
Signed-off-by: Kyeongil Kim <ki0351.kim@samsung.com>
---
drivers/input/keyboard/Kconfig | 6 ++++++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig
index a6b989a..4a04553 100644
--- a/drivers/input/keyboard/Kconfig
+++ b/drivers/input/keyboard/Kconfig
@@ -361,4 +361,10 @@ config KEYBOARD_XTKBD
To compile this driver as a module, choose M here: the
module will be called xtkbd.
+config KEYBOARD_S3C
+ tristate "Samsung S3C keypad support"
+ depends on PLAT_S3C
+ help
+ Enable support for Samsung S3C keypad interface.
+
endif
--
1.6.2.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 01/03] [INPUT][KEYBOARD] Kconfig support for s3c matrix keypad
2009-09-05 13:54 [PATCH 01/03] [INPUT][KEYBOARD] Kconfig support for s3c matrix keypad 양진성
@ 2009-09-05 15:56 ` Mark Brown
0 siblings, 0 replies; 2+ messages in thread
From: Mark Brown @ 2009-09-05 15:56 UTC (permalink / raw)
To: 양진성
Cc: linux-input, laforge, ben-linux,
김경일/AP개발팀(SYS.LSI)/E3(사원)/삼성전자,
'김국진/AP개발팀(SYS.LSI)/E5(책임)/삼성전자'
On Sat, Sep 05, 2009 at 10:54:44PM +0900, 양진성 wrote:
> This patch supports to select configuration for s3c matrix keypad.
> Signed-off-by: Jinsung Yang <jsgood.yang@samsung.com>
> Signed-off-by: Kyeongil Kim <ki0351.kim@samsung.com>
A relatively technical point but you should always try to order your
patch series so that the kernel always builds, even with the patch
series partly applied. This means that if you've split out the build
system changes that should be the last patch in the series.
Doing this is useful when people use tools such as 'git bisect' which
walk through history and also means that it's possible to do things like
partially apply the series (for example, if you get a situation where
the earlier patches are all OK but there's some issues that need to be
resolved in the later ones).
It'd probably also help to combine more of your changes together - in
general it's better if patches achieve some goal in a self-contained
fashion. This normally makes things easier to review. Things like
doing the build system separately are normally there in order to split a
very large driver into managable chunks.
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-09-05 15:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-09-05 13:54 [PATCH 01/03] [INPUT][KEYBOARD] Kconfig support for s3c matrix keypad 양진성
2009-09-05 15:56 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).