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

lenovo/thinkpad/t495: init from e495. #149

Merged
merged 1 commit into from Mar 12, 2020
Merged

Conversation

danderson
Copy link
Contributor

The E495 and T495 are based on the same core Ryzen platform, so the
E495 tweaks apply similarly to the T495.

Tested on my own T495, works well.

Copy link

@etu etu left a comment

Choose a reason for hiding this comment

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

I have a T495 as well and when I got it I had all kinds of weird issues, but since newer kernels it's fine.

So this profile should be fine.

I haven't actually had the time to pull it in on my T495 but I apply the same setting in my config already.

@Mic92
Copy link
Member

Mic92 commented Mar 11, 2020

Please link to the profile in the README table

The E495 and T495 are based on the same core Ryzen platform, so the
E495 tweaks apply similarly to the T495.

Tested on my own T495, works well.
@danderson
Copy link
Contributor Author

@Mic92 added to README, and rebased. Thanks for the review, and sorry I missed the instructions!

@jonringer
Copy link

the linked issue also mentions adding:

hardware.enableRedistributableFirmware = true;

is that needed as well?

@danderson
Copy link
Contributor Author

Empirically no: it was off on my laptop before I started using nixos-hardware, and the machine booted to X11 and worked just fine. It means we miss out on newer Intel wifi/bluetooth firmware and AMD microcode, but it looks like the burned-in firmware and microcode that ships with the T495 is "good enough".

I don't know what our policy is for that kind of thing? I think it's a good idea to enableRedistributableFirmware=true, but it's not strictly required and it might annoy people who don't want non-free blobs in their NixOS.

@Mic92 Mic92 merged commit 9f6d639 into NixOS:master Mar 12, 2020
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.

None yet

4 participants