Add support for STM32F42x/STM32F43x rev. 5/B #15407
Conversation
sfalexrog
force-pushed
the
stm32f427_rev5_support
branch
from
5ff2546
to
7f673c5
Compare
According to ST, both 5 and B share the same REV_ID. Signed-off-by: Alexey Rogachevskiy <sfalexrog@gmail.com>
sfalexrog
force-pushed
the
stm32f427_rev5_support
branch
from
7f673c5
to
2108f9d
Compare
| @@ -151,8 +153,10 @@ int board_mcu_version(char *rev, const char **revstr, const char **errata) | |||
| chip_errata = NULL; | |||
| break; | |||
|
|
|||
| case MCU_REV_STM32F7_REV_V: | |||
| *rev = 'V'; | |||
| case MCU_REV_STM32F4_REV_5: | |||
There was a problem hiding this comment.
Are these gaps we could be preemptively filling in before someone hits them?
There was a problem hiding this comment.
I'm sorry, I could not quite understand what you've meant there. Could you elaborate?
There was a problem hiding this comment.
@dagar I believe it is a sparse array.
This is the latese published in R18 of https://www.st.com/resource/en/reference_manual/dm00031020-stm32f405-415-stm32f407-417-stm32f427-437-and-stm32f429-439-advanced-arm-based-32-bit-mcus-stmicroelectronics.pdf
0x1000 = Revision A
0x1003 = Revision Y
0x1007 = Revision 1
0x2001= Revision 3
0x2003= Revision 5 and B
| case MCU_REV_STM32F4_REV_5: | ||
| // MCU_REV_STM32F4_REV_B shares the same REV_ID | ||
| // MCU_REV_STM32F7_REV_V shares the same REV_ID | ||
| *rev = chip_version == STM32H74xx_75xx ? 'V' : '5'; |
There was a problem hiding this comment.
The comment does not match the code
STM32H74xx_75xx
There was a problem hiding this comment.
Yes, sorry, I got confused by surrounding code.
Judging by the ST errata sheets, there's no V or X revisions for STM32F7 series, only STM32H7. I've changed the enum names to reflect that.
There was a problem hiding this comment.
comment says MCU_REV_STM32F7_REV_V shares the same REV_ID code has STM32H74xx_75xx
There was a problem hiding this comment.
Yes, I've changed the comment to say H7.
The comment was about the enum name of MCU_REV_STM32F7_REV_V, which is itself somewhat confusing, since no F7 to date seems to have a REV_ID of 0x2003.
|
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
|
Thanks! |
This P/R adds
REV_IDs for STM32F427 rev. 5 and B.Note that, accoding to https://www.st.com/resource/en/errata_sheet/dm00068628-stm32f427437-and-stm32f429439-line-limitations-stmicroelectronics.pdf, both 5 and B share the same
REV_ID. This patch reports both as rev. 5.Fixes #15406