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

Bump QuickLogic HAL PR module #28821

Merged

Conversation

kowalewskijan
Copy link
Contributor

@kowalewskijan kowalewskijan commented Sep 30, 2020

This PR is just for testing purposes to prove the QuickLogic HAL PR will not break the Zephyr build. To do so, I changed the remote and commit in the west manifest to use HAL with changes planned to be merged to HAL repository.

The HAL PR: zephyrproject-rtos/hal_quicklogic#2

EDIT: I transformed this PR from test to bump PR

west.yml Outdated Show resolved Hide resolved
@kowalewskijan kowalewskijan force-pushed the DNM-test-quicklogic-hal branch from ddf299b to ab6bd6f Compare October 1, 2020 07:42
This commit bumps module SHA to the newest one

Signed-off-by: Jan Kowalewski <[email protected]>
@kowalewskijan kowalewskijan force-pushed the DNM-test-quicklogic-hal branch from ab6bd6f to b529131 Compare October 2, 2020 08:50
@kowalewskijan kowalewskijan changed the title [DNM] Test QuickLogic HAL PR Bump QuickLogic HAL PR module Oct 2, 2020
@kowalewskijan
Copy link
Contributor Author

Since the PR in HAL repository was merged, I changed the revision to point to the newest commit. I think we can merge this to bump the module.

@kowalewskijan kowalewskijan requested a review from galak October 2, 2020 08:52
@kowalewskijan
Copy link
Contributor Author

@galak Can we merge this bump in this PR or should I open another one to request more reviewers? This would enable us to re-run checks in other QuickLogic PRs such as #28762 and #28765

Copy link
Member

@carlescufi carlescufi left a comment

Choose a reason for hiding this comment

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

Checked SHA

@carlescufi carlescufi merged commit 83b6fa2 into zephyrproject-rtos:master Nov 26, 2020
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.

4 participants