Received: by 2002:a05:6a10:a841:0:0:0:0 with SMTP id d1csp1986667pxy; Sat, 24 Apr 2021 01:28:05 -0700 (PDT) X-Google-Smtp-Source: ABdhPJybhhPOxkYJGd3LlkiYElJ3wYss1IWnihpRfeLHTJrXUIuXp/LaHwDT+nYSF3ICT576O5u3 X-Received: by 2002:a17:906:9a81:: with SMTP id ag1mr8125263ejc.464.1619252885739; Sat, 24 Apr 2021 01:28:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1619252885; cv=none; d=google.com; s=arc-20160816; b=Q2uITLqvR4M33d8sZqvYtVGgz27OY2QjDVrfBp/tB63btD0j3qHZWOOHJPsOTZmtwL Zuuw1POuobZ6HTwCEuMWkHlT0mmvPYUa0k/yYYimkIkRSFNGjX1fXDYTNol43hebsXub f8aUT1dFYwj+i9Q0C6mboKeasGcuogollRK4ARRiO8jYbctfYyazVijGm+5/Efeaq1X+ fLziLuX13WBZ9rUgH4elLpaX2l9UeTJ4Sn85rAVqvEbTZzl3DtqgVncdRYQj/4CVbkUI 0IFsBPUexOJXK6WSvUxae9/Alen9nwwzIslAIRjm1fZPs8oNG5o67Ocs/5hIIPAmL2Pi yEMA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-disposition:mime-version:message-id :subject:cc:to:from:date:dkim-signature; bh=SNOBsMH6hsVrtOZBfES+XFjoqfZ4tZXMmDAM/d9yCW0=; b=qEsNv0EkCGO3t/9HzLBcLcc6kkbh/SGqaHve8IlRuLGIHhpwy//zNKBoDPMbKFNYMr IMrTfnimTqA8/U2yfXY5QX+odLT7oAhezGYZlAkGNPHqlJTTK8gXb2mc+v2Yr4fHmxnz 3GOvOTii8svfK9L+RknnXzf4XJNEFWBD9Ph8VMvX1gUrOO8kuGxtTtLA33u+a4mrjatz VawDb1yZiZPWHY8MWOwAwoIr+IfnCO7ThMSeG+V20SS6ZNayGdbpvg+5swfiEtFyc3A2 ItmPMtB52ldwc9zAQybF6bKjlr1Mlc+qmbbC8dHljW84PrExyFIiCKMJrqNA/McoZ/yP NmLw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=nDfAjvSa; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id c22si998380edr.378.2021.04.24.01.27.42; Sat, 24 Apr 2021 01:28:05 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=nDfAjvSa; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231919AbhDXI0b (ORCPT + 99 others); Sat, 24 Apr 2021 04:26:31 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50024 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233250AbhDXIZ2 (ORCPT ); Sat, 24 Apr 2021 04:25:28 -0400 Received: from mail-pj1-x102d.google.com (mail-pj1-x102d.google.com [IPv6:2607:f8b0:4864:20::102d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 62CE9C06174A; Sat, 24 Apr 2021 01:17:00 -0700 (PDT) Received: by mail-pj1-x102d.google.com with SMTP id t13so219396pji.4; Sat, 24 Apr 2021 01:17:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:mime-version:content-disposition; bh=SNOBsMH6hsVrtOZBfES+XFjoqfZ4tZXMmDAM/d9yCW0=; b=nDfAjvSaBukuzHQZFqayoYjcWAVJZvfAsIWufYv/XTRHe8J8tEG7xvd7V0buVXE/6g qcMc4XbAjSrbdUH4CoradY9JACfaHlWQu9y/0tgHKs3DYT0v2pKUvFH5iMZ+fqPpI9nz YT8NcCLaeNPoJJC5jAJOndlsh5Hbixscv+hdtCxr9oPgKinwF7oEs69R2xMUEy7mmz6I N9F1Bo5M7hmkd8fNnbQV2R6nGn3nle1KUfQTU7iyvpmF5EB5XdngoTjQyO8imOtFpl2z UTS1SKqMg4z1khlrmOZbKwgzxY+p8r9JBPAJ92WksCu2R7UJxmmcfy2ahq6yqHFt/H1V QK/g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:mime-version :content-disposition; bh=SNOBsMH6hsVrtOZBfES+XFjoqfZ4tZXMmDAM/d9yCW0=; b=K/qqdUjoG4hPh22zIrClGbZNhgHfXZ1w7NstrpAUy81+q8o3Q/fIlJVzlf5fht/5+f lSATa+k5gwJQ/6Gcqi0+X7ptwsTwokBmErt+TW13JfY1NHEnjTUgJF0YLb9O6IPpMlj7 z9Re9/Mhem5TbBJY9lx6oxpoz6cVeMUtppVTwBSIpbvVp7ZF2/yzyygzNNlWbQ+Jma34 Q4hvFaZdMbDUJowDCcwaB18+aEg8EOSFujQYhOPAqEsb67Z+SOMaBtj9EagF6ThE+FSr mIevOieI4EVgJgyQBBIjeQihlhHSKhLsIoO+gNHvmp+bTau/v1qkrhJfWxztK1R6uqEV fqgA== X-Gm-Message-State: AOAM533RPNU5RPSb7SWYcojgi/MoYeMyHGrprjPLnyoXjkklPPXLC29h knEPPr2ATMSHT9rFN2Zr32s= X-Received: by 2002:a17:90b:388a:: with SMTP id mu10mr9870600pjb.203.1619252219831; Sat, 24 Apr 2021 01:16:59 -0700 (PDT) Received: from soma ([121.99.145.49]) by smtp.gmail.com with ESMTPSA id d4sm9157893pjz.49.2021.04.24.01.16.55 (version=TLS1 cipher=AES128-SHA bits=128/128); Sat, 24 Apr 2021 01:16:59 -0700 (PDT) Received: by soma (sSMTP sendmail emulation); Sat, 24 Apr 2021 20:16:52 +1200 Date: Sat, 24 Apr 2021 20:16:52 +1200 From: Daniel Beer To: ulf.hansson@linaro.org Cc: linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, Adrian Hunter , Ben Chuang , Daniel Beer Subject: [PATCH] mmc: sdhci-pci-gli: increase 1.8V regulator wait Message-ID: <20210424081652.GA16047@nyquist.nev> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Inserting an SD-card on an Intel NUC10i3FNK4 (which contains a GL9755) results in the message: mmc0: 1.8V regulator output did not become stable Following this message, some cards work (sometimes), but most cards fail with EILSEQ. This behaviour is observed on Debian 10 running kernel 4.19.188, but also with 5.8.18 and 5.11.15. The driver currently waits 5ms after switching on the 1.8V regulator for it to become stable. Increasing this to 10ms gets rid of the warning about stability, but most cards still fail. Increasing it to 20ms gets some cards working (a 32GB Samsung micro SD works, a 128GB ADATA doesn't). At 50ms, the ADATA works most of the time, and at 100ms both cards work reliably. Signed-off-by: Daniel Beer --- drivers/mmc/host/sdhci-pci-gli.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/mmc/host/sdhci-pci-gli.c b/drivers/mmc/host/sdhci-pci-gli.c index 592d79082f58..061618aa247f 100644 --- a/drivers/mmc/host/sdhci-pci-gli.c +++ b/drivers/mmc/host/sdhci-pci-gli.c @@ -627,8 +627,13 @@ static void sdhci_gli_voltage_switch(struct sdhci_host *host) * * Wait 5ms after set 1.8V signal enable in Host Control 2 register * to ensure 1.8V signal enable bit is set by GL9750/GL9755. + * + * ...however, the controller in the NUC10i3FNK4 (a 9755) requires + * slightly longer than 5ms before the control register reports that + * 1.8V is ready, and far longer still before the card will actually + * work reliably. */ - usleep_range(5000, 5500); + usleep_range(100000, 110000); } static void sdhci_gl9750_reset(struct sdhci_host *host, u8 mask) -- 2.20.1