From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.7 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 55677C432C3 for ; Thu, 14 Nov 2019 13:11:05 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 3673120709 for ; Thu, 14 Nov 2019 13:11:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726378AbfKNNLE (ORCPT ); Thu, 14 Nov 2019 08:11:04 -0500 Received: from gloria.sntech.de ([185.11.138.130]:34068 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726202AbfKNNLE (ORCPT ); Thu, 14 Nov 2019 08:11:04 -0500 Received: from wf0530.dip.tu-dresden.de ([141.76.182.18] helo=phil.localnet) by gloria.sntech.de with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1iVEtU-0002Qj-R0; Thu, 14 Nov 2019 14:10:40 +0100 From: Heiko Stuebner To: Markus Reichl Cc: Christoph Muellner , robh+dt@kernel.org, mark.rutland@arm.com, shawn.lin@rock-chips.com, devicetree@vger.kernel.org, Jeffy Chen , linux-kernel@vger.kernel.org, Viresh Kumar , Brian Norris , Douglas Anderson , Vicente Bergas , Kishon Vijay Abraham I , linux-rockchip@lists.infradead.org, Tony Xie , Klaus Goger , Enric Balletbo i Serra , Randy Li , Philipp Tomsich , Ezequiel Garcia , linux-arm-kernel@lists.infradead.org, "linux-mmc@vger.kernel.org" Subject: Re: arm64: dts: rockchip: Disable HS400 for mmc on rk3399-roc-pc Date: Thu, 14 Nov 2019 14:10:39 +0100 Message-ID: <2766673.iMURPl8gB5@phil> In-Reply-To: <367bf78a-f079-f0b4-68fe-52c86823c174@fivetechno.de> References: <20190301153348.29870-1-christoph.muellner@theobroma-systems.com> <20190301153348.29870-2-christoph.muellner@theobroma-systems.com> <367bf78a-f079-f0b4-68fe-52c86823c174@fivetechno.de> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Markus, $subject is missing the [PATCH] prefix Am Montag, 11. November 2019, 10:51:04 CET schrieb Markus Reichl: > Working with rootfs on two 128GB mmcs on rk3399-roc-pc. > > One (mmc name 128G72, one screw hole) works fine in HS400 mode. > Other (mmc name DJNB4R, firefly on pcb, two screw holes) gets lots of > mmc1: "running CQE recovery", even hangs with damaged fs, > when running under heavy load, e.g. compiling kernel. > Both run fine with HS200. > > Disabling CQ with patch mmc: core: Add MMC Command Queue Support kernel parameter [0] did not help. > [0] https://gitlab.com/ayufan-repos/rock64/linux-mainline-kernel/commit/54e264154b87dfe32a8359b2726e2d5611adbaf3 I'm hoping for some input from other people in Cc but your mail headers also referenced the drive-impendance series from Christoph [0], which it seems we need to poke the phy maintainer again. Did you check if changing the impedance helped (like the signal dampening Philipp described in one of the replies there). [0] https://patchwork.kernel.org/patch/10835567/ most current v2 it seems is https://patchwork.kernel.org/patch/10842421/ > Therefore I propose to disable HS400 mode on roc-pc for now. Hoping for more input :-) Heiko > Signed-off-by: Markus Reichl > --- > arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dtsi | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dtsi > index 29a50a083c42..33df95e384b4 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dtsi > @@ -660,8 +660,6 @@ > > &sdhci { > bus-width = <8>; > - mmc-hs400-1_8v; > - mmc-hs400-enhanced-strobe; > non-removable; > status = "okay"; > }; >