Received: by 2002:a6b:fb09:0:0:0:0:0 with SMTP id h9csp217557iog; Fri, 24 Jun 2022 02:40:19 -0700 (PDT) X-Google-Smtp-Source: AGRyM1sOzYq6IgUbohlAV8pRsMxwGd32r0EcX3F/5aZxeJWEaXH9Xp/BJoj0B+G9YsEDBJtEmOMQ X-Received: by 2002:a05:6402:3507:b0:436:446c:4fc1 with SMTP id b7-20020a056402350700b00436446c4fc1mr4217780edd.31.1656063618929; Fri, 24 Jun 2022 02:40:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1656063618; cv=none; d=google.com; s=arc-20160816; b=QV1lcYFhTfbs1smp2BJ36GP/4A6CCG5+jCFGO10cYHv92aecYGBwO7YyYVe7nk5yM+ LGDoXi3/sJlrnR4ffQ92EhLfRDojp+1hVfmcSJs4onqbmkNdgV8GcCs8T4jvecUb0q2/ wegb1QD6bVAvxJHkseVq4onDyMUF0wHNYOX8LMJujIUHYV59tUrGFDhaNlb+/AdcJqSm 80ub6ZOWmWu1RA8vJqBnfy3tuWX+7rELtDgNxSlLCaSs00J/1+4b+YBear7Sqkr2rbB+ Vtucxg/KMc8VUFnJ+o1XUKWaDNEDuif8JIYOAZEyW5MdeYu/SjKM2fDiYiS21HBmGMu5 eYxg== 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=MP9RQT0mw2gx9xecm41XDjO0xQp/A9AAK0cYbd+v7PM=; b=lm3ia0KWwuHLyIkqrXkydNFZTUA+Klb09gKUGU/TwWIu9F8zdawRTDV+LTsmsKyF0t 0Vtu/rJrAV21321kALoIOr/rwxy6nyP9RpuZ6Zgp8ukGslG3Tp2kGz1m/JSc+bM0Y1T9 MKBZPXiVPWqqn2QnOUbBrHDHnbjIMj7Mu8UJ1mIQU+WfBV01v8/sTYRkmEBaxuP+qzkM Td8C0r21pRM31Ub7u/8eJgRLZl7rNPHAoeSTAGWOc+BcjBymrF/qfFZHLlNNYjpxmgag Dzn1j0Z092QFK1w8BJSk1WAggkXcnvK4rzb12JyqdtphPIsO7bQPq6I6O9wSwMnS2wHf xthg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=XOSjJie8; 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 y5-20020a056402134500b0043580b38b6csi2411394edw.505.2022.06.24.02.39.52; Fri, 24 Jun 2022 02:40:18 -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=XOSjJie8; 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 S230333AbiFXJeu (ORCPT + 99 others); Fri, 24 Jun 2022 05:34:50 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51974 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229469AbiFXJet (ORCPT ); Fri, 24 Jun 2022 05:34:49 -0400 Received: from mail-io1-xd2d.google.com (mail-io1-xd2d.google.com [IPv6:2607:f8b0:4864:20::d2d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9638B699BF; Fri, 24 Jun 2022 02:34:48 -0700 (PDT) Received: by mail-io1-xd2d.google.com with SMTP id p69so2092342iod.10; Fri, 24 Jun 2022 02:34:48 -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=MP9RQT0mw2gx9xecm41XDjO0xQp/A9AAK0cYbd+v7PM=; b=XOSjJie8EVvJ68P8cDkEuXXdIP7YNykFHD5ucaFHa23sPTgdW5s+vEyYglwP5ITADf de6ih0EGdaikqxaCCoMYFOazeMTR+HJ2Od3FO0aVrMGZ9Ao0YQc3IGKpdK1KsldTwlvh mNgixo6yDK8vv6W4zP9rFZEgeSROQH0wRlhznnuez0r+LK3zlRzbpSDq8zkiXpRGsUdD wlBFXBP+v/yudkNVKvRuUvTnfyVEt9q4H+CcHi3K0tEYuY/us9b2jbsRIceZSdwT323h bCW6ZtoKL/DDgyl9mBYWdF/TeTYV9OJiSRhfvnTTdEl5HuLFgATDlbAUJ79bLaqNIqyo pjrg== 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=MP9RQT0mw2gx9xecm41XDjO0xQp/A9AAK0cYbd+v7PM=; b=hTAwqwnpuD0bg8/et7uTMOZKNM6Jv3GlxUPHmRqiY5BPoi+LOY0oBU6SMxoD9ZITW7 bSuFV3mfndskomSI+cNkjpWG9DLQCS+O1zx6GROfdejgo0+U0Int8kVR0hHISku+Kb1p /2rLNceaWpnkvE4oNF7oSvrCyNG5fq6bb405EaUJfDHIjiLp6Q/VTwys6lfNd0XYF6Po kbvIqC5LCFWflKy6lo3HM6jVbc63MOOoPHy++FhBM0YxYEEng+lk3la3kJ8PgH2XGVof 04eUcvPzwZ+FUUi4X0bxOEnIzQyhRe/WOBb2HRDE7XinMHgKS743ZgfGki+AYIbOxnK7 VSkg== X-Gm-Message-State: AJIora+kPWz4/GuYVOPISnffJ+3zolxj/ChUizbAmgtKayi0/soZjcBz nMNvrZUHp3/mb2kSsjujN1lv0a9JqL16dkIFp+E= X-Received: by 2002:a05:6638:348d:b0:331:d8f0:fd9d with SMTP id t13-20020a056638348d00b00331d8f0fd9dmr7928714jal.165.1656063287244; Fri, 24 Jun 2022 02:34:47 -0700 (PDT) MIME-Version: 1.0 References: <20220623115631.22209-1-peterwu.pub@gmail.com> <20220623115631.22209-6-peterwu.pub@gmail.com> In-Reply-To: From: ChiaEn Wu Date: Fri, 24 Jun 2022 17:34:36 +0800 Message-ID: Subject: Re: [PATCH v3 05/14] dt-bindings: backlight: Add Mediatek MT6370 backlight To: Joe Simmons-Talbott Cc: Lee Jones , Daniel Thompson , jingoohan1@gmail.com, pavel@ucw.cz, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, matthias.bgg@gmail.com, sre@kernel.org, chunfeng.yun@mediatek.com, gregkh@linuxfoundation.org, Jonathan Cameron , lars@metafoo.de, lgirdwood@gmail.com, broonie@kernel.org, linux@roeck-us.net, heikki.krogerus@linux.intel.com, deller@gmx.de, ChiaEn Wu , Alice Chen , ChiYuan Huang , dri-devel@lists.freedesktop.org, linux-leds@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-usb@vger.kernel.org, linux-iio@vger.kernel.org, linux-fbdev@vger.kernel.org, szunichen@gmail.com 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 Hi Joe, Joe Simmons-Talbott =E6=96=BC 2022=E5=B9=B46=E6=9C= =8823=E6=97=A5 =E9=80=B1=E5=9B=9B =E6=99=9A=E4=B8=8A9:17=E5=AF=AB=E9=81=93= =EF=BC=9A > > On Thu, Jun 23, 2022 at 07:56:22PM +0800, ChiaEn Wu wrote: > > From: ChiYuan Huang > > > > Add mt6370 backlight binding documentation. > > > > Signed-off-by: ChiYuan Huang > > --- > > > > v3 > > - Rename "mediatek,bled-pwm-hys-input-threshold-steps" to > > "mediatek,bled-pwm-hys-input-th-steps" > > - Refine "bled-pwm-hys-input-th-steps", "bled-ovp-microvolt", > > "bled-ocp-microamp" enum values > > --- > > .../leds/backlight/mediatek,mt6370-backlight.yaml | 92 ++++++++++++++= ++++++++ > > 1 file changed, 92 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/leds/backlight/me= diatek,mt6370-backlight.yaml > > > > diff --git a/Documentation/devicetree/bindings/leds/backlight/mediatek,= mt6370-backlight.yaml b/Documentation/devicetree/bindings/leds/backlight/me= diatek,mt6370-backlight.yaml > > new file mode 100644 > > index 0000000..26563ae > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/leds/backlight/mediatek,mt6370-= backlight.yaml > > @@ -0,0 +1,92 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/leds/backlight/mediatek,mt6370-back= light.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Mediatek MT6370 Backlight > > + > > +maintainers: > > + - ChiaEn Wu > > + > > +description: | > > + This module is part of the MT6370 MFD device. > > + The MT6370 Backlight WLED driver supports up to a 29V output voltage= for > > + 4 channels of 8 series WLEDs. Each channel supports up to 30mA of cu= rrent > > + capability with 2048 current steps (11 bits) in exponential or linea= r > > + mapping curves. > > + > > +allOf: > > + - $ref: common.yaml# > > + > > +properties: > > + compatible: > > + const: mediatek,mt6370-backlight > > + > > + default-brightness: > > + minimum: 0 > > + maximum: 2048 > > + > > + max-brightness: > > + minimum: 0 > > + maximum: 2048 > > + > > + enable-gpios: > > + description: External backlight 'enable' pin > > + maxItems: 1 > > + > > + mediatek,bled-pwm-enable: > > + description: | > > + Enable external PWM input for backlight dimming > > + type: boolean > > + > > + mediatek,bled-pwm-hys-enable: > > + description: | > > + Enable the backlight input-hysteresis for PWM mode > > + type: boolean > > + > > + mediatek,bled-pwm-hys-input-th-steps: > > + $ref: /schemas/types.yaml#/definitions/uint8 > > + enum: [1, 4, 16, 64] > > + description: | > > + The selection of the upper and lower bounds threshold of backlig= ht > > + PWM resolution. If we choose selection 64, the variation of PWM > > + resolution needs over than 64 steps. > > more than? > > Thanks, > Joe > Thanks for your helpful comments! I will revise this in the next patch. Thanks! > > + > > + mediatek,bled-ovp-shutdown: > > + description: | > > + Enable the backlight shutdown when OVP level triggered > > + type: boolean > > + > > + mediatek,bled-ovp-microvolt: > > + enum: [17000000, 21000000, 25000000, 29000000] > > + description: | > > + Backlight OVP level selection. > > + > > + mediatek,bled-ocp-shutdown: > > + description: | > > + Enable the backlight shutdown when OCP level triggerred. > > + type: boolean > > + > > + mediatek,bled-ocp-microamp: > > + enum: [900000, 1200000, 1500000, 1800000] > > + description: | > > + Backlight OC level selection. > > + > > + mediatek,bled-channel-use: > > + $ref: /schemas/types.yaml#/definitions/uint8 > > + description: | > > + Backlight LED channel to be used. > > + Each bit mapping to: > > + - 0: CH4 > > + - 1: CH3 > > + - 2: CH2 > > + - 3: CH1 > > + minimum: 1 > > + maximum: 15 > > + > > +required: > > + - compatible > > + - mediatek,bled-channel-use > > + > > +additionalProperties: false > > -- > > 2.7.4 > > Best regards, ChiaEn Wu