Update board_types.txt #236
Conversation
Add: TARGET_HW_ZEROONE_X6 101 Reserved “ZEROONE_X6" 101 #IDs 5600-5699 reserved for ZeroOne AP_HW_ZeroOne_X6 5600 AP_HW_ZeroOne_PMU 5601 AP_HW_ZeroOne_GNSS 5602
board_types.txt
Outdated
| @@ -49,6 +49,7 @@ TARGET_HW_PDW_MAS_MAIN-V1 86 | |||
| TARGET_HW_ATL_MANTIS_EDU 97 | |||
| TARGET_HW_THE_PEACH_K1 212 | |||
| TARGET_HW_THE_PEACH_R1 213 | |||
| TARGET_HW_ZEROONE_X6 101 | |||
There was a problem hiding this comment.
remove this - these are for F4-F7 bootloaders H7 are built in tree
There was a problem hiding this comment.
Ok,I have removed this.
| @@ -206,6 +209,11 @@ AP_HW_CUBEBLACK_PERIPH 1401 | |||
| AP_HW_PIXRACER_PERIPH 1402 | |||
| AP_HW_SWBOOMBOARD_PERIPH 1403 | |||
|
|
|||
| #IDs 5600-5699 reserved for ZeroOne | |||
| AP_HW_ZeroOne_X6 5600 | |||
There was a problem hiding this comment.
is AP_HW_ZeroOne_X6 and ZEROONE_X6 the same board? If so since you already have the AP id in Ardupilot, just add it here. Do not "Create" Ardupilot ID here that do not exist in Ardupilot yet.
There was a problem hiding this comment.
Yes, this line already exists in the Ardupilot.
| AP_HW_ZeroOne_PMU 5601 | ||
| AP_HW_ZeroOne_GNSS 5602 |
There was a problem hiding this comment.
| AP_HW_ZeroOne_PMU 5601 | |
| AP_HW_ZeroOne_GNSS 5602 | |
| AP_HW_ZeroOne_PMU 5601 | |
| AP_HW_ZeroOne_GNSS 5602 |
Add to AP first then document here.
There was a problem hiding this comment.
Yes, these two lines have been added to the Ardupilot.
remove:TARGET_HW_ZEROONE_X6 101
board_types.txt
Outdated
| @@ -64,6 +64,8 @@ Reserved "NXP ucans32k146" 34 | |||
| Reserved "NXP FMURT1170-V1" 35 # AKA PX4 FMU V6XRT | |||
| Reserved "NXP Tropic" 36 | |||
|
|
|||
| Reserved “ZEROONE_X6" 101 | |||
There was a problem hiding this comment.
We only want one entry - so delete this
There was a problem hiding this comment.
Do you mean we only need to add the following sentences?
#IDs 5600-5699 reserved for ZeroOne
AP_HW_ZeroOne_X6 5600
AP_HW_ZeroOne_PMU 5601
AP_HW_ZeroOne_GNSS 5602
There was a problem hiding this comment.
Ok,I have removed this line:
Reserved “ZEROONE_X6" 101
There was a problem hiding this comment.
@davids5 Do you think the modified version is OK?
Removed this line: Reserved “ZEROONE_X6" 101
Add:
TARGET_HW_ZEROONE_X6 101
Reserved “ZEROONE_X6" 101
#IDs 5600-5699 reserved for ZeroOne
AP_HW_ZeroOne_X6 5600
AP_HW_ZeroOne_PMU 5601
AP_HW_ZeroOne_GNSS 5602