8000 doc: Castor release notes by mchack-work · Pull Request #348 · tillitis/tillitis-key1 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

doc: Castor release notes #348

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

Merged
merged 2 commits into from
May 23, 2025
Merged

doc: Castor release notes #348

merged 2 commits into from
May 23, 2025

Conversation

mchack-work
Copy link
Member
@mchack-work mchack-work commented May 16, 2025

Description

Tracking the release notes for Castor.

Fixes #316

Type of change

  • Documentation (a change to documentation)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • I have squashed my work to relevant commits and rebased on main for linear history
  • I have added a "Co-authored-by: x" if several people contributed, either pair programming or by squashing commits from different authors.
  • I have updated the documentation where relevant (readme, dev.tillitis.se etc.)
  • QEMU is updated to reflect changes

@mchack-work mchack-work changed the title doc: Update release notes doc: Castor release notes May 16, 2025
@mchack-work
Copy link
Member Author
mchack-work commented May 21, 2025

Please don't just review the diff. Review the entire release notes since the last tag.

Copy link
Member
@SallSim SallSim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The whole document reviewed.
Two proposals, but ok if you don't want to change.

of a second app.

- `defaultapp`: An app that immediately resets the TKey to load an app
from the client, just like earlier releases.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
from the client, just like earlier releases.
from the client, mimicking Bellatrix behaviour.

different USB endpoints in the USB controller.

- Support a filesystem on flash: There's space for two pre-loaded
apps and four storage areas for device apps.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I realize that I get a bit confused by "two pre-loaded apps". Is the loader app one of these?
Maybe be more explicit? Proposal:

  • Support a filesystem on flash with space for: one area for 1st boot stage (verified boot loader), two areas (A and B for upgrade purposes) for pre-loaded app and four storage areas for use by device apps.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are two app slots: 0 and 1. Slot 0 will be used for the loader app. Slot 1 will have the tkey-fido2 app. There are no A/B areas for upgrade purposes.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added some more explanation of the typical use we imagine and the fact that the app in slot 0 is explicitly trusted by the firmware.

@mchack-work mchack-work marked this pull request as ready for review May 23, 2025 11:50
mchack-work and others added 2 commits May 23, 2025 14:05
Use current information in Developer Handbook,
https://dev.tillitis.se/ instead of relying on this old description.
Co-authored-by: Mikael Ågren <mikael@tillitis.se>
@mchack-work mchack-work merged commit 916c37e into main May 23, 2025
6 checks passed
@mchack-work mchack-work deleted the release_notes branch May 23, 2025 12:11
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.

Broken links in documentation
2 participants
0