From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.5 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 31A11C2D0EF for ; Sat, 18 Apr 2020 09:26:27 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id F34F821D82 for ; Sat, 18 Apr 2020 09:26:26 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=mg.codeaurora.org header.i=@mg.codeaurora.org header.b="YRRzxSYE" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org F34F821D82 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id EF9AF6E185; Sat, 18 Apr 2020 09:26:16 +0000 (UTC) Received: from mail27.static.mailgun.info (mail27.static.mailgun.info [104.130.122.27]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6C5336EBA4 for ; Fri, 17 Apr 2020 11:54:16 +0000 (UTC) DKIM-Signature: a=rsa-sha256; v=1; c=relaxed/relaxed; d=mg.codeaurora.org; q=dns/txt; s=smtp; t=1587124456; h=Message-ID: References: In-Reply-To: Subject: Cc: To: From: Date: Content-Transfer-Encoding: Content-Type: MIME-Version: Sender; bh=ynJ1oiNNtDGo8iLhGl7ZCdUICxiaTN7OWhxymrToTpI=; b=YRRzxSYEjxeomO1w+Vy+apY5Sz+B2GWhH8hyTlw537MaTNTW4BTBoqC810wS3xDViTkRx7dK n3bLaZR3+J03xagC43/rmsCBca+tgSCbDTZaPFotuSPrk4GU+vqmdL7afgQJF3WbFqQoc8Is ZzQZuHGVyV18VYSbYX8MpjRYg9E= X-Mailgun-Sending-Ip: 104.130.122.27 X-Mailgun-Sid: WyJkOTU5ZSIsICJkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnIiwgImJlOWU0YSJd Received: from smtp.codeaurora.org (ec2-35-166-182-171.us-west-2.compute.amazonaws.com [35.166.182.171]) by mxa.mailgun.org with ESMTP id 5e9998e7.7f6ee28eff80-smtp-out-n02; Fri, 17 Apr 2020 11:54:15 -0000 (UTC) Received: by smtp.codeaurora.org (Postfix, from userid 1001) id 5F4E8C433CB; Fri, 17 Apr 2020 11:54:15 +0000 (UTC) Received: from mail.codeaurora.org (localhost.localdomain [127.0.0.1]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) (Authenticated sender: kgunda) by smtp.codeaurora.org (Postfix) with ESMTPSA id 91D97C433F2; Fri, 17 Apr 2020 11:54:14 +0000 (UTC) MIME-Version: 1.0 Date: Fri, 17 Apr 2020 17:24:14 +0530 From: kgunda@codeaurora.org To: Rob Herring Subject: Re: [PATCH V5 3/4] backlight: qcom-wled: Add WLED5 bindings In-Reply-To: <20200415151234.GA25862@bogus> References: <1586274430-28402-1-git-send-email-kgunda@codeaurora.org> <1586274430-28402-4-git-send-email-kgunda@codeaurora.org> <20200415151234.GA25862@bogus> Message-ID: X-Sender: kgunda@codeaurora.org User-Agent: Roundcube Webmail/1.3.9 X-Mailman-Approved-At: Sat, 18 Apr 2020 09:26:15 +0000 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, daniel.thompson@linaro.org, b.zolnierkie@samsung.com, jingoohan1@gmail.com, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, bjorn.andersson@linaro.org, jacek.anaszewski@gmail.com, pavel@ucw.cz, linux-arm-msm@vger.kernel.org, Subbaraman Narayanamurthy , lee.jones@linaro.org, linux-leds@vger.kernel.org, Dan Murphy Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On 2020-04-15 20:42, Rob Herring wrote: > On Tue, Apr 07, 2020 at 09:17:09PM +0530, Kiran Gunda wrote: >> Add WLED5 specific bindings. >> > > checkpatch.pl complains about some trailing whitespace. The previous > patch too. > >> Signed-off-by: Kiran Gunda >> Signed-off-by: Subbaraman Narayanamurthy >> --- >> .../bindings/leds/backlight/qcom-wled.yaml | 60 >> ++++++++++++++++++++-- >> 1 file changed, 57 insertions(+), 3 deletions(-) >> >> diff --git >> a/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml >> b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml >> index 770e780..5714631 100644 >> --- a/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml >> +++ b/Documentation/devicetree/bindings/leds/backlight/qcom-wled.yaml >> @@ -21,6 +21,7 @@ properties: >> - qcom,pm8941-wled >> - qcom,pmi8998-wled >> - qcom,pm660l-wled >> + - qcom,pm8150l-wled >> >> reg: >> maxItems: 1 >> @@ -28,12 +29,13 @@ properties: >> default-brightness: >> description: >> brightness value on boot. >> - minimum: 0 >> - maximum: 4095 >> - default: 2048 >> >> label: true >> >> + max-brightness: >> + description: >> + Maximum brightness level. >> + >> qcom,cs-out: >> description: >> enable current sink output. >> @@ -130,6 +132,31 @@ properties: >> This feature is not supported for WLED3. >> type: boolean >> >> + qcom,modulator-sel: >> + description: > > Need a '|' at the end to preserve formatting. > >> + Selects the modulator used for brightness modulation. >> + Allowed values are, >> + 0 - Modulator A >> + 1 - Modulator B >> + This property is applicable only to WLED5 peripheral. >> + allOf: >> + - $ref: /schemas/types.yaml#/definitions/uint32 >> + - enum: [ 0, 1 ] >> + - default: 0 >> + >> + qcom,cabc-sel: >> + description: > > Need a '|'. > >> + Selects the CABC pin signal used for brightness modulation. >> + Allowed values are, >> + 0 - CABC disabled >> + 1 - CABC 1 >> + 2 - CABC 2 >> + 3 - External signal (e.g. LPG) is used for dimming >> + This property is applicable only to WLED5 peripheral. >> + allOf: >> + - $ref: /schemas/types.yaml#/definitions/uint32 >> + - enum: [ 0, 1, 2, 3 ] >> + >> allOf: >> - if: >> properties: >> @@ -179,6 +206,33 @@ allOf: >> - const: ovp >> - const: short >> >> + - if: >> + properties: >> + compatible: >> + contains: >> + enum: >> + - qcom,pm8150l-wled >> + >> + then: >> + properties: >> + default-brightness: >> + minimum: 0 >> + maximum: 32767 >> + >> + max-brightness: >> + minimum: 0 >> + maximum: 32767 >> + >> + else: >> + properties: >> + default-brightness: >> + minimum: 0 >> + maximum: 4095 > > Wrong indentation. > >> + >> + max-brightness: >> + minimum: 0 >> + maximum: 4095 >> + >> required: >> - compatible >> - reg >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora >> Forum, >> a Linux Foundation Collaborative Project >> Thanks for reviewing. I will submit the next revision with all the fixes. _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel