Skip to content

Fix WDT hanging too long on setup#299

Merged
jgfoster merged 4 commits intoOpen-Acidification:mainfrom
pianistrevor:Bug294
Mar 11, 2022
Merged

Fix WDT hanging too long on setup#299
jgfoster merged 4 commits intoOpen-Acidification:mainfrom
pianistrevor:Bug294

Conversation

@pianistrevor
Copy link
Copy Markdown
Collaborator

Starts the WDT after printing serial messages in case additional messages that take longer than the WDT failsafe are inserted later.
Removes the printout of the root directory of the SD card.

Copy link
Copy Markdown
Member

@jgfoster jgfoster left a comment

Choose a reason for hiding this comment

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

Well, that wasn't too complicated!
@prestoncarman, do you have comments?

@jgfoster jgfoster requested a review from prestoncarman March 10, 2022 05:36
@pianistrevor
Copy link
Copy Markdown
Collaborator Author

So somehow removing a function call and reordering it adds 2 bytes of static variables? Odd

Copy link
Copy Markdown
Member

@prestoncarman prestoncarman left a comment

Choose a reason for hiding this comment

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

LGTM, Probably should have been this way to start with.

@jgfoster jgfoster merged commit 8edf48c into Open-Acidification:main Mar 11, 2022
@pianistrevor pianistrevor deleted the Bug294 branch March 11, 2022 03:35
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