Skip to content

boards: st: stm32h745i_disco: m7: Fix sdram available size #99714

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

boards: st: stm32h745i_disco: m7: Fix sdram available size #99714

wants to merge 3 commits into from
+4 −2

Conversation

@rvosk
Copy link

@rvosk rvosk commented Nov 20, 2025

According to UM2488 section 6.12[1], the available SDRAM size is only 8MB by hardware design, though 128Bit SDRAM is connected.

Link: https://www.st.com/resource/en/user_manual/um2488-discovery-kits-with-stm32h745xi-and-stm32h750xb-mcus-stmicroelectronics.pdf [1]

@xhackerustc
boards: st: stm32h745i_disco: m7: Fix sdram available size
1a8bd48 
According to UM2488 section 6.12[1], the available SDRAM size is only 8MB
by hardware design, though 128Bit SDRAM is connected.

Link: https://www.st.com/resource/en/user_manual/um2488-discovery-kits-with-stm32h745xi-and-stm32h750xb-mcus-stmicroelectronics.pdf [1]
Signed-off-by: Jisheng Zhang <[email protected]>
@xhackerustc
boards: st: stm32h750b-dk: Fix sdram available size
6c6739f 
According to UM2488 section 6.12[1], the available SDRAM size is only 8MB
by hardware design, though 128Bit SDRAM is connected.

Link: https://www.st.com/resource/en/user_manual/um2488-discovery-kits-with-stm32h745xi-and-stm32h750xb-mcus-stmicroelectronics.pdf [1]
Signed-off-by: Jisheng Zhang <[email protected]>
@zephyrbot zephyrbot added area: Boards/SoCs size: XS A PR changing only a single line of code platform: STM32 ST Micro STM32 labels Nov 20, 2025
erwango
erwango previously approved these changes Nov 20, 2025
View reviewed changes
Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

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

Good catch

etienne-lms
etienne-lms reviewed Nov 20, 2025
View reviewed changes
Copy link
Contributor

@etienne-lms etienne-lms 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 also fix stm32h750b_dk board (stm32h750b_dk-common.dtsi) that has the same constraint (mentioned in referred UM2488)?

@rvosk
Copy link
Author

rvosk commented Nov 21, 2025

Could you also fix stm32h750b_dk board (stm32h750b_dk-common.dtsi) that has the same constraint (mentioned in referred UM2488)?

Will do. But I don't have stm32h750b disc board, so will be compiled test only.

@rvosk
Copy link
Author

rvosk commented Nov 21, 2025 •
edited
Loading

hmm, I dunno what's wrong, any hint is appreciated.

INFO: Potential tag based filters: {'cmsis_dsp', 'test_framework', 'wifi', 'mcumgr', 'net', 'bluetooth', 'kernel', 'posix'}
INFO: found boards: stm32h750b_dk,stm32h750b_dk/stm32h750xx/ext_flash_app,stm32h745i_disco/stm32h745xx/m7,stm32h745i_disco/stm32h745xx/m4
Traceback (most recent call last):
  File "/home/runner/work/zephyr/zephyr/zephyr/./scripts/ci/test_plan.py", line 549, in <module>
Changed files:
=========
boards/st/stm32h745i_disco/stm32h745i_disco_stm32h745xx_m7.dts
boards/st/stm32h750b_dk/stm32h750b_dk-common.dtsi
=========
    f.process()
  File "/home/runner/work/zephyr/zephyr/zephyr/./scripts/ci/test_plan.py", line 1[45](https://github.com/zephyrproject-rtos/zephyr/actions/runs/19566115838/job/56028408833?pr=99714#step:7:46), in process
    self.find_boards()
  File "/home/runner/work/zephyr/zephyr/zephyr/./scripts/ci/test_plan.py", line 292, in find_boards
    self.resolved_files.extend(list(filter(resolved_filter)))
                                    ^^^^^^^^^^^^^^^^^^^^^^^
TypeError: filter expected 2 arguments, got 1
Error: Process completed with exit code 1.

@etienne-lms
Copy link
Contributor

etienne-lms commented Nov 21, 2025

Could you also fix stm32h750b_dk board (stm32h750b_dk-common.dtsi) that has the same constraint (mentioned in referred UM2488)?

Will do. But I don't have stm32h750b disc board, so will be compiled test only.

Still a valuable change IMHO.
I'll test locally...

As for the CI failure, I can't tell. It looks unrelated. Could you rebase your series?

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 21, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@rvosk rvosk closed this Nov 21, 2025
@rvosk rvosk deleted the stm32h745idk branch November 21, 2025 10:19
@rvosk
Copy link
Author

rvosk commented Nov 21, 2025

there's network issue here, I mis-operated the rebase, so I create a new PR here:

#99799

sorry for inconvenience

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

Reviewers

@etienne-lms etienne-lms etienne-lms left review comments

@erwango erwango erwango left review comments

@djiatsaf-st djiatsaf-st Awaiting requested review from djiatsaf-st

@FRASTM FRASTM Awaiting requested review from FRASTM

@gautierg-st gautierg-st Awaiting requested review from gautierg-st

@GeorgeCGV GeorgeCGV Awaiting requested review from GeorgeCGV

@juickar juickar Awaiting requested review from juickar

@mathieuchopstm mathieuchopstm Awaiting requested review from mathieuchopstm

Assignees

@erwango erwango

Labels

area: Boards/SoCs platform: STM32 ST Micro STM32 size: XS A PR changing only a single line of code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@rvosk @etienne-lms @erwango @zephyrbot @xhackerustc