CMUAbstract / tab

TAB reference implementations, examples, and documentation
Other
0 stars 0 forks source link

Add support for Bootloader Write Addr32 #22

Open mablem8 opened 1 year ago

mablem8 commented 1 year ago
ChadTaylor17 commented 1 year ago

Support tests results:

In app mode: txcmd: bootloader_write_page_addr32 hw_id:0x1234 msg_id:0x000a src:gnd dst:cdh Address: 0x08008000 hex_data:0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000 reply: common_nack hw_id:0x1234 msg_id:0x000a src:cdh dst:gnd

In bootloader mode: txcmd: bootloader_write_page_addr32 hw_id:0x1234 msg_id:0x000a src:gnd dst:cdh Address: 0x08008000 hex_data:0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000 reply: bootloader_ack hw_id:0x1234 msg_id:0x000a src:cdh dst:gnd reason:0x08008000(addr)

Changes can be seen on this branch

mablem8 commented 1 year ago

Link to the commit that addresses the issue instead of the branch (since the branch will eventually be pruned after merging). Commit 852a7284639f6c0005d5be6abd9b0bf5a461aea6 fails when testing with tx_example.py. Commit 71684fc2a2523c217c736946740f6932f2cd6cf4 rectifies the problem. Changes look good but the issue remains open while waiting on COM support.