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

introduce zephyr into firmware monorepo #4449

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

TychoVrahe
Copy link
Contributor

@TychoVrahe TychoVrahe commented Dec 16, 2024

This PR introduces zephyr, or rather NCS, to firmware monorepo. Zephyr is used for implementing a BLE gateway.

There is still work to be done, but i would like to get this into main soon, as it will allow more incremental work and ease review of things that affect both core and BLE, like communication and so.

But this should be good enough to:

  • run on both T3W1 boards, the devkit1 and revA
  • should be clear from readme itself how to build and flash this.

Reviewers:
@matejcik please check the structure of the added stuff
@cepetr general architectural comments, no need to deep dive into the code, unless you want to.
@hiviah please look at the BLE specific code in /nordic/trezor/trezor-ble

fixes #4301

@TychoVrahe TychoVrahe self-assigned this Dec 16, 2024
Copy link

github-actions bot commented Dec 16, 2024

core UI changes device test click test persistence test
T2T1 Model T test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
T3B1 Safe 3 test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
T3T1 Safe 5 test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
All main(screens)

@TychoVrahe TychoVrahe force-pushed the tychovrahe/zephyr/introduction branch from bc3bbe8 to 09a894e Compare December 16, 2024 10:22
@TychoVrahe TychoVrahe force-pushed the tychovrahe/zephyr/introduction branch 4 times, most recently from 10ed2af to a260f74 Compare January 20, 2025 16:03
@TychoVrahe TychoVrahe force-pushed the tychovrahe/zephyr/introduction branch 2 times, most recently from cae09d4 to cc950f7 Compare January 23, 2025 09:19
@TychoVrahe TychoVrahe marked this pull request as ready for review January 23, 2025 09:41
@TychoVrahe TychoVrahe force-pushed the tychovrahe/zephyr/introduction branch from cc950f7 to 58b321a Compare January 24, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🔎 Needs review
Development

Successfully merging this pull request may close these issues.

Advertise colour together with name of the device over BT
3 participants