From 1b0b84259fe01ff150c9483f3af3e5fb789fd47c Mon Sep 17 00:00:00 2001 From: Guillaume Cugnet Date: Wed, 18 Mar 2026 10:21:30 +0100 Subject: [PATCH] framework/12th-gen-intel: fix blacklistedKernelModules using lib.mkIf lib.mkIf returns an attrset, not a string, so it cannot be used as a list element. Replace with lib.optional. Fixes: error: expected a string but found a set: { _type = "if"; ... } --- framework/13-inch/12th-gen-intel/default.nix | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/framework/13-inch/12th-gen-intel/default.nix b/framework/13-inch/12th-gen-intel/default.nix index f8ff0c01..c2707f5d 100644 --- a/framework/13-inch/12th-gen-intel/default.nix +++ b/framework/13-inch/12th-gen-intel/default.nix @@ -25,10 +25,10 @@ # This enables the brightness and airplane mode keys to work # https://community.frame.work/t/12th-gen-not-sending-xf86monbrightnessup-down/20605/11 "hid-sensor-hub" - # This fixes controller crashes during sleep - # https://community.frame.work/t/tracking-fn-key-stops-working-on-popos-after-a-while/21208/32 - (lib.mkIf (config.hardware.framework.enableKmod == false) "cros_ec_lpcs") - ]; + ] + # This fixes controller crashes during sleep + # https://community.frame.work/t/tracking-fn-key-stops-working-on-popos-after-a-while/21208/32 + ++ lib.optional (config.hardware.framework.enableKmod == false) "cros_ec_lpcs"; boot.kernelParams = [ # For Power consumption