Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 13 Jan 2023 13:22:17 -0600
From:   Rob Herring <robh@kernel.org>
To:     Wadim Egorov <W.Egorov@phytec.de>
Cc:     "a.zummo@towertech.it" <a.zummo@towertech.it>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "riku.voipio@iki.fi" <riku.voipio@iki.fi>,
        "pavel@ucw.cz" <pavel@ucw.cz>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "alexandre.belloni@bootlin.com" <alexandre.belloni@bootlin.com>,
        "linux-rtc@vger.kernel.org" <linux-rtc@vger.kernel.org>,
        "lee@kernel.org" <lee@kernel.org>,
        "linux-leds@vger.kernel.org" <linux-leds@vger.kernel.org>,
        "upstream@lists.phytec.de" <upstream@lists.phytec.de>
Subject: Re: [PATCH v2 1/2] dt-bindings: leds: Convert PCA9532 to dtschema
Message-ID: <20230113192217.GA2772523-robh@kernel.org>
References: <20230111163404.3526248-1-w.egorov@phytec.de>
 <167346265519.1103752.16521267476598663004.robh@kernel.org>
 <CAL_JsqJ0e__gb6xQxJ-XhJfyBx3mYdhqipqi9i1AoBaxoaBHiA@mail.gmail.com>
 <9e544e49-3f28-c628-196f-2e0fed6584ec@phytec.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <9e544e49-3f28-c628-196f-2e0fed6584ec@phytec.de>
Precedence: bulk

On Fri, Jan 13, 2023 at 04:18:26PM +0000, Wadim Egorov wrote:
> Am 11.01.23 um 19:51 schrieb Rob Herring:
> > On Wed, Jan 11, 2023 at 12:47 PM Rob Herring <robh@kernel.org> wrote:
> >>
> >> On Wed, 11 Jan 2023 17:34:03 +0100, Wadim Egorov wrote:
> >>> Convert the PCA9532 LED dimmer to dtschema.
> >>> While at it, also update and the the example to match
> >>> recommended node names and the link to the product datasheet.
> >>>
> >>> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
> >>> ---
> >>> v2:
> >>>    - Rename yaml file to match compatibles, nxp,pca953x.yaml
> >>>    - Remove Jacek Anaszewski from maintainers list
> >>>    - Remove color labels in example
> >>>    - Restore labels/default-states from original example
> >>>    - Drop reg description
> >>>    - Add unevaluatedProperties to patternProperties scope
> >>>    - Update description of type property & set default to 0
> >>>    - Fix indentation in example
> >>> ---
> >>>   .../devicetree/bindings/leds/leds-pca9532.txt | 49 -----------
> >>>   .../devicetree/bindings/leds/nxp,pca953x.yaml | 85 +++++++++++++++++++
> >>>   2 files changed, 85 insertions(+), 49 deletions(-)
> >>>   delete mode 100644 Documentation/devicetree/bindings/leds/leds-pca9532.txt
> >>>   create mode 100644 Documentation/devicetree/bindings/leds/nxp,pca953x.yaml
> >>>
> >> Running 'make dtbs_check' with the schema in this patch gives the
> >> following warnings. Consider if they are expected or the schema is
> >> incorrect. These may not be new warnings.
> >>
> >> Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> >> This will change in the future.
> >>
> >> Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20230111163404.3526248-1-w.egorov@phytec.de
> >>
> >>
> >> leddimmer@62: 'led1', 'led2', 'led3', 'led4' do not match any of the regexes: '^led-[0-9a-z]+$', 'pinctrl-[0-9]+'
> >>          arch/arm/boot/dts/rk3288-phycore-rdk.dtb
> >>
> >> leds@62: 'led1', 'led2', 'led3' do not match any of the regexes: '^led-[0-9a-z]+$', 'pinctrl-[0-9]+'
> >>          arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-rdk.dtb
> >>
> >> pca9530@61: 'tft-panel@0' does not match any of the regexes: '^led-[0-9a-z]+$', 'pinctrl-[0-9]+'
> >>          arch/arm/boot/dts/am335x-pdu001.dtb
> 
> How do I comply with the LED label warnings?
> I am not sure about the proper convention. I was assuming led-<name> is correct.
> The first two warnings can be fixed by dropping the '-' in the regex.
> But I don't see how the last warning should comply with a proper LED node name. 
> Or can this be ignored?

Those dts files should be fixed, not the schema. You don't have to do 
that.

Rob