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

Add ME HAP disable option to Protectli platforms #593

Open
wants to merge 11 commits into
base: dasharo
Choose a base branch
from

Conversation

philipandag
Copy link
Contributor

No description provided.

@philipandag philipandag force-pushed the add-me-hap-disable-protectli branch from cc9ed57 to ad1b97e Compare November 28, 2024 11:49
@philipandag philipandag marked this pull request as ready for review November 28, 2024 14:44
@philipandag philipandag requested a review from miczyg1 November 28, 2024 14:44
@miczyg1 miczyg1 force-pushed the add-me-hap-disable-protectli branch from e3e1ef0 to a008e47 Compare November 29, 2024 08:53
@miczyg1
Copy link
Contributor

miczyg1 commented Nov 29, 2024

I also pushed a commit to disable HECI if ME is in any disabled mode to match the behavior of other platforms. I did it only for Elkhart Lake and Meteor Lake where the FSP is known to leave HECI enabled in such cases.

@mkopec cc, want to look at it as well?

miczyg1
miczyg1 previously approved these changes Nov 29, 2024
@miczyg1 miczyg1 force-pushed the add-me-hap-disable-protectli branch from a008e47 to ce0dc83 Compare November 29, 2024 08:56
@miczyg1 miczyg1 force-pushed the add-me-hap-disable-protectli branch from ce0dc83 to be6ead1 Compare November 29, 2024 09:02
miczyg1
miczyg1 previously approved these changes Nov 29, 2024
…bled mode

Disable/hdie HECI if ME is in debug/HAP or soft disabled mode. The HECI should
be hidden if ME is disabled using HAP or HECI message. Match the behavior of
other platforms which disable HECI in FSP.

Signed-off-by: Michał Żygowski <[email protected]>
@miczyg1 miczyg1 force-pushed the add-me-hap-disable-protectli branch from be6ead1 to 609e958 Compare November 29, 2024 09:44
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.

2 participants