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.

Sign up for GitHub

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

Jump to bottom

Crc checksum #284

Draft
wants to merge 29 commits into
base: main
Choose a base branch
from
Draft

Crc checksum #284

wants to merge 29 commits into from

Conversation

PratyushMakkar
Copy link
Contributor

Purpose

Explain the purpose of the PR here, including references to any existing Notion tasks.

New Changes

  • Explain new changes

Testing

  • Explain tests that you ran to verify code functionality.
  • Any functions that can be unit-tested should include a unit test in the PR. Otherwise, explain why it cannot be unit-tested.

Outstanding Changes

  • If there are non-critical changes (i.e. additional features) that can be made to this feature in the future, indicate them here.

@github-actions GitHub Actions
Copy link

Pull reviewers stats

Stats of the last 120 days for UWOrbital:

User Total reviews Time to review Total comments
dgobalak 28 2d 6h 21m 144
Navtajh04 20 4d 7h 34m 117
Yarik-Popov 12 4d 17h 46m 85
PratyushMakkar 5 1d 6h 8m 21
Gjjjiang 4 4d 15h 52m 14
manumanuk 2 35d 14h 41m 5
twilkhoo 1 4m 0
JoelManYunLee 1 17h 57m 4
kepler452b123 1 7d 4h 25m 11
Krisphy 1 111d 20h 21m 1
⚡️ Pull request stats

PratyushMakkar and others added 23 commits March 28, 2024 01:19
@PratyushMakkar
Added tests
e13429f
@PratyushMakkar
bl commit
268df5a
@kepler452b123
Generated HAL code in hal/source directories
97bced3 
The new HAL files in the source directories are
the same as the ones in the shared directories but
it has new CRC changes.
@kepler452b123
Typo in launchpad HAL
c443b98
@PratyushMakkar
Made changes to bootloader CRC module
6fca382
@PratyushMakkar
Merge branch 'main' into crc_checksum
40faa9e 
Merging main.
@PratyushMakkar
Revert "Typo in launchpad HAL"
523be7c 
This reverts commit c443b98.
@PratyushMakkar
Revert "Generated HAL code in hal/source directories"
eb5f0d1 
This reverts commit 97bced3.
@PratyushMakkar
Placeholder for tests
6b11dd3
@PratyushMakkar
Finished bl_common
81ea36c
@PratyushMakkar
Removed cmake bl_utils
2fb596d
@PratyushMakkar
Modfied bl_crc
d69ea40
@PratyushMakkar
Modified bl_common
479fa2b
@PratyushMakkar
Mistaken const
cc33388
@PratyushMakkar
Cast to uin32_t
45c9408
@PratyushMakkar
error code
c51f42c
@PratyushMakkar
crc -> blCrc
c0a767c
@PratyushMakkar
CommitMesg
af3c62d
@PratyushMakkar
yeat
b2c6a6e
@PratyushMakkar
yeat
6668e6e
@PratyushMakkar
yeat
ff333af
@PratyushMakkar
initializing static variable
41191f4
@PratyushMakkar
flags
2e65ea7
@PratyushMakkar PratyushMakkar requested a review from Gjjjiang April 13, 2024 23:23
PratyushMakkar and others added 3 commits April 14, 2024 17:46
@PratyushMakkar
Changes to crc
ba4d378
@PratyushMakkar
Null check
76839e2
@kepler452b123
Modified shared hcg files to have CRC settings
9d23520 
Found error in SPI1 for all hcg files (baudrate of 286 might be too low for data format 0)
@Navtajh04
Copy link
Contributor

whats the status on this @PratyushMakkar ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Navtajh04 Navtajh04 Awaiting requested review from Navtajh04

@Yarik-Popov Yarik-Popov Awaiting requested review from Yarik-Popov

@JoelManYunLee JoelManYunLee Awaiting requested review from JoelManYunLee

@Gjjjiang Gjjjiang Awaiting requested review from Gjjjiang

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@PratyushMakkar @Navtajh04 @kepler452b123