STM32CubeF7 icon indicating copy to clipboard operation
STM32CubeF7 copied to clipboard

Basically all of the stm32f7xx_hal_eth.h macros are just copy-pasta'd from H7 and using H7 registers

Open HTRamsey opened this issue 2 years ago • 4 comments

F7 registers != H7 registers

HTRamsey avatar Aug 28 '23 15:08 HTRamsey

Hello @holden-zenith,

Thank you for this report. We will get back to you as soon as we analyze it further. This may take some time. Thank you for your comprehension.

With Regards,

TOUNSTM avatar Aug 29 '23 13:08 TOUNSTM

Hello @holden-zenith,

I've tried to do an analysis on my own and I can't find the problem you mentioned. In order to enable a better analysis of this problem, could you please give us more details.

Thank you again for your contribution.

With regards,

TOUNSTM avatar Nov 28 '23 09:11 TOUNSTM

for example #define ETH_DMA_NORMAL_IT ETH_DMACIER_NIE There is no ETH_DMACIER_NIE in stm32f767xx.h, it is ETH_DMAIER_NISE.

HTRamsey avatar Nov 29 '23 14:11 HTRamsey

ST Internal Reference: 169492

TOUNSTM avatar Dec 22 '23 09:12 TOUNSTM

Can someone clarify which macros are affected by this?

mek363 avatar Apr 26 '24 20:04 mek363

I think this fix was merged

HTRamsey avatar Apr 26 '24 20:04 HTRamsey

@HTRamsey thanks, was it this commit? https://github.com/STMicroelectronics/stm32f7xx_hal_driver/commit/5af4f7192a9331d27e5e90feed5e7cebc3f1356c

Is there any explanation of what incorrect behavior we might see without this fix? Or would we see compile-time errors because the referenced defines are not found?

mek363 avatar Apr 26 '24 23:04 mek363

Should just be compile-time errors because they were all referencing H7 registers.

HTRamsey avatar Apr 27 '24 01:04 HTRamsey

Fixed in commit 5ff42d9979458a6426d67783c8ea2b2c7baee8d6

ALABSTM avatar May 27 '24 15:05 ALABSTM