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

Adding support for ga403 #1255

Draft
wants to merge 17 commits into
base: master
Choose a base branch
from
Draft

Adding support for ga403 #1255

wants to merge 17 commits into from

Conversation

Mydien27
Copy link

@Mydien27 Mydien27 commented Nov 18, 2024

Description of changes
  • Added zephyrus G14 2024 support (ga403)
  • NOTE: my model is specifically the ga403UV
  • heavily based on the 402x due to hardware similarity
  • Forced the use of nvidia and amd video drivers (the one set in the common nvidia module prevent the display manager from starting see Asus ga401 broken #689)
  • Modified nvidia sleep/hibernate/resume services to trigger on systemctl suspend-then-hibernate and hybrid-sleep (otherwise the gpu does not suspend and prevents sleep/hibernation)
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

flake.nix Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@Mydien27
Copy link
Author

Mydien27 commented Dec 3, 2024

I apologize for the silly mistakes

boot = {
# Note: it seems being on the latest kernel is important for CPPC to work, which is required for the amd_pstate
# https://bugzilla.kernel.org/show_bug.cgi?id=218686
kernelPackages = mkDefault pkgs.linuxPackages_latest;
Copy link
Member

Choose a reason for hiding this comment

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

Please set a minimum kernel version instead:

kernelPackages = lib.mkIf (lib.versionOlder pkgs.linux.version "6.X") pkgs.linuxPackages_latest;

Replace x with the kernel version that works for you. In doubt the current latest kernel version if you don't want to test others.

# https://wiki.archlinux.org/title/Iwd#iwd_keeps_roaming
#
# But NixOS doesn't have the tweaks for IWD, yet.
networking.wireless.iwd.settings =
Copy link
Member

Choose a reason for hiding this comment

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

This was copied from a different module. Can you move this to a common module?

@Mic92 Mic92 marked this pull request as draft January 7, 2025 06:52
@Mic92
Copy link
Member

Mic92 commented Jan 7, 2025

Please undraft when you are ready.

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.

2 participants