Poc x280 - #2189
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). |
There was a problem hiding this comment.
First thourough review, missing linux and coreboot configs, saving in oldconfigs and validating linux config not depending on t480 linux one are relevant. Please beat me at it and check at helpers under modules/coreboot and modules/linux to save in oldconfig and have a _defconfig sitting in repo alongside the oldconfig version so board configs can easily be compared. Good work here @NobodyNo0ne and @AlguienSasaki
Specific helpers (./docker_repro.sh make BOARD=xyz [helper] where [helper] might create changes to commit:
- coreboot.save_in_defconfig_format_backup
- coreboot.save_in_oldconfig_format_in_place
- linux.save_in_olddefconfig_format_in_place
|
Please create PR against heads-wiki to be merged at the same time as this PR final review+merge for disasssembly, and reuse template of t480/t480s there see https://osresearch.net/T480s-maximized-flashing/ https://osresearch.net/T480-maximized-flashing/ and https://osresearch.net/Contributing-to-Heads-wiki/ Note anything needing to change under https://osresearch.net/Porting/ per your porting effort to help other ports and making me redundant, thanks! |
I think the main issue is that I am just bad at reading documentation... doing most of my work at 10pm-5am is not helpful either, but that is also my fault. |
|
Looked at the CircleCI logs, No rule to make target '../../../blobs/xx80/x280_ifd.bin' Either I figure out how to derive the IFD from https://download.lenovo.com/pccbbs/mobiles/n20uj44w.exe or I have an epiphany. It's just a YAML file why am I fighting for my life over it I have unironically read the docs I know this exists I somehow, not only failed to fix CircleCI for the X280, but also broke it for other boards.
are you sure? CircleCI no longer works, it seems to expect it, and there are blobs present in upstream repo. https://github.com/linuxboot/heads/tree/master/blobs/xx80 |
6b0b529 to
cb2a634
Compare
gbe and ifd considered "cofiguration blobs", gbe can be generated and are under blob dir, me can't. ifd are historically coming from a donor board as can be seen with git history for all blobs files. Sorry if wasn't clear and made you loose time. |
Signed-off-by: NobodyNo0ne <akbarov2007@gmail.com>
Oh. |
EDIT:
CircleCI has ACTUALLY been looked at
Restored the rest of Heads
No longer 152 behind
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.
In the meantime, I have noticed an issue where it only shows 4GB of RAM. In my case, my X280 has 16GB. Suboptimal.
From my testing, TOTP should be fully functional. The only changes I made between the config for the heads payload for T480 to the X280 related to i915, since otherwise it would complain about not finding the display, and being unuseable
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