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

Rebased X230 maximized fhd (edp connector based mods) #1299

Closed

Conversation

lethedata
Copy link

@lethedata lethedata commented Jan 29, 2023

With #1201 having been merged, rebased #967 on master.

Compiled and flashed to hardware successfully

tlaurion and others added 4 commits August 15, 2022 09:55
Otherwise binary patches cannot be patched/created

Additional fixes needed
- flashrom patch was invalid and got catched by git apply. Correcting
- gpg2-2.2.21.patch was pointing to bad target. Correcting
- add x230-maximized-fhd_edp and x230-hotp-maximized-fhd_edp board configs
- add/rework coreboot patch for x230 fhd variant to be applied on top of 4.13
- add coreboot config to point to x230-edp variant, fixing path to vbt file since default path is wrong under. Comment made upstream https://review.coreboot.org/c/coreboot/+/28950/22#message-4904ce82f01ba0505b391e072e4537b6a9f1a229
  - remove no gfx init and replace with libgfxinit(defonfig default), set internal display as default
- add x230-hotp-maximized-fhd_edp and x230-maximized-fhd_edp to CircleCI builds
@lethedata
Copy link
Author

First time pulling a PR into a branch to be rebased, let me know if something didn't pull right

@tlaurion
Copy link
Collaborator

tlaurion commented Jan 29, 2023

@echo-84 thanks for the reminder.

I guess I should rebase #967 on master ;) will try to do tomorrow or otherwise monday. Otherwise 2 unneeded commits from this PR.

@lethedata
Copy link
Author

Got it, I'll go ahead and close this PR and leave a note on what I changed to get it working on #967

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