lkml.org 
[lkml]   [2024]   [Jan]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [RFC PATCH 00/13] Add tuning algorithm for delay chain
From
On 1/31/24 7:35 AM, Raghavendra, Vignesh wrote:
> Hi,
>
> On 1/31/2024 6:07 AM, Judith Mendez wrote:
>> This patch series introduces a new tuning algorithm for
>> mmc. The new algorithm should be used when delay chain is
>> enabled. The ITAPDLY is selected from the largest passing
>> window and the buffer is not viewed as a circular buffer.
>> The new tuning algorithm is implemented as per the paper
>> published here [0] and has been tested on the following
>> platforms: AM62x SK, AM62A SK, AM62p SK, AM64x SK, and AM64x
>> EVM.
>>
>> The series also includes a few fixes in the sdhci_am654
>> driver on OTAPDLYEN/ITAPDLYEN and ITAPDELSEL. There are
>> also device tree node fixes for missing mmc nodes,
>> modifying DLL properties, and fixes for OTAP/ITAP delay
>> values.
>>
>> MMC0/MMC2 nodes are introduced for AM62ax in this series.
>>
>> This series is sent as a RFC mostly to get some feedback
>> and/or comments on the new tuning algorithm implementation.
>>
>> [0] https://www.ti.com/lit/an/spract9/spract9.pdf
>>
>
>
>> Judith Mendez (11):
>> drivers: mmc: host: sdhci_am654: Add tuning algorithm for delay chain
>> drivers: mmc: host: sdhci_am654: Write ITAPDLY for DDR52 timing
>> drivers: mmc: host: sdhci_am654: Add missing OTAP/ITAP enable
>> drivers: mmc: host: sdhci_am654: Add ITAPDLYSEL in
>> sdhci_j721e_4bit_set_clock
>> drivers: mmc: host: sdhci_am654: Fix ITAPDLY for HS400 timing
>
> These patches needs to have Fixes: tag as they are bug fixes IMO.

Understood, will add.

>
>> arm64: dts: ti: k3-am62a-main: Add sdhci2 instance
>> arm64: dts: ti: k3-am64-main: Update ITAP/OTAP values for MMC
>> arm64: dts: ti: k3-am62-main: Update ITAP/OTAP values for MMC
>> arm64: dts: ti: k3-am62p: Add missing properties for MMC
>> arm64: dts: ti: k3-am6*: Remove DLL properties for soft phys
>> arm64: dts: ti: k3-am6*: Reorganize MMC properties
>>
>> Nitin Yadav (2):
>> arm64: dts: ti: k3-am62a-main: Add sdhci0 instance
>> arm64: dts: ti: k3-am62a7-sk: Enable eMMC support
>>
>
> Can the driver changes be merged independent of DT changes? Or are they
> meant to go together? Latter would be problematic as it creates cross
> tree dependencies.

The driver changes can be merged independently.

>
>> arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 57 +++--
>> .../arm64/boot/dts/ti/k3-am625-beagleplay.dts | 5 -
>> arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 45 +++-
>> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 27 ++-
>> arch/arm64/boot/dts/ti/k3-am62p-main.dtsi | 44 +++-
>> arch/arm64/boot/dts/ti/k3-am62p5-sk.dts | 7 +-
>> .../arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 4 +-
>> arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 17 +-
>> arch/arm64/boot/dts/ti/k3-am642-evm.dts | 4 +-
>> arch/arm64/boot/dts/ti/k3-am642-sk.dts | 2 -
>> drivers/mmc/host/sdhci_am654.c | 215 ++++++++++++++----
>> 11 files changed, 321 insertions(+), 106 deletions(-)
>>


\
 
 \ /
  Last update: 2024-05-27 14:43    [W:0.199 / U:3.820 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site