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

Update/2405 #303

Merged
merged 26 commits into from
Jun 21, 2024
Merged

Update/2405 #303

merged 26 commits into from
Jun 21, 2024

Conversation

gus33000
Copy link
Collaborator

@gus33000 gus33000 commented May 30, 2024

Rebase repository on the latest upstream code.

The latest code allows for more modularity regarding configuration maps

@gus33000
Copy link
Collaborator Author

Note: Don't merge just yet as a few devices still need to be migrated.

Worth pointing out a lot of devices never actually used their uefiplat.cfg for creating the configuration map headers. Im migrating those as is but at the same time it was very obvious when they include microsoft surface duo hob addr values...

@gus33000
Copy link
Collaborator Author

Note: This also adds support for MTP 8150 directly

@gus33000
Copy link
Collaborator Author

PR compiles fine for 8150, other variants untested for compilation
Functionality remains untested

@gus33000 gus33000 added the enhancement New feature or request label May 30, 2024
@gus33000
Copy link
Collaborator Author

S-Error on MTP8150-AC (Hercules)

@mohahack

This comment was marked as outdated.

@mohahack

This comment was marked as duplicate.

@gus33000
Copy link
Collaborator Author

The update is nowhere near ready to be merged or functional, we know about this already. Stick to stable releases from the release page. This is not intended to be consumed just yet 👍

@sunflower2333
Copy link
Member

need a new way to pass display address in memory map to framebuffer serial library,
Hob does not work. also pcd is not acceptable in this case..

@gus33000
Copy link
Collaborator Author

PASS: https://github.com/woa-msmnile/msmnilePkg/actions/runs/9554518500

With workaround for the new config / mem map libs (we force the sec ones in, which equals to the old behavior entirely, no hob)

@gus33000
Copy link
Collaborator Author

Artefacts as part of the test run are only available for a few days, if anyone want to test, feel free while time allows.

@gus33000
Copy link
Collaborator Author

image
We may need to fix the kona zip name though..

@gus33000
Copy link
Collaborator Author

Failure observed as well:

No files were found with the provided path: Build/LahainaPkg/ci_artifacts. No artifacts will be uploaded.

And indeed, no 8350 zip file dropped, need investigation as to why it looks up this path and not the correct one (SurfaceDuo2Pkg)

@gus33000
Copy link
Collaborator Author

lgtm, i guess this can be merged if you want to

@sunflower2333
Copy link
Member

@gus33000 have to merge main into new br becuase new devices/changes were added in this period.

@sunflower2333 sunflower2333 merged commit 960e6b6 into main Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants