From: Linus Walleij <linus.walleij@linaro.org> To: Rob Herring <robh@kernel.org> Cc: Lee Jones <lee.jones@linaro.org>, Daniel Thompson <daniel.thompson@linaro.org>, Jingoo Han <jingoohan1@gmail.com>, dri-devel <dri-devel@lists.freedesktop.org>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, Sam Ravnborg <sam@ravnborg.org> Subject: Re: [PATCH 1/3 v2] dt-bindings: backlight: Add some common backlight properties Date: Wed, 19 Aug 2020 22:46:41 +0200 [thread overview] Message-ID: <CACRpkdYOym=0BOsTr=bh4Zax5euj9RcRzZ_keUL3Z1EN+UykWw@mail.gmail.com> (raw) In-Reply-To: <CAL_JsqLR3HEsbuNq7i+N3ETYVoMtUP90_Ev=tO8GJr+fF4QHWQ@mail.gmail.com> On Wed, Aug 12, 2020 at 5:46 PM Rob Herring <robh@kernel.org> wrote: > On Wed, Aug 12, 2020 at 2:58 AM Linus Walleij <linus.walleij@linaro.org> wrote: > > > > Let's use a common.yaml include for the backlight like we do with > > the LEDs. The LEDs are inherently incompatible so their bindings > > cannot be reused for backlight. > > > > Cc: devicetree@vger.kernel.org > > Suggested-by: Sam Ravnborg <sam@ravnborg.org> > > Signed-off-by: Linus Walleij <linus.walleij@linaro.org> > > --- > > ChangeLog v1->v2: > > - New patch as suggested by Sam. > > --- > > .../bindings/leds/backlight/common.yaml | 42 +++++++++++++++++++ > > 1 file changed, 42 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/leds/backlight/common.yaml > > I'd expect some refactoring here with existing backlight schemas > including the ones I just added for 5.9. Yeah if it takes off I can certainly make a slew of refactorings, I would like to do that once this is applied. > > + LED powered by a boost converter. > > + > > +properties: > > + default-on: > > + description: > > + The initial state of the backlight can be set to be on with this > > + property. This is a state applied by the operating system so that the > > + backlight is always turned on at boot. > > Needs a type. Dropping this property because the subsystem maintainer doubts this is needed. Fixed the rest! Yours, Linus Walleij
WARNING: multiple messages have this Message-ID (diff)
From: Linus Walleij <linus.walleij@linaro.org> To: Rob Herring <robh@kernel.org> Cc: "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, Daniel Thompson <daniel.thompson@linaro.org>, Jingoo Han <jingoohan1@gmail.com>, dri-devel <dri-devel@lists.freedesktop.org>, Lee Jones <lee.jones@linaro.org>, Sam Ravnborg <sam@ravnborg.org> Subject: Re: [PATCH 1/3 v2] dt-bindings: backlight: Add some common backlight properties Date: Wed, 19 Aug 2020 22:46:41 +0200 [thread overview] Message-ID: <CACRpkdYOym=0BOsTr=bh4Zax5euj9RcRzZ_keUL3Z1EN+UykWw@mail.gmail.com> (raw) In-Reply-To: <CAL_JsqLR3HEsbuNq7i+N3ETYVoMtUP90_Ev=tO8GJr+fF4QHWQ@mail.gmail.com> On Wed, Aug 12, 2020 at 5:46 PM Rob Herring <robh@kernel.org> wrote: > On Wed, Aug 12, 2020 at 2:58 AM Linus Walleij <linus.walleij@linaro.org> wrote: > > > > Let's use a common.yaml include for the backlight like we do with > > the LEDs. The LEDs are inherently incompatible so their bindings > > cannot be reused for backlight. > > > > Cc: devicetree@vger.kernel.org > > Suggested-by: Sam Ravnborg <sam@ravnborg.org> > > Signed-off-by: Linus Walleij <linus.walleij@linaro.org> > > --- > > ChangeLog v1->v2: > > - New patch as suggested by Sam. > > --- > > .../bindings/leds/backlight/common.yaml | 42 +++++++++++++++++++ > > 1 file changed, 42 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/leds/backlight/common.yaml > > I'd expect some refactoring here with existing backlight schemas > including the ones I just added for 5.9. Yeah if it takes off I can certainly make a slew of refactorings, I would like to do that once this is applied. > > + LED powered by a boost converter. > > + > > +properties: > > + default-on: > > + description: > > + The initial state of the backlight can be set to be on with this > > + property. This is a state applied by the operating system so that the > > + backlight is always turned on at boot. > > Needs a type. Dropping this property because the subsystem maintainer doubts this is needed. Fixed the rest! Yours, Linus Walleij _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-08-19 20:46 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-08-12 8:58 [PATCH 1/3 v2] dt-bindings: backlight: Add some common backlight properties Linus Walleij 2020-08-12 8:58 ` Linus Walleij 2020-08-12 8:58 ` [PATCH 2/3 v2] dt-bindings: backlight: Add Kinetic KTD253 bindings Linus Walleij 2020-08-12 8:58 ` Linus Walleij 2020-08-13 10:48 ` Daniel Thompson 2020-08-13 10:48 ` Daniel Thompson 2020-08-12 8:58 ` [PATCH 3/3 v2] backlight: Add Kinetic KTD253 backlight driver Linus Walleij 2020-08-13 10:30 ` Daniel Thompson 2020-08-12 15:46 ` [PATCH 1/3 v2] dt-bindings: backlight: Add some common backlight properties Rob Herring 2020-08-12 15:46 ` Rob Herring 2020-08-19 20:46 ` Linus Walleij [this message] 2020-08-19 20:46 ` Linus Walleij 2020-08-13 11:04 ` Daniel Thompson 2020-08-13 11:04 ` Daniel Thompson
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CACRpkdYOym=0BOsTr=bh4Zax5euj9RcRzZ_keUL3Z1EN+UykWw@mail.gmail.com' \ --to=linus.walleij@linaro.org \ --cc=daniel.thompson@linaro.org \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=jingoohan1@gmail.com \ --cc=lee.jones@linaro.org \ --cc=robh@kernel.org \ --cc=sam@ravnborg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.