feat: move amdgpu-specific config to its own role, since i now also have nvidia systems

modules/core/hardware.nix

@@ -1,18 +1,3 @@
 { pkgs, ... }:
 {  
-  hardware = {
-    graphics = {
-      enable = true;
-    };
-    enableRedistributableFirmware = true;
-    opengl = {
-      extraPackages = with pkgs; [
-        amdvlk
-      ];
-      # For 32 bit applications as well
-      extraPackages32 = with pkgs; [
-        driversi686Linux.amdvlk
-      ];
-    };
-  };
 }

roles/amdgpu.nix

@@ -0,0 +1,31 @@
+{ lib, pkgs, config, ... }:
+with lib;
+let
+  cfg = config.liv.amdgpu;
+in {
+  options.liv.amdgpu = {
+    enable = mkEnableOption "Enable amdgpu drivers";
+  };
+
+  config = mkIf cfg.enable {
+    hardware = {
+      graphics = {
+        enable = true;
+      };
+      enableRedistributableFirmware = true;
+      opengl = {
+        extraPackages = with pkgs; [
+          amdvlk
+        ];
+        # For 32 bit applications as well
+        extraPackages32 = with pkgs; [
+          driversi686Linux.amdvlk
+        ];
+      };
+    };
+
+    environment.systemPackages = with pkgs; [
+      amdvlk
+    ];
+  };
+}

roles/default.nix

@@ -2,6 +2,8 @@
 {
   imports =
        [(import ./laptop.nix)]
+    ++ [(import ./amdgpu.nix)]
+    ++ [(import ./nvidia.nix)]
     ++ [(import ./desktop.nix)]
     ++ [(import ./creative.nix)];
 }