Re: [PATCH v4 1/2] dt-bindings: memory-controllers: Add MediaTek DRAM controller interface

Krzysztof Kozlowski
2025-04-06T12:35:26Z

On Thu, Apr 03, 2025 at 02:48:47PM GMT, Crystal Guo wrote:
> +maintainers:
> + - Crystal Guo <crystal.guo@mediatek.com>
> +
> +description:
> + A MediaTek DRAM controller interface to provide the current data rate of DRAM.

DRAM controller does not offer scaling? Or PHY/timing configuration?
This binding looks pretty incomplete.

> +
> +properties:
> + compatible:
> + items:
> + - enum:
> + - mediatek,mt8196-dramc
> +
> + reg:
> + items:
> + - description: anaphy registers
> + - description: ddrphy registers
> +
> +additionalProperties: false

If there is going to be any resend then this goes after required: block.

> +
> +required:
> + - compatible
> + - reg

Best regards,
Krzysztof