radxa / kernel

BSP kernel source
Other
146 stars 174 forks source link

Rock 5*A* fan pwm #88

Closed Rippanda12 closed 1 year ago

Rippanda12 commented 1 year ago

Hello on rock 5B fan pwm is /sys/devices/platform/fd8b0010.pwm looking using ls /sys/devices/platform/*pwm* results in nothing

CodeChenL commented 1 year ago

try/sys/class/pwm

RadxaYuntian commented 1 year ago

Fan support has been added in the following commit: https://github.com/radxa/kernel/commit/d3e9cd5cda3ac169c663682f29af1e688519798c.

Rippanda12 commented 1 year ago

Doesn't work the fan doesnt spin up (the chip almost throthles 70C) image image meanwhile on rock 5B theres a pwm* folder which allows the controll image

RadxaYuntian commented 1 year ago

Once the PWM signal is registered as pwm-fan it cannot be accessed by standard PWM interface since it is now part of thermal subsystem. You should set thermal governors to step_wise so the fan can be controlled according to the temperature targets.

Rippanda12 commented 1 year ago

then why isnt the same done for rock 5b?

RadxaYuntian commented 1 year ago

This is part of the requirement for our new systems, that all PWM fan has to be properly registered as thermal cooling devices. 5B's system was created before this change, so it will be updated later. We expect to release new 5B image along with 5A.