Re: [PATCH 4/4] dts: arm64: imx8dxl-evk: add lpuart1 and cm40 uart

From: Fabio Estevam
Date: Wed Feb 28 2024 - 15:05:17 EST


On Wed, Feb 28, 2024 at 4:55 PM Frank Li <Frank.Li@xxxxxxx> wrote:

> + modem_reset: modem-reset {
> + compatible = "gpio-reset";

I don't see this compatible documented.

> + reset-gpios = <&pca6416_2 0 GPIO_ACTIVE_LOW>;
> + reset-delay-us = <2000>;
> + reset-post-delay-ms = <40>;
> + #reset-cells = <0>;
> + };
> +
> + m2_uart1_sel: fixedregulator@101 {

Why is this @101 here? You should remove it.

Passing a unit address without a corresponding reg entry triggers a
dt-schema warning.

We want less warnings dt-schema warnings, not more :-)