From patchwork Sat May 12 13:37:36 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Guennadi Liakhovetski X-Patchwork-Id: 158738 Return-Path: X-Original-To: incoming-imx@patchwork.ozlabs.org Delivered-To: patchwork-incoming-imx@bilbo.ozlabs.org Received: from merlin.infradead.org (merlin.infradead.org [IPv6:2001:4978:20e::2]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id DB4F1B7005 for ; Sat, 12 May 2012 23:40:31 +1000 (EST) Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1STCW3-000316-8e; Sat, 12 May 2012 13:37:47 +0000 Received: from moutng.kundenserver.de ([212.227.17.8]) by merlin.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1STCVy-00030g-Kn for linux-arm-kernel@lists.infradead.org; Sat, 12 May 2012 13:37:44 +0000 Received: from axis700.grange (dslb-088-076-068-194.pools.arcor-ip.net [88.76.68.194]) by mrelayeu.kundenserver.de (node=mreu3) with ESMTP (Nemesis) id 0MPeS5-1SXtLc0Vpx-004rMJ; Sat, 12 May 2012 15:37:37 +0200 Received: by axis700.grange (Postfix, from userid 1000) id 8D146189B6B; Sat, 12 May 2012 15:37:36 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by axis700.grange (Postfix) with ESMTP id 7FDA2189AEB; Sat, 12 May 2012 15:37:36 +0200 (CEST) Date: Sat, 12 May 2012 15:37:36 +0200 (CEST) From: Guennadi Liakhovetski X-X-Sender: lyakh@axis700.grange To: Olof Johansson Subject: Re: [GIT PULL] Urgent fixes (for v3.4 if possible) for Renesas ARM-based platforms In-Reply-To: Message-ID: References: <201205112025.13478.rjw@sisk.pl> MIME-Version: 1.0 Content-ID: X-Provags-ID: V02:K0:Za1SC9oGc3sRt0+EZLsJs19E4T3doGzhx5vmqCiTdyc L1t350BPWNScn8/0O0duiGkF1ZLp7c7n83gJOLA4FYEnxsKOpX l+QUZFcrQpbyvcvgolclpXOckG2z1diabWZnhUL3ZFBYObNXFi usiI4P6YlRbU746xp+90Rf+Sb1SHI1UpRySxGXfWSit02NllcX YbfyOL+tNFxDap4vlq31f9X+6Cwl+AQqIgEZifE6NiPQ+YVqul im34M2yvkmiO/lStmW6iLoyDQglj6B1Mju1jKNvvUZMx/dQHRi e6vhSK3+Q0ze+xjban6g0lE0j5CgwMcwDMNPyKZKzFian2JAGl dNs7cM1iTB1Im79sJtXWStPTx0wc7uYGWtBGLMKIL0XoYoDUzI CUZS5boRoQsOQ== X-Spam-Note: CRM114 invocation failed X-Spam-Score: -1.9 (-) X-Spam-Report: SpamAssassin version 3.3.2 on merlin.infradead.org summary: Content analysis details: (-1.9 points) pts rule name description ---- ---------------------- -------------------------------------------------- 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider (g.liakhovetski[at]gmx.de) -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/, no trust [212.227.17.8 listed in list.dnswl.org] -0.0 SPF_HELO_PASS SPF: HELO matches SPF record -0.0 T_RP_MATCHES_RCVD Envelope sender domain matches handover relay domain -0.0 SPF_PASS SPF: sender matches SPF record -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Cc: Arnd Bergmann , Linux-sh list , Magnus Damm , "Rafael J. Wysocki" , "arm@kernel.org" , linux-arm-kernel@lists.infradead.org X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.14 Precedence: list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org List-Id: linux-imx-kernel.lists.patchwork.ozlabs.org On Fri, 11 May 2012, Olof Johansson wrote: > On Fri, May 11, 2012 at 11:37 PM, Guennadi Liakhovetski > wrote: > > On Fri, 11 May 2012, Olof Johansson wrote: > > > >> Hej Magnus, > >> > >> On Fri, May 11, 2012 at 9:28 PM, Magnus Damm wrote: > >> > >> >>>  git://git.kernel.org/pub/scm/linux/kernel/git/rafael/renesas.git fixes > >> >> > >> >> No such branch (nor tag). > >> > > >> > I believe Rafael will get back to you about that when he gets to his computer. > >> > >> Yep, good. > >> > >> >>> Guennadi Liakhovetski (2): > >> >>>      ARM: mach-shmobile: convert ag5evm to use the generic MMC GPIO hotplug helper > >> >>>      ARM: mach-shmobile: convert mackerel to use the generic MMC GPIO hotplug helper > >> >> > >> >> These seem to fix the long-standing build errors on the two platforms, > >> >> but that's impossible to tell from reading the commit messages (they > >> >> are empty). Care to add one describing why they're important to go in? > >> > > >> > I only know that a) some boards don't build without these fixes and b) > >> > they used to build just fine. > >> > > >> > Perhaps Guennadi [CC:ed] could be so kind to fill us in with the blanks? > >> > >> It was pretty obvious once I tried applying them and building that > >> they did indeed resolve the build errors. However, the reason I'm > >> asking for a better description is that when looking at just the pull > >> request, or even the patch full commit description, it was impossible > >> to tell why the patch was urgent as a fix. So, while I am not > >> questioning that the patch should be included, I'm just requesting to > >> make sure it's properly described. That's also good for historical > >> purposes when someone is reading the git logs a year from now, etc. > > > > Ok, sorry, I wasn't sure from the previous mail - would you prefer to > > actually extend their commit messages? As described here > > > > http://thread.gmane.org/gmane.linux.kernel.mmc/12744/focus=12748 > > > > These patches are a part of a patch-series, so, when they were submitted > > in that context, it was pretty obvious how they relate to the rest. > > However, unfortunately, they are now applied later than the main patches > > in that series, so, their purpose has become much less obvious, and their > > main feature now seems to be not the conversion of the affected platforms, > > but fixing the build. > > Yeah, that seems to be what happened here, so indeed by now the > commits are a bit out of context. > > > So, I think, we could use something like this in the > > patch descriptions: > > > > This also fixes modular mmc build on this platform by eliminating the use > > of an inline function, which calls into the mmc core. > > Sounds good, with a cut-and-paste of the build error that happens > without the patch for extra credit. > > Rafael, can you add the above or something like it to the commit > messages when you push out the fixes branch? Let me try to make it a bit easier yet: here are the biuld failures from both platforms: mackerel: arch/arm/mach-shmobile/built-in.o: In function `mackerel_sdhi0_gpio_cd': pfc-sh7372.c:(.text+0x1138): undefined reference to `mmc_detect_change' ag5evm: arch/arm/mach-shmobile/built-in.o: In function `ag5evm_sdhi0_gpio_cd': pfc-sh73a0.c:(.text+0x7c0): undefined reference to `mmc_detect_change' Also attaching to this email the patches with added patch descriptions and these error-logs. Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ From f777bac2a06ac4c08dea6a815a340e000ee0bdf6 Mon Sep 17 00:00:00 2001 From: Guennadi Liakhovetski Date: Mon, 16 Apr 2012 23:09:19 +0200 Subject: [PATCH 2/2] ARM: mach-shmobile: convert ag5evm to use the generic MMC GPIO hotplug helper This also fixes the following modular mmc build failure: arch/arm/mach-shmobile/built-in.o: In function `mackerel_sdhi0_gpio_cd': pfc-sh7372.c:(.text+0x1138): undefined reference to `mmc_detect_change' on this platform by eliminating the use of an inline function, which calls into the mmc core. Signed-off-by: Guennadi Liakhovetski Reviewed-by: Simon Horman Acked-by: Magnus Damm Signed-off-by: Rafael J. Wysocki --- arch/arm/mach-shmobile/board-ag5evm.c | 22 ++-------------------- 1 files changed, 2 insertions(+), 20 deletions(-) diff --git a/arch/arm/mach-shmobile/board-ag5evm.c b/arch/arm/mach-shmobile/board-ag5evm.c index cb224a3..0891ec6 100644 --- a/arch/arm/mach-shmobile/board-ag5evm.c +++ b/arch/arm/mach-shmobile/board-ag5evm.c @@ -365,23 +365,13 @@ static struct platform_device mipidsi0_device = { }; /* SDHI0 */ -static irqreturn_t ag5evm_sdhi0_gpio_cd(int irq, void *arg) -{ - struct device *dev = arg; - struct sh_mobile_sdhi_info *info = dev->platform_data; - struct tmio_mmc_data *pdata = info->pdata; - - tmio_mmc_cd_wakeup(pdata); - - return IRQ_HANDLED; -} - static struct sh_mobile_sdhi_info sdhi0_info = { .dma_slave_tx = SHDMA_SLAVE_SDHI0_TX, .dma_slave_rx = SHDMA_SLAVE_SDHI0_RX, - .tmio_flags = TMIO_MMC_HAS_IDLE_WAIT, + .tmio_flags = TMIO_MMC_HAS_IDLE_WAIT | TMIO_MMC_USE_GPIO_CD, .tmio_caps = MMC_CAP_SD_HIGHSPEED, .tmio_ocr_mask = MMC_VDD_27_28 | MMC_VDD_28_29, + .cd_gpio = GPIO_PORT251, }; static struct resource sdhi0_resources[] = { @@ -557,7 +547,6 @@ static void __init ag5evm_init(void) lcd_backlight_reset(); /* enable SDHI0 on CN15 [SD I/F] */ - gpio_request(GPIO_FN_SDHICD0, NULL); gpio_request(GPIO_FN_SDHIWP0, NULL); gpio_request(GPIO_FN_SDHICMD0, NULL); gpio_request(GPIO_FN_SDHICLK0, NULL); @@ -566,13 +555,6 @@ static void __init ag5evm_init(void) gpio_request(GPIO_FN_SDHID0_1, NULL); gpio_request(GPIO_FN_SDHID0_0, NULL); - if (!request_irq(intcs_evt2irq(0x3c0), ag5evm_sdhi0_gpio_cd, - IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING, - "sdhi0 cd", &sdhi0_device.dev)) - sdhi0_info.tmio_flags |= TMIO_MMC_HAS_COLD_CD; - else - pr_warn("Unable to setup SDHI0 GPIO IRQ\n"); - /* enable SDHI1 on CN4 [WLAN I/F] */ gpio_request(GPIO_FN_SDHICLK1, NULL); gpio_request(GPIO_FN_SDHICMD1_PU, NULL); -- 1.7.2.5