-
Notifications
You must be signed in to change notification settings - Fork 139
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
Update DS-019 Pixhawk versions and revisions.md #53
Conversation
I would like to apply for a version of the flight control board, can I choose the value of 6?
@@ -150,7 +150,7 @@ HWTYPE = {v5/6x}{VER}{REV} | |||
| 0x003 | resistors | NXP T1 PHY | | |||
| 0x004 | resistors | HB CM4 | | |||
| 0x005 | resistors | HB mini | | |||
| 0x006 | resistors | RESERVED DRONECODE | | |||
| 0x006 | resistors | ZeroOne X6 | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is already reserved you will need to use and EEPROM code point
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want to determine the two resistance values on the base board, which value of base board versions can I use?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this even apply if they are using their own build target? @davids5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mrpollo I am assuming this is for a BASE board only. In that case they should use code 0x007 (46.4K (R1) and 84.5K for R2) Then Drone would provide the files and instructions to assign a EEPROM code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we use EEPROM code, does it mean that the resistors on the baseboard and FMU board should use code 0x007 (46.4K (R1) and 84.5K for R2)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The FMUM has an id: HW FMUM ID: 0xXXX
The Base has an id: HW BASE ID: 0xXXX
Since you are adding board support for your FMUM you can choose any ID you want for the FMUM - Rev 0 is nop pop of resistors. The code that runs on your FMUM will have to deal with any FMUM - Revs you ship. This may be sensor changes etc.
In contrast: If you included your FMUM board support in the px4_fmu-v6x board, you would use EEPROM code point resistors. Dronecode would give you a tool and an FMUM_ID to program on the board in manufacturing.
If you base board is different: Adds some feature not supported on other base board. (Your BASE is not 100% compatible with any of the already defined ID) you would use EEPROM code point resistors. Dronecode would give you a tool and an BASE_ID to program on the board in manufacturing.
In contrast: If the BASE is exactly the same (electrically and connectors ) as a Auterion v5x base board
or `Holybro base Board' you can use Config 0 (nop pop)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sensors on our FMU and IMU include: 2 ICM45686, 1 BMI088, 2 ICP20100, and 1 RM3100, which we are ready to use Rev 0(nop pop of resistors).
Then the base board has some functional updates, such as PWM output 3.3V and 5V switching, and we are ready to use EEPROM code point resistors.
Do you think that's okay?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PX4/PX4-Autopilot#23477
In this link, after I updated the pab_manifest.c, I encountered an error when uploading it to PR. Could you please help me find out the reason?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PX4/PX4-Autopilot#23477 In this link, after I updated the pab_manifest.c, I encountered an error when uploading it to PR. Could you please help me find out the reason?
I have modified it according to your suggestion, please help to review it.
Dear developers, we plan to use No pop for the resistor on the FMU board and EEPROM code on the baseboard. I have modified the bottom line, please help review it. | 0x150 | EEPROM | ZeroOne X6 Baseboard |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mrpollo Does the assignment of 0x150 work or would you suggest another value?
The value is changed to 0x110: | 0x110 | EEPROM | ZeroOne X6 Baseboard |
Thank you for your reply, we would like to change it to 0x110.I have submitted the modification, please kindly review it. |
Hey @ZeroOne-Aero, I think |
Change it to:0x150
Ok, I've changed it to 0x150. |
I would like to apply for a version of the flight control board, can I choose the value of 6?