Received: by 2002:a25:d7c1:0:0:0:0:0 with SMTP id o184csp4785190ybg; Tue, 29 Oct 2019 12:20:45 -0700 (PDT) X-Google-Smtp-Source: APXvYqzf6grsaPYfjxMVI05evaaR5rvI+T8aprPYA2OTsSfN1N+RCu9+B9U75VQcTt8PRnoTQdb+ X-Received: by 2002:a50:b685:: with SMTP id d5mr27999560ede.276.1572376845187; Tue, 29 Oct 2019 12:20:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1572376845; cv=none; d=google.com; s=arc-20160816; b=aPnveqckGB6jnhYgs4tsNCog53qx0bwjDJDU74JsVmLW5ZtQAYGYfXIMjuEj2z9jub UEtpOAhsu93eR1aTfTfvwFph9E+zFiWZhoLBqGQkPMQ0rOaWsPgyNzJyUR92F8fx0iAo cvH+yUSsNr/gL0sjs01Xp9uejTCfv6/txOSU1ft5fe4oTGopi3EVyI/3zvLeN3+vnaaQ 7Frw+03d21y6Oe9H84bR6EdrtZSLC4ZjXp3PjWS6gJnv64yq03RoaMXXOsrmFBZWJaZn IQW//rsVZg8R17znrp9FYtWdizAOphWx0K5FguwCrp7uRFd/GzoUsZb39BvHmZmX8/Go rkyg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:dkim-signature; bh=ZZ+5u4A+qnpHl/68OvUYQcD2pzD+zlilhEVKLONYfvU=; b=F6/tj9/vFwvmpZLJlzc+y0nx4B8DHc/ZlLfV2J8BnclVJGr4CMT/Vl/3+GWrF/Iaf7 CPYNcAlk7BbUvIDOw5Lsk0B86K4jMEnw3PaEbrtUarTrLvYAu0lABdQ940XAYO2sF+lo gjk8jlL7c1C94s3PLLiqcGugNR1TmzDYLg8pwThvaI+qW9p9g1364wYsgc155yJMq4g1 //RyhLwn7bAwstuCBgvmXJy0svAGMNPCVs2uE8QSUACL1oc/0wjqW1exRmcEoGxma6Yo LvwwbCk2+jBXBl7iz4nco2GjIJ1mWxKviPo9cExuILl9aZePJ6Zmmapm7i09/edZu01I N92w== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@kemnade.info header.s=20180802 header.b=VYUWPhKL; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l10si4944065edk.212.2019.10.29.12.20.21; Tue, 29 Oct 2019 12:20:45 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@kemnade.info header.s=20180802 header.b=VYUWPhKL; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732679AbfJ2MSP (ORCPT + 99 others); Tue, 29 Oct 2019 08:18:15 -0400 Received: from mail.andi.de1.cc ([85.214.55.253]:33640 "EHLO mail.andi.de1.cc" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728359AbfJ2MSP (ORCPT ); Tue, 29 Oct 2019 08:18:15 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=kemnade.info; s=20180802; h=Content-Type:MIME-Version:References: In-Reply-To:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=ZZ+5u4A+qnpHl/68OvUYQcD2pzD+zlilhEVKLONYfvU=; b=VYUWPhKLnHoCNMor4xDtDak84 Gc/1BVOIz1agTd2AssvOHtWX1qyIRf20wQPlrnniW8E8/lXx1TpWSe0umZOi1PWcr/vMzEdA4h7aZ L5s9KOne9OZruFeFIyKU0kubi9SkXw4KrOU+oV5c6JX4PoCD1njo33Wwr3XZgoc8Znvy4=; Received: from [2a02:790:ff:919:7ee9:d3ff:fe1f:a246] (helo=localhost) by mail.andi.de1.cc with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1iPQRl-0007Aq-Qi; Tue, 29 Oct 2019 13:18:02 +0100 Received: from [::1] (helo=localhost) by eeepc with esmtp (Exim 4.89) (envelope-from ) id 1iPQRj-0000pw-SV; Tue, 29 Oct 2019 13:17:59 +0100 Date: Tue, 29 Oct 2019 13:16:26 +0100 From: Andreas Kemnade To: Rob Herring Cc: Lee Jones , Daniel Thompson , Jingoo Han , Jacek Anaszewski , Pavel Machek , Dan Murphy , Mark Rutland , Bartlomiej Zolnierkiewicz , dri-devel , Linux LED Subsystem , devicetree@vger.kernel.org, "linux-kernel@vger.kernel.org" , Linux Fbdev development list Subject: Re: [PATCH v4 1/2] dt-bindings: backlight: lm3630a: add enable_gpios Message-ID: <20191029131626.2a18d27e@kemnade.info> In-Reply-To: References: <20190912213257.24147-1-andreas@kemnade.info> <20190912213257.24147-2-andreas@kemnade.info> X-Mailer: Claws Mail 3.14.1 (GTK+ 2.24.31; i686-pc-linux-gnu) MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; boundary="Sig_/DGHuyPjjOiEtDx6AE.MMu8p"; protocol="application/pgp-signature" X-Spam-Score: -1.0 (-) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --Sig_/DGHuyPjjOiEtDx6AE.MMu8p Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi, On Mon, 28 Oct 2019 18:25:56 -0500 Rob Herring wrote: > On Thu, Sep 12, 2019 at 4:33 PM Andreas Kemnade wr= ote: > > > > add enable-gpios to describe HWEN pin > > > > Signed-off-by: Andreas Kemnade > > Acked-by: Daniel Thompson =20 >=20 > This breaking linux-next now... >=20 oops, sorry.=20 > > --- > > changes in v2: added example > > changes in v3: added Acked-by > > changes in v4: moved enable-gpios to the right position > > in the example > > .../bindings/leds/backlight/lm3630a-backlight.yaml | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/leds/backlight/lm3630a-b= acklight.yaml b/Documentation/devicetree/bindings/leds/backlight/lm3630a-ba= cklight.yaml > > index dc129d9a329e..c8470628fe02 100644 > > --- a/Documentation/devicetree/bindings/leds/backlight/lm3630a-backligh= t.yaml > > +++ b/Documentation/devicetree/bindings/leds/backlight/lm3630a-backligh= t.yaml > > @@ -29,6 +29,10 @@ properties: > > '#size-cells': > > const: 0 > > > > + enable-gpios: > > + description: GPIO to use to enable/disable the backlight (HWEN pin= ). > > + maxItems: 1 > > + > > required: > > - compatible > > - reg > > @@ -96,6 +100,7 @@ examples: > > led-controller@38 { > > compatible =3D "ti,lm3630a"; > > reg =3D <0x38>; > > + enable-gpios =3D <&gpio2 5 GPIO_ACTIVE_HIGH>; =20 >=20 > Error: Documentation/devicetree/bindings/leds/backlight/lm3630a-backlight= .example.dts:24.46-47 > syntax error > FATAL ERROR: Unable to parse input tree > make[1]: *** [Documentation/devicetree/bindings/leds/backlight/lm3630a-ba= cklight.example.dt.yaml] > Error 1 > scripts/Makefile.lib:314: recipe for target > 'Documentation/devicetree/bindings/leds/backlight/lm3630a-backlight.examp= le.dt.yaml' > failed >=20 > You need the include for the define. gpio/gpio.h is missing. Yes, was not aware of that these things will be compiled and the automatic check did not work on my system at all. So I decided not to fix that for ju= st this simple thing which was a not so good idea. Will send a fixup. Regards, Andreas --Sig_/DGHuyPjjOiEtDx6AE.MMu8p Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEPIWxmAFyOaBcwCpFl4jFM1s/ye8FAl24LZoACgkQl4jFM1s/ ye/XiA/8DVVyxxx109wsfNmna3iGryDIzfVOZVxZKmh2SUGenfnGGggrA2vM9QJx GRYg0Wi7kkXEW2EmYrgI0AddYBxTsuEI82ANE4BO6umA+1nCSedjttxZ2OptyZCH sZqcV+nKvtROpTwSWOdG4cYH+3vpL7Is9meLNlyFbiDUjznJ67K0tgXuOCE6Iq69 xvw1xhnaomCXazrimAlht1JNDQc6Rgma7FT9AUSM9ngEW7tbfxSVhuYUCKaMlN17 M6DhV9PePIf1SZVelYIeOT8pkBHxpcb+Pza61FpQYrXF2iRwsmfMtzNx8LOiCobp 5qnkyz2YIRZ5CIz7gc5EkZim+pIPCAJekZz3HEWsS3ypkhw3pLabRBOekX2r57iO NwqStsPFM71+xZnElqULvQ5o331G3aCbJEFkCsG9sZp98qNN0yI0UYABVl+MIeqT euvtkNHzvntSBsqYUm7MjWW8uUiFxuS0eL+AIiXBOofvJ6nsvuQHyEDvHsMNec1R dokvq/TxLbKe81rZY1OBGT/UGzMKKGuwkH0BetpuJ0eOYZJ+nHCXA20JWr5fCoPS 06G2W/MLRyTTuL7tGJ8Xa6RcKvj9pCxO+upE3eRGp02ZxyGefs68gHprOS8UiclL TMCcxD8d3k9LmR2e089kZwoA+9JhgUK33pCZBQwVVIBGuSxl1nY= =c3AX -----END PGP SIGNATURE----- --Sig_/DGHuyPjjOiEtDx6AE.MMu8p--