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

config: remove x200 linux config #387

Closed
wants to merge 1 commit into from
Closed

config: remove x200 linux config #387

wants to merge 1 commit into from

Conversation

merge
Copy link
Contributor

@merge merge commented Apr 20, 2018

It's not used anymore, I think. I might be wrong. Please review. thanks!

It's not used anymore.
osresearch added a commit that referenced this pull request Jul 18, 2018
@osresearch
Copy link
Collaborator

Merged, thanks!

@osresearch osresearch closed this Jul 18, 2018
@tlaurion
Copy link
Collaborator

@osresearch @merge
If I remember well, work have been done to have a lightweight linux config and board config that deactivates TPM support. That may need to be maintained a bit, but I do not think it should be deleted.

The goal of such config would be to have heads able to boot signed tail isos directly from a usb keys, Tails keys being bundled in Heads recently.

@merge: what is inteded by "not used anymore?"

@merge
Copy link
Contributor Author

merge commented Jul 23, 2018

Thanks for the feedback. "Not used" means that there's no board / make target that uses said config. In that case this was missing and thus not readable. Could you add a "tails" target and re-add the config if that's appropriate?

@tlaurion
Copy link
Collaborator

tlaurion commented Jul 31, 2018

@persmule changes were to permit Heads to become a more generic coreboot payload, extending to x200 and others that doesn't have a TPM, or for which the TPM was not yet supported (eg. kgpe-d16)

There is no x200 board configuration in Heads repository right now, but it's inspired inclusion should prevail current linux config file deletion.

@herbsmn
Copy link

herbsmn commented Oct 29, 2020

Is there any chance the x200 linux config could be added back?

@tlaurion
Copy link
Collaborator

@herbsmn absolutely. The problem here is that the device has no TPM, so #836 would be required for additional security, where inclusion of the HOTP toolchain + GPG2 will probably fit really tight.

Also note that not all x200 are equal. If I remember well, only the labeled as "PRO" (AMT PRO on intel label?) on the tests i've done in the past had a 8mb(?) SPI chip.... Would need to find traces of that in the backlog...

@herbsmn
Copy link

herbsmn commented Oct 29, 2020

My larger hope is that the Montevina / G45 / Centrino 2 devices that can be run without Intel ME can be supported by heads. I noticed that @persmule was running heads on this x301 based on the text at the bottom of this page: https://doc.coreboot.org/mainboard/lenovo/x301.html

When I asked persmule about this they said that the heads x200 board configs in question here were used to build the x301 Rom.

In their original heads patch set that added the x200 configs it says they also tested the configs on an x200s device: baa30a2

Regarding which x200 models and, more broadly, Montevina models have 4MiB vs 8MiB SPI chips, you can maybe check the ThinkPad RAW Image Versioning's Device IDs using this handy table here: https://notabug.org/libreboot/libreboot/issues/731

You are correct that the vPro variants have 8MiB SPI chips while all others have 4MiB SPI chips.

@tlaurion
Copy link
Collaborator

@herbsmn have not tested but see and comment #878

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.

4 participants