Skip to content

Conversation

@hasslesstech
Copy link
Contributor

No description provided.

@hasslesstech hasslesstech linked an issue Aug 12, 2025 that may be closed by this pull request
6 tasks
Copy link
Member

@AksonovSergei AksonovSergei left a comment

Choose a reason for hiding this comment

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

Please fix my comments before the next meeting. If you can't, I will help you.

return errors;
}

unsigned int Buzzer_Pulse(const unsigned int on_time_ms, const unsigned int period_time_ms, const unsigned int total_active_time_ms)
Copy link
Member

Choose a reason for hiding this comment

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

This is a blocking function. If you call it with total_active_time_ms = 5000, it will block the thread that calls it for 5 seconds.

Copy link
Member

Choose a reason for hiding this comment

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

One more test case from Gemini:
Test Case: Disabling an Active Buzzer
Objective
Verify that the buzzer stops immediately when Buzzer_Disable() is called, even if the Buzzer_Pulse() function is still active.

Steps
Call the Buzzer_Pulse() function with a long duration, for example, total_active_time_ms = 5000.

Wait a short time (e.g., 500 milliseconds) to confirm the buzzer is on.

Call the Buzzer_Disable() function.

Expected Result
The buzzer should turn off right after Buzzer_Disable() is called. The Buzzer_Pulse() function's 5-second timer should be ignored. The device should be silent.

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.

[ECU STM32] Implement Buzzer Driver

3 participants