diff options
| author | Frieder Schrempf <frieder.schrempf@kontron.de> | 2026-09-21 12:16:44 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2026-09-21 14:15:03 +0200 |
| commit | 2c68b2403acd1cbf0b19c71dab9858cd2ca6bbb5 (patch) | |
| tree | 0a27b3b44af42c8aaf5466662e08c7231d4dfbc8 /tools/perf/scripts/python | |
| parent | 93f51579e7df248780214094418f205253383cc5 (diff) | |
spi: dt-bindings: Clarify what rx-sample-delay-ns describes
The property is documented as an offset from the controller's default
sampling time. That says what the value does, but not what it is meant to
describe, and it leaves room for two readings: a delay that belongs to
the board, or a stand-in for the peripheral's own clock-to-output-valid
time, which has no other expression in a device tree today.
The first reading is the original one. The property was introduced for
Rockchip in commit 76b17e6e4923 ("spi/rockchip: Add device tree property
to configure Rx Sample Delay") to deal with "boards with high-capacitance
SPI lines", where "the controller samples the Rx data line too early".
The current wording arrived later, in commit 5ce78f4456a9 ("dt-bindings:
snps, dw-apb-ssi: Add sparx5 support, plus rx-sample-delay-ns property"),
where it described a DesignWare register and was qualified as such. That
qualification was dropped when the property was moved to the generic
schema in commit b658be56e867 ("spi: dt-bindings: Move
'rx-sample-delay-ns' to spi-peripheral-props.yaml"), leaving a register
description standing in for a definition.
The distinction matters because the two compose. A board delay is
specific to one design, while a datasheet timing parameter is the same
wherever the chip is soldered, so a mechanism that derives the chip side
from the chip would add to a value that already accounts for it.
Spell out that the property describes the board. Nothing changes in what
the value means or in how existing device trees are interpreted.
Assisted-by: Claude:claude-opus-5
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
Link: https://patch.msgid.link/20260921-spi-sample-delay-cleanup-v2-1-b460d572fd32@kontron.de
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions
