MichaIng / DietPi

Lightweight justice for your single-board computer!
https://dietpi.com/
GNU General Public License v2.0
4.68k stars 491 forks source link

Radxa Zero LED permanently on #6372

Open Jargendas opened 1 year ago

Jargendas commented 1 year ago

Required Information

DietPi 8.17.2 | bulleye | 6.1.11-meson64 | Radxa Zero (arch64) (1 GB memory, V1.51)

Additional Information (if applicable)

Steps to reproduce

Boot up DietPi on Radxa Zero.

Expected behaviour

The power LED next to the power USB should be showing some activity, e.g. when the SD card is active, like on the Raspberry Pi.

Actual behaviour

The LED is permanently on.

Extra details

The problem might only exist for hardware revision 1.51, as the pin of the power LED was changed. Also, I found a post explaining how to modify the device tree in Radxa's OS version to match the correct pin. However, the device tree used by DietPi seems to be different, at least I could not find the referenced line.

Joulinar commented 1 year ago

Did you tried dietpi-led_control to change behaviour of LED?

Jargendas commented 1 year ago

I have now, it fails with [FAILED] DietPi-LED_control | No LED devices found in /sys/class/leds/. Exiting....

Jargendas commented 1 year ago

This probably means the LED is not present in the device tree altogether, so all versions of the Zero have this problem.

MichaIng commented 1 year ago

Just to verify:

ls -l /sys/class/leds/

Possibly another regression with Linux 6.1.

Jargendas commented 1 year ago

Yeah, that's empty. Does this have to be fixed upstream in the Linux Kernel?

MichaIng commented 1 year ago

Not sure. At least we should have a look into the Armbian kernel patches: https://github.com/armbian/build/tree/main/patch/kernel/archive/meson64-6.1

Here the upstream device tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dts?h=linux-6.1.y

pyavitz commented 1 year ago

Depending on the revision of the board 1.4 (GPIOAO_8) 1.51 (GPIOAO_10) the LED node needs to be adjusted. So really it is better to not set it in the DTS, but to use an overlay to get it working.

A working example can be found here: https://github.com/pyavitz/debian-image-builder/blob/feature/patches/amlogic/radxazero/6.1/002-arch-arm64-dts-meson-g12a-radxa-zero-overlays.patch#L23