Fix notification LED on jasmine

[p4block]

The HAL behaves in mysterious ways and requires this blink file to be writable. Also sync permissions with stock ROM just in case.

[phhusson]

You tested that modified rw-system.sh right?

[p4block]

I patched my current rw-system instead of building a new system, but yes, the files get successfully modified.

There still seems to be an issue where the LED is randomly initialized to be "on" on android start.
It seems to be related to charging. If the LED has booted as off, charging will not turn it on. If the LED has booted as on, (and manually turned off), charging will turn it back on.

I will edit this once I find a final fix.

[suwakowww]

Feedback for wayne:
I tried to modified like this on my phone. (append these commands on the same file but append it after line 137) But it still "always on" after several reboots, (and turn back normal after reboot twice.)

[tytydraco]

The HAL behaves in mysterious ways and requires this blink file to be writable. Also sync permissions with stock ROM just in case.

I had my tester give this a shot on the AEX Pie GSI (arm64). It doesn't seem to work. His duty_pcts was root:root so I had him chown system:system like stock. No dice.

[tytydraco]

Some more feedback: It breaks itself when plugged into power. However, If one reboots twice (with the cable unplugged), the issue is resolved. However, charging the device will no longer trigger the LED. So it seems that there's most likely an issue with the LED receiving events..?

[tytydraco]

This device tree file includes all the proper commands necessary for this to work. Please update it based on this:

https://github.com/xiaomi-sdm660/android_device_xiaomi_jasmine/blob/41c4682966532c1195585ed4e40c52a3ed06fc1c/light/android.hardware.light%402.0-service.xiaomi_jasmine.rc

on boot
    # Notification LED
    chown system system /sys/class/leds/red/blink
    chown system system /sys/class/leds/red/brightness
    chown system system /sys/class/leds/red/duty_pcts
    chown system system /sys/class/leds/red/pause_hi
    chown system system /sys/class/leds/red/pause_lo
    chown system system /sys/class/leds/red/ramp_step_ms
    chown system system /sys/class/leds/red/start_idx

    chmod 660 /sys/class/leds/red/duty_pcts
    chmod 660 /sys/class/leds/red/pause_hi
    chmod 660 /sys/class/leds/red/pause_lo
    chmod 660 /sys/class/leds/red/ramp_step_ms
    chmod 660 /sys/class/leds/red/start_idx

[penn5]

@p4block this fixes on Oreo vendor, Pie vendor or both?

[likeadragonmaid]

@tytydraco Yes. I was the tester. Btw are there any updates on this issue? @phhusson

[tytydraco]

Please close this thread. A proper fix was merged into the main branch.