From 440088677b4e6f9527466cbc5b02abf2bf3f863d Mon Sep 17 00:00:00 2001
From: Yegor Timoshenko <yegortimoshenko@gmail.com>
Date: Fri, 22 Dec 2017 02:56:28 +0000
Subject: [PATCH] lenovo: minor cosmetic changes

---
 lenovo/ideapad/z510.nix           |  6 ++----
 lenovo/thinkpad/general-intel.nix |  2 --
 lenovo/thinkpad/general.nix       |  6 ++++--
 lenovo/thinkpad/t410.nix          | 21 ++++++++++++++++-----
 lenovo/thinkpad/t460s.nix         |  8 ++++----
 lenovo/thinkpad/x140e.nix         |  2 +-
 lenovo/thinkpad/x220i-tablet.nix  |  2 +-
 7 files changed, 28 insertions(+), 19 deletions(-)

diff --git a/lenovo/ideapad/z510.nix b/lenovo/ideapad/z510.nix
index 4c45cac..6cc6dc6 100644
--- a/lenovo/ideapad/z510.nix
+++ b/lenovo/ideapad/z510.nix
@@ -1,10 +1,8 @@
-# NOTE: this doesn't inherit from the `general.nix`
-# as z510 is not a ThinkPad
-
 { config, pkgs, ... }:
+
 {
   hardware.cpu.intel.updateMicrocode = true;
-  
+
   # see https://github.com/NixOS/nixpkgs/issues/18356
   # found buggy driver with method https://wiki.ubuntu.com/DebuggingKernelSuspend
   boot.blacklistedKernelModules = [ "nouveau" ];
diff --git a/lenovo/thinkpad/general-intel.nix b/lenovo/thinkpad/general-intel.nix
index 5aeec18..5b94124 100644
--- a/lenovo/thinkpad/general-intel.nix
+++ b/lenovo/thinkpad/general-intel.nix
@@ -1,5 +1,3 @@
-{ ... }:
-
 {
   boot.kernelModules = mkDefault [ "kvm-intel" ];
   services.xserver.videoDrivers = [ "intel" ];
diff --git a/lenovo/thinkpad/general.nix b/lenovo/thinkpad/general.nix
index 71db23a..00f3c3a 100644
--- a/lenovo/thinkpad/general.nix
+++ b/lenovo/thinkpad/general.nix
@@ -1,6 +1,8 @@
-{ pkgs, lib, ... }:
+{ lib, pkgs, ... }:
 
-with lib;
+let
+  inherit (lib) mkDefault;
+in
 
 {
   hardware.trackpoint = mkDefault {
diff --git a/lenovo/thinkpad/t410.nix b/lenovo/thinkpad/t410.nix
index 5737cc6..53c731f 100644
--- a/lenovo/thinkpad/t410.nix
+++ b/lenovo/thinkpad/t410.nix
@@ -6,11 +6,22 @@
   boot = {
     kernelParams = [
       # Kernel GPU Savings Options (NOTE i915 chipset only)
-      "drm.debug=0" "drm.vblankoffdelay=1" "i915.semaphores=1" "i915.modeset=1"
-      "i915.use_mmio_flip=1" "i915.powersave=1" "i915.enable_ips=1"
-      "i915.disable_power_well=1" "i915.enable_hangcheck=1"
-      "i915.enable_cmd_parser=1" "i915.fastboot=0" "i915.enable_ppgtt=1"
-      "i915.reset=0" "i915.lvds_use_ssc=0" "i915.enable_psr=0" "vblank_mode=0"
+      "drm.debug=0"
+      "drm.vblankoffdelay=1"
+      "i915.semaphores=1"
+      "i915.modeset=1"
+      "i915.use_mmio_flip=1"
+      "i915.powersave=1"
+      "i915.enable_ips=1"
+      "i915.disable_power_well=1"
+      "i915.enable_hangcheck=1"
+      "i915.enable_cmd_parser=1"
+      "i915.fastboot=0"
+      "i915.enable_ppgtt=1"
+      "i915.reset=0"
+      "i915.lvds_use_ssc=0"
+      "i915.enable_psr=0"
+      "vblank_mode=0"
       "i915.i915_enable_rc6=1"
     ];
     blacklistedKernelModules = [
diff --git a/lenovo/thinkpad/t460s.nix b/lenovo/thinkpad/t460s.nix
index d6f2ba3..dc09fd2 100644
--- a/lenovo/thinkpad/t460s.nix
+++ b/lenovo/thinkpad/t460s.nix
@@ -1,10 +1,10 @@
 { config, pkgs, ... }:
 
 {
-  imports =
-    [ ../lib/kernel-version.nix
-      ./general-intel.nix
-    ];
+  imports = [
+    ../lib/kernel-version.nix
+    ./general-intel.nix
+  ];
 
   ## BEGIN from generated hardware-configuration
   ## Probably better to just use a freshly generated hardware.configuration.nix
diff --git a/lenovo/thinkpad/x140e.nix b/lenovo/thinkpad/x140e.nix
index df1f7f3..efa5106 100644
--- a/lenovo/thinkpad/x140e.nix
+++ b/lenovo/thinkpad/x140e.nix
@@ -15,5 +15,5 @@
   };
 
   # video card
-  services.xserver.videoDrivers = ["ati"];
+  services.xserver.videoDrivers = [ "ati" ];
 }
diff --git a/lenovo/thinkpad/x220i-tablet.nix b/lenovo/thinkpad/x220i-tablet.nix
index c87df3f..9f9638a 100644
--- a/lenovo/thinkpad/x220i-tablet.nix
+++ b/lenovo/thinkpad/x220i-tablet.nix
@@ -8,7 +8,7 @@
 
   boot = {
     kernelModules = [ "tp_smapi" ];
-    extraModulePackages = [ config.boot.kernelPackages.tp_smapi ];
+    extraModulePackages = with config.boot.kernelPackages; [ tp_smapi ];
   };
 
   # hard disk protection if the laptop falls