From 53ca456f6013f3c6762943a172ee0d4cc656e26f Mon Sep 17 00:00:00 2001
From: Tom Bereknyei <tomberek@gmail.com>
Date: Sat, 17 Aug 2019 02:08:08 -0400
Subject: [PATCH] Suggested fixes

---
 dell/xps/15-7590/README.wiki | 3 +++
 dell/xps/15-7590/default.nix | 3 ++-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/dell/xps/15-7590/README.wiki b/dell/xps/15-7590/README.wiki
index b583b19..7c74582 100644
--- a/dell/xps/15-7590/README.wiki
+++ b/dell/xps/15-7590/README.wiki
@@ -37,6 +37,9 @@ Update: The 48.ucode causes the Killer wifi card to crash. The iwlfwifi-cc-a0-46
   boot.loader.efi.canTouchEfiVariables = true;
   boot.kernelPackages = pkgs.linuxPackages_5_1;
 ```
+Disable the `canTouchEfiVariables` after a boot or two to prevent NVRAM wearout.
+
+
 === After installation ===
 
 * ''Add systemd-boot to UEFI boot list.'' The (uneditable anyways) settings mapping drive UUIDs to HD* work fine.
diff --git a/dell/xps/15-7590/default.nix b/dell/xps/15-7590/default.nix
index 4a9030a..0666373 100644
--- a/dell/xps/15-7590/default.nix
+++ b/dell/xps/15-7590/default.nix
@@ -4,10 +4,11 @@
   imports = [
     ../../../common/cpu/intel
     ../../../common/pc/laptop
+    ../../../common/pc/ssd
   ];
 
   # TODO: boot loader
-  boot.loader.systemd-boot.enable = lib.mkDefault true;
+  #boot.loader.systemd-boot.enable = lib.mkDefault true;
   boot.kernelPackages = pkgs.linuxPackages_5_1;
   boot.loader.efi.canTouchEfiVariables = lib.mkDefault true;