linux-kernel - Re: [PATCH] arm64: dts: qcom: sm8550: camss: Add CAMSS block definition

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: <c2a3e578-b098-450f-96f6-a3ae321f2b4c@kernel.org>
Date: Wed, 7 Aug 2024 14:43:09 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Depeng Shao <quic_depengs@...cinc.com>, andersson@...nel.org,
 robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
 bryan.odonoghue@...aro.org
Cc: linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, kernel@...cinc.com,
 Yongsheng Li <quic_yon@...cinc.com>
Subject: Re: [PATCH] arm64: dts: qcom: sm8550: camss: Add CAMSS block
 definition

On 07/08/2024 14:39, Krzysztof Kozlowski wrote:
> On 07/08/2024 14:33, Depeng Shao wrote:
>> Add CAMSS block definition for sm8550.
>>
>> This drop contains definitions for the following components on sm8550:
> 
> 1. Subject: there is no prefix camss. There is no such file, directory
> or module.
> 
> 2. You already sent this, so this should be v2 or v3 or vX. Provide
> changelog under ---.
> 
> If there is going to be resend, please fix above.
> 
> 3. If this was tested on aim300, I am surprised this being not enabled
> on aim300.

One more thing, bindings were not accepted, thus this patch should not
go in. There were no new bindings, so I assume patchset is using
rejected ones.

It's fine to send it to get some comments, although would be nice to
mention to maintainer that this cannot be picked up as is. :(

Best regards,
Krzysztof