Skip to contentSkip to conversation

misc: led: Add PCA9551 LED driver

PATCH Superseded Stefan Roese 3 participants 4 messages Mar 12, 2015 – May 10, 2015 Char and misc drivers

lore
Share & tools
lore

Stefan Roese

author
#0 lore raw

PATCH v2 1/1 misc: led: Add PCA9551 LED driver

This patch adds a driver for the PCA9551 LED controller.

Originated-by Timo Herbrecher <[email protected]>
Signed-off-by Stefan Roese <[email protected]>
Reviewed-by Tom Rini <[email protected]>
Cc Fabio Estevam <[email protected]>

---
v2:
- Removed __packed fromstruct pca9551_blink_rate as it doesn't seem to be needed
- Removed PCA9551 dependency from Kconfig to allow usage with other boards
- Better handling of return codes as suggested by Fabio

 drivers/misc/Kconfig       |  13 ++++
 drivers/misc/Makefile      |   1 +
 drivers/misc/pca9551_led.c | 155 +++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 169 insertions(+)
 create mode 100644 drivers/misc/pca9551_led.c
drivers/misc/Kconfig +13 -0
index 36a8f0d..4114be3 100644
@@ -61,3 +61,16 @@ config CONFIG_FSL_SEC_MON
 	  system states.
 	  Security Monitor can be transitioned on any security failures,
 	  like software violations or hardware security violations.
+
+config PCA9551_LED
+	bool "Enable PCA9551 LED driver"
+	help
+	  Enable driver for PCA9551 LED controller. This controller
+	  is connected via I2C. So I2C needs to be enabled.
+
+config PCA9551_I2C_ADDR
+	hex "I2C address of PCA9551 LED controller"
+	depends on PCA9551_LED
+	default 0x60
+	help
+	  The I2C address of the PCA9551 LED controller.
drivers/misc/Makefile +1 -0
index 6028cd4..149e686 100644
@@ -29,3 +29,4 @@ obj-$(CONFIG_STATUS_LED) += status_led.o
 obj-$(CONFIG_TWL4030_LED) += twl4030_led.o
 obj-$(CONFIG_FSL_IFC) += fsl_ifc.o
 obj-$(CONFIG_FSL_SEC_MON) += fsl_sec_mon.o
+obj-$(CONFIG_PCA9551_LED) += pca9551_led.o
drivers/misc/pca9551_led.c new file +155 -0

155 lines not on this page · raw message

Signature · 1 line
2.3.2
author
#2 lore raw

PATCH v2 1/1 misc: led: Add PCA9551 LED driver

Hi Tom,

Quoted 9 lines · Fabio Estevam
Quoted 6 lines · Stefan Roese

This patch adds a driver for the PCA9551 LED controller.

Originated-by Timo Herbrecher <[email protected]>
Signed-off-by Stefan Roese <[email protected]>
Reviewed-by Tom Rini <[email protected]>
Cc Fabio Estevam <[email protected]>
Reviewed-by Fabio Estevam <[email protected]>

Gentle ping on this patch. Could you please pull this one for the next release?

Thanks,
Stefan

#3 lore raw

PATCH v2 1/1 misc: led: Add PCA9551 LED driver

In reply to Stefan Roese
Quoted 7 lines · Stefan Roese

This patch adds a driver for the PCA9551 LED controller.

Originated-by Timo Herbrecher <[email protected]>
Signed-off-by Stefan Roese <[email protected]>
Reviewed-by Tom Rini <[email protected]>
Cc Fabio Estevam <[email protected]>
Reviewed-by Fabio Estevam <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150510/a74deb42/attachment.sig>