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.

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

drivers: ethernet: eth_stm32_hal allow proper priority assignment to rx_thread #87094

Open
clamattia opened this issue Mar 14, 2025 · 3 comments · May be fixed by #87116
Open

drivers: ethernet: eth_stm32_hal allow proper priority assignment to rx_thread #87094

clamattia opened this issue Mar 14, 2025 · 3 comments · May be fixed by #87116
Assignees
Labels
area: Ethernet bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32

Comments

@clamattia
Copy link
Contributor

clamattia commented Mar 14, 2025

Describe the bug
#77904 introduced the defect of coupling the cooperativeness of the driver-thread to the cooperativeness of the traffic-class queue.

It makes sense for an application to assign different cooperativenesses to these.

@descsny has already suggested in above PR to not couples those two priorities. Coupling the two priority cooperativeness levels is too restrictive.

To Reproduce
NA

Expected behavior
Please do as suggested by #77904 (comment)

Impact
Proper priorities can no longer be assigned.

Logs and console output
NA

Environment (please complete the following information):
NA

Additional context
NA

@clamattia clamattia added the bug The issue is a bug, or the PR is fixing a bug label Mar 14, 2025
@henrikbrixandersen henrikbrixandersen added the platform: STM32 ST Micro STM32 label Mar 14, 2025
@henrikbrixandersen
Copy link
Member

Please use our bug template when reporting bugs. You need to edit this issue to include the information requested in https://github.com/zephyrproject-rtos/zephyr/blob/main/.github/ISSUE_TEMPLATE/001_bug_report.md

@clamattia
Copy link
Contributor Author

Please use our bug template when reporting bugs. You need to edit this issue to include the information requested in https://github.com/zephyrproject-rtos/zephyr/blob/main/.github/ISSUE_TEMPLATE/001_bug_report.md

@henrikbrixandersen Thank you. I hope it fits better now.

@clamattia
Copy link
Contributor Author

Le me know, if this is considered an enhancement, and I am happy to adjust the wording.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Ethernet bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants