From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joonyoung Shim Subject: Re: [PATCH v6 3/3] input: samsung-keypad - Add samsung keypad driver Date: Wed, 21 Jul 2010 14:42:22 +0900 Message-ID: <4C4688BE.1080409@samsung.com> References: <1277724824-13308-1-git-send-email-jy0922.shim@samsung.com> <1277724824-13308-3-git-send-email-jy0922.shim@samsung.com> <4C32E21A.8080100@samsung.com> <013401cb1d66$5fbf3fb0$1f3dbf10$%kim@samsung.com> <4C33D859.9050708@samsung.com> <026901cb1f35$64c63470$2e529d50$%kim@samsung.com> <4C36D235.3040702@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7BIT Return-path: Received: from mailout3.samsung.com ([203.254.224.33]:22615 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751200Ab0GUFmZ (ORCPT ); Wed, 21 Jul 2010 01:42:25 -0400 In-reply-to: <4C36D235.3040702@samsung.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: dmitry.torokhov@gmail.com Cc: Kukjin Kim , linux-samsung-soc@vger.kernel.org, kyungmin.park@samsung.com, ben-linux@fluff.org, linux-input@vger.kernel.org, ch.naveen@samsung.com, linux-arm-kernel@lists.infradead.org Hi, Dmitry. On 7/9/2010 4:39 PM, Joonyoung Shim wrote: > On 7/9/2010 4:07 PM, Kukjin Kim wrote: >> Joonyoung Shim wrote: >>> On 7/7/2010 8:52 AM, Kukjin Kim wrote: >>>> Joonyoung Shim wrote: >>>>> Hi, Naveen. >>>>> >>>>> On 6/28/2010 8:33 PM, Joonyoung Shim wrote: >>>>>> This patch adds support for keypad driver running on Samsung cpus. This >>>>>> driver is tested on GONI and Aquila board using S5PC110 cpu. >>>>>> >>>>>> Signed-off-by: Joonyoung Shim >>>>>> Signed-off-by: Kyungmin Park >>>>>> --- >>>>>> drivers/input/keyboard/Kconfig | 9 + >>>>>> drivers/input/keyboard/Makefile | 1 + >>>>>> drivers/input/keyboard/samsung-keypad.c | 491 >>>>> +++++++++++++++++++++++++++++++ >>>>>> 3 files changed, 501 insertions(+), 0 deletions(-) >>>>>> create mode 100644 drivers/input/keyboard/samsung-keypad.c >>>>>> >>>>> Could you please test this v6 keypad driver on your various targets? >>>> Hi, >>>> >>>> I received the test result from Naveen. >>>> And now he is debugging about that. >>>> >>>> ..his e-mail client has some problem..so I replied... >>>> >>>> --- >>>> From: Naveen Ch >>>> >>>> I've tested with v6 patches and they are working fine on SMDK6410, SMDKC100 >>> and SMDKV210, except for some header inclusion (linux/sched.h) in samsung- >>> keypad.c. >>>> The build breaking error is like following: >>>> (make s5pc100_defconfig and select input->keypad->samsung keypad) >>>> >>>> drivers/input/keyboard/samsung-keypad.c: In function 'samsung_keypad_irq': >>>> drivers/input/keyboard/samsung-keypad.c:170: error: 'TASK_UNINTERRUPTIBLE' >>> undeclared (first use in this function) >>>> drivers/input/keyboard/samsung-keypad.c:170: error: (Each undeclared identifier >>> is reported only once >>>> drivers/input/keyboard/samsung-keypad.c:170: error: for each function it appears >>> in.) >>>> drivers/input/keyboard/samsung-keypad.c:170: error: implicit declaration of >>> function 'schedule_timeout' >>>> drivers/input/keyboard/samsung-keypad.c: In function 'samsung_keypad_stop': >>>> drivers/input/keyboard/samsung-keypad.c:202: error: 'TASK_NORMAL' >>> undeclared (first use in this function) >>>> make[3]: *** [drivers/input/keyboard/samsung-keypad.o] Error 1 >>>> >>> This error occurs on s5pc100_defconfig having disabled CONFIG_PREEMPT >>> option. I think sched.h should be included to remove like this >>> dependency in driver, are there other ways? >> I'm not sure root cause is that. But as you said, I think need inclusion it for prevent the problem. >> Actually didn't happen it before..so maybe the some core functions have moved between rc-3 and earlier versions... > > The sched.h is included by below path. > > interrupt.h -> hardirq.h -> smp_lock.h -> sched.h > > The smp_lock.h in hardirq.h is included if CONFIG_PREEMPT is defined. > The samsung keypad driver haven't needed including sched.h before > Dmitry's patch is applied.. > >> Anyway, if no any issues, could you please re-submit updated it soon? > > I haven't get yet Dmitry's review of v6 patch set, so i am waiting it. > Ping. Do you have any feedback? If ok, i want to go to input tree or samsung tree these added sched.h including. From mboxrd@z Thu Jan 1 00:00:00 1970 From: jy0922.shim@samsung.com (Joonyoung Shim) Date: Wed, 21 Jul 2010 14:42:22 +0900 Subject: [PATCH v6 3/3] input: samsung-keypad - Add samsung keypad driver In-Reply-To: <4C36D235.3040702@samsung.com> References: <1277724824-13308-1-git-send-email-jy0922.shim@samsung.com> <1277724824-13308-3-git-send-email-jy0922.shim@samsung.com> <4C32E21A.8080100@samsung.com> <013401cb1d66$5fbf3fb0$1f3dbf10$%kim@samsung.com> <4C33D859.9050708@samsung.com> <026901cb1f35$64c63470$2e529d50$%kim@samsung.com> <4C36D235.3040702@samsung.com> Message-ID: <4C4688BE.1080409@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, Dmitry. On 7/9/2010 4:39 PM, Joonyoung Shim wrote: > On 7/9/2010 4:07 PM, Kukjin Kim wrote: >> Joonyoung Shim wrote: >>> On 7/7/2010 8:52 AM, Kukjin Kim wrote: >>>> Joonyoung Shim wrote: >>>>> Hi, Naveen. >>>>> >>>>> On 6/28/2010 8:33 PM, Joonyoung Shim wrote: >>>>>> This patch adds support for keypad driver running on Samsung cpus. This >>>>>> driver is tested on GONI and Aquila board using S5PC110 cpu. >>>>>> >>>>>> Signed-off-by: Joonyoung Shim >>>>>> Signed-off-by: Kyungmin Park >>>>>> --- >>>>>> drivers/input/keyboard/Kconfig | 9 + >>>>>> drivers/input/keyboard/Makefile | 1 + >>>>>> drivers/input/keyboard/samsung-keypad.c | 491 >>>>> +++++++++++++++++++++++++++++++ >>>>>> 3 files changed, 501 insertions(+), 0 deletions(-) >>>>>> create mode 100644 drivers/input/keyboard/samsung-keypad.c >>>>>> >>>>> Could you please test this v6 keypad driver on your various targets? >>>> Hi, >>>> >>>> I received the test result from Naveen. >>>> And now he is debugging about that. >>>> >>>> ..his e-mail client has some problem..so I replied... >>>> >>>> --- >>>> From: Naveen Ch >>>> >>>> I've tested with v6 patches and they are working fine on SMDK6410, SMDKC100 >>> and SMDKV210, except for some header inclusion (linux/sched.h) in samsung- >>> keypad.c. >>>> The build breaking error is like following: >>>> (make s5pc100_defconfig and select input->keypad->samsung keypad) >>>> >>>> drivers/input/keyboard/samsung-keypad.c: In function 'samsung_keypad_irq': >>>> drivers/input/keyboard/samsung-keypad.c:170: error: 'TASK_UNINTERRUPTIBLE' >>> undeclared (first use in this function) >>>> drivers/input/keyboard/samsung-keypad.c:170: error: (Each undeclared identifier >>> is reported only once >>>> drivers/input/keyboard/samsung-keypad.c:170: error: for each function it appears >>> in.) >>>> drivers/input/keyboard/samsung-keypad.c:170: error: implicit declaration of >>> function 'schedule_timeout' >>>> drivers/input/keyboard/samsung-keypad.c: In function 'samsung_keypad_stop': >>>> drivers/input/keyboard/samsung-keypad.c:202: error: 'TASK_NORMAL' >>> undeclared (first use in this function) >>>> make[3]: *** [drivers/input/keyboard/samsung-keypad.o] Error 1 >>>> >>> This error occurs on s5pc100_defconfig having disabled CONFIG_PREEMPT >>> option. I think sched.h should be included to remove like this >>> dependency in driver, are there other ways? >> I'm not sure root cause is that. But as you said, I think need inclusion it for prevent the problem. >> Actually didn't happen it before..so maybe the some core functions have moved between rc-3 and earlier versions... > > The sched.h is included by below path. > > interrupt.h -> hardirq.h -> smp_lock.h -> sched.h > > The smp_lock.h in hardirq.h is included if CONFIG_PREEMPT is defined. > The samsung keypad driver haven't needed including sched.h before > Dmitry's patch is applied.. > >> Anyway, if no any issues, could you please re-submit updated it soon? > > I haven't get yet Dmitry's review of v6 patch set, so i am waiting it. > Ping. Do you have any feedback? If ok, i want to go to input tree or samsung tree these added sched.h including.