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

framework: Add Intel Core Ultra Series 1 #988

Closed

Conversation

JohnAZoidberg
Copy link
Member

Description of changes

Not selling to customers yet, but already announced: https://frame.work/tw/en/blog/introducing-the-new-framework-laptop-13-with-intel-core-ultra-series-1-processors

Things done
  • Tested the changes in your own NixOS Configuration
  • Tested the changes end-to-end by using your fork of nixos-hardware and
    importing it via <nixos-hardware> or Flake input

@quinn-dougherty
Copy link
Contributor

maybe shorter name? like ultra-1. Even better (maybe), 1-ultra to fit with <number>-gen pattern from previous intel regime.

@@ -150,6 +150,7 @@ See code for all available configurations.
| [Framework 11th Gen Intel Core](framework/13-inch/11th-gen-intel) | `<nixos-hardware/framework/13-inch/11th-gen-intel>` |
| [Framework 12th Gen Intel Core](framework/13-inch/12th-gen-intel) | `<nixos-hardware/framework/13-inch/12th-gen-intel>` |
| [Framework 13th Gen Intel Core](framework/13-inch/13th-gen-intel) | `<nixos-hardware/framework/13-inch/13th-gen-intel>` |
| [Framework Intel Core Ultra Series 1](framework/13-inch/intel-core-ultra-series1) | `<nixos-hardware/framework/13-inch/intel-core-ultra-series1>` |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also reference this in the flake.nix.

@umgefahren
Copy link

I would love to see this shipped :)

@umgefahren
Copy link

For anyone interested: I addressed the concerns raised above and submitted a PR to Framework. Here it is: FrameworkComputer#1
I have been running this patch ever since and I have no complaints. I would be very happy to see @FrameworkComputer merge it.
I could also just open a new PR to this Repo.

@Mic92
Copy link
Member

Mic92 commented Nov 2, 2024

@umgefahren Just open a new PR. @JohnAZoidberg if you want to continue working on this, also just re-open this pull request.

@Mic92 Mic92 closed this Nov 2, 2024
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