Re: [RFC PATCH 03/22] dt-bindings: memory: ti,j721s2-msmc: Add TI K3 MSMC binding

From: Krzysztof Kozlowski

Date: Wed Jul 15 2026 - 00:57:21 EST


On 14/07/2026 14:55, MANNURU VENKATESWARLU wrote:
> +required:
> + - compatible
> + - '#address-cells'
> + - '#size-cells'
> + - ranges
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/interrupt-controller/arm-gic.h>
> + #include <dt-bindings/soc/ti,sci_pm_domain.h>
> +
> + msmc0: msmc {
> + compatible = "ti,j721s2-msmc", "simple-bus";

NAK

> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> + intrlv-gran = <0>;
> + intrlv-size = <0>;
> + ecc-enable = <0>;
> + emif-config = <0>;
> + emif-active = <0>;

I do not accept downstream code sent to review.

Start doing proper internal reviews. This binding and DTS is absolutely
unacceptable, you just repeat all known mistakes ignoring any guidelines.

Best regards,
Krzysztof