This is the actual driver for the as3668.
There were no related drivers in the kernel until now,
so I created a new file. I've checked around for relatives
of the as3668 but all of them are quite different,
which is why wrote the driver just for the as3668.
Signed-off-by: Lukas Timmermann <linux@timmermann.space>
---
MAINTAINERS | 6 ++
drivers/leds/Kconfig | 14 +++
drivers/leds/Makefile | 1 +
drivers/leds/leds-as3668.c | 196 +++++++++++++++++++++++++++++++++++++
4 files changed, 217 insertions(+)
create mode 100644 drivers/leds/leds-as3668.c
diff --git a/MAINTAINERS b/MAINTAINERS
index c59316109e3f..5929c158a0e2 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3506,6 +3506,12 @@ L: linux-leds@vger.kernel.org
S: Maintained
F: drivers/leds/flash/leds-as3645a.c
+AS3668 LED DRIVER
+M: Lukas Timmermann <linux@timmermann.space>
+L: linux-leds@vger.kernel.org
+S: Maintained
+F: drivers/leds/leds-as3668.c
+
ASAHI KASEI AK7375 LENS VOICE COIL DRIVER
M: Tianshu Qiu <tian.shu.qiu@intel.com>
L: linux-media@vger.kernel.org
diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig
index a104cbb0a001..3fe7a0e98140 100644
--- a/drivers/leds/Kconfig
+++ b/drivers/leds/Kconfig
@@ -100,6 +100,20 @@ config LEDS_ARIEL
Say Y to if your machine is a Dell Wyse 3020 thin client.
+config LEDS_AS3668
+ tristate "LED support for AMS AS3668"
+ depends on LEDS_CLASS
+ depends on I2C
+ help
+ This option enables support for the AMS AS3668 LED controller.
+ The AS3668 provides up to four LED channels and is controlled via
+ the I2C bus. This driver offers basic brightness control for each
+ channel, without support for blinking or other advanced features.
+
+
+ To compile this driver as a module, choose M here: the module
+ will be called leds-as3668.
+
config LEDS_AW200XX
tristate "LED support for Awinic AW20036/AW20054/AW20072/AW20108"
depends on LEDS_CLASS
diff --git a/drivers/leds/Makefile b/drivers/leds/Makefile
index 2f170d69dcbf..983811384fec 100644
--- a/drivers/leds/Makefile
+++ b/drivers/leds/Makefile
@@ -14,6 +14,7 @@ obj-$(CONFIG_LEDS_ADP5520) += leds-adp5520.o
obj-$(CONFIG_LEDS_AN30259A) += leds-an30259a.o
obj-$(CONFIG_LEDS_APU) += leds-apu.o
obj-$(CONFIG_LEDS_ARIEL) += leds-ariel.o
+obj-$(CONFIG_LEDS_AS3668) += leds-as3668.o
obj-$(CONFIG_LEDS_AW200XX) += leds-aw200xx.o
obj-$(CONFIG_LEDS_AW2013) += leds-aw2013.o
obj-$(CONFIG_LEDS_BCM6328) += leds-bcm6328.o
diff --git a/drivers/leds/leds-as3668.c b/drivers/leds/leds-as3668.c
new file mode 100644
index 000000000000..e3f90078f6ec
--- /dev/null
+++ b/drivers/leds/leds-as3668.c
@@ -0,0 +1,196 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Osram AMS AS3668 LED Driver IC
+ *
+ * Copyright (C) 2025 Lukas Timmermann <linux@timmermann.space>
+ */
+
+#include <linux/i2c.h>
+#include <linux/leds.h>
+#include <linux/module.h>
+#include <linux/uleds.h>
+
+#define AS3668_MAX_LEDS 4
+
+/* Chip Registers */
+#define AS3668_CHIP_ID1 0x3e
+#define AS3668_CHIP_ID2 0x3f
+
+#define AS3668_CHIP_ID2_SERIAL_MASK GENMASK(7, 4)
+#define AS3668_CHIP_ID2_REV_MASK GENMASK(3, 0)
+
+#define AS3668_CURRX_CONTROL 0x01
+#define AS3668_CURR1 0x02
+#define AS3668_CURR2 0x03
+#define AS3668_CURR3 0x04
+#define AS3668_CURR4 0x05
+
+/* Constants */
+#define AS3668_CHIP_IDENT 0xa5
+#define AS3668_CHIP_REV1 0x01
+
+struct as3668_led {
+ struct led_classdev cdev;
+ struct as3668 *chip;
+ struct fwnode_handle *fwnode;
+
+ int num;
+};
+
+struct as3668 {
+ struct i2c_client *client;
+ struct as3668_led leds[AS3668_MAX_LEDS];
+};
+
+static int as3668_read_value(struct i2c_client *client, u8 reg)
+{
+ return i2c_smbus_read_byte_data(client, reg);
+}
+
+static int as3668_write_value(struct i2c_client *client, u8 reg, u8 value)
+{
+ int err = i2c_smbus_write_byte_data(client, reg, value);
+
+ if (err)
+ dev_err(&client->dev, "error writing to reg 0x%tx, returned %d", reg, err);
+
+ return err;
+}
+
+static enum led_brightness as3668_brightness_get(struct led_classdev *cdev)
+{
+ struct as3668_led *led = container_of(cdev, struct as3668_led, cdev);
+
+ return as3668_read_value(led->chip->client, AS3668_CURR1 + led->num);
+}
+
+static void as3668_brightness_set(struct led_classdev *cdev, enum led_brightness brightness)
+{
+ struct as3668_led *led = container_of(cdev, struct as3668_led, cdev);
+
+ as3668_write_value(led->chip->client, AS3668_CURR1 + led->num, brightness);
+}
+
+static int as3668_dt_init(struct as3668 *as3668)
+{
+ struct device *dev = &as3668->client->dev;
+ struct as3668_led *led;
+ struct led_init_data init_data = {};
+ int err;
+ u32 reg;
+ int i = 0;
+
+ for_each_available_child_of_node_scoped(dev_of_node(dev), child) {
+ err = of_property_read_u32(child, "reg", ®);
+ if (err)
+ dev_err(dev, "unable to read device tree led reg, err %d", err);
+
+ i = reg;
+
+ if (i < 0 || i > AS3668_MAX_LEDS) {
+ dev_err(dev, "unsupported led reg %d\n", i);
+ return -EOPNOTSUPP;
+ }
+
+ led = &as3668->leds[i];
+ led->fwnode = of_fwnode_handle(child);
+
+ led->num = i;
+ led->chip = as3668;
+
+ led->cdev.max_brightness = U8_MAX;
+ led->cdev.brightness_get = as3668_brightness_get;
+ led->cdev.brightness_set = as3668_brightness_set;
+
+ init_data.fwnode = led->fwnode;
+ init_data.default_label = ":";
+
+ err = devm_led_classdev_register_ext(dev, &led->cdev, &init_data);
+ if (err) {
+ dev_err(dev, "failed to register %d LED\n", i);
+ return err;
+ }
+ }
+
+ return 0;
+}
+
+static int as3668_probe(struct i2c_client *client)
+{
+ u8 chip_id1, chip_id2, chip_serial, chip_rev;
+ struct as3668 *as3668;
+
+ /* Check for sensible i2c address */
+ if (client->addr != 0x42)
+ return dev_err_probe(&client->dev, -EFAULT,
+ "unexpected address for as3668 device\n");
+
+ /* Read identifier from chip */
+ chip_id1 = as3668_read_value(client, AS3668_CHIP_ID1);
+
+ if (chip_id1 != AS3668_CHIP_IDENT)
+ return dev_err_probe(&client->dev, -ENODEV,
+ "chip reported wrong id: 0x%tx\n", chip_id1);
+
+ /* Check the revision*/
+ chip_id2 = as3668_read_value(client, AS3668_CHIP_ID2);
+ chip_serial = FIELD_GET(AS3668_CHIP_ID2_SERIAL_MASK, chip_id2);
+ chip_rev = FIELD_GET(AS3668_CHIP_ID2_REV_MASK, chip_id2);
+
+ if (chip_rev != AS3668_CHIP_REV1)
+ dev_warn(&client->dev, "unexpected chip revision\n");
+
+ /* Print out information about the chip */
+ dev_dbg(&client->dev,
+ "chip_id: 0x%tx | chip_id2: 0x%tx | chip_serial: 0x%tx | chip_rev: 0x%tx\n",
+ chip_id1, chip_id2, chip_serial, chip_rev);
+
+ as3668 = devm_kzalloc(&client->dev, struct_size(as3668, leds, AS3668_MAX_LEDS), GFP_KERNEL);
+ as3668->client = client;
+
+ as3668_dt_init(as3668);
+
+ /* Initialize the chip */
+ as3668_write_value(client, AS3668_CURRX_CONTROL, 0x55);
+ as3668_write_value(client, AS3668_CURR1, 0x00);
+ as3668_write_value(client, AS3668_CURR2, 0x00);
+ as3668_write_value(client, AS3668_CURR3, 0x00);
+ as3668_write_value(client, AS3668_CURR4, 0x00);
+
+ return 0;
+}
+
+static void as3668_remove(struct i2c_client *client)
+{
+ as3668_write_value(client, AS3668_CURRX_CONTROL, 0x0);
+}
+
+static const struct i2c_device_id as3668_idtable[] = {
+ {"as3668"},
+ {}
+};
+
+MODULE_DEVICE_TABLE(i2c, as3668_idtable);
+
+static const struct of_device_id as3668_match_table[] = {
+ {.compatible = "ams,as3668"},
+ {}
+};
+
+MODULE_DEVICE_TABLE(of, as3668_match_table);
+
+static struct i2c_driver as3668_driver = {
+ .driver = {
+ .name = "leds_as3668",
+ .of_match_table = as3668_match_table,
+ },
+ .probe = as3668_probe,
+ .remove = as3668_remove,
+ .id_table = as3668_idtable,
+};
+
+module_i2c_driver(as3668_driver);
+
+MODULE_AUTHOR("Lukas Timmermann <linux@timmermann.space>");
+MODULE_DESCRIPTION("AS3668 LED driver");
+MODULE_LICENSE("GPL");
--
2.49.0
Hi Lukas,
kernel test robot noticed the following build errors:
[auto build test ERROR on lee-leds/for-leds-next]
[also build test ERROR on linus/master v6.15 next-20250605]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Lukas-Timmermann/leds-as3668-Driver-for-the-ams-Osram-4-channel-i2c-LED-driver/20250605-112204
base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/leds.git for-leds-next
patch link: https://lore.kernel.org/r/20250604225838.102910-2-linux%40timmermann.space
patch subject: [PATCH v3 1/2] leds: as3668: Driver for the ams Osram 4-channel i2c LED driver
config: x86_64-kexec (attached as .config)
compiler: clang version 20.1.2 (https://github.com/llvm/llvm-project 58df0ef89dd64126512e4ee27b4ac3fd8ddf6247)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250605/202506052351.GK29s0TN-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202506052351.GK29s0TN-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/leds/Kconfig:116: syntax error
drivers/leds/Kconfig:116: unknown statement "The"
drivers/leds/Kconfig:117:warning: ignoring unsupported character '.'
drivers/leds/Kconfig:117: unknown statement "the"
drivers/leds/Kconfig:118:warning: ignoring unsupported character ','
drivers/leds/Kconfig:118:warning: ignoring unsupported character '.'
drivers/leds/Kconfig:118: unknown statement "channel"
drivers/leds/Kconfig:121:warning: ignoring unsupported character ','
drivers/leds/Kconfig:121:warning: ignoring unsupported character ':'
drivers/leds/Kconfig:121: unknown statement "To"
drivers/leds/Kconfig:122:warning: ignoring unsupported character '.'
drivers/leds/Kconfig:122: unknown statement "will"
make[3]: *** [scripts/kconfig/Makefile:85: oldconfig] Error 1
make[2]: *** [Makefile:731: oldconfig] Error 2
make[1]: *** [Makefile:248: __sub-make] Error 2
make[1]: Target 'oldconfig' not remade because of errors.
make: *** [Makefile:248: __sub-make] Error 2
make: Target 'oldconfig' not remade because of errors.
--
>> drivers/leds/Kconfig:116: syntax error
drivers/leds/Kconfig:116: unknown statement "The"
drivers/leds/Kconfig:117:warning: ignoring unsupported character '.'
drivers/leds/Kconfig:117: unknown statement "the"
drivers/leds/Kconfig:118:warning: ignoring unsupported character ','
drivers/leds/Kconfig:118:warning: ignoring unsupported character '.'
drivers/leds/Kconfig:118: unknown statement "channel"
drivers/leds/Kconfig:121:warning: ignoring unsupported character ','
drivers/leds/Kconfig:121:warning: ignoring unsupported character ':'
drivers/leds/Kconfig:121: unknown statement "To"
drivers/leds/Kconfig:122:warning: ignoring unsupported character '.'
drivers/leds/Kconfig:122: unknown statement "will"
make[3]: *** [scripts/kconfig/Makefile:85: olddefconfig] Error 1
make[2]: *** [Makefile:731: olddefconfig] Error 2
make[1]: *** [Makefile:248: __sub-make] Error 2
make[1]: Target 'olddefconfig' not remade because of errors.
make: *** [Makefile:248: __sub-make] Error 2
make: Target 'olddefconfig' not remade because of errors.
vim +116 drivers/leds/Kconfig
66
67 config LEDS_88PM860X
68 tristate "LED Support for Marvell 88PM860x PMIC"
69 depends on LEDS_CLASS
70 depends on MFD_88PM860X
71 help
72 This option enables support for on-chip LED drivers found on Marvell
73 Semiconductor 88PM8606 PMIC.
74
75 config LEDS_AN30259A
76 tristate "LED support for Panasonic AN30259A"
77 depends on LEDS_CLASS && I2C && OF
78 help
79 This option enables support for the AN30259A 3-channel
80 LED driver.
81
82 To compile this driver as a module, choose M here: the module
83 will be called leds-an30259a.
84
85 config LEDS_APU
86 tristate "Front panel LED support for PC Engines APU/APU2/APU3 boards"
87 depends on LEDS_CLASS
88 depends on X86 && DMI
89 help
90 This driver makes the PC Engines APU1 front panel LEDs
91 accessible from userspace programs through the LED subsystem.
92
93 If you're looking for APU2/3, use the pcengines-apu2 driver.
94 (symbol CONFIG_PCENGINES_APU2)
95
96 To compile this driver as a module, choose M here: the
97 module will be called leds-apu.
98
99 config LEDS_ARIEL
100 tristate "Dell Wyse 3020 status LED support"
101 depends on LEDS_CLASS
102 depends on (MACH_MMP3_DT && MFD_ENE_KB3930) || COMPILE_TEST
103 help
104 This driver adds support for controlling the front panel status
105 LEDs on Dell Wyse 3020 (Ariel) board via the KB3930 Embedded
106 Controller.
107
108 Say Y to if your machine is a Dell Wyse 3020 thin client.
109
110 config LEDS_AS3668
111 tristate "LED support for AMS AS3668"
112 depends on LEDS_CLASS
113 depends on I2C
114 help
115 This option enables support for the AMS AS3668 LED controller.
> 116 The AS3668 provides up to four LED channels and is controlled via
117 the I2C bus. This driver offers basic brightness control for each
118 channel, without support for blinking or other advanced features.
119
120
121 To compile this driver as a module, choose M here: the module
122 will be called leds-as3668.
123
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
© 2016 - 2025 Red Hat, Inc.