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

Update DS-019 Pixhawk versions and revisions.md #53

Merged
merged 4 commits into from
Aug 14, 2024

Conversation

ZeroOne-Aero
Copy link
Contributor

I would like to apply for a version of the flight control board, can I choose the value of 6?

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 |
Copy link
Collaborator

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

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 want to determine the two resistance values on the base board, which value of base board versions can I use?

Copy link
Collaborator

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

Copy link
Collaborator

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.

Copy link
Contributor Author

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)?

Copy link
Collaborator

@davids5 davids5 Aug 6, 2024

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)

Copy link
Contributor Author

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?

Copy link
Contributor Author

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?

Copy link
Contributor Author

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 |
Copy link
Collaborator

@davids5 davids5 left a 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 |
@ZeroOne-Aero
Copy link
Contributor Author

@mrpollo Does the assignment of 0x150 work or would you suggest another value?

Thank you for your reply, we would like to change it to 0x110.I have submitted the modification, please kindly review it.

@mrpollo
Copy link
Collaborator

mrpollo commented Aug 13, 2024

Hey @ZeroOne-Aero, I think 0x150 would be better for ZeroOne, this way you can have the whole 0x150/0x159 range available for future releases.

@ZeroOne-Aero
Copy link
Contributor Author

Ok, I've changed it to 0x150.

@mrpollo mrpollo merged commit 12dcd15 into pixhawk:master Aug 14, 2024
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.

3 participants