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: Updated the hal silabs version #87673

Closed

Conversation

ragurram26
Copy link
Contributor

@ragurram26 ragurram26 commented Mar 26, 2025

To ensure validation for changes in hal_silabs,
this commit updates west.yml to point to cb17629c3f00b6c547b6bc9f29863af33f8e1d65.

@github-actions github-actions bot added the size: XS A PR changing only a single line of code label Mar 26, 2025
Copy link

github-actions bot commented Mar 26, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_silabs zephyrproject-rtos/hal_silabs@9d32354 zephyrproject-rtos/hal_silabs@cb17629 (main) zephyrproject-rtos/hal_silabs@9d323543..cb17629c

All manifest checks OK

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

@jhedberg
Copy link
Member

@ragurram26 ragurram26 force-pushed the updated_latest_version branch from 0c60f5d to 39a4024 Compare March 26, 2025 14:21
@ragurram26
Copy link
Contributor Author

@jhedberg I have updated my PR

@ragurram26 ragurram26 force-pushed the updated_latest_version branch from 39a4024 to fc28ebf Compare March 28, 2025 06:13
@jerome-pouiller
Copy link
Collaborator

@ragurram26 Compliance fails because you need to fix your name in your gitconfig. Once done, git commit --amend --reset-author -s should update the author of the commit.

@jerome-pouiller jerome-pouiller self-requested a review March 28, 2025 15:59
@ragurram26 ragurram26 force-pushed the updated_latest_version branch from fc28ebf to 9584d84 Compare March 28, 2025 16:32
@ragurram26 ragurram26 force-pushed the updated_latest_version branch from 9584d84 to 21d35e3 Compare April 1, 2025 05:46
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Apr 1, 2025
@jhedberg
Copy link
Member

jhedberg commented Apr 1, 2025

@ragurram26 please remove the mention of "pull/94/head" in the commit message. That's not relevant since the HAL PR was already merged.

To ensure validation for changes in hal_silabs,
this commit updates west.yml to point to cb17629c3f00b6c547b6bc9f29863af33f8e1d65

Signed-off-by: Rahul Gurram <rahul.gurram@silabs.com>
@ragurram26 ragurram26 force-pushed the updated_latest_version branch from 21d35e3 to 8a9e208 Compare April 1, 2025 10:56
@ragurram26
Copy link
Contributor Author

@jhedberg I have updated the commit message

@jhedberg
Copy link
Member

jhedberg commented Apr 1, 2025

@ragurram26 there are a bunch of PRs currently open that update the hal_silabs reference. E.g. I just merged the sleeptimer PR in the HAL tree. I think it'd make sense to try to consolidate the main tree PRs - maybe close this one and let the changes be pulled in through the other open PRs that update hal_silabs in the manifest?

@jerome-pouiller
Copy link
Collaborator

Superseded by #87606

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
manifest manifest-hal_silabs platform: Silabs Silicon Labs size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants