Received: by 2002:a6b:fb09:0:0:0:0:0 with SMTP id h9csp2502280iog; Sun, 19 Jun 2022 20:21:56 -0700 (PDT) X-Google-Smtp-Source: AGRyM1uMgTXrOh41Zz+7vxAp/98CZBOzn8SSR6oeBebttmPZG/78YuyiDJWYxB1WyVq8VDPqUCmW X-Received: by 2002:a17:906:f2d7:b0:710:54d3:f052 with SMTP id gz23-20020a170906f2d700b0071054d3f052mr18951405ejb.599.1655695316363; Sun, 19 Jun 2022 20:21:56 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1655695316; cv=none; d=google.com; s=arc-20160816; b=kmEFxEPBVYrBNO5n2hsA8WJ4oAGVeWQYKsNOapMq9JeQ6kZKVZPpSL6NbbS/iKkvF/ hflgCEq/u5DWbZscit9IZkXY7ZIwex4LVuQuFLtD0bjZSUY4/OvggFdEn0ZCLvO3S/RM 6iC/bj3RW+KMjTqXdgrj5SD75c4D+TA3679du0z3XA5kgXu9yrK/Fu4W1iKFDjzoruB9 61gQ6rbp55rdrV2eSvGbdHpE3X5mz5tMqEaGgjSrYSJSvr8dCvJmFmPrPJNoHaNHaAE1 gIhpMR7kfJ2yMB15d7QNcgXccomMu7LPF3F/5ANWVBOkTd5BrCQhgYj9WklxRyKlOz1s /3qQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=ClA6mV7hozr21kuBkeCxjd9fGNxXJpc68c5A8HEJUUI=; b=h2b5swUhtsuYOv3LcUpmiieqMBNXb9LTDYJcloQuOOWNyfjEJgZruuq/jal+PCv1uC 70gKd0+6ppJEXr4DguJeeRNbzuqQIOwCERVvaDkO5fU3/xt+dMIWLdkuSDvz4cH6XBKl 5FFgEgGvpvqN77BdYKc9wJFoi+qMXZrN+sjx1uqbvvZodvO0NkbD5cpbXE16BPJh+uV9 FdQwHpaFlLs6X6RIgwU+G3oWl1FDzT5YMANlmdWP39EeLZQy3OaXoY5t5+aDCrNYW/4I VbQGMVjEM51rX6HuLMlKDODxs2tSZm502v1xQ2TNyqp6x52kL/xbp/S6OD049YSY9lIu 5BJw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=gOpv1108; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id di10-20020a170906730a00b006f3dae85820si10184437ejc.472.2022.06.19.20.21.30; Sun, 19 Jun 2022 20:21:56 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=gOpv1108; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237615AbiFTDHn (ORCPT + 99 others); Sun, 19 Jun 2022 23:07:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49682 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237246AbiFTDHl (ORCPT ); Sun, 19 Jun 2022 23:07:41 -0400 Received: from mail-il1-x12d.google.com (mail-il1-x12d.google.com [IPv6:2607:f8b0:4864:20::12d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B8B1865B2; Sun, 19 Jun 2022 20:07:40 -0700 (PDT) Received: by mail-il1-x12d.google.com with SMTP id m16so5055740ilf.6; Sun, 19 Jun 2022 20:07:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=ClA6mV7hozr21kuBkeCxjd9fGNxXJpc68c5A8HEJUUI=; b=gOpv1108ut5ySpAPndiTka0cdAPZs7Flmw6QF/XASUursDWrwbsAx9Vt+ZXF5kcdEw TCakBrMFGrsxiKWEOhenPFzr9LBdXhjmxW5370IMQcOOlgaFHdTNkBB+FXGyNxJcg61e E1CcWHlUQ+QDQ0SBGzNysb+dxHy8expVECKExkABctkP/dtbwAAACdK2L7UDO9Gbz48K ji3tgBrL0udfA8TkFrC8hp8qz2em39uxnCPqC9QkBkCfnnMqz+vlX4RNa8QIDVPQFIsS Tfem3Y9ER0V16TcCeQoXqA7MiKjEXoKy5//qqMQChDe5hPDjkyy9/RyTBkvqD/PR7kp3 aZZQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=ClA6mV7hozr21kuBkeCxjd9fGNxXJpc68c5A8HEJUUI=; b=GEfgbHzpmH0uUZ6uMTT+DbKYhbPVmaJQTSCZAFx1ZAdCCTzdFDIiDQr+wyqLTgzCDQ EfmgI6/LZEzcpopXMiOC795S5yI89SucZRRMjX38yKGBQ3m/puF7EWKfUoeP/A4twnbm xW/krzXvaoy1GJd1E9Fpd3AoW+o72yMqFtIkzTBUZJLn4eDL46ymkiPuSmtxrxyv57r1 0dKNHtKLi/YesODLsilCKAoRvrW64rjIhQAPystd8ztbtc3OZy1MrHuGDDdqDcj1ujD2 ei6ngSJe9/DBAXyAEAd9ALSdNgPh3fi8+6hSfBYbGzgN9sdNawzeRj6ud8aw9UyYmnfN HQ6A== X-Gm-Message-State: AJIora9OBmOY1Vekl6KDag8YTpQ0KCLzT41BZtBZFykqGL/dMCUCj+5v YgxbCS6jSXStBKCbx1uoSmzMZ9dkSXLVUYmt8zo= X-Received: by 2002:a05:6e02:1bc1:b0:2d1:96b3:b961 with SMTP id x1-20020a056e021bc100b002d196b3b961mr12302759ilv.213.1655694460188; Sun, 19 Jun 2022 20:07:40 -0700 (PDT) MIME-Version: 1.0 References: <20220613111146.25221-1-peterwu.pub@gmail.com> <20220613111146.25221-4-peterwu.pub@gmail.com> <91e9e3af-8208-7535-1864-08744f934593@linaro.org> In-Reply-To: <91e9e3af-8208-7535-1864-08744f934593@linaro.org> From: szuni chen Date: Mon, 20 Jun 2022 11:07:29 +0800 Message-ID: Subject: Re: [PATCH v2 03/15] dt-bindings: leds: mt6370: Add Mediatek mt6370 current sink type LED indicator To: Krzysztof Kozlowski Cc: ChiaEn Wu , Jonathan Cameron , Lars-Peter Clausen , Matthias Brugger , Lee Jones , Daniel Thompson , Jingoo Han , Pavel Machek , Rob Herring , Krzysztof Kozlowski , linux-iio , linux-arm Mailing List , "moderated list:ARM/Mediatek SoC support" , Linux Kernel Mailing List , dri-devel , Linux LED Subsystem , devicetree , Linux PM , USB , "open list:FRAMEBUFFER LAYER" , ChiYuan Huang Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dear Krzysztof, Thank you for the valuable command. Krzysztof Kozlowski =E6=96=BC 2022=E5=B9= =B46=E6=9C=8817=E6=97=A5 =E9=80=B1=E4=BA=94 =E6=B8=85=E6=99=A85:09=E5=AF=AB= =E9=81=93=EF=BC=9A > > On 13/06/2022 04:11, ChiaEn Wu wrote: > > From: ChiYuan Huang > > > > Add Mediatek mt6370 current sink type LED indicator binding documentati= on. > > > > Signed-off-by: ChiYuan Huang > > --- > > .../leds/mediatek,mt6370-indicator.yaml | 48 +++++++++++++++++++ > > 1 file changed, 48 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/leds/mediatek,mt6= 370-indicator.yaml > > > > diff --git a/Documentation/devicetree/bindings/leds/mediatek,mt6370-ind= icator.yaml b/Documentation/devicetree/bindings/leds/mediatek,mt6370-indica= tor.yaml > > new file mode 100644 > > index 000000000000..42b96c8047a3 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/leds/mediatek,mt6370-indicator.= yaml > > @@ -0,0 +1,48 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/leds/mediatek,mt6370-indicator.yaml= # > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: LED driver for MT6370 PMIC from MediaTek Integrated. > > + > > +maintainers: > > + - Alice Chen > > + > > +description: | > > + This module is part of the MT6370 MFD device. > > + Add MT6370 LED driver include 4-channel RGB LED support Register/PWM= /Breath Mode > > + > > +properties: > > + compatible: > > + const: mediatek,mt6370-indicator > > + > > + "#address-cells": > > + const: 1 > > + > > + "#size-cells": > > + const: 0 > > + > > +patternProperties: > > + "^(multi-)?led@[0-3]$": > > If this is multi-led, then you should reference > /schemas/leds/leds-pwm-multicolor.yaml > > See other examples using it. In my opinion, leds-pwm-multicolor.yaml is used for external pwm signals, and needs to assign an external pwm property. But our pwm leds use an internal clock to generate the pwm signal. Is leds-class-multicolor.yaml more appropriate? Sincerely, Alice > > > > Best regards, > Krzysztof