Skip to content

Conversation

ragurram26
Copy link
Contributor

si91x bus events.

Modified the osEventFlagsWait call to include the osFlagsNoClear option, which ensures that the events are not cleared automatically. After the events are received, manually clear them.

@jhedberg
Copy link
Member

jhedberg commented Oct 1, 2025

Seems like this should have an Upstream-status trailer. I assume the corresponding fix has been submitted to wiseconnect upstream?

@ragurram26 ragurram26 force-pushed the event_flag_issue branch 2 times, most recently from d8234fb to 8215c8a Compare October 2, 2025 07:57
Copy link
Collaborator

@jerome-pouiller jerome-pouiller left a comment

Choose a reason for hiding this comment

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

The commit log explain what you did. This is not super helpful.

I would rather like to see why this change fix the bug (which is not obvious).

…s events.

It was observed that when one event is set and another event is subsequently
set, the first event gets cleared prematurely before it can be processed. To
address this, the osEventFlagsWait call was updated to use the osFlagsNoClear
option, which prevents automatic clearing of events upon retrieval. Events
are now manually cleared after they are read, ensuring that all set events
are reliably processed and none are missed due to premature clearing caused
by subsequent event updates.

Signed-off-by: Rahul Gurram <[email protected]>
@ragurram26
Copy link
Contributor Author

ragurram26 commented Oct 6, 2025

The commit log explain what you did. This is not super helpful.

I would rather like to see why this change fix the bug (which is not obvious).

It was observed that when one event is set and another event is subsequently set, the first event gets cleared prematurely before it can be processed. To address this, the osEventFlagsWait call was updated to use the osFlagsNoClear option, which prevents automatic clearing of events upon retrieval. Events are now manually cleared after they are read, ensuring that all set events are reliably processed and none are missed due to premature clearing caused by subsequent event updates.

Same as been updated in the commit log

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