linux-kernel - Re: [PATCH] dt-bindings: leds: Describe optional 'reg' property used for Qualcomm LPG nodes

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: <CAA8EJppGS38aP7gyd1c3kNgraAVJDoqUef2cDfZpu2aL_iwW0g@mail.gmail.com>
Date:   Thu, 21 Jul 2022 23:19:10 +0300
From:   Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
To:     Bhupesh Sharma <bhupesh.sharma@...aro.org>
Cc:     linux-arm-msm@...r.kernel.org, linux-pm@...r.kernel.org,
        bhupesh.linux@...il.com, linux-kernel@...r.kernel.org,
        bjorn.andersson@...aro.org, robh@...nel.org, pavel@....cz
Subject: Re: [PATCH] dt-bindings: leds: Describe optional 'reg' property used
 for Qualcomm LPG nodes

On Thu, 21 Jul 2022 at 22:55, Bhupesh Sharma <bhupesh.sharma@...aro.org> wrote:
>
> As Bjorn noted in [1], it is useful to describe the optional
> 'reg' property for Qualcomm LPG nodes as it is used in
> some Qualcomm dts files.

I don't think this is correct. LPG block maps to several regions, so
using just one of them in reg doesn't look correct.

> This fixes the following 'make dtbs_check' error reported for
> pm8350c & sc8280xp pwm nodes:
>
> arch/arm64/boot/dts/qcom/sc7280-herobrine-herobrine-r1.dtb:
>  pwm@...0: 'reg' does not match any of the regexes:
>  '^led@[0-9a-f]$', 'pinctrl-[0-9]+'

I'd prefer to follow the existing schema and to drop the region from
those files.

>
> [1]. https://lore.kernel.org/linux-arm-msm/Ytg3tIaL5h5b9ewH@builder.lan/
>
> Cc: Bjorn Andersson <bjorn.andersson@...aro.org>
> Cc: robh@...nel.org
> Cc: pavel@....cz
> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@...aro.org>
> ---
>  Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml b/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
> index fe336fa16518..f394ab7a757b 100644
> --- a/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
> +++ b/Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml
> @@ -27,6 +27,9 @@ properties:
>        - qcom,pmi8994-lpg
>        - qcom,pmi8998-lpg
>
> +  reg:
> +    maxItems: 1
> +
>    "#pwm-cells":
>      const: 2
>
> --
> 2.35.3
>


-- 
With best wishes
Dmitry