[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <6f085aee-e1a3-4171-822e-1ad4f963515a@oss.qualcomm.com>
Date: Fri, 5 Dec 2025 14:50:23 +0100
From: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
To: David Heidelberg <david@...t.cz>, longnoserob@...il.com,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>
Cc: linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
phone-devel@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/3] arm64: dts: qcom: sdm845-xiaomi-beryllium: Enable
SLPI
On 12/5/25 2:45 PM, David Heidelberg wrote:
> On 05/12/2025 14:27, Konrad Dybcio wrote:
>
> [...]
>
>>> @@ -637,3 +643,4 @@ &wifi {
>>> vdd-3.3-ch0-supply = <&vreg_l25a_3p3>;
>>> vdd-3.3-ch1-supply = <&vreg_l23a_3p3>;
>>> };
>>> +
>>>
>>
>> Stray hunk?
>
> I think your comment goes into the category of "extreme nitpicks".
>
> Here as usual I guess there are three options:
> - another revision because of newline
> - maintainer merge it with it and nothing happens
> - while merge, maintainer remove the line
$ LC_ALL=C b4 shazam 5b1ad72d-bd85-4950-8148-266296a05db1@...t.cz -P 1
Grabbing thread from lore.kernel.org/all/5b1ad72d-bd85-4950-8148-266296a05db1@...t.cz/t.mbox.gz
Checking for newer revisions
Grabbing search results from lore.kernel.org
Analyzing 6 messages in the thread
Analyzing 0 code-review messages
Checking attestation on all messages, may take a moment...
---
[PATCH v2 1/3] arm64: dts: qcom: sdm845-xiaomi-beryllium: Enable SLPI
---
✗ No key: openpgp/longnoserob@...il.com
---
Total patches: 1 (cherrypicked: 1)
---
Base: using specified base-commit 6987d58a9cbc5bd57c983baa514474a86c945d56
Applying: arm64: dts: qcom: sdm845-xiaomi-beryllium: Enable SLPI
/<snip path>/.git/rebase-apply/patch:26: new blank line at EOF.
+
warning: 1 line adds whitespace errors.
It works best when both submitters and maintainers check their work..
Konrad
Powered by blists - more mailing lists