-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
west.yml: Updated the hal silabs version #87673
Conversation
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
@ragurram26 until zephyrproject-rtos/hal_silabs#94 is merged you need to reference that PR instead of any specific commit. See https://docs.zephyrproject.org/latest/develop/modules.html#process-for-submitting-changes-to-existing-modules |
0c60f5d
to
39a4024
Compare
@jhedberg I have updated my PR |
39a4024
to
fc28ebf
Compare
@ragurram26 Compliance fails because you need to fix your name in your |
fc28ebf
to
9584d84
Compare
9584d84
to
21d35e3
Compare
@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>
21d35e3
to
8a9e208
Compare
@jhedberg I have updated the commit message |
@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? |
Superseded by #87606 |
To ensure validation for changes in hal_silabs,
this commit updates west.yml to point to cb17629c3f00b6c547b6bc9f29863af33f8e1d65.