linux-kernel - Re: [PATCH 2/3] arm64: dts: qcom: sm8550: Add tz-log node

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] [day] [month] [year] [list]

Message-ID: <0003915e-26ee-4f8c-bb5d-f4823b88a2aa@oss.qualcomm.com>
Date: Tue, 10 Feb 2026 09:50:44 +0100
From: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
To: Rob Herring <robh@...nel.org>
Cc: webgeek1234@...il.com, Bjorn Andersson <andersson@...nel.org>,
        Konrad Dybcio <konradybcio@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley
 <conor+dt@...nel.org>,
        Kumar Sharma <quic_vksharma@...cinc.com>,
        linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, Lei Chen <quic_chenlei@...cinc.com>
Subject: Re: [PATCH 2/3] arm64: dts: qcom: sm8550: Add tz-log node

On 2/9/26 6:07 PM, Rob Herring wrote:
> On Fri, Jan 30, 2026 at 11:59:15AM +0100, Konrad Dybcio wrote:
>> On 1/29/26 8:46 AM, Aaron Kling via B4 Relay wrote:
>>> From: Lei Chen <quic_chenlei@...cinc.com>
>>>
>>> Add DT node to enable tz-log driver.
>>>
>>> Signed-off-by: Lei Chen <quic_chenlei@...cinc.com>
>>> Signed-off-by: Aaron Kling <webgeek1234@...il.com>
>>> ---
>>
>> It's nice that you preserved the original authorship.
>>
>> Please extend the rather lackluster commit message to explain the
>> "why", which is notably different from the original downstream
>> addition, since your goal here is to mainly appease a grumpy
>> bootloader.
> 
> Humm, that's certainly good to know. Though I don't think it will help 
> the case for this binding.

FYI I'm not really interested in having *that* binding upstream, the
patch contributor first believed that it'd be necessary for the
bootloader to consume it (refusing to boot otherwise), but as we figured
out in another branch of this(?) thread, simply a label was enough

Konrad