Re: [RESEND PATCH v7 15/16] arm64: dts: st: support Engicam MicroGEA-STM32MP257-RMM board
Alexandre TORGUE <[email protected]>
| Newsgroups | org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
Hi Dario On 7/15/26 08:20, Dario Binacchi wrote: > Support for Engicam MicroGEA-STM32MP257-RMM board with: > > - 8 GB eMMC Flash > - 2 GB LPDDR4 DRAM > - CAN > - LEDs > - LCD panel with touchscreen > - Micro SD card connector > - Audio codec > - Buzzer > > Signed-off-by: Dario Binacchi <[email protected]> > > --- > > (no changes since v5) > > Changes in v5: > - Fix touchscreen resolution to 480x854 > - Fix SPI1 CS0 polarity to GPIO_ACTIVE_LOW > > Changes in v2: > - Drop the clocks property from the sai1 node in stm32mp257-engicam-microgea-rmm.dts > to avoid overriding the peripheral bus clock reference defined in the base > SoC device tree. Suggested by Sashiko. > - Reference the existing labeled nodes directly at the root level using > &sai1a and &sai1b in stm32mp257-engicam-microgea-rmm.dts instead of > redefining the entire node structure and redeclaring the labels. Suggested by Sashiko. > - Drop the #clock-cells property from sai1a and remove the reference to sai1a from > the clocks array in sai1b, relying strictly on the st,sync property to handle > internal synchronization. > > arch/arm64/boot/dts/st/Makefile | 1 + > .../st/stm32mp257-engicam-microgea-rmm.dts | 319 ++++++++++++++++++ Series is ok, I just have a question regarding board name. Why you don't add the "d" soc information inside the board name as it is done for others STM32 boards ?