Skip to content

Conversation

@cowsed
Copy link
Contributor

@cowsed cowsed commented Jan 8, 2026

Description

Use runtime SFDP to allow sensor mod to detect that its 512 Mb and enter 32 bit addressing mode

w/o this, the addresses wrap around at 0x1000000 limiting us to using 16 MB of the flash and confusing FS implementations like littlefs

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

This is how grim reefer did it. Will test when i get back to roc using flash shell and we probably should write a file for a while and see if it freaks out.

Example of failing test w/ flash shell
image

Test Configuration:

Checklist:

  • New functionality is documented in the necessary spots (i.e new functions documented in the header)
  • Unit tests cover any new functionality or edge cases that the PR was meant to resolve (if applicable)
  • The CI checks are passing
  • I reviewed my own code in the GitHub diff and am sure that each change is intentional
  • I feel comfortable about this code flying in a rocket

@cowsed cowsed changed the title Tiny fixes for sensor mod (Correct flash addressing + more gyro ) Correct Flash Addressing w/ sensor mod Jan 12, 2026
@cowsed
Copy link
Contributor Author

cowsed commented Jan 24, 2026

image

@cowsed cowsed marked this pull request as ready for review January 24, 2026 17:16
@cowsed cowsed requested a review from a team as a code owner January 24, 2026 17:16
Copy link
Member

@AarC10 AarC10 left a comment

Choose a reason for hiding this comment

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

missing a few thousand lines of changes and 50+ files

@cowsed
Copy link
Contributor Author

cowsed commented Jan 24, 2026

maybe ill commit a diff of a dump of the entire 512 mbits

@cowsed cowsed merged commit dc48c7b into main Jan 24, 2026
6 checks passed
@cowsed cowsed deleted the cowsed/sensor_mod_storage_fix branch January 24, 2026 18:20
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.

3 participants