From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [PATCH] input/spear-keyboard: Remove kbd_set_plat_data() Date: Mon, 12 Mar 2012 12:24:44 +0530 Message-ID: <4F5D9DB4.1030404@st.com> References: <8acf668a2c8d67aec3dc79dfe2895b2fd80a18be.1330320826.git.viresh.kumar@st.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from eu1sys200aog109.obsmtp.com ([207.126.144.127]:33000 "EHLO eu1sys200aog109.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751269Ab2CLGzb (ORCPT ); Mon, 12 Mar 2012 02:55:31 -0400 In-Reply-To: <8acf668a2c8d67aec3dc79dfe2895b2fd80a18be.1330320826.git.viresh.kumar@st.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: "dmitry.torokhov@gmail.com" Cc: "linux-input@vger.kernel.org" , Armando VISCONTI , Shiraz HASHIM , Vipin KUMAR , Rajeev KUMAR , Deepak SIKRI , Vipul Kumar SAMAR , Amit VIRDI , Pratyush ANAND , Bhupesh SHARMA , "viresh.linux@gmail.com" , Bhavna YADAV , Vincenzo FRASCINO , Mirko GARDI , Salvatore DE DOMINICIS , "linux-kernel@vger.kernel.org" On 2/27/2012 11:09 AM, Viresh KUMAR wrote: > We must use platform_device_add_data() instead of kbd_set_plat_data(). Remove > definition of kbd_set_plat_data(). > > Signed-off-by: Viresh Kumar > --- > arch/arm/plat-spear/include/plat/keyboard.h | 7 ------- > 1 files changed, 0 insertions(+), 7 deletions(-) > > diff --git a/arch/arm/plat-spear/include/plat/keyboard.h b/arch/arm/plat-spear/include/plat/keyboard.h > index c16cc31..0562f13 100644 > --- a/arch/arm/plat-spear/include/plat/keyboard.h > +++ b/arch/arm/plat-spear/include/plat/keyboard.h > @@ -159,11 +159,4 @@ struct kbd_platform_data { > unsigned int mode; > }; > > -/* This function is used to set platform data field of pdev->dev */ > -static inline void > -kbd_set_plat_data(struct platform_device *pdev, struct kbd_platform_data *data) > -{ > - pdev->dev.platform_data = data; > -} > - > #endif /* __PLAT_KEYBOARD_H */ Hi Dmitry, Have you applied this patch? -- viresh