search

radxa-pkg / radxa-overlays

Additional device tree overlays to support different hardwares on Radxa products
GNU General Public License v2.0
43 stars 31 forks source link

feat: open pwm1-m2 for rock-5b-plus #348

Closed nascs closed 4 months ago

nascs commented 4 months ago

另外 PWM1-M2 这个是否有必要打开？或者关闭 Overlay, 在 mraa 上面也关闭？之前 5B 是没有开 PWM1-M2 这个 Overlay 的

https://github.com/radxa/kernel/blob/6ca933c58c3f32d8ddd47db50af7d189120fc955/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts#L943 https://github.com/radxa/kernel/blob/6ca933c58c3f32d8ddd47db50af7d189120fc955/arch/arm64/boot/dts/rockchip/rk3588-rock-5b-plus.dts#L924

nascs commented 4 months ago

可以支持的话就打开呗。之前为啥关闭？

详情，不记得了，记得应该有问过你。这个打开的话，就得看看其他产品的 mraa 有无类似情况。

RadxaYuntian commented 4 months ago

那估计是产品上什么设备用到了这个引脚或者这个pwm控制器，你看看

RadxaYuntian commented 4 months ago

哦我看到了是设备树默认打开。还是检查一下这个是否有实际设备在使用，没有的话设备树应该默认关闭，然后添加对应的overlay

nascs commented 4 months ago

哦我看到了是设备树默认打开。还是检查一下这个是否有实际设备在使用，没有的话设备树应该默认关闭，然后添加对应的overlay

也就是在板级 dts 里面移除风扇配置，然后加一个打开的 overlay ?

RadxaYuntian commented 4 months ago

这个给风扇用了那就别折腾了

nascs commented 4 months ago

所以这个到底在5b+上能打开么？还是也是给风扇用了？

pwm1-m0 给风扇用了，

https://github.com/radxa/kernel/blob/6ca933c58c3f32d8ddd47db50af7d189120fc955/arch/arm64/boot/dts/rockchip/rk3588-rock-5b-plus.dts#L924