Add x280 port, based on coreboot 25.12 + atop SPD upstream patch - #2189
Add x280 port, based on coreboot 25.12 + atop SPD upstream patch#2189NobodyNo0ne wants to merge 4 commits into
Conversation
|
Looking at what GitHub has just told me here CircleCI in fact does NOT work (I did NOT realise drafted PRs can be seen) |
|
Ouch @NobodyNo0ne some cleanup will need to take place here. No clue why you impact 282 files with 33 commits and glancing over, i cannot explain how you come to this. You deleted blobs/* files, which circleci depends on to download blobs needed by buikds (ME etc) which are not found which is why CircleCI fails. Also ou have not read porting guide https://osresearch.net/Porting/ nor https://osresearch.net/Development please do so first then add question here. |
I did read it but if I am honest with you I just completely ignored CircleCI while porting. Many files impacted because I removed many files that were not X280 related over porting. ^ |
|
I'm working on making this PR not an absolute mess, sorry about all this I thought it worked like an E-Mail draft. |
|
PR is now no longer an absolute mess. I think.
|
4b23454 to
9f42a3b
Compare
|
@NobodyNo0ne this pr is not based on origin/master of this repo, please git fetch origin/master, git rebase --signoff origin/master |
|
on ab932f9 As discussed on matrix,
As per matrix thread disscussion (for posterity) Use Makefile helpers to reduce copile time and pain points; by using Makefile comments: Lines 1148 to 1165 in 7902f1f |
0f174bc to
dd719aa
Compare
|
From OP
Nitrokey: @nestire @jans23 some problems shipping nk3 outside of EU? Purism: @JonathonHall-Purism some Librem keys devices in production? Thanks for updating, if there is issues tell me I'll open and pin issue under Heads, was not aware. Cc @NobodyNo0ne |
|
Nitrokey does ship worldwide but temporarily you have to request a quote for outside of the EU. |
|
You reported on matrix thread that 4gb ram only was seen, reported by https://ticket.coreboot.org/issues/627 and fixed by https://review.coreboot.org/c/coreboot/+/91170 This means you will need to drop a patch on top of 25.12 since 25.12 was not really complete for x280, through gerrit's "Download ->Format patch" instructions Attention: this will clone coreboot into working directory. Something like the following, where XXXX needs to be the next patch number to be applied in proper order from the coreboot patches dir (change paths as needed): And then |
|
No problem having it staging in this PR until heads-wiki pr is created and changes moved there, but once ready, you will have to clean the git history (git squash/amend/fixup+signoff) so that the final PR doesn't include those changes in it's final state (git keeps all files history, so your jpg and whatnot would otherwise make the repository size bigger for everyone in the future to download even it those were never needed). |
Waiting x280-maximized will produce results: CircleCi imposes concurrency limits. |
|
have to go for now, but will read your tests later Was confused because origin/master forces modules and cpio rebuilds since beginning of august.... maybe I missed something but no kernel modules in modules.cpio never supposed to happen :/ |
yep kernel modules there https://output.circle-artifacts.com/output/job/8e40dd3a-6918-4181-ba87-cdfbfe26e078/artifacts/0/build/x86/EOL_x280-maximized/hashes.txt : (you can see in filename: 689bae5) |
|
modified PR name; OP (PR description needs some cleanup to represent current state of the branch) |
|
Let's see with af4fa60 if circleci produces rom with kernel modules in modules.cpio before flashing this time? If there is issue with that build, this means something is yet missing in Makefile modules/linux to make things clean on warm cache rebuilts (prior build from circleci was on a clean, no cache, build) |
|
On warm cache build for af4fa60 https://app.circleci.com/pipelines/github/linuxboot/heads/1465/workflows/c8f63b47-2b53-4be5-b258-5cf24ec0fb66 (will edit)
|
@NobodyNo0ne https://output.circle-artifacts.com/output/job/ec5cfeec-6c92-4691-96e5-e23537780b0c/artifacts/0/build/x86/EOL_x280-maximized/heads-EOL_x280-maximized-202608271655-v0.2.1-3153-gaf4fa60.zip contains proper modules.cpio with usb modules as showed in previous comment Please comment :) |
I thought you went so I was in a terminal the whole time |
Couldn't leave without my head empty of doubts, heading out now :) |
You in fact fixed USB. |
|
#2198 So far in testing, Re-Ownership works etc. Booted into OS WiFi Works RAM patch works. |
Modify OP, maybe even mofigy porting.md but https://osresearch.net/Porting/#testing provides clear checkboxes to add in OP. Thanks |
I was just testing the immediate USB issue. Will move to going through the checklist. I will add the checklist to first comment/ Opening PR comment <- you beat me to it I have also never had a non graphical installer work, which is rather unfortunate/ a bit annoying. Seems like a known/documented issue though (I have read the docs about this) |
|
modified OP in case it wasn't clear what was needed there from link. |
I think the main issue with testing is user error (me). Seems to all work. I had an isomount issue but it seems to have been a one-off? Cannot replicate it. This port of heads has also been my first time using it, which probably does not help. Squashed commits.
I do not see this in the current repo or locally |
…ocking, and 16GB models of X280 for Coreboot 25.12. TLaurion fixes for splash/display, without i915. Signed-off-by: NobodyNo0ne <akbarov2007@gmail.com>
Signed-off-by: NobodyNo0ne <akbarov2007@gmail.com>
NobodyNo0ne
left a comment
There was a problem hiding this comment.
I have gone through this. It finally doesn't say anything is left but I feel like I already did most of it. I was probably doing something wrong.
| paths: | ||
| - build/x86/<< parameters.coreboot_dir >> | ||
| - build/x86/musl-cross-make-227df8b99103f9c59f6570babf892978e293082f | ||
| - build/x86/musl-cross-make-fd6be58297ee21fcba89216ccd0d4aca1e3f1c5c |
There was a problem hiding this comment.
This should be fixed but github hasnt marked as outdated <- it now has
|
Oh why is circleci failing NOW But I did not change modules/coreboot in this commit :(
I am hoping it was just the patch file. Fixed! :D |
…ged nothing in modules/coreboot for it to fail finding 3rdparty Signed-off-by: NobodyNo0ne <akbarov2007@gmail.com>
tlaurion
left a comment
There was a problem hiding this comment.
LGTM! Didn't try to optimize the blob script, thought there were common functions to reuse now, but good for merge either way.
Oh. Add yourself as board tester and add board under doc/BOARD_OWNER_AND_TESTERS.md
Thanks!
'LGTM'? When I can, I will get a NitroKey and test that to check that box (also to use it for myself). I see Libreboot has started selling preflashed X280 laptops as well, I wonder if X280 based NitroPads will happen? :) Is there a separate .md where I attribute credit/processes, or is that not needed? |
…RDS_AND_TESTERS.md Signed-off-by: NobodyNo0ne <akbarov2007@gmail.com>
EDIT:
CoreBoot version specified to 25.12
Deguard tool updated to version that added X280 support (for X280 delta)
HOTP is on hold. Mine doesn't work, and Librem keys are out of stock, I need to ask for a quote from NitroKey.
X280 models with 16GB RAM now work as intended with patch.
TOTP should be fully functional.
Display issues rectified, thanks to @tlaurion, now no longer bodged with i915
CircleCI now uses X280 as seed for 25.12, thanks to @tlaurion
I think this PR is finally at a stage where I would have created the draft, if I knew they were public.
Now aware Draft PRs are visible
Original Post:
I have a port of heads for the x280. PR as draft since I have not:
Tested HOTP (I do not have a dongle YET)
Tested reproducibility (circleci)
I also need to change a couple things, since the coreboot version is currently labelled simply as 'x280'.
I had to use a different coreboot version since 25.09 did not seem to support the x280, and I am not cut out for backporting.
Do tell me if I am missing anything.
I would also like to thank @AlguienSasaki for helping me, and for his LibreBoot port.
Edits by @tlaurion
Some notes for doc input:
Testing
af4fa60using external programmer modelRaspberry Pi Pico (pico-serprog)on3.3VVoltage modeFedorainstall and rebootX.Y.Z.hardwarekey e.g. nk3Devuan<- Note from @NobodyNo0ne Installing Devuan with heads is a massive PITApatches/coreboot-25.12and coreboot config contain proper preparation of the platform)