From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shawn Guo Subject: Re: [PATCH v4 02/21] ARM: dts: imx7-colibri: disable HS400 Date: Mon, 19 Aug 2019 13:06:46 +0200 Message-ID: <20190819110645.GL5999@X250> References: <20190812142105.1995-1-philippe.schenker@toradex.com> <20190812142105.1995-3-philippe.schenker@toradex.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20190812142105.1995-3-philippe.schenker@toradex.com> Sender: linux-kernel-owner@vger.kernel.org To: Philippe Schenker Cc: Marcel Ziswiler , Max Krummenacher , "stefan@agner.ch" , "devicetree@vger.kernel.org" , Rob Herring , Mark Rutland , Michal =?utf-8?B?Vm9rw6HEjQ==?= , Fabio Estevam , Stefan Agner , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Pengutronix Kernel Team , NXP Linux Team , Sascha Hauer List-Id: devicetree@vger.kernel.org On Mon, Aug 12, 2019 at 02:21:17PM +0000, Philippe Schenker wrote: > From: Stefan Agner > > Force HS200 by masking bit 63 of the SDHCI capability register. > The i.MX ESDHC driver uses SDHCI_QUIRK2_CAPS_BIT63_FOR_HS400. With > that the stack checks bit 63 to descide whether HS400 is available. > Using sdhci-caps-mask allows to mask bit 63. The stack then selects > HS200 as operating mode. > > This prevents rare communication errors with minimal effect on > performance: > sdhci-esdhc-imx 30b60000.usdhc: warning! HS400 strobe DLL > status REF not lock! > > Signed-off-by: Stefan Agner > Signed-off-by: Philippe Schenker Applied, thanks.