Re: [PATCH v5 2/9] documention: leds: Add multicolor class documentation

From: Pavel Machek
Date: Thu Sep 12 2019 - 16:55:56 EST


Hi!

> +Directory Layout Example
> +========================
> +root:/sys/class/leds/rgb:grouped_leds# ls -lR colors/
> +colors/:
> +drwxr-xr-x 2 root root 0 Jun 28 20:21 blue
> +drwxr-xr-x 2 root root 0 Jun 28 20:21 green
> +drwxr-xr-x 2 root root 0 Jun 28 20:21 red
> +-rw------- 1 root root 4096 Jun 28 20:21 color_mix
> +
> +colors/blue:
> +-rw------- 1 root root 4096 Jun 28 20:21 intensity
> +-r-------- 1 root root 4096 Jun 28 20:27 max_intensity
> +-r-------- 1 root root 4096 Jun 28 20:21 color_id

I don't really like the directories... A bit too much complexity, and
it will have a memory footprint, too.

I'd expect max_intensity to be same for all the leds in
rgb:grouped_leds... Could we simply rely on max_brightness file?

[If not, would one "max_intensity" file in rgb:grouped_leds be
enough?]

Best regards,
Pavel

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachment: signature.asc
Description: Digital signature