From: Adrian Hunter <adrian.hunter@intel.com>
To: Thierry Reding <thierry.reding@gmail.com>
Cc: Aapo Vienamo <avienamo@nvidia.com>,
Ulf Hansson <ulf.hansson@linaro.org>,
Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Jonathan Hunter <jonathanh@nvidia.com>,
Mikko Perttunen <mperttunen@nvidia.com>,
linux-mmc@vger.kernel.org, devicetree@vger.kernel.org,
linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/8] Tegra SDHCI support HS400 on Tegra210 and Tegra186
Date: Mon, 27 Aug 2018 14:47:49 +0300 [thread overview]
Message-ID: <aed782d2-9bf1-f750-88eb-3501fab5ecd3@intel.com> (raw)
In-Reply-To: <095142c4-2697-d9e1-4930-a39e27b7ef40@intel.com>
On 27/08/18 13:26, Adrian Hunter wrote:
> On 27/08/18 13:08, Thierry Reding wrote:
>> On Fri, Aug 10, 2018 at 09:13:57PM +0300, Aapo Vienamo wrote:
>>> Hi all,
>>> This series implements support for HS400 signaling on Tegra210 and
>>> Tegra186. This includes programming the DQS trimmer values, implementing
>>> enhanced strobe and HS400 delay line calibration.
>>>
>>> This series depends on the "Tegra SDHCI add support for HS200 and UHS
>>> signaling" series.
>>>
>>> Changelog:
>>> v2:
>>> - Document in dt-bindings which controllers support HS400
>>> - Use val instead of reg in tegra_sdhci_set_dqs_trim()
>>> - Change "dt" to "DT" in "mmc: tegra: Parse and program DQS trim
>>> value" commit message
>>> - Add spaces around << in tegra_sdhci_set_dqs_trim()
>>> - Make the "mmc: tegra: Implement HS400 enhanced strobe" commit
>>> message more detailed
>>> - Remove a debug print from tegra_sdhci_hs400_enhanced_strobe()
>>> - Add blank lines around if-else-block in
>>> tegra_sdhci_hs400_enhanced_strobe()
>>> - Use val instead of reg in tegra_sdhci_hs400_enhanced_strobe()
>>> - Make commit message of "mmc: tegra: Implement HS400 delay line
>>> calibration" more detailed
>>>
>>> Aapo Vienamo (8):
>>> dt-bindings: mmc: Add DQS trim value to Tegra SDHCI
>>> mmc: tegra: Parse and program DQS trim value
>>> mmc: tegra: Implement HS400 enhanced strobe
>>> mmc: tegra: Implement HS400 delay line calibration
>>> arm64: dts: tegra186: Add SDMMC4 DQS trim value
>>> arm64: dts: tegra210: Add SDMMC4 DQS trim value
>>> arm64: dts: tegra186: Enable HS400
>>> arm64: dts: tegra210: Enable HS400
>>>
>>> .../bindings/mmc/nvidia,tegra20-sdhci.txt | 4 ++
>>> arch/arm64/boot/dts/nvidia/tegra186.dtsi | 2 +
>>> arch/arm64/boot/dts/nvidia/tegra210.dtsi | 2 +
>>> drivers/mmc/host/sdhci-tegra.c | 84 +++++++++++++++++++++-
>>> 4 files changed, 89 insertions(+), 3 deletions(-)
>>
>> Adrian,
>>
>> any chance you could take a brief look at these? They are a prerequisite
>> for the 2-patch series ("[PATCH 0/2] Tegra SDHCI rerun pad calibration
>> periodically") that you already acked.
>
> Sure, I'll try and have a look today.
>
These are fine. For the sdhci-tegra patches (patches 2 - 4):
Acked-by: Adrian Hunter <adrian.hunter@intel.com>
next prev parent reply other threads:[~2018-08-27 11:50 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-10 18:13 [PATCH v2 0/8] Tegra SDHCI support HS400 on Tegra210 and Tegra186 Aapo Vienamo
2018-08-10 18:13 ` [PATCH v2 1/8] dt-bindings: mmc: Add DQS trim value to Tegra SDHCI Aapo Vienamo
2018-08-13 19:26 ` Rob Herring
2018-08-10 18:13 ` [PATCH v2 2/8] mmc: tegra: Parse and program DQS trim value Aapo Vienamo
2018-08-10 18:14 ` [PATCH v2 3/8] mmc: tegra: Implement HS400 enhanced strobe Aapo Vienamo
2018-08-10 18:14 ` [PATCH v2 4/8] mmc: tegra: Implement HS400 delay line calibration Aapo Vienamo
2018-08-10 18:14 ` [PATCH v2 5/8] arm64: dts: tegra186: Add SDMMC4 DQS trim value Aapo Vienamo
2018-08-10 18:14 ` [PATCH v2 6/8] arm64: dts: tegra210: " Aapo Vienamo
2018-08-10 18:14 ` [PATCH v2 7/8] arm64: dts: tegra186: Enable HS400 Aapo Vienamo
2018-08-10 18:14 ` [PATCH v2 8/8] arm64: dts: tegra210: " Aapo Vienamo
2018-08-24 9:11 ` [PATCH v2 0/8] Tegra SDHCI support HS400 on Tegra210 and Tegra186 Thierry Reding
2018-08-27 9:50 ` Ulf Hansson
2018-08-27 10:08 ` Thierry Reding
2018-08-27 10:26 ` Adrian Hunter
2018-08-27 11:47 ` Adrian Hunter [this message]
2018-08-31 14:02 ` Thierry Reding
2018-09-03 6:14 ` Ulf Hansson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aed782d2-9bf1-f750-88eb-3501fab5ecd3@intel.com \
--to=adrian.hunter@intel.com \
--cc=avienamo@nvidia.com \
--cc=devicetree@vger.kernel.org \
--cc=jonathanh@nvidia.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mperttunen@nvidia.com \
--cc=robh+dt@kernel.org \
--cc=thierry.reding@gmail.com \
--cc=ulf.hansson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).