From ef811636cc847355688804593282078bac7758d4 Mon Sep 17 00:00:00 2001
From: Lena Fuhrimann <6780471+cloudlena@users.noreply.github.com>
Date: Sat, 13 Jan 2024 12:58:05 +0100
Subject: [PATCH] Add fwupd to Dell XPS 13 7390 config

---
 dell/xps/13-7390/README.wiki | 8 +-------
 dell/xps/13-7390/default.nix | 6 +++++-
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/dell/xps/13-7390/README.wiki b/dell/xps/13-7390/README.wiki
index 64b8af6..c16f342 100644
--- a/dell/xps/13-7390/README.wiki
+++ b/dell/xps/13-7390/README.wiki
@@ -17,10 +17,4 @@ to your <code>configuration.nix</code> if you cannot wake from "Sleep". See refe
 == Firmware upgrades ==
 
 Note that this device is supported by [https://fwupd.org/ fwupd].
-To perform firmware upgrades just activate the service
-
-<code>
-services.fwupd.enable = true;
-</code>
-
-Then use <code>fwupdmgr</code> to perform updates.
+To perform updates, simply use <code>fwupdmgr</code>.
diff --git a/dell/xps/13-7390/default.nix b/dell/xps/13-7390/default.nix
index 1aa0c4b..1f1c456 100644
--- a/dell/xps/13-7390/default.nix
+++ b/dell/xps/13-7390/default.nix
@@ -13,5 +13,9 @@
   # - https://github.com/NixOS/nixos-hardware/issues/173
   boot.kernelPackages = lib.mkIf (lib.versionOlder pkgs.linux.version "5.6") pkgs.linuxPackages_latest;
 
-  services.thermald.enable = true;
+  # Cooling management
+  services.thermald.enable = lib.mkDefault true;
+
+  # Allows for updating firmware via `fwupdmgr`.
+  services.fwupd.enable = lib.mkDefault true;
 }