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

Denzeel Oliva posted 3 patches 10 months, 1 week ago
.../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(+)
[PATCH v1 0/3] arm64: dts: exynos990: define all PERIC0/1 USI nodes
Posted by Denzeel Oliva 10 months, 1 week ago
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(+)

-- 
2.48.1
Re: [PATCH v1 0/3] arm64: dts: exynos990: define all PERIC0/1 USI nodes
Posted by Krzysztof Kozlowski 10 months, 1 week ago
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