Re: [PATCH v4 1/8] dt-bindings: mmc: add Cadence SD6HC binding

From: Krzysztof Kozlowski

Date: Wed Aug 12 2026 - 06:32:03 EST


On Mon, Aug 10, 2026 at 12:00:15PM -0700, Tanmay Kathpalia wrote:
> + cdns,iocell-input-delay-ps:
> + description:
> + Input delay across the IO cells, used for SD6HC PHY timing calculations.
> + minimum: 0
> + maximum: 20000
> +
> + cdns,iocell-output-delay-ps:
> + description:
> + Output delay across the IO cells, used for SD6HC PHY timing calculations.
> + minimum: 0
> + maximum: 20000
> +
> + cdns,delay-element-ps:
> + description:
> + DLL delay element size, used for SD6HC PHY timing calculations.
> + minimum: 1
> + maximum: 1000

I missed this in previous reviews:

It seems you have defaults for these three, so "default: xyz"

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>

Best regards,
Krzysztof