search

RadxaYuntian / mraa

Linux Library for low speed IO Communication in C with bindings for C++, Python, Node.js & Java. Supports generic io platforms, as well as Intel Edison, Intel Joule, Raspberry Pi and many more.
http://mraa.io
MIT License
0 stars 0 forks source link

Pwm issue #12

Closed nascs closed 6 months ago

nascs commented 8 months ago

5a/5b/3c/cm3 pwm 都用 led 重新测了一遍，3c pin 19 那个 pwm15_m0 被 fan0 占用

RadxaYuntian commented 8 months ago

所以是必须要加上这个pwm.pinmap才能使用pwm么？这个应该是在结构分配内存时自动赋0了吧？

nascs commented 8 months ago

所以是必须要加上这个pwm.pinmap才能使用pwm么？这个应该是在结构分配内存时自动赋0了吧？

上周， 看 ROCK 3C 的 pin 7 脚 pwm 无法使用，后面加上 pwm.pinmap 就能用，但是今天的情况就是加不加都能用，打印了 board->pins[pin].pwm.pinmap 默认会初始化为 0，所以应该是不加也行

RadxaYuntian commented 8 months ago

所以还是没搞清楚为什么pwm不行咯

RadxaYuntian commented 8 months ago

已有文件的顺序就别改了吧，回头可以单独弄一个formatting的PR，这里还是主要修复使用问题

RadxaYuntian commented 6 months ago

这边也要处理啊

nascs commented 6 months ago

这边也要处理啊

等 ROCK PI E 的 PR 合并

RadxaYuntian commented 6 months ago

这两个不影响，你可以先改，然后rock pi e的合并了再rebase

nascs commented 6 months ago

这两个不影响，你可以先改，然后rock pi e的合并了再rebase

好的，下周一

nascs commented 6 months ago

这边也要处理啊

所有板子的顺序要不要再这一块处理了？

RadxaYuntian commented 6 months ago

你要弄就先有个单独的commit处理格式问题。

nascs commented 6 months ago

pwm修正的两个提交合并一下，然后pwm初始化的提交消息就写fix而不是unified。okdo那边已经发现之前的初始化是有问题的

指的是不加 pinmap 会有问题？

RadxaYuntian commented 5 months ago

这个可以提交给上游了

nascs commented 5 months ago

这个可以提交给上游了

eclipse/mraa/pull/1129

RadxaYuntian commented 5 months ago

一般链接PR/issue是用eclipse/mraa#1129这样的格式，效果：eclipse/mraa#1129