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

Broadwell-DE FSP #5

Closed
zaolin opened this issue Nov 6, 2018 · 5 comments
Closed

Broadwell-DE FSP #5

zaolin opened this issue Nov 6, 2018 · 5 comments

Comments

@zaolin
Copy link

zaolin commented Nov 6, 2018

@nate-desimone-intel Currently headers and sample code are missing in the repository. Can you add the missing ones?

@zaolin
Copy link
Author

zaolin commented Nov 6, 2018

Also maybe update them?

@nate-desimone
Copy link
Contributor

Hi @zaolin, I see the following: https://github.com/IntelFsp/FSP/blob/master/BroadwellDEFspBinPkg/include/fspvpd.h

Is there something more you are looking for?

@zaolin
Copy link
Author

zaolin commented Nov 6, 2018

@nate-desimone
Copy link
Contributor

Hi @zaolin, when comparing the contents of both repos, they are identical:
https://github.com/IntelFsp/FSP/tree/master/BroadwellDEFspBinPkg/include
https://review.coreboot.org/cgit/coreboot.git/tree/src/vendorcode/intel/fsp1_0/broadwell_de/include

diff

The copy of fspvpd.h in github is newer than the version in the coreboot repo:

file_diff

The fspsupport.c file is also present in github, just under a different directory structure. I am closing this issue.

@zaolin
Copy link
Author

zaolin commented Nov 6, 2018

@nate-desimone-intel Thanks, got it. Just was a little bit confused.

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

No branches or pull requests

2 participants