From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx3.wp.pl ([212.77.101.10]:39106 "EHLO mx3.wp.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750789AbbGaKxi (ORCPT ); Fri, 31 Jul 2015 06:53:38 -0400 Date: Fri, 31 Jul 2015 12:53:31 +0200 From: Jakub Kicinski To: Greg KH Cc: stable@vger.kernel.org Subject: Re: mmc: sdhci: Fix FSL ESDHC reset handling quirk Message-ID: <20150731125331.57c5e4cb@north> In-Reply-To: <20150731010739.GB31169@kroah.com> References: <20150602163709.71d1460c@north> <20150603004629.GA24127@kroah.com> <20150603065438.4fb78645@north> <20150731010739.GB31169@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: stable-owner@vger.kernel.org List-ID: On Thu, 30 Jul 2015 18:07:39 -0700, Greg KH wrote: > On Wed, Jun 03, 2015 at 06:54:38AM +0200, Jakub Kicinski wrote: > > On Wed, 3 Jun 2015 09:46:29 +0900, Greg KH wrote: > > > On Tue, Jun 02, 2015 at 04:37:09PM +0200, Jakub Kicinski wrote: > > > > Please include following patch from Linus' tree in the 3.18 branch. > > > > Without it SDHC is unusable on Freescale P2020. Thanks! > > > > > > > > > > > > commit 556b5f33513f16450e8bb803c82a24377b94a34e > > > > > > I don't see this commit anywhere in Linus's tree. Where did you find > > > it? > > > > Argh, stupidly I sent you the commit id from after I applied this to > > 3.18... In Linus's tree it's 304f0a98d168, sorry. > > > > > > commit 304f0a98d168563b5f69c53a7544c36e6d221ac4 > > Author: Alessio Igor Bogani > > Date: Tue Dec 9 09:40:38 2014 +0100 > > > > mmc: sdhci: Fix FSL ESDHC reset handling quirk > > > > The commit 0718e59ae259 ("mmc: sdhci: move FSL ESDHC reset handling quirk into > > esdhc code") states that Freescale esdhc is the only controller which needs > > the interrupt registers restored after a reset. So it moves > > SDHCI_QUIRK_RESTORE_IRQS_AFTER_RESET quirk handling code into the > > esdhc-imx driver only. Unfortunately the same controller is used in > > other boards which use the of-esdhc driver instead (like powerpc P2020). > > > > Restore interrupts after reset in the sdhci-of-esdhc driver also. > > > > Signed-off-by: Alessio Igor Bogani > > Signed-off-by: Ulf Hansson > > > I don't remember tree what you wanted this applied to, so I tried > 3.14-stable, and that didn't work. Can you be more specific? OK, let me try again. I would like the following commit to be applied to 3.18: commit 304f0a98d168563b5f69c53a7544c36e6d221ac4 Author: Alessio Igor Bogani Date: Tue Dec 9 09:40:38 2014 +0100 mmc: sdhci: Fix FSL ESDHC reset handling quirk The commit 0718e59ae259 ("mmc: sdhci: move FSL ESDHC reset handling quirk into esdhc code") states that Freescale esdhc is the only controller which needs the interrupt registers restored after a reset. So it moves SDHCI_QUIRK_RESTORE_IRQS_AFTER_RESET quirk handling code into the esdhc-imx driver only. Unfortunately the same controller is used in other boards which use the of-esdhc driver instead (like powerpc P2020). Restore interrupts after reset in the sdhci-of-esdhc driver also. Signed-off-by: Alessio Igor Bogani Signed-off-by: Ulf Hansson Thanks!