Problem increasing the number of RPM sensors

111153 · February 4, 2022, 10:03am

I am currently modifying AP_RPM.cpp, where only two RPM sensors are supported.

However, the problems discovered after the build are as follows.

BRD_PWM_COUNT Parameters Disappeared
In RPM parameter, RPM 1 was originally RPM, but there’s 1 attached to it.

Before changing the code, RPM measurement using Hall sensor is possible by expanding up to two.

However, after changing the code, the BRD_PWM_COUNT parameter, which can adjust the number of GPIOs, has disappeared and the RPM measurement function is not also available.

If anyone has expanded the RPM sensor to four, please comment.

amilcarlucas · February 4, 2022, 10:39am

Are you using master branch?
Can you post a github pull request so that we can review you code?

hoijo · February 5, 2022, 6:21am

Here! This ID is my different account.

github.com/ArduPilot/ardupilot

increasing the number of RPM sensor

ArduPilot:master ← hoijo:rpm_add_branch

opened 06:19AM - 05 Feb 22 UTC

hoijo

+61 -36

I am currently modifying AP_RPM.cpp, where only two RPM sensors are supported. … However, the problems discovered after the build are as follows. 1. BRD_PWM_COUNT Parameters Disappeared 2. In RPM parameter, RPM 1 was originally RPM, but there’s 1 attached to it. Before changing the code, RPM measurement using Hall sensor is possible by expanding up to two. However, after changing the code, the BRD_PWM_COUNT parameter, which can adjust the number of GPIOs, has disappeared and the RPM measurement function is not also available. If anyone has expanded the RPM sensor to four, please comment.