-
Notifications
You must be signed in to change notification settings - Fork 13.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PR #6619 continued #7200
PR #6619 continued #7200
Conversation
@bkueng Can you rebase and get this in? |
And when you do, please reword the chinese commit messages in english 8) |
64614c9
to
fee6129
Compare
Rebased & cleaned up the commits a bit |
2e841e0
to
8dc5096
Compare
@davidaroyer I merged the ocpoc_mmap_pwm_out driver into a more generic linux_pwm_out and fixed the I2C LED device path (last 5 commits). Can you please test it (preferable including ESC calibration)? |
@bkueng I ran a quick flight after testing the ESC calibration; everything worked well for me. Thanks! |
Cool! |
@bkueng @LorenzMeier On nano pi plantform , 64 bit linux sytem is avalible( I'm running 64 bit Ubuntu and Debian on it) http://wiki.friendlyarm.com/wiki/index.php/Main_Page#GettingStarted |
Great, thanks! Looking forward to it. |
Both 8 channels PPM encoder and 8 channels revicer are required. Before launch px4, ppmdeocde programe should be launched. To download ppmdecode programe, visit https://github.com/crossa/raspberry-pi-px4firmware. Pxfmini and navio are not popular autopilot hardware in china, I can handly to purchase it. So that I use raspberry pi to build autopilot separately. This dirver help us to decode ppm single to pwm and pushlish it
Now I've fixed style of the code
2. Rewrite and rebase pca9685 driver 3. Try to fix issue when push the stick of channel 3 to the maxmum position, 0uswill be output to channel 1, should be maxmum pwm signal 4. Fix the code style
Add -D__DF_RPI in posix_rpi_common.cmake Add rpi_rc_in and rpi_pca9685_pwm_out driver in posix_rpi_common.cmake Rename posix_rpi_cross_without_navio_or_pxfmini to posix_rpi_cross_no_shield.cmake
2. Remove printf in the cpp
This is already added to the global list, and we already removed this flag previously, but some of them sneaked back in.
previously, get_devname() was used as the I2C device path, but on NuttX, get_devname() is the device file which the driver creates. This patch changes it, so the sematics are the same as on NuttX: both now use _bus to decide to which I2C bus device to talk to. I did not see any other use-cases than the led on ocpoc.
09eb118
to
96383aa
Compare
Rebased & merged |
Main changes: code cleanup & merging of navio_sysfs_pwm output driver with the new pwm driver.