Skip to content

stm32cube: stm32f4, stm32f7: Fix errata workaround#362

Open
icsys-omh wants to merge 1 commit into
zephyrproject-rtos:mainfrom
icsys-omh:fix-errata-workaround
Open

stm32cube: stm32f4, stm32f7: Fix errata workaround#362
icsys-omh wants to merge 1 commit into
zephyrproject-rtos:mainfrom
icsys-omh:fix-errata-workaround

Conversation

@icsys-omh
Copy link
Copy Markdown

The errata for STM32F4 says successive writes to MACCR may be ignored, and recommends reading back, waiting and rewriting as a workaround. This workaround has been done correctly for writes to TE, but has mistakenly been removed after enabling RE, which means that receiving sometimes ended up not working. The errata recommends doing multiple writes, then a final wait and rewrite at the end, which seems to work correctly.

Even though this is not mentioned in the F7 errata, the same fix is required there.

Fixes zephyrproject-rtos/zephyr#107024.

Copy link
Copy Markdown
Collaborator

@juickar juickar left a comment

Choose a reason for hiding this comment

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

Could you mention the changes in both
stm32cube/stm32f4xx/README
and
stm32cube/stm32f7xx/README

The errata for STM32F4 says successive writes to MACCR may be ignored,
and recommends reading back, waiting and rewriting as a workaround. This
workaround has been done correctly for writes to TE, but has mistakenly
been removed after enabling RE, which means that receiving sometimes
ended up not working. The errata recommends doing multiple writes, then
a final wait and rewrite at the end, which seems to work correctly.

Even though this is not mentioned in the F7 errata, the same fix is
required there.

Fixes zephyrproject-rtos/zephyr#107024.

Signed-off-by: Ole Morten Haaland <omh@ixys.no>
@icsys-omh icsys-omh force-pushed the fix-errata-workaround branch from d636b32 to 8cb1f3d Compare April 21, 2026 20:06
@juickar juickar requested review from erwango and etienne-lms April 22, 2026 07:19
@erwango erwango added the DNM label Apr 23, 2026
@erwango
Copy link
Copy Markdown
Member

erwango commented Apr 23, 2026

@juickar Do we know if this will be fixed in "upstream" Cube packages ?

@juickar
Copy link
Copy Markdown
Collaborator

juickar commented Apr 23, 2026

@juickar Do we know if this will be fixed in "upstream" Cube packages ?

I opened a ticket but the assignee is OoO until Monday

@erwango
Copy link
Copy Markdown
Member

erwango commented Apr 24, 2026

I opened a ticket but the assignee is OoO until Monday

Ok. Let's wait for an initial feedback before merging.

@erwango
Copy link
Copy Markdown
Member

erwango commented May 4, 2026

For tracking purpose: HAL1-27528. Still no update

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ethernet: stm32: Data reception not working until register rewrite

4 participants