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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <11b9864c-56a7-48c2-9dc9-9567ca24d975@kernel.org>
Date: Fri, 14 Feb 2025 09:20:06 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Denzeel Oliva <wachiturroxd150@...il.com>, gregkh@...uxfoundation.org,
 jirislaby@...nel.org, robh@...nel.org, krzk+dt@...nel.org,
 conor+dt@...nel.org, alim.akhtar@...sung.com, semen.protsenko@...aro.org,
 linux-kernel@...r.kernel.org, linux-serial@...r.kernel.org,
 devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-samsung-soc@...r.kernel.org
Subject: Re: [PATCH v1 0/3] arm64: dts: exynos990: define all PERIC0/1 USI
 nodes

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