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

Date:   Thu, 8 Dec 2022 10:57:45 +0000
From:   Lee Jones <lee@...nel.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc:     Pavel Machek <pavel@....cz>, 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 Thu, 08 Dec 2022, Krzysztof Kozlowski wrote:

> 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.

Sounds good:

Acked-by: Lee Jones <lee@...nel.org>

-- 
Lee Jones [李琼斯]