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: peci: xec: Re-add error recovery handling for PECI driver #23701

Merged
merged 1 commit into from
Mar 24, 2020

Conversation

albertofloyd
Copy link
Collaborator

@albertofloyd albertofloyd commented Mar 23, 2020

PR #23690 fixes #23586, however breaks error recovery.

Use temp variable to avoid coverity issue without breaking error recovery.

Signed-off-by: Jose Alberto Meza jose.a.meza.arellano@intel.com

@nashif
Copy link
Member

nashif commented Mar 23, 2020

PR #23690 fixes #23586, however breaks error recovery.

oops, sorry.

@zephyrbot
Copy link
Collaborator

zephyrbot commented Mar 23, 2020

All checks passed.

Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.

Using temp variable to avoid coverity issue without breakin
error recovery.

Signed-off-by: Jose Alberto Meza <jose.a.meza.arellano@intel.com>
@andrewboie andrewboie merged commit 04e190a into zephyrproject-rtos:master Mar 24, 2020
@albertofloyd albertofloyd deleted the peci_cov branch March 26, 2020 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Coverity CID :208914] Self assignment in drivers/peci/peci_mchp_xec.c
5 participants