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

Add support for Bootloader Write Addr32 #22

Open
mablem8 opened this issue Feb 20, 2023 · 2 comments
Open

Add support for Bootloader Write Addr32 #22

mablem8 opened this issue Feb 20, 2023 · 2 comments
Assignees
Milestone

Comments

@mablem8
Copy link
Collaborator

mablem8 commented Feb 20, 2023

  • Add a description of Bootloader Write Addr32 to DOCUMENTATION.md
  • Add Bootloader Write Addr32 to c-implementation/tab.c with an external handler
  • Add a handler implementation of Bootloader Write Addr32 to cdh-monolithic.c
  • Add a handler implementation of Bootloader Write Addr32 to com-monolithic.c
  • Add an implementation of Bootloader Write Addr32 to python-implementation/tab.py
  • Add testing of Bootloader Write Addr32 in python-examples/tx/tx_example.py
  • Test support for Bootloader Write Addr32 with c-examples/cdh-monolithic and python-examples/tx
@mablem8 mablem8 added this to the Sprint 3 milestone Feb 20, 2023
@ChadTaylor17
Copy link

  • Add a description of Bootloader Write Addr32 to DOCUMENTATION.md
  • Add Bootloader Write Addr32 to c-implementation/tab.c with an external handler
  • Add a handler implementation of Bootloader Write Addr32 to cdh-monolithic.c
  • Add a handler implementation of Bootloader Write Addr32 to com-monolithic.c
  • Add an implementation of Bootloader Write Addr32 to python-implementation/tab.py
  • Add testing of Bootloader Write Addr32 in python-examples/tx/tx_example.py
  • Test support for Bootloader Write Addr32 with c-examples/cdh-monolithic and python-examples/tx

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

mablem8 commented Feb 24, 2023

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

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

No branches or pull requests

2 participants