From: Dan Murphy <dmurphy-l0cyMroinI0@public.gmane.org> To: Jingoo Han <jingoohan1-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, 'Pavel Machek' <pavel-+ZI9xUNit7I@public.gmane.org> Cc: 'Jacek Anaszewski' <jacek.anaszewski-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, rpurdie-Fm38FmjxZ/leoWH0uzbU5w@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-leds-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, 'Lee Jones' <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>, 'Daniel Thompson' <daniel.thompson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Subject: Re: [PATCH v3 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver Date: Fri, 17 Nov 2017 12:02:39 -0600 [thread overview] Message-ID: <17b8e1ce-092a-0449-4733-533162b26499@ti.com> (raw) In-Reply-To: <000b01d35fc1$57ad0500$07070f00$@gmail.com> Pavel and Jacek On 11/17/2017 10:30 AM, Jingoo Han wrote: > On Friday, November 17, 2017 6:21 AM, Pavel Machek wrote: >> >> Hi! >> >>>>> If that does not yet exist... you want to create glue layer to be >> able >>>>> to use LED as a display backlight. (It may already exist, no idea). >>>>> >>>>> ... >>>>> >>>>> Actually or maybe a LED trigger. Just set LED's trigger to "this is >>>>> display backlight". >>>> >>>> There is one already: >>>> >>>> drivers/leds/trigger/ledtrig-backlight.c >>>> >>>> It adds a LED class device to the fb_notifier_list >>>> (drivers/video/fbdev/core/fb_notify.c) >>>> >>>> using fb_register_client(). The same is used in >>>> backlight_device_register (drivers/video/backlight/backlight.c). >>>> >>>> Actually why do you want to have this driver in the LED subsystem, >>>> if it is advertised as "designed for LCD display backlighting"? >>> >>> Well this is also advertised as a driver for Smart phone and tablet >> devices. And having worked with the Android lighting >>> solutions this is the preferred subsystem for Android. The Android OS >> manages the led brightness based on ALS values and in >>> turn calls into the driver to control the brightness register through >> the vendor provided lighting HAL. >>> >> >> Well.. if it can control other LEDs than just backlight, I believe it >> can stay in the LED subsystem. > > I also agree with your opinion. I will make the necessary changes for v4. Just a heads up I won't be posting v4 until after the US holiday so you don't think I abandoned everything. Dan > > Best regards, > Jingoo Han > >> >>> I am going to look at the backlight source to figure out how to get the >> same functionality using the backlight subsystem. >>> Otherwise I will plug in this driver to the backlight subsystem through >> the notifier. >>> >> >> The backlight trigger should be ok. >> >> Best regards, >> > Pavel >> -- >> (english) http://www.livejournal.com/~pavelmachek >> (cesky, pictures) >> http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html > -- ------------------ Dan Murphy -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Dan Murphy <dmurphy@ti.com> To: Jingoo Han <jingoohan1@gmail.com>, "'Pavel Machek'" <pavel@ucw.cz> Cc: "'Jacek Anaszewski'" <jacek.anaszewski@gmail.com>, <robh+dt@kernel.org>, <mark.rutland@arm.com>, <rpurdie@rpsys.net>, <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <linux-leds@vger.kernel.org>, "'Lee Jones'" <lee.jones@linaro.org>, "'Daniel Thompson'" <daniel.thompson@linaro.org> Subject: Re: [PATCH v3 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver Date: Fri, 17 Nov 2017 12:02:39 -0600 [thread overview] Message-ID: <17b8e1ce-092a-0449-4733-533162b26499@ti.com> (raw) In-Reply-To: <000b01d35fc1$57ad0500$07070f00$@gmail.com> Pavel and Jacek On 11/17/2017 10:30 AM, Jingoo Han wrote: > On Friday, November 17, 2017 6:21 AM, Pavel Machek wrote: >> >> Hi! >> >>>>> If that does not yet exist... you want to create glue layer to be >> able >>>>> to use LED as a display backlight. (It may already exist, no idea). >>>>> >>>>> ... >>>>> >>>>> Actually or maybe a LED trigger. Just set LED's trigger to "this is >>>>> display backlight". >>>> >>>> There is one already: >>>> >>>> drivers/leds/trigger/ledtrig-backlight.c >>>> >>>> It adds a LED class device to the fb_notifier_list >>>> (drivers/video/fbdev/core/fb_notify.c) >>>> >>>> using fb_register_client(). The same is used in >>>> backlight_device_register (drivers/video/backlight/backlight.c). >>>> >>>> Actually why do you want to have this driver in the LED subsystem, >>>> if it is advertised as "designed for LCD display backlighting"? >>> >>> Well this is also advertised as a driver for Smart phone and tablet >> devices. And having worked with the Android lighting >>> solutions this is the preferred subsystem for Android. The Android OS >> manages the led brightness based on ALS values and in >>> turn calls into the driver to control the brightness register through >> the vendor provided lighting HAL. >>> >> >> Well.. if it can control other LEDs than just backlight, I believe it >> can stay in the LED subsystem. > > I also agree with your opinion. I will make the necessary changes for v4. Just a heads up I won't be posting v4 until after the US holiday so you don't think I abandoned everything. Dan > > Best regards, > Jingoo Han > >> >>> I am going to look at the backlight source to figure out how to get the >> same functionality using the backlight subsystem. >>> Otherwise I will plug in this driver to the backlight subsystem through >> the notifier. >>> >> >> The backlight trigger should be ok. >> >> Best regards, >> > Pavel >> -- >> (english) http://www.livejournal.com/~pavelmachek >> (cesky, pictures) >> http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html > -- ------------------ Dan Murphy
next prev parent reply other threads:[~2017-11-17 18:02 UTC|newest] Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-11-15 19:42 [PATCH v3 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver Dan Murphy 2017-11-15 19:42 ` Dan Murphy [not found] ` <20171115194203.13572-1-dmurphy-l0cyMroinI0@public.gmane.org> 2017-11-15 19:42 ` [PATCH v3 2/2] leds: lm3692x: Introduce LM3692x dual string driver Dan Murphy 2017-11-15 19:42 ` Dan Murphy [not found] ` <20171115194203.13572-2-dmurphy-l0cyMroinI0@public.gmane.org> 2017-11-15 21:12 ` Pavel Machek 2017-11-15 21:12 ` Pavel Machek 2017-11-15 21:25 ` Dan Murphy 2017-11-15 21:25 ` Dan Murphy 2017-11-18 14:19 ` Jacek Anaszewski 2017-11-18 14:19 ` Jacek Anaszewski 2017-11-28 12:53 ` Dan Murphy 2017-11-28 12:53 ` Dan Murphy 2017-11-15 20:15 ` [PATCH v3 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver Jacek Anaszewski 2017-11-15 20:15 ` Jacek Anaszewski [not found] ` <cb5c5a65-d5a0-86a3-36b4-cf06baae9ec4-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2017-11-15 20:31 ` Dan Murphy 2017-11-15 20:31 ` Dan Murphy [not found] ` <0a78c883-f074-cc19-3c0c-fc05607400e9-l0cyMroinI0@public.gmane.org> 2017-11-15 22:23 ` Pavel Machek 2017-11-15 22:23 ` Pavel Machek 2017-11-16 20:14 ` Jacek Anaszewski [not found] ` <82336eb7-2b89-d37b-d688-4e4302766346-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2017-11-16 21:42 ` Dan Murphy 2017-11-16 21:42 ` Dan Murphy 2017-11-17 2:19 ` Jingoo Han 2017-11-17 2:19 ` Jingoo Han 2017-11-17 11:20 ` Pavel Machek 2017-11-17 16:30 ` Jingoo Han 2017-11-17 16:30 ` Jingoo Han 2017-11-17 18:02 ` Dan Murphy [this message] 2017-11-17 18:02 ` Dan Murphy 2017-11-17 23:58 ` Pavel Machek 2017-11-28 17:27 ` Dan Murphy 2017-11-28 17:27 ` Dan Murphy 2017-11-16 15:41 ` Rob Herring 2017-11-16 15:41 ` Rob Herring 2017-11-16 15:45 ` Dan Murphy 2017-11-16 15:45 ` Dan Murphy [not found] ` <d3483e49-650d-1d68-bec6-eeb84d8ac48a-l0cyMroinI0@public.gmane.org> 2017-11-16 15:58 ` Rob Herring 2017-11-16 15:58 ` Rob Herring 2017-11-16 20:11 ` dts: fun with chip names " Pavel Machek 2017-11-16 20:36 ` Rob Herring [not found] ` <CAL_Jsq+aysVR9U2Wquwwv7cjjO7wO8EXBg1Ja82u6s5t1yeJUw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-11-16 21:40 ` Dan Murphy 2017-11-16 21:40 ` Dan Murphy 2017-11-18 14:19 ` Jacek Anaszewski
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=17b8e1ce-092a-0449-4733-533162b26499@ti.com \ --to=dmurphy-l0cymroini0@public.gmane.org \ --cc=daniel.thompson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \ --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=jacek.anaszewski-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=jingoohan1-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \ --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-leds-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \ --cc=pavel-+ZI9xUNit7I@public.gmane.org \ --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=rpurdie-Fm38FmjxZ/leoWH0uzbU5w@public.gmane.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.