Skip to content

Conversation

is-qian
Copy link
Contributor

@is-qian is-qian commented Mar 30, 2025

No description provided.

@beastoin
Copy link
Collaborator

beastoin commented Mar 31, 2025

wow, welcome @is-qian , does this PR ready to merge or just your draft ?

@is-qian
Copy link
Contributor Author

is-qian commented Mar 31, 2025

The new board still has many issues with the old code, and I think it will require modifying the src part of the code because the new hardware uses the nrf5340, which differs in many places from the src in the old code, so it currently won’t compile. Therefore, I think this is still just a draft. However, I can upload our test code here later to make it more convenient for subsequent testing and modifying the src code, so that the src code can be made compatible with the new hardware.

@beastoin
Copy link
Collaborator

i see, so just create a new folder in the firmware, e.g. omi > firmware > omi2. what do you think ? if this makes sense to you, go ahead and do it.

anw, could you build the firmware on the devkit2 and give it a try with our omi ai app ? feel free to ask if you need my helps.

@is-qian

@is-qian
Copy link
Contributor Author

is-qian commented Apr 2, 2025

I have added test code and optimized the directory, hoping it will be helpful for your subsequent development.

@beastoin
Copy link
Collaborator

beastoin commented Apr 2, 2025

great @is-qian !

i understood that all tests are done at the component level, right?

could you help with some basic tests functions related to the BLE/WIFI communication, for example:

  • read/write the button's state via BLE.
  • read the sd-card info via WIFI

@is-qian
Copy link
Contributor Author

is-qian commented Apr 2, 2025

In the test project, for the convenience of the whole machine testing, BLE is used as a shell terminal, and you can use it just like a serial port. Of course, you can use it to read the button status. However, if you only want to broadcast the button status, you may need to adjust your code accordingly. For WiFi, I think you can refer to this demo.

@beastoin
Copy link
Collaborator

beastoin commented Apr 3, 2025

yes, thank man, and currently, we use https://github.com/adafruit/Adafruit_nRF52_Bootloader as the boot loader:
1/ could we use the adafruit's boot loader for the new device ?
2/ if no, what would you suggest ?

@is-qian

@is-qian
Copy link
Contributor Author

is-qian commented Apr 7, 2025

This repository does not support nrf5340 yet, you can refer to this. You can also use mucboot.

@beastoin
Copy link
Collaborator

beastoin commented Apr 7, 2025

thank @is-qian , lgtm & congratulation 🚀

@beastoin beastoin merged commit b61f26f into BasedHardware:main Apr 7, 2025
skywinder added a commit that referenced this pull request Apr 7, 2025
Rutik7066 pushed a commit to Rutik7066/omi that referenced this pull request Apr 10, 2025
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.

2 participants