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

fix(PeriphDrivers): Fix -wstrict-prototype and other Misc. MAX78002 Warnings #1077

Merged
merged 6 commits into from
Jul 9, 2024

Conversation

Jake-Carter
Copy link
Contributor

@Jake-Carter Jake-Carter commented Jul 8, 2024

Description

Primarily fixes -wtrict-prototype warnings for the MAX78002 peripheral driver builds. While I was at it I also fixed double promotion and some other misc. warnings in the examples.

The build system now skips CSI2 and mxc_lock driver files for RISC-V builds (i.e. when RISCV_CORE = 1) to avoid those warnings as well.

Checklist Before Requesting Review

  • PR Title follows correct guidelines.
  • Description of changes and all other relevant information.
  • (Optional) Link any related GitHub issues using a keyword
  • (Optional) Provide info on any relevant functional testing/validation. For API changes or significant features, this is not optional.

- Change MXC_CSI2_CaptureFrame prototype to no longer accept the number
  of data lanes as an argument.  The number of data lanes is configured
at initialization time.  This is a low risk change, as it was
previously non-functional and unused.
@github-actions github-actions bot added the MAX78002 Related to the MAX78002 (AI87) label Jul 8, 2024
@sihyung-maxim
Copy link
Contributor

Can you wait to merge until this PR is in: #1076? Want to test if changes from a forked branch are properly pushed into the zephyr-hal repo.

@Jake-Carter Jake-Carter merged commit fca56e9 into analogdevicesinc:main Jul 9, 2024
9 of 10 checks passed
@Jake-Carter Jake-Carter deleted the fix/ai87-warnings branch July 9, 2024 21:57
sihyung-maxim pushed a commit to analogdevicesinc/hal_adi that referenced this pull request Jul 9, 2024
EricB-ADI pushed a commit that referenced this pull request Aug 21, 2024
ozersa pushed a commit to analogdevicesinc/hal_adi that referenced this pull request Sep 11, 2024
ozersa pushed a commit to analogdevicesinc/hal_adi that referenced this pull request Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX78002 Related to the MAX78002 (AI87)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants