Skip to content
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

AP_Bootloader: Reserve 4 boards ID for MFE_PDB_CAN、MFE_POS3_CAN、MFE_RTK_CAN、MFE_AirSpeed_CAN. #28483

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mikefenghao
Copy link
Contributor

AP_Bootloader: Reserve a board ID for MFE_PDB_CAN、MFE_POS3_CAN、MFE_RTK_CAN、MFE_AirSpeed_CAN.
Here are some CAN devices, and hardware definitions will be submitted later.

@@ -390,6 +390,12 @@ AP_HW_DroneBuild_G2 5701

# IDs 6000-6099 reserved for SpektreWorks

# IDs 6100-6199 reserved for MFE
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think our new standard is to reserve at most 10 ID's at a time now.

@@ -400,6 +406,8 @@ AP_HW_CUAV-7-NANO 7000
# IDs 7100-7109 reserved for VIEWPRO
AP_HW_VUAV-V7pro 7100

# IDs 7100-7109 reserved for VIEWPRO
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this a typo?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have changed it to 10 ID'S and removed the TYPO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants