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

drivers: ambiq: Correct peripheral instance calculation #87253

Merged

Conversation

AlessandroLuo
Copy link
Contributor

This commit corrected ambiq peripheral instance calculations

aaronyegx
aaronyegx previously approved these changes Mar 18, 2025
@AlessandroLuo AlessandroLuo force-pushed the ambiq-peripheral-inst-cal branch 3 times, most recently from 5c74eee to 906338e Compare March 19, 2025 02:52
swift-tk
swift-tk previously approved these changes Mar 19, 2025
aaronyegx
aaronyegx previously approved these changes Mar 19, 2025
@kartben
Copy link
Collaborator

kartben commented Mar 20, 2025

@AlessandroLuo this needs to be rebased - thx!

@RichardSWheatley
Copy link
Contributor

@AlessandroLuo this needs to be rebased - thx!

I'm sure he will be happy to rebase but for future reference can you specify exactly why?

@AlessandroLuo AlessandroLuo force-pushed the ambiq-peripheral-inst-cal branch from 906338e to c0a81ac Compare March 20, 2025 15:39
@AlessandroLuo
Copy link
Contributor Author

@AlessandroLuo this needs to be rebased - thx!

Done

aaronyegx
aaronyegx previously approved these changes Mar 21, 2025
@AlessandroLuo AlessandroLuo requested a review from swift-tk March 24, 2025 08:12
swift-tk
swift-tk previously approved these changes Mar 24, 2025
This commit corrected ambiq peripheral instance calculations

Signed-off-by: Hao Luo <hluo@ambiq.com>
@AlessandroLuo AlessandroLuo dismissed stale reviews from swift-tk and aaronyegx via cc09000 March 24, 2025 09:15
@AlessandroLuo AlessandroLuo force-pushed the ambiq-peripheral-inst-cal branch from c0a81ac to cc09000 Compare March 24, 2025 09:15
@kartben kartben merged commit a460479 into zephyrproject-rtos:main Mar 25, 2025
21 checks passed
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.

6 participants