From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751905AbcI1IZC (ORCPT ); Wed, 28 Sep 2016 04:25:02 -0400 Received: from mga14.intel.com ([192.55.52.115]:53694 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751566AbcI1IYw (ORCPT ); Wed, 28 Sep 2016 04:24:52 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.30,409,1470726000"; d="scan'208";a="766730336" Subject: Re: [RESEND PATCH v2 4/4] mmc: sdhci-of-arasan: add sdhci_arasan_voltage_switch for arasan,5.1 To: Shawn Lin , Ulf Hansson References: <1474589577-11193-1-git-send-email-shawn.lin@rock-chips.com> <1474589577-11193-4-git-send-email-shawn.lin@rock-chips.com> Cc: Jaehoon Chung , linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org From: Adrian Hunter Organization: Intel Finland Oy, Registered Address: PL 281, 00181 Helsinki, Business Identity Code: 0357606 - 4, Domiciled in Helsinki Message-ID: Date: Wed, 28 Sep 2016 11:20:08 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <1474589577-11193-4-git-send-email-shawn.lin@rock-chips.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 23/09/16 03:12, Shawn Lin wrote: > Per the vendor's requirement, we shouldn't do any setting for > 1.8V Signaling Enable, otherwise the interaction/behaviour between > phy and controller will be undefined. Mostly it works fine if we do > that, but we still see failures. Anyway, let's fix it to meet the > vendor's requirement. The error log looks like: > > [ 93.405085] mmc1: unexpected status 0x800900 after switch > [ 93.408474] mmc1: switch to bus width 1 failed > [ 93.408482] mmc1: mmc_select_hs200 failed, error -110 > [ 93.408492] mmc1: error -110 during resume (card was removed?) > [ 93.408705] PM: resume of devices complete after 213.453 msecs > > Signed-off-by: Shawn Lin Acked-by: Adrian Hunter > --- > > Changes in v2: None > > drivers/mmc/host/sdhci-of-arasan.c | 24 ++++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c > index da8e40a..1573a8d 100644 > --- a/drivers/mmc/host/sdhci-of-arasan.c > +++ b/drivers/mmc/host/sdhci-of-arasan.c > @@ -265,6 +265,28 @@ void sdhci_arasan_reset(struct sdhci_host *host, u8 mask) > } > } > > +static int sdhci_arasan_voltage_switch(struct mmc_host *mmc, > + struct mmc_ios *ios) > +{ > + switch (ios->signal_voltage) { > + case MMC_SIGNAL_VOLTAGE_180: > + /* > + * Plese don't switch to 1V8 as arasan,5.1 doesn't > + * actually refer to this setting to indicate the > + * signal voltage and the state machine will be broken > + * actually if we force to enable 1V8. That's something > + * like broken quirk but we could work around here. > + */ > + return 0; > + case MMC_SIGNAL_VOLTAGE_330: > + case MMC_SIGNAL_VOLTAGE_120: > + /* We don't support 3V3 and 1V2 */ > + break; > + } > + > + return -EINVAL; > +} > + > static struct sdhci_ops sdhci_arasan_ops = { > .set_clock = sdhci_arasan_set_clock, > .get_max_clock = sdhci_pltfm_clk_get_max_clock, > @@ -661,6 +683,8 @@ static int sdhci_arasan_probe(struct platform_device *pdev) > > host->mmc_host_ops.hs400_enhanced_strobe = > sdhci_arasan_hs400_enhanced_strobe; > + host->mmc_host_ops.start_signal_voltage_switch = > + sdhci_arasan_voltage_switch; > } > > ret = sdhci_add_host(host); >