From: Florian Vaussard <florian.vaussard@gmail.com>
To: Jacek Anaszewski <jacek.anaszewski@gmail.com>,
Pavel Machek <pavel@ucw.cz>
Cc: linux-leds@vger.kernel.org, linux-kernel@vger.kernel.org,
Florian Vaussard <florian.vaussard@gmail.com>
Subject: [PATCH v4 0/2] leds: Add driver for NCP5623 3-channel I2C LED driver
Date: Wed, 21 Feb 2018 22:46:52 +0100 [thread overview]
Message-ID: <20180221214654.3306-1-florian.vaussard@gmail.com> (raw)
Hello Jacek and Pavel,
Winter came, then spring, summer and automn went away. Snow came again,
and now here is the v4 of the NCP5623 patch... Never too late!
This series add a new driver for On Semiconductor NCP5623, a 3-channel I2C
LED driver. It is used in our design to drive a RGB LED.
The first patch introduces the device tree binding, while the second patch
adds the driver itself.
Best regards,
Florian
---
v3 -> v4:
- Rebased on Linux 4.15
- Moved ncp5623_of_match[] after probe
- Fixed computation of max_brightness
- Some more simplifications
- Updated copyright and SPDX license
v2 -> v3:
- Rebased on latest leds/for-next
- Minor fixes to the binding documentation
- Removed ncp5623_set_pwm() by inlining it directly inside the caller
- Removed ncp5623_destroy_devices() as we are already using devm_ flavours
- Got rid of the 'active' state variable by using 'led_no' instead
- Some other cosmetic fixes
v1 -> v2:
- Adapted the DT binding (led-max-microamp for each LED node)
- Removed underscores from node names in the example
- Use brightness_set_blocking to avoid workqueue
- Introduced LED_to_CMD macro to avoid switch statement
- Various other fixes
Florian Vaussard (2):
leds: ncp5623: Add device tree binding documentation
leds: Add driver for NCP5623 3-channel I2C LED driver
.../devicetree/bindings/leds/leds-ncp5623.txt | 60 ++++++
drivers/leds/Kconfig | 11 +
drivers/leds/Makefile | 1 +
drivers/leds/leds-ncp5623.c | 228 +++++++++++++++++++++
4 files changed, 300 insertions(+)
create mode 100644 Documentation/devicetree/bindings/leds/leds-ncp5623.txt
create mode 100644 drivers/leds/leds-ncp5623.c
--
2.13.6
next reply other threads:[~2018-02-21 21:47 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-21 21:46 Florian Vaussard [this message]
2018-02-21 21:46 ` [PATCH v4 1/2] leds: ncp5623: Add device tree binding documentation Florian Vaussard
2018-02-24 8:52 ` Pavel Machek
2018-02-25 14:30 ` Jacek Anaszewski
2018-02-21 21:46 ` [PATCH v4 2/2] leds: Add driver for NCP5623 3-channel I2C LED driver Florian Vaussard
2018-02-21 22:20 ` Andy Shevchenko
2018-02-24 8:53 ` Pavel Machek
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=20180221214654.3306-1-florian.vaussard@gmail.com \
--to=florian.vaussard@gmail.com \
--cc=jacek.anaszewski@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-leds@vger.kernel.org \
--cc=pavel@ucw.cz \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox