On Sun, Dec 16, 2018 at 06:23:53PM +0100, Roger Pueyo Centelles | Guifi.net
wrote:
>
>
> Hi,
Hello Roger!
> [...]
>
> leds {
> compatible = "gpio-leds";
> pinctrl-1 = <&led_rssilow_pin &led_rssimediumhigh_pin
> &led_rssihigh_pin>;
>
> [...]
>
> &pinmux {
> led_rssilow_pin: pinmux_rssilow_pin {
> pinctrl-single,bits = <0x8 0x0 0xff000000>;
> };
>
> led_rssimediumhigh_pin: pinmux_rssimediumhigh_pin {
> pinctrl-single,bits = <0xc 0x0 0x00ff0000>;
> };
>
> led_rssihigh_pin: pinmux_rssihigh_pin {
> pinctrl-single,bits = <0x10 0x0 0x000000ff>;
> };
> };
>
> [...]
The pinmux part looks OK to me. Could you change the leds part to the
below and try again?
leds {
compatible = "gpio-leds";
pinctrl-names = "default";
pinctrl-0 = <&jtag_disable_pins &led_rssilow_pin
&led_rssimediumhigh_pin &led_rssihigh_pin>;
[...]
I added the jtag bit because I saw that you use it under keys. You have
to remove
pinctrl-0 = <&jtag_disable_pins>;
under keys. Just put them all in one place.
>From my testing, when defining pinctrl-0 and pinctrl-1, the second one
doesn't do anything. For example:
pinctrl-0 = <&jtag_disable_pins>; // works
pinctrl-1 = <&enable_gpio_11>; // nothing happens
But
pinctrl-0 = <&jtag_disable_pins &enable_gpio_11>; // both are applied - works
Kind regards,
Seb
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel
