Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752341AbaLSTCb (ORCPT ); Fri, 19 Dec 2014 14:02:31 -0500 Received: from mail-vc0-f181.google.com ([209.85.220.181]:47865 "EHLO mail-vc0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751431AbaLSTC2 (ORCPT ); Fri, 19 Dec 2014 14:02:28 -0500 MIME-Version: 1.0 In-Reply-To: References: <1417563767-32181-1-git-send-email-dianders@chromium.org> Date: Fri, 19 Dec 2014 11:02:26 -0800 X-Google-Sender-Auth: 18lBf32lWk1mRArA1fdfQJEep_Y Message-ID: Subject: Re: [PATCH v5 0/4] Fixes for SDIO interrupts for dw_mmc From: Doug Anderson To: Ulf Hansson Cc: Jaehoon Chung , Seungwon Jeon , Alim Akhtar , Sonny Rao , Andrew Bresticker , Heiko Stuebner , Russell King - ARM Linux , H Hartley Sweeten , Tony Lindgren , Sascha Hauer , Wolfram Sang , linux-mmc , "linux-kernel@vger.kernel.org" , Chris Ball , =?UTF-8?B?R3LDqWdvcnkgU291dGFkw6k=?= , Joe Perches , Axel Lin , linux-omap , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Ulf, On Fri, Dec 19, 2014 at 2:17 AM, Ulf Hansson wrote: > On 3 December 2014 at 00:42, Doug Anderson wrote: >> Bing Zhao at Marvell found a problem with dw_mmc where interrupts >> weren't firing sometimes. He tracked it down to a read-modify-write >> problem with the INTMASK. These patches fix the problem. >> >> Note: I've picked up a > 1-year old series here to make another >> attempt at landing it upstream. These patches have been in shipping >> Chromebooks for the last year. Note that v3 to v4 has no changes >> other than a rebase and a small commit message update. >> >> The first two patches extend the "init_card()" mechanism of MMC core >> to actually be called for all card types, not just SDIO. That could >> be applied any time and should fix at least one longstanding bug >> (untested). >> >> The third patch is a cleanup patch to use init_card() to move things >> around a bit so we don't need to handle SDIO cards in such a strange >> place. On earlier versions of this patch Seungwon brought up a few >> points which I have _not_ addressed. See >> . Other than talk of >> cards with out of band interrupts maybe being able to gate their >> clocks, he wanted to use MMC_QUIRK_BROKEN_CLK_GATING. I didn't do >> that because of the ordering of init_card() and when the quirks are >> set. Some users of init_card() like pandora_wl1251_init_card() rely >> on it being called very early in the process. >> pandora_wl1251_init_card() hardcodes a vendor and device and thus need >> to be called super early. On the other hand the code that adds quirks >> _reads_ the vendor and device. It can't possibly move before >> init_card(). If folks are willing to take an additional host op of >> init_card_late() I can certainly go that way, though. >> >> The fourth patch is (I think) reviewed and ready to go assuming the >> other two land. > > I have queued this up for 3.20. Thanks! > It was a bit hard to follow the > updated the revisions, please don't send patches "in-reply-to" for > future sets. Very strange. I didn't send out anything in-reply-to other than what git-send-email usually does. I believe I had: [0] - no in reply to. [1] - in reply to [0] [2] - in reply to [0] [3] - in reply to [0] [4] - in reply to [0] Is there some other way you'd prefer? Looking full headers in , I confirm it is "in-reply-to" "1417563767-32181-1-git-send-email-dianders@chromium.org". Patchwork doesn't keep cover letters, but you can see at ) that there is no in-reply-to. I'm more than happy to adjust my workflow if you can give me some specifics. Thanks! :) > In v5, I don't find a patch 1/4. Anyway, I have taken patch 2->4. Ah, maybe because it wasn't sent to linux-mmc? I messed that up and will try to do better in the future. Sorry. :( You were in the To line, though. You can see at . Do you want me to repost it and CC linux-mmc with Tony's Ack? --- Note: patchwork seems to find all my patches: pwclient list -w dianders@chromium.org -p "" 5425241 New [v5,1/4] ARM: OMAP2+: Make sure pandora_wl1251_init_card() applies to SDIO only 5425291 New [v5,1/4] ARM: OMAP2+: Make sure pandora_wl1251_init_card() applies to SDIO only 5425311 New [v5,1/4] ARM: OMAP2+: Make sure pandora_wl1251_init_card() applies to SDIO only 5425231 New [v5,2/4] mmc: core: Support the optional init_card() callback for MMC and SD 5425301 New [v5,2/4] mmc: core: Support the optional init_card() callback for MMC and SD 5425271 New [v5,3/4] mmc: dw_mmc: Cleanup disable of low power mode w/ SDIO interrupts 5425281 New [v5,3/4] mmc: dw_mmc: Cleanup disable of low power mode w/ SDIO interrupts 5425251 New [v5,4/4] mmc: dw_mmc: Protect read-modify-write of INTMASK with a lock 5425261 New [v5,4/4] mmc: dw_mmc: Protect read-modify-write of INTMASK with a lock -- 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/