Skip to content

introduce the nucleo_h7s3l78 nucleo144 board from STMicroelectronics #81651

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

Merged
merged 3 commits into from
Jan 6, 2025

Conversation

FRASTM
Copy link
Collaborator

@FRASTM FRASTM commented Nov 20, 2024

Add the nucleo_h7s3l8 board from STMicroelectronics
This Nucleo 144 board has stm32h7S3L8 mcu.

Refer to the https://www.st.com/en/evaluation-tools/nucleo-h7s3l8.html

$ west build -p auto -b nucleo_h7s3l8 samples/hello_world/

*** Booting Zephyr OS build v4.0.0-rc3-49-g7500cdb33f48 ***                     
Hello World! nucleo_h7s3l8/stm32h7s3xx         

Copy link
Collaborator

Choose a reason for hiding this comment

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

trim space around image, (optionally) make background transparent, save as webp, put webp through https://tinypng.com/

Copy link
Collaborator

Choose a reason for hiding this comment

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

Seems unresolved?

Copy link
Collaborator

Choose a reason for hiding this comment

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

3x seems unresolved, to put this into perspective, I save this as webp in gimp, it drops to 45KiB, I then put it through tinypng, it drops to 36KiB, under half the jpg size

@FRASTM
Copy link
Collaborator Author

FRASTM commented Nov 20, 2024

add comments from @nordicjm
Fix CI error on wdg_api and adc_api

@FRASTM FRASTM force-pushed the nucleo_h7s3l8 branch 2 times, most recently from 3f21277 to 16b6b3d Compare November 21, 2024 08:07
@FRASTM FRASTM marked this pull request as ready for review November 21, 2024 09:50
@zephyrbot zephyrbot added platform: STM32 ST Micro STM32 area: Watchdog Watchdog area: ADC Analog-to-Digital Converter (ADC) labels Nov 21, 2024
@FRASTM FRASTM requested a review from nordicjm November 21, 2024 15:29
Copy link
Collaborator

Choose a reason for hiding this comment

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

3x seems unresolved, to put this into perspective, I save this as webp in gimp, it drops to 45KiB, I then put it through tinypng, it drops to 36KiB, under half the jpg size

@FRASTM
Copy link
Collaborator Author

FRASTM commented Dec 11, 2024

reducing the image size with nucleo_h7s3l8.webp file

@FRASTM FRASTM requested a review from nordicjm December 11, 2024 13:11
nordicjm
nordicjm previously approved these changes Dec 11, 2024
FRASTM added 3 commits January 2, 2025 12:47
Add the nucleo_h7s3l8 board from STMicroelectronics
This Nucleo 144 board has stm32h7s3L8 mcu.

Signed-off-by: Francois Ramu <[email protected]>
Add the overlay file to run the tests/drivers/adc/adc_api
on the nucleo_h7s3l8

Signed-off-by: Francois Ramu <[email protected]>
Add the overlay file to run tests/drivers/watchdog/wdt_basic_api
on the nucleo_h7s3l8 and stm32h7s78_dk target boards

Signed-off-by: Francois Ramu <[email protected]>
@@ -45,6 +45,8 @@ tests:
filter: dt_compat_enabled("st,stm32-window-watchdog") or dt_compat_enabled("st,stm32-watchdog")
extra_args: DTC_OVERLAY_FILE="boards/stm32_wwdg_h7.overlay"
platform_allow:
- nucleo_h7s3l8
- stm32h7s78_dk
Copy link
Collaborator

Choose a reason for hiding this comment

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

Adding stm32h7s78_dk to this test suite is not necessarily part of introducing the nucleo_h7s3l78 but it is very minor so I think its fine :)

@kartben kartben merged commit 0488044 into zephyrproject-rtos:main Jan 6, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC) area: Watchdog Watchdog platform: STM32 ST Micro STM32
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants