Hi Rob, On Mon, 2022-12-19 at 09:38 -0600, Rob Herring wrote: > On Mon, Dec 19, 2022 at 10:40:15AM +0800, Xiangsheng Hou wrote: > > Add mediatek,rx-latch-latency-ns property which adjust data read > > latch latency in the unit of nanoseconds. > > > > Signed-off-by: Xiangsheng Hou <xiangsheng.hou@xxxxxxxxxxxx> > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > > Reviewed-by: AngeloGioacchino Del Regno < > > angelogioacchino.delregno@xxxxxxxxxxxxx> > > --- > > .../devicetree/bindings/spi/mediatek,spi-mtk-snfi.yaml | 3 > > +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/spi/mediatek,spi- > > mtk-snfi.yaml b/Documentation/devicetree/bindings/spi/mediatek,spi- > > mtk-snfi.yaml > > index bab23f1b11fd..1e5e89a693c3 100644 > > --- a/Documentation/devicetree/bindings/spi/mediatek,spi-mtk- > > snfi.yaml > > +++ b/Documentation/devicetree/bindings/spi/mediatek,spi-mtk- > > snfi.yaml > > @@ -45,6 +45,9 @@ properties: > > description: device-tree node of the accompanying ECC engine. > > $ref: /schemas/types.yaml#/definitions/phandle > > > > + mediatek,rx-latch-latency-ns: > > + description: Data read latch latency, unit is nanoseconds. > > Doesn't the common 'rx-sample-delay-ns' work for you? The driver need two timing related parameter, one for sample delay which have been used by rx-sample-delay-ns. Another is read latency, just introduce this private timing property since the common spi-rx- delay-us is microsecond in unit. Thanks XIangsheng Hou