Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754741Ab1GAC2I (ORCPT ); Thu, 30 Jun 2011 22:28:08 -0400 Received: from mailout3.samsung.com ([203.254.224.33]:24037 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754523Ab1GAC2G (ORCPT ); Thu, 30 Jun 2011 22:28:06 -0400 X-AuditID: cbfee61b-b7bfdae000006d51-15-4e0d30afcbec Date: Fri, 01 Jul 2011 11:27:59 +0900 From: =?utf-8?B?7KCE7Iq57JuQ?= Subject: RE: [PATCH] mmc: dw_mmc: Add the function call for board-specific initialization. In-reply-to: To: "'James Hogan'" , "'Jaehoon Chung'" Cc: linux-mmc@vger.kernel.org, cjb@laptop.org, linux-kernel@vger.kernel.org Message-id: <008801cc3796$7eb6c0f0$7c2442d0$%jun@samsung.com> MIME-version: 1.0 X-Mailer: Microsoft Office Outlook 12.0 Content-type: text/plain; charset=utf-8 Content-language: ko Content-transfer-encoding: 7BIT Thread-index: Acw3AcI64PFRlfokS8W+J2Q1XNFM9wAlBmsg References: <1309402665-29228-1-git-send-email-tgih.jun@samsung.com> <4E0C18BD.8000608@samsung.com> <4E0C31DF.5090200@samsung.com> X-Brightmail-Tracker: AAAAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4389 Lines: 153 Hi, Thank you for comment. > -----Original Message----- > From: linux-mmc-owner@vger.kernel.org [mailto:linux-mmc- > owner@vger.kernel.org] On Behalf Of James Hogan > Sent: Thursday, June 30, 2011 5:43 PM > To: Jaehoon Chung > Cc: Seungwon Jeon; linux-mmc@vger.kernel.org; cjb@laptop.org; linux- > kernel@vger.kernel.org > Subject: Re: [PATCH] mmc: dw_mmc: Add the function call for board-specific > initialization. > > Hi, > > On 30 June 2011 09:20, Jaehoon Chung wrote: > > Hi James > > > > James Hogan wrote: > >> On 30 June 2011 07:33, Jaehoon Chung wrote: > >>> Hi Mr.Jeon > >>> > >>> This patch is the compiler error. > >>> > >>> didn't find host->card_work.. > >>> > >>> do you want this > >>> queue_work(dw_mci_card_workqueue, &slot->host->card_work); ? > >> > >> I think the patch is based on mmc-next, which has the following commit: > >> 35ba9d8 mmc: dw_mmc: convert card tasklet to workqueue > > > > I also tested this patch based on mmc-next. > > But my mean is that there is not host structure in > dw_mci_detect_interrupt(). > > > > drivers/mmc/host/dw_mmc.c: In function 'dw_mci_detect_interrupt': > > drivers/mmc/host/dw_mmc.c:1592: error: 'host' undeclared (first use in > this function) > > drivers/mmc/host/dw_mmc.c:1592: error: (Each undeclared identifier is > reported only once > > drivers/mmc/host/dw_mmc.c:1592: error: for each function it appears in.) > > drivers/mmc/host/dw_mmc.c:1590: warning: unused variable 'slot' > > make[3]: *** [drivers/mmc/host/dw_mmc.o] Error 1 > > make[2]: *** [drivers/mmc/host] Error 2 > > Sorry, I misunderstood. You are correct. queue_work(dw_mci_card_workqueue, &host->card_work); 'slot' is omitted. (&slot->host->card_work) Sorry for my mistake. I'll fix it. Best regards, Seungwon. > Cheers > James > > > > > Did you really compile fine? > > > > Regards, > > Jaehoon Chung > > > >> > >> Cheers > >> James > >> > >>> > >>> Regards, > >>> Jaehoon Chung > >>> > >>> > >>> Seungwon Jeon wrote: > >>>> Need to call init() function of dw_mci_board. > >>>> It seems to be omitted to call this function. > >>>> > >>>> Signed-off-by: Seungwon Jeon > >>>> --- > >>>> drivers/mmc/host/dw_mmc.c | 11 +++++++++++ > >>>> 1 files changed, 11 insertions(+), 0 deletions(-) > >>>> > >>>> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c > >>>> index a524416..073c420 100644 > >>>> --- a/drivers/mmc/host/dw_mmc.c > >>>> +++ b/drivers/mmc/host/dw_mmc.c > >>>> @@ -1585,6 +1585,15 @@ static void dw_mci_work_routine_card(struct > work_struct *work) > >>>> } > >>>> } > >>>> > >>>> +static irqreturn_t dw_mci_detect_interrupt(int irq, void *dev_id) > >>>> +{ > >>>> + struct dw_mci_slot *slot = dev_id; > >>>> + > >>>> + queue_work(dw_mci_card_workqueue, &host->card_work); > >>>> + > >>>> + return IRQ_HANDLED; > >>>> +} > >>>> + > >>>> static int __init dw_mci_init_slot(struct dw_mci *host, unsigned int > id) > >>>> { > >>>> struct mmc_host *mmc; > >>>> @@ -1657,6 +1666,8 @@ static int __init dw_mci_init_slot(struct > dw_mci *host, unsigned int id) > >>>> } else > >>>> regulator_enable(host->vmmc); > >>>> > >>>> + host->pdata->init(id, dw_mci_detect_interrupt, host); > >>>> + > >>>> if (dw_mci_get_cd(mmc)) > >>>> set_bit(DW_MMC_CARD_PRESENT, &slot->flags); > >>>> else > >>>> -- > >>>> 1.7.0.4 > >>>> > >>>> -- > >>>> To unsubscribe from this list: send the line "unsubscribe linux-mmc" > in > >>>> the body of a message to majordomo@vger.kernel.org > >>>> More majordomo info at http://vger.kernel.org/majordomo-info.html > >>>> > >>> -- > >>> To unsubscribe from this list: send the line "unsubscribe linux-mmc" > in > >>> the body of a message to majordomo@vger.kernel.org > >>> More majordomo info at http://vger.kernel.org/majordomo-info.html > >>> > >> > >> > >> > > > > > > > > -- > James Hogan > -- > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/