Re: leds-is31fl319x

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Thomas,

On 9/20/24 16:48, Weber, Thomas wrote:
Hello,
you are listed as module authors for the leds-is31fl319x driver.

Should the color property and the pattern property be functional in this module?

I added this to my device tree

is31fl@64 {
     compatible = "issi,is31fl3199";
     reg = <0x64>;
     #address-cells = <1>;
     #size-cells = <0>;
     pinctrl-names = "default";
     pinctrl-0 = <&pinctrl_is31_sdb>;
     shutdown-gpios = <&gpio5 6 GPIO_ACTIVE_HIGH>;

     led@1 {
       reg = <1>;
       label = "IND_LED_WH_1";
       color = <LED_COLOR_ID_WHITE>;
       linux,default-trigger = "pattern";
       led-pattern = <0 1000 255 2000>;
     };

     led@2 {
       reg = <2>;
       label = "IND_LED_BU_1";
       color = <LED_COLOR_ID_BLUE>;
     };

But in the rootfs I get this:

root@vsm:/sys/class/leds/IND_LED_BU_1 cat color
white

Are you working with mainline kernel? LED class does not create 'color'
sysfs file AFAICS, but uses that DT property to construct LED class
device name (and in LED class multicolor).

And anyway, this driver does not use
devm_led_classdev_register_ext(), which is required to parse color
property, so the LED class device name is created using what
is provided in 'label' DT property.

root@vsm:/sys/class/leds/IND_LED_WH_1 cat trigger
[none] timer heartbeat cpu cpu0 cpu1 cpu2 cpu3 default-on panic mmc2 sbs-2-000b-charging-or-full sbs-2-000b-charging sbs-2-000b-full sbs-2-000b-charging-blink-full-solid rfkill-any rfkill-none bluetooth-power


What am I doing wrong?

You seem to have LEDS_TRIGGER_PATTERN turned off in kernel config.

Best regards
Thomas


--
Best regards,
Jacek Anaszewski




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux