Hi Philippe, Thank you for the patch. On Wed, Jan 18, 2023 at 11:52:51AM +0100, Philippe Schenker wrote: > From: Philippe Schenker <philippe.schenker@xxxxxxxxxxx> > > Follow the change that has been done on imx8mm-verdin-dev.dtsi and > remove the include from dahlia to be consistent. Put back all nodes that > previously had been included. > > Signed-off-by: Philippe Schenker <philippe.schenker@xxxxxxxxxxx> > > --- > > .../boot/dts/freescale/imx8mp-verdin-dev.dtsi | 116 +++++++++++++++++- > 1 file changed, 114 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-verdin-dev.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-verdin-dev.dtsi > index cefabe65b252..361426c0da0a 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp-verdin-dev.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mp-verdin-dev.dtsi > @@ -3,8 +3,6 @@ > * Copyright 2022 Toradex > */ > > -#include "imx8mp-verdin-dahlia.dtsi" > - Makes sense. Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > / { > /* TODO: Audio Codec */ > > @@ -21,16 +19,99 @@ reg_eth2phy: regulator-eth2phy { > }; > }; > > +&backlight { > + power-supply = <®_3p3v>; > +}; > + > +/* Verdin SPI_1 */ > +&ecspi1 { > + status = "okay"; > +}; > + > +/* EEPROM on display adapter boards */ > +&eeprom_display_adapter { > + status = "okay"; > +}; > + > +/* EEPROM on Verdin Development board */ > +&eeprom_carrier_board { > + status = "okay"; > +}; > + > +&eqos { > + status = "okay"; > +}; > + > &fec { > phy-supply = <®_eth2phy>; > status = "okay"; > }; > > +&flexcan1 { > + status = "okay"; > +}; > + > +&flexcan2 { > + status = "okay"; > +}; > + > +/* Verdin QSPI_1 */ > +&flexspi { > + status = "okay"; > +}; > + > &gpio_expander_21 { > status = "okay"; > vcc-supply = <®_1p8v>; > }; > > +/* Current measurement into module VCC */ > +&hwmon { > + status = "okay"; > +}; > + > +&hwmon_temp { > + vs-supply = <®_1p8v>; > + status = "okay"; > +}; > + > +/* Verdin I2C_2_DSI */ > +&i2c2 { > + status = "okay"; > +}; > + > +&i2c3 { > + status = "okay"; > +}; > + > +/* Verdin I2C_1 */ > +&i2c4 { > + status = "okay"; > + > + /* TODO: Audio Codec */ > +}; > + > +/* TODO: Verdin PCIE_1 */ > + > +/* Verdin PWM_1 */ > +&pwm1 { > + status = "okay"; > +}; > + > +/* Verdin PWM_2 */ > +&pwm2 { > + status = "okay"; > +}; > + > +/* Verdin PWM_3_DSI */ > +&pwm3 { > + status = "okay"; > +}; > + > +®_usdhc2_vmmc { > + vin-supply = <®_3p3v>; > +}; > + > /* TODO: Verdin I2C_1 with Audio Codec */ > > /* Verdin UART_1, connector X50 through RS485 transceiver */ > @@ -38,9 +119,40 @@ &uart1 { > linux,rs485-enabled-at-boot-time; > rs485-rts-active-low; > rs485-rx-during-tx; > + status = "okay"; > +}; > + > +/* Verdin UART_2 */ > +&uart2 { > + status = "okay"; > +}; > + > +/* Verdin UART_3, used as the Linux Console */ > +&uart3 { > + status = "okay"; > +}; > + > +/* Verdin USB_1 */ > +&usb3_0 { > + status = "okay"; > +}; > + > +&usb3_phy0 { > + status = "okay"; > +}; > + > +/* Verdin USB_2 */ > +&usb3_1 { > + fsl,permanently-attached; > + status = "okay"; > +}; > + > +&usb3_phy1 { > + status = "okay"; > }; > > /* Limit frequency on dev board due to long traces and bad signal integrity */ > &usdhc2 { > max-frequency = <100000000>; > + status = "okay"; > }; -- Regards, Laurent Pinchart