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

west.yml: Update Nordic HAL to pull GCC 12 warning fixes #48661

Merged

Conversation

stephanosio
Copy link
Member

This commit updates the Nordic HAL (hal_nordic) to pull in the fixes
for the warnings observed when building with the GCC 12.

Signed-off-by: Stephanos Ioannidis [email protected]

@zephyrbot zephyrbot added manifest west manifest-hal_nordic DNM This PR should not be merged (Do Not Merge) labels Aug 3, 2022
@zephyrbot
Copy link
Collaborator

zephyrbot commented Aug 3, 2022

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hal_nordic zephyrproject-rtos/hal_nordic@f6628a3 zephyrproject-rtos/hal_nordic@2e1c828 (master) zephyrproject-rtos/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@carlescufi
Copy link
Member

Thanks @stephanosio. Use 2e1c828cf4efb71679aeaec94128708a5353e031

@stephanosio stephanosio force-pushed the hal_nordic_fix_gcc12_warn branch from ed3260f to 2c311bc Compare August 8, 2022 10:21
@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label Aug 8, 2022
anangl
anangl previously approved these changes Aug 8, 2022
carlescufi
carlescufi previously approved these changes Aug 8, 2022
@carlescufi
Copy link
Member

@stephanosio any idea why CI is failing here?

@stephanosio
Copy link
Member Author

@stephanosio any idea why CI is failing here?

The failures have nothing to do with this PR.

Something is wrong with the twister: https://discord.com/channels/720317445772017664/733037890514321419/1006166543413018664

This commit updates the Nordic HAL (hal_nordic) to pull in the fixes
for the warnings observed when building with the GCC 12.

Signed-off-by: Stephanos Ioannidis <[email protected]>
@stephanosio stephanosio force-pushed the hal_nordic_fix_gcc12_warn branch from 2c311bc to 0e147be Compare August 8, 2022 14:37
@stephanosio
Copy link
Member Author

Rebased to pull in the hotfix #48781

@nashif nashif dismissed stale reviews from carlescufi and anangl via 0e147be August 8, 2022 15:17
@carlescufi carlescufi merged commit 38afa78 into zephyrproject-rtos:main Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants