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

modules: debug: mipi-sys-t PR 14 "potential error warning removed" #85394

Merged

Conversation

LoveKarlsson
Copy link
Contributor

Adjusted code where a deliberate assignment happened in a q-mark operator, which caused IAR to give a warning
"did you mean '==' instead of '=' ?"

@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Feb 7, 2025
@zephyrbot
Copy link
Collaborator

zephyrbot commented Feb 7, 2025

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

Name Old Revision New Revision Diff
mipi-sys-t zephyrproject-rtos/mipi-sys-t@71ace1f zephyrproject-rtos/mipi-sys-t@33e5c23 (zephyr) zephyrproject-rtos/mipi-sys-t@71ace1f5..33e5c23c

All manifest checks OK

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

@zephyrbot zephyrbot added manifest manifest-mipi-sys-t DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Feb 7, 2025
@LoveKarlsson LoveKarlsson changed the title mipi-sys-t PR 14 "potential error warning removed" modules: debug: mipi-sys-t PR 14 "potential error warning removed" Feb 8, 2025
@nashif
Copy link
Member

nashif commented Feb 8, 2025

zephyrproject-rtos/mipi-sys-t#14 was merged

@nashif nashif self-assigned this Feb 8, 2025
Adjusted code where a deliberate assignment happened in a q-mark
operator, which caused IAR to give a warning
"did you mean '==' instead of '=' ?"

Signed-off-by: Lars-Ove Karlsson <lars-ove.karlsson@iar.com>
@kartben kartben force-pushed the mipi-warning-removed branch from 54b03ba to 9051c27 Compare February 14, 2025 13:54
@kartben
Copy link
Collaborator

kartben commented Feb 14, 2025

manifest updated to point to merged commit

@zephyrbot zephyrbot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Feb 14, 2025
@fabiobaltieri fabiobaltieri merged commit 1619fc9 into zephyrproject-rtos:main Feb 14, 2025
24 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
manifest manifest-mipi-sys-t size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants