Hi Pavel, On Wed, Jul 15, 2020 at 7:33 AM Pavel Machek <pavel@xxxxxx> wrote: > > Hi! > > > Add support for the LED feature of the NCT6795D chip found on some > > motherboards, notably MSI ones. The LEDs are typically used using a > > RGB connector so this driver creates one LED device for each color > > component. > > Ok, let me take a look. What entries does it present in /sys? Right now these 3 directories in /sys/class/leds: nct6795d:blue: nct6795d:green: nct6795d:red: with the usual suspects `brightness` and `max_brightness` in each. I am not 100% sure I got the names right so please let me know if that is not correct. > > We'll want you to switch to multicolor framework. Most definitely. Last time I checked it was not merged yet though? > > > Also add self as maintainer. > > That will need to be separate patch, I'd be asking for trouble if I > took that. Note that I am also comfortable if you prefer to take care of this, just didn't want to look like I was throwing this code at you without following up on its maintenance. Cheers, Alex.