[PATCH RFC v4 0/1] leds: asus-aura-scsi: Add ASUS Aura RGB LED driver for ROG NVMe enclosures
From: Liang Haowen
Date: Tue Sep 15 2026 - 10:51:57 EST
Hello,
v4, sent as its own thread this time, addressing the third sashiko
review round (the Medium naming item is a real bug, the two Low ones
are false positives again).
Changes since v3:
- LED class device names follow the devicename:color:function scheme
again. v3 embedded the sdev's H:C:T:L verbatim
(asus-arion-0:0:0:0:led0); the extra colons break the single
separator userspace parses LED names with. The colons are now
flattened to dashes (asus-arion-0-0-0-0:led0), keeping the
per-attachment uniqueness while leaving exactly one separator.
The two low-severity items are the same false positives as before:
- kzalloc_obj() exists in include/linux/slab.h since v7.0 (Kees
Cook's overflow-refactor series); this driver builds against 7.2.
- blk_rq_map_kern() takes four arguments on current kernels
(rq, buf, len, gfp); drivers/scsi/scsi_lib.c calls it exactly this
way from scsi_execute_cmd().
v4 was re-verified on hardware: per-LED colours, 100 sequential
updates, concurrent updates from four writers, and unplug under load
(zero splats, zero leaked LED nodes, clean rmmod).
Everything else is unchanged: the hardware description, the
scsi_device_handler that does not claim the sdev, the multicolor LED
interface, the protocol handling and the known caveats (manual attach
until a notifier lands; SAVE on every update writes the enclosure
flash, wear uncharacterized; NULL-parent LED registration to avoid
the sdev reference cycle).
The open question from v3 stands: the driver is deliberately not
wired into Kconfig/Makefile/MAINTAINERS yet, because the agreed
direction with the SCSI side is a split into a SCSI transport helper
and a shared ASUS Aura LED interface, and the wiring would follow
that shape. Is deferring the wiring to that split acceptable for an
RFC, or would you rather have the driver buildable in-tree from this
series already?
Comments on the interface shape and on folding this into the shared
Aura work with Denis remain very welcome.
Signed-off-by: Liang Haowen <nbg2974@xxxxxxxxx>