linux-kernel - Re: [PATCH] leds: qcom,pm8058-led: Convert to DT schema

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: <947e47d3-4235-8f16-d3ef-c3ddd7d51acf@linaro.org>
Date:   Thu, 8 Dec 2022 11:19:47 +0100
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Lee Jones <lee@...nel.org>, Pavel Machek <pavel@....cz>
Cc:     Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <andersson@...nel.org>,
        Konrad Dybcio <konrad.dybcio@...aro.org>,
        Satya Priya <quic_c_skakit@...cinc.com>,
        linux-leds@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org
Subject: Re: [PATCH] leds: qcom,pm8058-led: Convert to DT schema

On 08/12/2022 11:09, Lee Jones wrote:
> On Wed, 07 Dec 2022, Pavel Machek wrote:
> 
>> On Thu 2022-12-01 18:08:58, Rob Herring wrote:
>>> On Thu, Dec 01, 2022 at 02:15:05PM +0100, Krzysztof Kozlowski wrote:
>>>> Convert the Qualcomm PM8058 PMIC LED bindings to DT schema.
>>>>
>>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>>>> ---
>>>>  .../devicetree/bindings/leds/leds-pm8058.txt  | 67 -------------------
>>>>  .../bindings/leds/qcom,pm8058-led.yaml        | 57 ++++++++++++++++
>>>>  .../devicetree/bindings/mfd/qcom-pm8xxx.yaml  |  4 ++
>>>>  3 files changed, 61 insertions(+), 67 deletions(-)
>>>>  delete mode 100644 Documentation/devicetree/bindings/leds/leds-pm8058.txt
>>>>  create mode 100644 Documentation/devicetree/bindings/leds/qcom,pm8058-led.yaml
>>>
>>> Reviewed-by: Rob Herring <robh@...nel.org>
>>>
>>> Or should I apply it?
>>
>> Thanks for ACK, let me take it, I guess.
> 
> Did you see Krzysztof's replies to this patch?
> 
> Sounded like he was going to re-work it, which is why I left it.

The only rework needed was to add "dt-bindings:" prefix in the subject.
If you could add it while applying/amending commit, that would be great
and spare me resend. Otherwise, let me know if you dropped this patch
and expect a resend.

Best regards,
Krzysztof