All of lore.kernel.org
 help / color / mirror / Atom feed
From: kgunda@codeaurora.org
To: Daniel Thompson <daniel.thompson@linaro.org>
Cc: bjorn.andersson@linaro.org, jingoohan1@gmail.com,
	lee.jones@linaro.org, b.zolnierkie@samsung.com,
	dri-devel@lists.freedesktop.org, jacek.anaszewski@gmail.com,
	pavel@ucw.cz, robh+dt@kernel.org, mark.rutland@arm.com,
	linux-leds@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	linux-arm-msm-owner@vger.kernel.org
Subject: Re: [PATCH V4 0/8] backlight: qcom-wled: Support for QCOM wled driver
Date: Mon, 06 Aug 2018 16:28:47 +0530	[thread overview]
Message-ID: <405f3406c933b93d84a554f4552941f4@codeaurora.org> (raw)
In-Reply-To: <20180803074503.xvjvllsejw7xq3ox@holly.lan>

On 2018-08-03 13:15, Daniel Thompson wrote:
> On Fri, Aug 03, 2018 at 12:49:34PM +0530, kgunda@codeaurora.org wrote:
>> Hi Bjorn,
>> Can you please help review this patch series ?
>> 
>> Pavel, Rob, Daniel reviewed the series except the "auto string 
>> detection"
>> patch.
> 
> I did take a glance at the last patch... but in the end decided to wait
> for v5 to look in depth.
> 
> 
> Daniel.
> 
> 
>> 
Thanks Daniel! I will push the V5 series then.

>> Thanks,
>> Kiran
>> 
>> On 2018-07-09 15:52, Kiran Gunda wrote:
>> > This patch series renames the pm8941-wled.c driver to qcom-wled.c to add
>> > the support for multiple PMICs supported by qualcomm. This patch series
>> > supports both PM8941 and PMI8998 WLED. The PMI8998 WLED has the support
>> > to handle the OVP (over voltage protection) and the SC (short circuit
>> > protection)
>> > interrupts. It also has the auto string detection algorithm support to
>> > configure the right strings if the user specified string configuration
>> > is in-correct. These three features are added in this series for
>> > PMI8998.
>> >
>> > changes from v1:
>> >    - Fixed the commit message for
>> >    - backlight: qcom-wled: Rename pm8941-wled.c to qcom-wled.c
>> >
>> > Changes from v2:
>> >    - Fixed bjorn and other reviewer's comments
>> >    - Seperated the device tree bindings
>> >    - Splitted out the WLED4 changes in seperate patch
>> >    - Merged OVP and auto string detection patch
>> >
>> > Changes from v3:
>> >   - Added Reviewed-by/Acked-by tags
>> >   - Fixed comments from Bjorn/Vinod/Rob
>> >   - Splitting the "backlight: qcom-wled: Add support for WLED4
>> > peripheral" patch
>> >     to seperate the WLED3 specific restructure.
>> >
>> > Kiran Gunda (8):
>> >   backlight: qcom-wled: Rename pm8941-wled.c to qcom-wled.c
>> >   backlight: qcom-wled: restructure the qcom-wled bindings
>> >   backlight: qcom-wled: Add new properties for PMI8998
>> >   backlight: qcom-wled: Rename PM8941* to WLED3
>> >   backlight: qcom-wled: Restructure the driver for WLED3
>> >   backlight: qcom-wled: Add support for WLED4 peripheral
>> >   backlight: qcom-wled: add support for short circuit handling
>> >   backlight: qcom-wled: Add auto string detection logic
>> >
>> >  .../bindings/leds/backlight/pm8941-wled.txt        |   42 -
>> >  .../bindings/leds/backlight/qcom-wled.txt          |  150 +++
>> >  drivers/video/backlight/Kconfig                    |    8 +-
>> >  drivers/video/backlight/Makefile                   |    2 +-
>> >  drivers/video/backlight/pm8941-wled.c              |  432 -------
>> >  drivers/video/backlight/qcom-wled.c                | 1298
>> > ++++++++++++++++++++
>> >  6 files changed, 1453 insertions(+), 479 deletions(-)
>> >  delete mode 100644
>> > Documentation/devicetree/bindings/leds/backlight/pm8941-wled.txt
>> >  create mode 100644
>> > Documentation/devicetree/bindings/leds/backlight/qcom-wled.txt
>> >  delete mode 100644 drivers/video/backlight/pm8941-wled.c
>> >  create mode 100644 drivers/video/backlight/qcom-wled.c
> --
> To unsubscribe from this list: send the line "unsubscribe 
> linux-arm-msm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

      reply	other threads:[~2018-08-06 10:58 UTC|newest]

Thread overview: 59+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-09 10:22 [PATCH V4 0/8] backlight: qcom-wled: Support for QCOM wled driver Kiran Gunda
2018-07-09 10:22 ` [PATCH V4 1/8] backlight: qcom-wled: Rename pm8941-wled.c to qcom-wled.c Kiran Gunda
2018-07-09 10:34   ` Kiran Gunda
2018-07-09 10:22 ` [PATCH V4 2/8] backlight: qcom-wled: restructure the qcom-wled bindings Kiran Gunda
2018-07-09 10:22 ` [PATCH V4 3/8] backlight: qcom-wled: Add new properties for PMI8998 Kiran Gunda
2018-07-11 15:12   ` Rob Herring
2018-07-20 13:15   ` Daniel Thompson
2018-07-20 13:15     ` Daniel Thompson
2018-08-07  5:23   ` Bjorn Andersson
2018-08-16  5:23     ` kgunda
2018-09-03 18:55       ` Bjorn Andersson
2018-09-26  5:31         ` kgunda
2018-07-09 10:22 ` [PATCH V4 4/8] backlight: qcom-wled: Rename PM8941* to WLED3 Kiran Gunda
2018-07-09 10:34   ` Kiran Gunda
2018-07-20 13:23   ` Daniel Thompson
2018-07-20 13:23     ` Daniel Thompson
2018-07-23  5:55     ` kgunda
2018-07-23  5:55       ` kgunda
2018-08-07  5:41   ` Bjorn Andersson
2018-08-07  5:41     ` Bjorn Andersson
2018-08-16  5:30     ` kgunda
2018-08-16  5:42       ` kgunda
2018-07-09 10:22 ` [PATCH V4 5/8] backlight: qcom-wled: Restructure the driver for WLED3 Kiran Gunda
2018-07-09 10:34   ` Kiran Gunda
2018-07-20 13:40   ` Daniel Thompson
2018-07-20 13:40     ` Daniel Thompson
2018-07-20 13:40     ` Daniel Thompson
2018-08-07  5:37   ` Bjorn Andersson
2018-08-07  5:37     ` Bjorn Andersson
2018-08-16  5:28     ` kgunda
2018-08-16  5:40       ` kgunda
2018-07-09 10:22 ` [PATCH V4 6/8] backlight: qcom-wled: Add support for WLED4 peripheral Kiran Gunda
2018-07-09 10:34   ` Kiran Gunda
2018-07-20 13:47   ` Daniel Thompson
2018-07-20 13:47     ` Daniel Thompson
2018-07-20 13:47     ` Daniel Thompson
2018-07-20 21:21     ` Pavel Machek
2018-07-20 21:21       ` Pavel Machek
2018-07-23  6:16       ` kgunda
2018-07-23  6:28         ` kgunda
2018-08-07  5:45   ` Bjorn Andersson
2018-08-07  5:45     ` Bjorn Andersson
2018-07-09 10:22 ` [PATCH V4 7/8] backlight: qcom-wled: add support for short circuit handling Kiran Gunda
2018-07-09 10:34   ` Kiran Gunda
2018-07-20 14:07   ` Daniel Thompson
2018-07-20 14:07     ` Daniel Thompson
2018-07-20 14:07     ` Daniel Thompson
2018-07-23  6:05     ` kgunda
2018-07-23  6:17       ` kgunda
2018-07-09 10:22 ` [PATCH V4 8/8] backlight: qcom-wled: Add auto string detection logic Kiran Gunda
2018-07-09 10:34   ` Kiran Gunda
2018-08-07  6:32   ` Bjorn Andersson
2018-08-07  6:32     ` Bjorn Andersson
2018-08-16  5:33     ` kgunda
2018-08-16  5:45       ` kgunda
2018-08-03  7:19 ` [PATCH V4 0/8] backlight: qcom-wled: Support for QCOM wled driver kgunda
2018-08-03  7:45   ` Daniel Thompson
2018-08-03  7:45     ` Daniel Thompson
2018-08-06 10:58     ` kgunda [this message]

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=405f3406c933b93d84a554f4552941f4@codeaurora.org \
    --to=kgunda@codeaurora.org \
    --cc=b.zolnierkie@samsung.com \
    --cc=bjorn.andersson@linaro.org \
    --cc=daniel.thompson@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jacek.anaszewski@gmail.com \
    --cc=jingoohan1@gmail.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-arm-msm-owner@vger.kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-leds@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=pavel@ucw.cz \
    --cc=robh+dt@kernel.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: link
Be 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.