From nobody Wed Dec 8 12:29:13 2021 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 22C9D18C81F0 for ; Wed, 8 Dec 2021 12:29:25 +0000 (UTC) (envelope-from wma@semihalf.com) Received: from mail-yb1-xb2e.google.com (mail-yb1-xb2e.google.com [IPv6:2607:f8b0:4864:20::b2e]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1D4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4J8Ggm6ypsz3PQ5 for ; Wed, 8 Dec 2021 12:29:24 +0000 (UTC) (envelope-from wma@semihalf.com) Received: by mail-yb1-xb2e.google.com with SMTP id e136so5604915ybc.4 for ; Wed, 08 Dec 2021 04:29:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=semihalf-com.20210112.gappssmtp.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=4zF4++3jN5rj2/1mC21lqHTwQwEx1WQvJo7O/fPeG0E=; b=Wmsa13AxQ/gmY7iTwupVLxC/HO9C5tBJ7vJhmcOMYHrGhVR4Q83JpWlbG9TEpL10eL gvOPtOn38IzPVhNEPhjNzKd+dEfnbD2nDf7lD/EB1At4SBGGr8cavG7J/cIjnqmmCFnS M4eOKlPSrjlrVN4wkEoCiiTOc35Z3ZIaSMgectlqzimkhb2RPL8z2GluGFWR82lHSXOx FZFSfwcIuWKbdE+ObMDBPTDSZaPoWe7bKAOANIbts6NUQrXwpkQeAb+OR5TvIay+9ftG zFlZHdddGpa7cNDx/hj/puKv8wCVF01iC+r4soaqLmPzMUN5ub9nDJApJ31kcYe0tEoj DnCw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=4zF4++3jN5rj2/1mC21lqHTwQwEx1WQvJo7O/fPeG0E=; b=lWGia+wGFlUiNV+vN5D0pFWJ2uJwgEvmnaKoHE06tbrigzjd3vv+S/+Ov+CoqVCY5M goGRDxCHCeyp50I/wjxqSvvkErSmUyr6r64cG4as6Bxuf7rpYsm+bKL3wpYzlV0OtADE rZgKJhsQngFsn3GPMfMAWl9xV/O50PYGfEX7LzCHH4quCE0RuBWwZtwtCSx+JNKDOQSr Ex5Si0JB0GSX/gkyN0ZaFbmKc5dZhbeGeM4qd1brpnnhj4KT4kXUm8t+fd+SPLA0GlR2 uvLT5hIIBIFHs4Of6JB71YM/wT2TxJz1HBiA0zKGaV0WCUyiw5SiB5HdLDqicmqPxKNU Shiw== X-Gm-Message-State: AOAM53383DiV7cdUcBFgN4dwL9FXsh9AP/CfvVaaAa2lJFXTynrzgxzU 3SgtjD6gQR79XZpcMNZ7gy+DUor9fHPyq8coJn5clQ== X-Google-Smtp-Source: ABdhPJwng+8go6jMObg6LgqDTIQZau29VTD/xQ5UqUNJSVk1a4VNGHGG89NP0XpqueezxWTNarZkE0q6K2ptvyX59PE= X-Received: by 2002:a25:e746:: with SMTP id e67mr57765271ybh.476.1638966564447; Wed, 08 Dec 2021 04:29:24 -0800 (PST) List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 References: <202112081021.1B8ALYH0041586@gitrepo.freebsd.org> <20211208113328.295de39f3c1b976d5c07531f@bidouilliste.com> In-Reply-To: <20211208113328.295de39f3c1b976d5c07531f@bidouilliste.com> From: Wojciech Macek Date: Wed, 8 Dec 2021 13:29:13 +0100 Message-ID: Subject: Re: git: de9c000cedfe - main - sdhci_fsl_fdt: Add support for HS200/HS400 modes To: Emmanuel Vadot Cc: Wojciech Macek , src-committers , dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Content-Type: multipart/alternative; boundary="000000000000df7ea905d2a1a36e" X-Rspamd-Queue-Id: 4J8Ggm6ypsz3PQ5 X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-ThisMailContainsUnwantedMimeParts: N --000000000000df7ea905d2a1a36e Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Ah, sorry, I thought this one was ready as well. Please let me know if you think sth needs to be changed, we'll do a follow up then Wojtek =C5=9Br., 8 gru 2021 o 11:33 Emmanuel Vadot napisa= =C5=82(a): > On Wed, 8 Dec 2021 10:21:34 GMT > Wojciech Macek wrote: > > > The branch main has been updated by wma: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=3Dde9c000cedfe77c6da4929178fee571= b1fd0cc80 > > > > commit de9c000cedfe77c6da4929178fee571b1fd0cc80 > > Author: Hubert Mazur > > AuthorDate: 2021-11-03 09:38:05 +0000 > > Commit: Wojciech Macek > > CommitDate: 2021-12-08 10:21:02 +0000 > > > > sdhci_fsl_fdt: Add support for HS200/HS400 modes > > > > The controller requires some custom logic to perform MMC tuning > > and to later switch to HS400 mode. Implement it supplying mmcbr_tun= e > > and sdhci_set_uhs_timing devmethods respectivly. Since the latter > > is called unconditionally when the ios is updated we need to keep > > track of the tuning state in sc and execute the HS400 switch logic > > only when required. > > > > Two HS200/HS400 related errata were implemented. > > 1. In HS400 modes the clock divisors are limited to 4, 8, 12. > > Apply it by falling back to the closes, higher divider when > needed. > > 2. Hardware tuning procedure can sometimes fails. If that is the ca= se > > fallback to the software tuning. > > > > Reviewed by: manu, mw > > I did not review that one, it's way larger than the other ones so I > wanted to have a look later. > > > Obtained from: Semihalf > > Sponsored by: Alstom Group > > Differential revision: https://reviews.freebsd.org/D33320 > > > -- > Emmanuel Vadot > --000000000000df7ea905d2a1a36e--