From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrian Hunter Subject: Re: [PATCH V4 01/15] mmc: host: Add facility to support re-tuning Date: Tue, 14 Apr 2015 16:13:53 +0300 Message-ID: <552D1291.3010809@intel.com> References: <1427489863-9050-1-git-send-email-adrian.hunter@intel.com> <1427489863-9050-2-git-send-email-adrian.hunter@intel.com> <551BDAEA.20405@intel.com> <551D6BCA.7040708@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: Received: from mga11.intel.com ([192.55.52.93]:13478 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753805AbbDNNQB (ORCPT ); Tue, 14 Apr 2015 09:16:01 -0400 In-Reply-To: Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Ulf Hansson Cc: linux-mmc , Aaron Lu , Philip Rakity , Al Cooper , Arend van Spriel On 13/04/15 15:30, Ulf Hansson wrote: > On 2 April 2015 at 18:18, Adrian Hunter wrote: >> On 2/04/2015 4:05 p.m., Ulf Hansson wrote: >>> >>> [...] >>> >>>>> >>>>>> - ability to enable / disable re-tuning >>>>> >>>>> >>>>> Handled internally by the mmc core. >>>> >>>> >>>> The host controller driver enables re-tuning based on whether the host >>>> controller requires it for that transfer mode. For example, only the >>>> SDHCI >>>> host controller knows if tuning is required for SDR50 mode according to >>>> the >>>> SDHCI capability register bit 45. >>> >>> >>> That seems a bit silly. >>> >>> All hosts wants the re-tuning to be "enabled" if the current used >>> speed mode requires it. It's not a host driver thing to deal with, >>> just the core. >> >> >> No it is up to the host controller. That is how it is in the >> SD Host Controller Specification. Both whether to tune SDR50 and >> whether to run a re-tuning timer. >> >> Tuning is inherently a host controller problem. The card can always >> receive correctly from the host, but the host has to adjust its >> "sampling point" to receive from the card. Only the host knows its >> capabilities in this regard. > > The requirement of doing re-tuning is stated in the eMMC/SD specs. The > SDHCI spec is just adopting to what these specs already states. Then, > I think the mmc core is the only one, who shall be able of > enable/disable re-tuning and thus we shouldn't need any APIs for that. > > Hosts needs to be able to flag that re-tuning is needed and to > configure the re-tuning timeout (optional). OK sent patch set V5