[v1,0/3] arm64: dts: exynos990: define all PERIC0/1 USI nodes

Message

Denzeel Oliva Feb. 12, 2025, 11:40 p.m. UTC

This series adds device tree bindings and nodes for the
Universal Serial Interface (USI) and UART controllers on
Samsung Exynos990 SoC.

The Exynos990 USI block supports multiple protocols (UART, SPI, I2C)
through shared hardware resources.

And also add dt-schema for USI and UART compatibility.

Denzeel Oliva (3):
  dt-bindings: samsung: usi: add exynos990-usi compatible
  dt-bindings: serial: samsung: add Exynos990 compatible
  arm64: dts: exynos990: define all PERIC USI nodes

 .../bindings/serial/samsung_uart.yaml         |    6 +
 .../bindings/soc/samsung/exynos-usi.yaml      |    1 +
 arch/arm64/boot/dts/exynos/exynos990.dtsi     | 1693 +++++++++++++++++
 3 files changed, 1700 insertions(+)

Comments

Krzysztof Kozlowski Feb. 14, 2025, 8:20 a.m. UTC | #1

On 13/02/2025 00:40, Denzeel Oliva wrote:
> This series adds device tree bindings and nodes for the
> Universal Serial Interface (USI) and UART controllers on
> Samsung Exynos990 SoC.
> 
> The Exynos990 USI block supports multiple protocols (UART, SPI, I2C)
> through shared hardware resources.
> 
> And also add dt-schema for USI and UART compatibility.
> 
> Denzeel Oliva (3):
>   dt-bindings: samsung: usi: add exynos990-usi compatible
>   dt-bindings: serial: samsung: add Exynos990 compatible
>   arm64: dts: exynos990: define all PERIC USI nodes

Within  few days you sent more than one patchset targeting soc, but this
one also targets serial.

1. Split your patchsets per subsystem.
2. Don't send one DTS patch each day, but collect them and send one
patchset with all of them. At least within some time frame.

You also got comment on your previous v1 (and this is v2? any
changelog?) that all this looks unmergeable, but nothing in commit msg
explained that. I don't know how to treat the rest of your submissions.

Best regards,
Krzysztof