treewide: mark things that have to be done

Author: lukateras

Diff for: acer/aspire/4810t/default.nix

@@ -14,14 +14,15 @@
       "vga=0x317"
       "video=vesafb:ywrap"
 
-      # Important, to disable Kernel Mode Setting for the graphics card
-      # This will allow backlight regulation
+      # Important, disable KMS to fix backlight regulation:
       "nomodeset"
     ];
   };
 
+  # TODO: reverse compat
   hardware.opengl.driSupport = false;
 
+  # TODO: reverse compat
   services.xserver = {
     defaultDepth = lib.mkDefault 24;
   };

Diff for: apple/macbook-pro/10-1/default.nix

@@ -10,7 +10,7 @@
   boot.loader.systemd-boot.enable = true;
   boot.loader.efi.canTouchEfiVariables = true;
 
-  # TODO: drop
+  # TODO: reverse compat
   hardware.opengl.driSupport32Bit = true;
 
   services.xserver = {

Diff for: audio-gd/compass2.nix

@@ -1,8 +1,6 @@
-{ config, pkgs, ... }:
-
 {
   boot = {
-    extraModprobeConfig = ''
+    extraModprobeConfig = lib.mkDefault ''
       options snd slots=snd_usb_audio,snd-hda-intel
     '';
   };

Diff for: lenovo/thinkpad/t410/default.nix

@@ -8,6 +8,8 @@
   ];
 
   boot = {
+    # TODO: this configuration seems to be very aggressive.
+    # Ask @peti if it's stable or not.
     kernelParams = [
       "drm.debug=0"
       "drm.vblankoffdelay=1"

Diff for: lenovo/thinkpad/t440p/default.nix

@@ -10,6 +10,7 @@
     extraModprobeConfig = lib.mkDefault ''
       options bbswitch use_acpi_to_detect_card_state=1
     '';
+    # TODO: probably enable tcsd? Is this line necessary?
     kernelModules = [ "tpm-rng" ];
   };
 }

Diff for: lenovo/thinkpad/x220/default.nix

@@ -4,7 +4,7 @@
   imports = [
     ../.
     ../../../common/cpu/intel
-    ../../../common/pc/laptop/hdd
+    ../../../common/pc/laptop/hdd # TODO: reverse compat
     ../tp-smapi.nix
   ];
 }

Diff for: microsoft/surface-pro/3/default.nix

@@ -4,5 +4,6 @@
   # to use the type cover in the initrd
   boot.kernelModules = [ "hid-microsoft" ];
 
+  # TODO: reverse compat
   networking.wireless.enable = lib.mkDefault true;
 }

Diff for: samsung/np900x3c/default.nix

@@ -1,3 +1,5 @@
+# TODO: use ../../common/pc/laptop
+
 { lib, ... }:
 
 {