Skip to content
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

Stm32h503 does not enable the driver if instance is not present #181

Merged
merged 4 commits into from
Nov 17, 2023

Conversation

FRASTM
Copy link
Collaborator

@FRASTM FRASTM commented Nov 16, 2023

The stm32h503 soc does not embed delayblock (DLYB) or SAI block or DCACHE instances.
Consequently the corresponding drivers should be included.

FRASTM added a commit to FRASTM/zephyr that referenced this pull request Nov 16, 2023
Update the stm32h5xx/drivers/include/
to enable the SAI, DelayBlock and DCACHE only if the instance
exists in the soc. Which is not the case for stm32h503x device
zephyrproject-rtos/hal_stm32#181

Signed-off-by: F. Ramu <[email protected]>
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.

Please provide ST internal reference and document fix in README file

Include the stm32H5 HAL SAI driver only if instance
SAI_block A or B  exist in the stm32H5 soc (not on stm32h50x)

Signed-off-by: F. Ramu <[email protected]>
Include the stm32H5 HAL DCACHE driver only if instance
DCACHE1 exists in the stm32H5 soc (not on stm32h50x)

Signed-off-by: F. Ramu <[email protected]>
Include the stm32H5 HAL DLYB driver only if instance
SDMMC or OCTOSPI exist in the stm32H5 soc (not on stm32h50x)

Signed-off-by: F. Ramu <[email protected]>
List the changes to the stm32CubeH5 driver for this PR

Signed-off-by: F. Ramu <[email protected]>
@FRASTM
Copy link
Collaborator Author

FRASTM commented Nov 17, 2023

Updating the stm32cube/stm32h5xx/README

@erwango erwango merged commit af2d314 into zephyrproject-rtos:main Nov 17, 2023
4 checks passed
FRASTM added a commit to FRASTM/zephyr that referenced this pull request Nov 20, 2023
Update the stm32h5xx/drivers/include/
to enable the SAI, DelayBlock and DCACHE only if the instance
exists in the soc. Which is not the case for stm32h503x device
zephyrproject-rtos/hal_stm32#181

Signed-off-by: F. Ramu <[email protected]>
FRASTM added a commit to FRASTM/zephyr that referenced this pull request Nov 20, 2023
Update the stm32h5xx/drivers/include/
to enable the SAI, DelayBlock and DCACHE only if the instance
exists in the soc. Which is not the case for stm32h503x device
zephyrproject-rtos/hal_stm32#181

Signed-off-by: F. Ramu <[email protected]>
FRASTM added a commit to FRASTM/zephyr that referenced this pull request Nov 20, 2023
Update the stm32h5xx/drivers/include/
to enable the SAI, DelayBlock and DCACHE only if the instance
exists in the soc. Which is not the case for stm32h503x device
zephyrproject-rtos/hal_stm32#181

Signed-off-by: Francois Ramu <[email protected]>
carlescufi pushed a commit to zephyrproject-rtos/zephyr that referenced this pull request Nov 22, 2023
Update the stm32h5xx/drivers/include/
to enable the SAI, DelayBlock and DCACHE only if the instance
exists in the soc. Which is not the case for stm32h503x device
zephyrproject-rtos/hal_stm32#181

Signed-off-by: Francois Ramu <[email protected]>
coreboot-org-bot pushed a commit to coreboot/zephyr-cros that referenced this pull request Nov 22, 2023
Update the stm32h5xx/drivers/include/
to enable the SAI, DelayBlock and DCACHE only if the instance
exists in the soc. Which is not the case for stm32h503x device
zephyrproject-rtos/hal_stm32#181

(cherry picked from commit 4ddbf51)

Original-Signed-off-by: Francois Ramu <[email protected]>
GitOrigin-RevId: 4ddbf51
Change-Id: I55ca96bd043ba6b6c729d359ad1d45211d34efd7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/zephyr/+/5053132
Tested-by: Tristan Honscheid <[email protected]>
Tested-by: ChromeOS Prod (Robot) <[email protected]>
Reviewed-by: Tristan Honscheid <[email protected]>
Commit-Queue: Tristan Honscheid <[email protected]>
yunheQin pushed a commit to yunheQin/zephyr that referenced this pull request Dec 6, 2023
Update the stm32h5xx/drivers/include/
to enable the SAI, DelayBlock and DCACHE only if the instance
exists in the soc. Which is not the case for stm32h503x device
zephyrproject-rtos/hal_stm32#181

Signed-off-by: Francois Ramu <[email protected]>
panxxhub pushed a commit to panxxhub/zephyr that referenced this pull request Dec 7, 2023
Update the stm32h5xx/drivers/include/
to enable the SAI, DelayBlock and DCACHE only if the instance
exists in the soc. Which is not the case for stm32h503x device
zephyrproject-rtos/hal_stm32#181

Signed-off-by: Francois Ramu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants