Update v5.4 patches
Changes: - Move surface_hotplug driver to own repository. - SAM: - Clean up code, update documentation Links: - SAM:aaad476104
- hotplug:fcc8c8740b
- kernel:734db2d782
This commit is contained in:
parent
bebac23f28
commit
9c6d2f704d
|
@ -1,4 +1,4 @@
|
||||||
From c4a532b01ec70f86cf9b9bedc3204dfc1c540d4f Mon Sep 17 00:00:00 2001
|
From 52c2874d22fc6557a35645b70d372c0fad4f4daf Mon Sep 17 00:00:00 2001
|
||||||
From: qzed <qzed@users.noreply.github.com>
|
From: qzed <qzed@users.noreply.github.com>
|
||||||
Date: Tue, 17 Sep 2019 17:17:56 +0200
|
Date: Tue, 17 Sep 2019 17:17:56 +0200
|
||||||
Subject: [PATCH] platform/x86: Surface 3 battery platform operation region
|
Subject: [PATCH] platform/x86: Surface 3 battery platform operation region
|
||||||
|
|
|
@ -1,4 +1,4 @@
|
||||||
From 61cb0d8cd2357c097d2677e7750a84cd8b5586a8 Mon Sep 17 00:00:00 2001
|
From 5da9a0d8e0cc445f15e63ee059d5300f370ab9bb Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Sun, 18 Oct 2020 16:42:44 +0900
|
Date: Sun, 18 Oct 2020 16:42:44 +0900
|
||||||
Subject: [PATCH] (surface3-oemb) add DMI matches for Surface 3 with broken DMI
|
Subject: [PATCH] (surface3-oemb) add DMI matches for Surface 3 with broken DMI
|
||||||
|
|
|
@ -1,4 +1,4 @@
|
||||||
From 3af00ec1d18257ce796983eed0300f2004218577 Mon Sep 17 00:00:00 2001
|
From e29776ef0efa91f4df58fbbfc1df9ef97e1b603d Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Thu, 24 Sep 2020 18:02:06 +0900
|
Date: Thu, 24 Sep 2020 18:02:06 +0900
|
||||||
Subject: [PATCH] mwifiex: pcie: skip cancel_work_sync() on reset failure path
|
Subject: [PATCH] mwifiex: pcie: skip cancel_work_sync() on reset failure path
|
||||||
|
@ -156,7 +156,7 @@ index f7ce9b6db6b4..72d0c01ff359 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From d45b400ba9f478a658e707e3b67da8ae7df8d4d1 Mon Sep 17 00:00:00 2001
|
From 51b872674a148db8adf5d51721419fd4fd98f0cc Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Mon, 28 Sep 2020 17:46:49 +0900
|
Date: Mon, 28 Sep 2020 17:46:49 +0900
|
||||||
Subject: [PATCH] mwifiex: pcie: add DMI-based quirk impl for Surface devices
|
Subject: [PATCH] mwifiex: pcie: add DMI-based quirk impl for Surface devices
|
||||||
|
@ -364,7 +364,7 @@ index 000000000000..5326ae7e5671
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From 050654a714fdcd741635bc0ea0491654571d27a5 Mon Sep 17 00:00:00 2001
|
From 2a65d4ec8d47f91a1c11ebded54ffe51e5a58a5d Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Tue, 29 Sep 2020 17:25:22 +0900
|
Date: Tue, 29 Sep 2020 17:25:22 +0900
|
||||||
Subject: [PATCH] mwifiex: pcie: add reset_d3cold quirk for Surface gen4+
|
Subject: [PATCH] mwifiex: pcie: add reset_d3cold quirk for Surface gen4+
|
||||||
|
@ -565,7 +565,7 @@ index 5326ae7e5671..8b9dcb5070d8 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From accc6c892c0e450aaa911e9e4ebd8158083e7e0b Mon Sep 17 00:00:00 2001
|
From bce45dfcd08b9080b9496d2c6c44f5aae9c23dfc Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Tue, 29 Sep 2020 17:32:22 +0900
|
Date: Tue, 29 Sep 2020 17:32:22 +0900
|
||||||
Subject: [PATCH] mwifiex: pcie: add reset_wsid quirk for Surface 3
|
Subject: [PATCH] mwifiex: pcie: add reset_wsid quirk for Surface 3
|
||||||
|
@ -744,7 +744,7 @@ index 8b9dcb5070d8..3ef7440418e3 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From a2d4cbba0b60bd83eb6027c33a265511c88916d9 Mon Sep 17 00:00:00 2001
|
From fd6bd9437b7080a34c6f3895b3a7ce9031ddbbc0 Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Wed, 30 Sep 2020 18:08:24 +0900
|
Date: Wed, 30 Sep 2020 18:08:24 +0900
|
||||||
Subject: [PATCH] mwifiex: pcie: (OEMB) add quirk for Surface 3 with broken DMI
|
Subject: [PATCH] mwifiex: pcie: (OEMB) add quirk for Surface 3 with broken DMI
|
||||||
|
@ -806,7 +806,7 @@ index f0a6fa0a7ae5..34dcd84f02a6 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From 8b6918e998bc1eaf86c7ef98c4992318a7e4902b Mon Sep 17 00:00:00 2001
|
From f7622047c79ecdf22102c12354c0953ee5f8fe6d Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Thu, 24 Sep 2020 01:56:29 +0900
|
Date: Thu, 24 Sep 2020 01:56:29 +0900
|
||||||
Subject: [PATCH] mwifiex: fix mwifiex_shutdown_sw() causing sw reset failure
|
Subject: [PATCH] mwifiex: fix mwifiex_shutdown_sw() causing sw reset failure
|
||||||
|
@ -883,7 +883,7 @@ index d14e55e3c9da..5894566ec480 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From faf56fcc7e8efdb3c819e93b3971a16444dfca4b Mon Sep 17 00:00:00 2001
|
From d5d2ce93e71d797606729289a8ffcfe4064bbaf9 Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Thu, 24 Sep 2020 01:56:34 +0900
|
Date: Thu, 24 Sep 2020 01:56:34 +0900
|
||||||
Subject: [PATCH] mwifiex: pcie: use shutdown_sw()/reinit_sw() on
|
Subject: [PATCH] mwifiex: pcie: use shutdown_sw()/reinit_sw() on
|
||||||
|
@ -1025,7 +1025,7 @@ index 263d918767bd..bd6791dc3a0f 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From d744decd5b3a73945d0b0b22084df57c0bf321bb Mon Sep 17 00:00:00 2001
|
From 2a4b3c5e7cf6eed56279041ce122e5b2e5edcb55 Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Mon, 24 Aug 2020 17:11:35 +0900
|
Date: Mon, 24 Aug 2020 17:11:35 +0900
|
||||||
Subject: [PATCH] mwifiex: pcie: add enable_device_dump module parameter
|
Subject: [PATCH] mwifiex: pcie: add enable_device_dump module parameter
|
||||||
|
@ -1074,7 +1074,7 @@ index bd6791dc3a0f..d7ff898c1767 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From 3e6912a7af65f5e6de84fca22ed86d30c901422d Mon Sep 17 00:00:00 2001
|
From d928e920630da4c59074377d41056b0ff95f3572 Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Sun, 4 Oct 2020 00:11:49 +0900
|
Date: Sun, 4 Oct 2020 00:11:49 +0900
|
||||||
Subject: [PATCH] mwifiex: pcie: disable bridge_d3 for Surface gen4+
|
Subject: [PATCH] mwifiex: pcie: disable bridge_d3 for Surface gen4+
|
||||||
|
@ -1229,7 +1229,7 @@ index 3ef7440418e3..a95ebac06e13 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From 091916139c8f402926e816771b0e0fb7bfc1e87a Mon Sep 17 00:00:00 2001
|
From 6979ff4d83954a9d17bed4928ab9cb0e15fdaa6d Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Sun, 4 Oct 2020 00:25:48 +0900
|
Date: Sun, 4 Oct 2020 00:25:48 +0900
|
||||||
Subject: [PATCH] mwifiex: add allow_ps_mode module parameter
|
Subject: [PATCH] mwifiex: add allow_ps_mode module parameter
|
||||||
|
@ -1291,7 +1291,7 @@ index 9e6dc289ec3e..20f5ee3fe7e3 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From 3caeb635372d3d2b3fbe9e8bf6563c52d4143210 Mon Sep 17 00:00:00 2001
|
From e7b042cdef3990529eb9f7ad5556b91058fceada Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Sun, 4 Oct 2020 00:38:48 +0900
|
Date: Sun, 4 Oct 2020 00:38:48 +0900
|
||||||
Subject: [PATCH] mwifiex: print message when changing ps_mode
|
Subject: [PATCH] mwifiex: print message when changing ps_mode
|
||||||
|
@ -1326,7 +1326,7 @@ index 20f5ee3fe7e3..8020a2929069 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From c2becaf4c8c640844ece032ea3daf38d9be21c4e Mon Sep 17 00:00:00 2001
|
From d58d4c6f79e11e795f26b43b5bb620b13f281277 Mon Sep 17 00:00:00 2001
|
||||||
From: Tsuchiya Yuto <kitakar@gmail.com>
|
From: Tsuchiya Yuto <kitakar@gmail.com>
|
||||||
Date: Sun, 4 Oct 2020 00:59:37 +0900
|
Date: Sun, 4 Oct 2020 00:59:37 +0900
|
||||||
Subject: [PATCH] mwifiex: disable ps_mode explicitly by default instead
|
Subject: [PATCH] mwifiex: disable ps_mode explicitly by default instead
|
||||||
|
|
|
@ -1,4 +1,4 @@
|
||||||
From 352847b758c4214c6f71ff8f0f7c84572a7421ae Mon Sep 17 00:00:00 2001
|
From 3e1fa51523c86fc89ff26e59ceca1a7872d775cc Mon Sep 17 00:00:00 2001
|
||||||
From: Dorian Stoll <dorian.stoll@tmsp.io>
|
From: Dorian Stoll <dorian.stoll@tmsp.io>
|
||||||
Date: Mon, 27 Jan 2020 21:16:20 +0100
|
Date: Mon, 27 Jan 2020 21:16:20 +0100
|
||||||
Subject: [PATCH] mei: Add IPTS device IDs
|
Subject: [PATCH] mei: Add IPTS device IDs
|
||||||
|
@ -53,7 +53,7 @@ index 75ab2ffbf235..78790904d77c 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From ec05c68a677969b6476ab4cd7211195b1abf97e3 Mon Sep 17 00:00:00 2001
|
From 44562d2dccd6e03ea3a90c808d31bb7ad63c0f02 Mon Sep 17 00:00:00 2001
|
||||||
From: Dorian Stoll <dorian.stoll@tmsp.io>
|
From: Dorian Stoll <dorian.stoll@tmsp.io>
|
||||||
Date: Fri, 20 Dec 2019 23:15:58 +0100
|
Date: Fri, 20 Dec 2019 23:15:58 +0100
|
||||||
Subject: [PATCH] uapi: Add MEI bus ID
|
Subject: [PATCH] uapi: Add MEI bus ID
|
||||||
|
@ -79,7 +79,7 @@ index 9a61c28ed3ae..47fc20975245 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From cf6108463298a4dc6e3c9438dad00ffc1e64c992 Mon Sep 17 00:00:00 2001
|
From 96669608c79d3edce6e80cf03747e7eeb4473d80 Mon Sep 17 00:00:00 2001
|
||||||
From: Dorian Stoll <dorian.stoll@tmsp.io>
|
From: Dorian Stoll <dorian.stoll@tmsp.io>
|
||||||
Date: Mon, 27 Jan 2020 21:22:42 +0100
|
Date: Mon, 27 Jan 2020 21:22:42 +0100
|
||||||
Subject: [PATCH] input: Add support for Intel Precise Touch & Stylus
|
Subject: [PATCH] input: Add support for Intel Precise Touch & Stylus
|
||||||
|
|
|
@ -1,4 +1,4 @@
|
||||||
From 3e6ed2b40967944ca27b9bbffda276abd9f618ec Mon Sep 17 00:00:00 2001
|
From 87ce2ab4da170e84491125c5be6eaf1cb646069b Mon Sep 17 00:00:00 2001
|
||||||
From: Maximilian Luz <luzmaximilian@gmail.com>
|
From: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
Date: Sun, 16 Aug 2020 23:39:56 +0200
|
Date: Sun, 16 Aug 2020 23:39:56 +0200
|
||||||
Subject: [PATCH] platform/x86: Add Driver to set up lid GPEs on MS Surface
|
Subject: [PATCH] platform/x86: Add Driver to set up lid GPEs on MS Surface
|
||||||
|
|
|
@ -1,4 +1,4 @@
|
||||||
From 09c9a18f997393f1eaff9eae7302b7a72a70affa Mon Sep 17 00:00:00 2001
|
From a73578ba1e914ddbc3f4f8ce3bc81c8f3711e4e3 Mon Sep 17 00:00:00 2001
|
||||||
From: Maximilian Luz <luzmaximilian@gmail.com>
|
From: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
Date: Sat, 25 Jul 2020 17:19:53 +0200
|
Date: Sat, 25 Jul 2020 17:19:53 +0200
|
||||||
Subject: [PATCH] i2c: acpi: Implement RawBytes read access
|
Subject: [PATCH] i2c: acpi: Implement RawBytes read access
|
||||||
|
@ -109,7 +109,7 @@ index c70983780ae7..1c90651161a6 100644
|
||||||
--
|
--
|
||||||
2.29.2
|
2.29.2
|
||||||
|
|
||||||
From ae9241298ad53418b7d858f761a825f186ea55c8 Mon Sep 17 00:00:00 2001
|
From 9d32382041fc72fc26fef7eda4c22cc0643b6944 Mon Sep 17 00:00:00 2001
|
||||||
From: Maximilian Luz <luzmaximilian@gmail.com>
|
From: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
Date: Sun, 6 Sep 2020 04:01:19 +0200
|
Date: Sun, 6 Sep 2020 04:01:19 +0200
|
||||||
Subject: [PATCH] platform/x86: Add driver for Surface Book 1 dGPU switch
|
Subject: [PATCH] platform/x86: Add driver for Surface Book 1 dGPU switch
|
||||||
|
|
File diff suppressed because it is too large
Load diff
550
patches/5.4/0008-surface-hotplug.patch
Normal file
550
patches/5.4/0008-surface-hotplug.patch
Normal file
|
@ -0,0 +1,550 @@
|
||||||
|
From 9f137648cd3fe0ab12f824140cda963933f9e3ce Mon Sep 17 00:00:00 2001
|
||||||
|
From: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
|
Date: Thu, 29 Oct 2020 22:04:38 +0100
|
||||||
|
Subject: [PATCH] PCI: Allow D3cold for hot-plug ports on Surface Books
|
||||||
|
|
||||||
|
The Microsoft Surface Book series of devices have a tablet part (so
|
||||||
|
called clipboard) that can be detached from the base of the device.
|
||||||
|
While the clipboard contains the CPU, the base can contain a discrete
|
||||||
|
GPU (dGPU). This dGPU is connected via a PCIe hot-plug port.
|
||||||
|
|
||||||
|
Currently D3cold is disallowed for all hot-plug ports. On the Surface
|
||||||
|
Book 2 and 3, this leads to increased power consumption during suspend
|
||||||
|
and when the dGPU is not used (i.e. runtime suspended). This can be
|
||||||
|
observed not only in battery drain, but also by the dGPU getting notably
|
||||||
|
warm while suspended and not in D3cold.
|
||||||
|
|
||||||
|
Testing shows that the Surface Books behave well with D3cold enabled for
|
||||||
|
hot-plug ports, alleviating the aforementioned issues. Thus white-list
|
||||||
|
D3cold for hot-plug ports on those devices.
|
||||||
|
|
||||||
|
Note: PCIe hot-plug signalling while the device is in D3cold is handled
|
||||||
|
via ACPI, out-of-band interrupts, and the surface_hotplug driver
|
||||||
|
(combined). The device will work without the surface_hotplug driver,
|
||||||
|
however, device removal/addition will only be detected on device resume.
|
||||||
|
|
||||||
|
Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
|
Patchset: surface-hotplug
|
||||||
|
---
|
||||||
|
drivers/pci/pci.c | 31 +++++++++++++++++++++++++++++--
|
||||||
|
1 file changed, 29 insertions(+), 2 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
|
||||||
|
index b1b2c8ddbc92..15566ec8f75d 100644
|
||||||
|
--- a/drivers/pci/pci.c
|
||||||
|
+++ b/drivers/pci/pci.c
|
||||||
|
@@ -2611,6 +2611,32 @@ static const struct dmi_system_id bridge_d3_blacklist[] = {
|
||||||
|
{ }
|
||||||
|
};
|
||||||
|
|
||||||
|
+static const struct dmi_system_id bridge_d3_hotplug_whitelist[] = {
|
||||||
|
+#ifdef CONFIG_X86
|
||||||
|
+ {
|
||||||
|
+ /*
|
||||||
|
+ * Microsoft Surface Books have a hot-plug root port for the
|
||||||
|
+ * discrete GPU (the device containing it can be detached form
|
||||||
|
+ * the top-part, containing the cpu).
|
||||||
|
+ *
|
||||||
|
+ * If this discrete GPU is not transitioned into D3cold for
|
||||||
|
+ * suspend, the device will become notably warm and also
|
||||||
|
+ * consume a lot more power than desirable.
|
||||||
|
+ *
|
||||||
|
+ * We assume that since those devices have been confirmed
|
||||||
|
+ * working with D3, future Surface devices will too. So let's
|
||||||
|
+ * keep this match generic.
|
||||||
|
+ */
|
||||||
|
+ .ident = "Microsoft Surface",
|
||||||
|
+ .matches = {
|
||||||
|
+ DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Microsoft Corporation"),
|
||||||
|
+ DMI_MATCH(DMI_PRODUCT_NAME, "Surface"),
|
||||||
|
+ },
|
||||||
|
+ },
|
||||||
|
+#endif
|
||||||
|
+ { }
|
||||||
|
+};
|
||||||
|
+
|
||||||
|
/**
|
||||||
|
* pci_bridge_d3_possible - Is it possible to put the bridge into D3
|
||||||
|
* @bridge: Bridge to check
|
||||||
|
@@ -2651,10 +2677,11 @@ bool pci_bridge_d3_possible(struct pci_dev *bridge)
|
||||||
|
/*
|
||||||
|
* Hotplug ports handled natively by the OS were not validated
|
||||||
|
* by vendors for runtime D3 at least until 2018 because there
|
||||||
|
- * was no OS support.
|
||||||
|
+ * was no OS support. Explicitly whitelist systems that have
|
||||||
|
+ * been confirmed working.
|
||||||
|
*/
|
||||||
|
if (bridge->is_hotplug_bridge)
|
||||||
|
- return false;
|
||||||
|
+ return dmi_check_system(bridge_d3_hotplug_whitelist);
|
||||||
|
|
||||||
|
if (dmi_check_system(bridge_d3_blacklist))
|
||||||
|
return false;
|
||||||
|
--
|
||||||
|
2.29.2
|
||||||
|
|
||||||
|
From a6ed52ecf177d01eda912e252c51dbaae6007488 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
|
Date: Mon, 9 Nov 2020 14:23:00 +0100
|
||||||
|
Subject: [PATCH] PCI: Run platform power transition on initial D0 entry
|
||||||
|
|
||||||
|
On some devices and platforms, the initial platform power state is not
|
||||||
|
in sync with the power state of the PCI device.
|
||||||
|
|
||||||
|
pci_enable_device_flags() updates the state of a PCI device by reading
|
||||||
|
from the the PCI_PM_CTRL register. This may change the stored power
|
||||||
|
state of the device without running the appropriate platform power
|
||||||
|
transition.
|
||||||
|
|
||||||
|
Due to the stored power-state being changed, the later call to
|
||||||
|
pci_set_power_state(..., PCI_D0) in do_pci_enable_device() can evaluate
|
||||||
|
to a no-op if the stored state has been changed to D0 via that. This
|
||||||
|
will then prevent the appropriate platform power transition to be run,
|
||||||
|
which can on some devices and platforms lead to platform and PCI power
|
||||||
|
state being entirely different, i.e. out-of-sync. On ACPI platforms,
|
||||||
|
this can lead to power resources not being turned on, even though they
|
||||||
|
are marked as required for D0.
|
||||||
|
|
||||||
|
Specifically, on the Microsoft Surface Book 2 and 3, some ACPI power
|
||||||
|
regions that should be "on" for the D0 state (and others) are
|
||||||
|
initialized as "off" in ACPI, whereas the PCI device is in D0. As the
|
||||||
|
state is updated in pci_enable_device_flags() without ensuring that the
|
||||||
|
platform state is also updated, the power resource will never be
|
||||||
|
properly turned on. Instead, it lives in a sort of on-but-marked-as-off
|
||||||
|
zombie-state, which confuses things down the line when attempting to
|
||||||
|
transition the device into D3cold: As the resource is already marked as
|
||||||
|
off, it won't be turned off and the device does not fully enter D3cold,
|
||||||
|
causing increased power consumption during (runtime-)suspend.
|
||||||
|
|
||||||
|
By replacing pci_set_power_state() in do_pci_enable_device() with
|
||||||
|
pci_power_up(), we can force pci_platform_power_transition() to be
|
||||||
|
called, which will then check if the platform power state needs updating
|
||||||
|
and appropriate actions need to be taken.
|
||||||
|
|
||||||
|
Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
|
Patchset: surface-hotplug
|
||||||
|
---
|
||||||
|
drivers/pci/pci.c | 4 +---
|
||||||
|
1 file changed, 1 insertion(+), 3 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
|
||||||
|
index 15566ec8f75d..9b0a591fc60b 100644
|
||||||
|
--- a/drivers/pci/pci.c
|
||||||
|
+++ b/drivers/pci/pci.c
|
||||||
|
@@ -1596,9 +1596,7 @@ static int do_pci_enable_device(struct pci_dev *dev, int bars)
|
||||||
|
u16 cmd;
|
||||||
|
u8 pin;
|
||||||
|
|
||||||
|
- err = pci_set_power_state(dev, PCI_D0);
|
||||||
|
- if (err < 0 && err != -EIO)
|
||||||
|
- return err;
|
||||||
|
+ pci_power_up(dev);
|
||||||
|
|
||||||
|
bridge = pci_upstream_bridge(dev);
|
||||||
|
if (bridge)
|
||||||
|
--
|
||||||
|
2.29.2
|
||||||
|
|
||||||
|
From 2137639d4ed25d4d5535c8648ab17c23f60f6412 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
|
Date: Sat, 31 Oct 2020 20:46:33 +0100
|
||||||
|
Subject: [PATCH] PCI: Add sysfs attribute for PCI device power state
|
||||||
|
|
||||||
|
While most PCI power-states can be queried from user-space via lspci,
|
||||||
|
this has some limits. Specifically, lspci fails to provide an accurate
|
||||||
|
value when the device is in D3cold as it has to resume the device before
|
||||||
|
it can access its power state via the configuration space, leading to it
|
||||||
|
reporting D0 or another on-state. Thus lspci can, for example, not be
|
||||||
|
used to diagnose power-consumption issues for devices that can enter
|
||||||
|
D3cold or to ensure that devices properly enter D3cold at all.
|
||||||
|
|
||||||
|
To alleviate this issue, introduce a new sysfs device attribute for the
|
||||||
|
PCI power state, showing the current power state as seen by the kernel.
|
||||||
|
|
||||||
|
Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
|
Patchset: surface-hotplug
|
||||||
|
---
|
||||||
|
Documentation/ABI/testing/sysfs-bus-pci | 9 +++++++++
|
||||||
|
drivers/pci/pci-sysfs.c | 12 ++++++++++++
|
||||||
|
2 files changed, 21 insertions(+)
|
||||||
|
|
||||||
|
diff --git a/Documentation/ABI/testing/sysfs-bus-pci b/Documentation/ABI/testing/sysfs-bus-pci
|
||||||
|
index 8bfee557e50e..460032b4e950 100644
|
||||||
|
--- a/Documentation/ABI/testing/sysfs-bus-pci
|
||||||
|
+++ b/Documentation/ABI/testing/sysfs-bus-pci
|
||||||
|
@@ -347,3 +347,12 @@ Description:
|
||||||
|
If the device has any Peer-to-Peer memory registered, this
|
||||||
|
file contains a '1' if the memory has been published for
|
||||||
|
use outside the driver that owns the device.
|
||||||
|
+
|
||||||
|
+What: /sys/bus/pci/devices/.../power_state
|
||||||
|
+Date: November 2020
|
||||||
|
+Contact: Linux PCI developers <linux-pci@vger.kernel.org>
|
||||||
|
+Description:
|
||||||
|
+ This file contains the current PCI power state of the device.
|
||||||
|
+ The value comes from the PCI kernel device state and can be one
|
||||||
|
+ of: "unknown", "error", "D0", D1", "D2", "D3hot", "D3cold".
|
||||||
|
+ The file is read only.
|
||||||
|
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c
|
||||||
|
index e401f040f157..418927872ae6 100644
|
||||||
|
--- a/drivers/pci/pci-sysfs.c
|
||||||
|
+++ b/drivers/pci/pci-sysfs.c
|
||||||
|
@@ -124,6 +124,17 @@ static ssize_t cpulistaffinity_show(struct device *dev,
|
||||||
|
}
|
||||||
|
static DEVICE_ATTR_RO(cpulistaffinity);
|
||||||
|
|
||||||
|
+/* PCI power state */
|
||||||
|
+static ssize_t power_state_show(struct device *dev,
|
||||||
|
+ struct device_attribute *attr, char *buf)
|
||||||
|
+{
|
||||||
|
+ struct pci_dev *pci_dev = to_pci_dev(dev);
|
||||||
|
+ pci_power_t state = READ_ONCE(pci_dev->current_state);
|
||||||
|
+
|
||||||
|
+ return sprintf(buf, "%s\n", pci_power_name(state));
|
||||||
|
+}
|
||||||
|
+static DEVICE_ATTR_RO(power_state);
|
||||||
|
+
|
||||||
|
/* show resources */
|
||||||
|
static ssize_t resource_show(struct device *dev, struct device_attribute *attr,
|
||||||
|
char *buf)
|
||||||
|
@@ -598,6 +609,7 @@ static ssize_t driver_override_show(struct device *dev,
|
||||||
|
static DEVICE_ATTR_RW(driver_override);
|
||||||
|
|
||||||
|
static struct attribute *pci_dev_attrs[] = {
|
||||||
|
+ &dev_attr_power_state.attr,
|
||||||
|
&dev_attr_resource.attr,
|
||||||
|
&dev_attr_vendor.attr,
|
||||||
|
&dev_attr_device.attr,
|
||||||
|
--
|
||||||
|
2.29.2
|
||||||
|
|
||||||
|
From 1c9053c2ce86a2c9f9906ee98101202682f058e1 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
|
Date: Mon, 14 Dec 2020 20:50:59 +0100
|
||||||
|
Subject: [PATCH] platform/x86: Add Surface Hotplug driver
|
||||||
|
|
||||||
|
Add a driver to handle out-of-band hot-plug signaling for the discrete
|
||||||
|
GPU (dGPU) on Microsoft Surface Book 2 and 3 devices. This driver is
|
||||||
|
required to properly detect hot-plugging of the dGPU and relay the
|
||||||
|
appropriate signal to the PCIe hot-plug driver core.
|
||||||
|
|
||||||
|
Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
|
Patchset: surface-hotplug
|
||||||
|
---
|
||||||
|
drivers/platform/x86/Kconfig | 12 ++
|
||||||
|
drivers/platform/x86/Makefile | 1 +
|
||||||
|
drivers/platform/x86/surface_hotplug.c | 267 +++++++++++++++++++++++++
|
||||||
|
3 files changed, 280 insertions(+)
|
||||||
|
create mode 100644 drivers/platform/x86/surface_hotplug.c
|
||||||
|
|
||||||
|
diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
|
||||||
|
index f06a0bf92890..3353aa9a9617 100644
|
||||||
|
--- a/drivers/platform/x86/Kconfig
|
||||||
|
+++ b/drivers/platform/x86/Kconfig
|
||||||
|
@@ -756,6 +756,18 @@ config ACPI_WMI
|
||||||
|
It is safe to enable this driver even if your DSDT doesn't define
|
||||||
|
any ACPI-WMI devices.
|
||||||
|
|
||||||
|
+config SURFACE_HOTPLUG
|
||||||
|
+ tristate "Surface Hot-Plug System Driver"
|
||||||
|
+ depends on ACPI
|
||||||
|
+ default m
|
||||||
|
+ help
|
||||||
|
+ Driver for the Surface discrete GPU (dGPU) hot-plug system.
|
||||||
|
+
|
||||||
|
+ This driver provides support for out-of-band hot-plug event signaling
|
||||||
|
+ on Surface Book 2 and 3 devices. This out-of-band signaling is
|
||||||
|
+ required to notify the kernel of any hot-plug events when the dGPU is
|
||||||
|
+ powered off, i.e. in D3cold.
|
||||||
|
+
|
||||||
|
config WMI_BMOF
|
||||||
|
tristate "WMI embedded Binary MOF driver"
|
||||||
|
depends on ACPI_WMI
|
||||||
|
diff --git a/drivers/platform/x86/Makefile b/drivers/platform/x86/Makefile
|
||||||
|
index a1e70973257c..7a6b5ce38cc0 100644
|
||||||
|
--- a/drivers/platform/x86/Makefile
|
||||||
|
+++ b/drivers/platform/x86/Makefile
|
||||||
|
@@ -87,6 +87,7 @@ obj-$(CONFIG_SURFACE_PRO3_BUTTON) += surfacepro3_button.o
|
||||||
|
obj-$(CONFIG_SURFACE_3_BUTTON) += surface3_button.o
|
||||||
|
obj-$(CONFIG_SURFACE_3_POWER_OPREGION) += surface3_power.o
|
||||||
|
obj-$(CONFIG_SURFACE_GPE) += surface_gpe.o
|
||||||
|
+obj-$(CONFIG_SURFACE_HOTPLUG) += surface_hotplug.o
|
||||||
|
obj-$(CONFIG_SURFACE_BOOK1_DGPU_SWITCH) += sb1_dgpu_sw.o
|
||||||
|
obj-$(CONFIG_INTEL_PUNIT_IPC) += intel_punit_ipc.o
|
||||||
|
obj-$(CONFIG_INTEL_BXTWC_PMIC_TMU) += intel_bxtwc_tmu.o
|
||||||
|
diff --git a/drivers/platform/x86/surface_hotplug.c b/drivers/platform/x86/surface_hotplug.c
|
||||||
|
new file mode 100644
|
||||||
|
index 000000000000..572fba30cd77
|
||||||
|
--- /dev/null
|
||||||
|
+++ b/drivers/platform/x86/surface_hotplug.c
|
||||||
|
@@ -0,0 +1,267 @@
|
||||||
|
+// SPDX-License-Identifier: GPL-2.0+
|
||||||
|
+/*
|
||||||
|
+ * Surface Book (gen. 2 and later) hot-plug driver.
|
||||||
|
+ *
|
||||||
|
+ * Surface Book devices (can) have a hot-pluggable discrete GPU (dGPU). This
|
||||||
|
+ * driver is responsible for out-of-band hot-plug event signaling on these
|
||||||
|
+ * devices. It is specifically required when the hot-plug device is in D3cold
|
||||||
|
+ * and can thus not generate PCIe hot-plug events itself.
|
||||||
|
+ *
|
||||||
|
+ * Event signaling is handled via ACPI, which will generate the appropriate
|
||||||
|
+ * device-check notifications to be picked up by the PCIe hot-plug driver.
|
||||||
|
+ *
|
||||||
|
+ * Copyright (C) 2019-2020 Maximilian Luz <luzmaximilian@gmail.com>
|
||||||
|
+ */
|
||||||
|
+
|
||||||
|
+#include <linux/acpi.h>
|
||||||
|
+#include <linux/gpio.h>
|
||||||
|
+#include <linux/interrupt.h>
|
||||||
|
+#include <linux/kernel.h>
|
||||||
|
+#include <linux/module.h>
|
||||||
|
+#include <linux/mutex.h>
|
||||||
|
+#include <linux/platform_device.h>
|
||||||
|
+
|
||||||
|
+static const struct acpi_gpio_params shps_base_presence_int = { 0, 0, false };
|
||||||
|
+static const struct acpi_gpio_params shps_base_presence = { 1, 0, false };
|
||||||
|
+static const struct acpi_gpio_params shps_device_power_int = { 2, 0, false };
|
||||||
|
+static const struct acpi_gpio_params shps_device_power = { 3, 0, false };
|
||||||
|
+static const struct acpi_gpio_params shps_device_presence_int = { 4, 0, false };
|
||||||
|
+static const struct acpi_gpio_params shps_device_presence = { 5, 0, false };
|
||||||
|
+
|
||||||
|
+static const struct acpi_gpio_mapping shps_acpi_gpios[] = {
|
||||||
|
+ { "base_presence-int-gpio", &shps_base_presence_int, 1 },
|
||||||
|
+ { "base_presence-gpio", &shps_base_presence, 1 },
|
||||||
|
+ { "device_power-int-gpio", &shps_device_power_int, 1 },
|
||||||
|
+ { "device_power-gpio", &shps_device_power, 1 },
|
||||||
|
+ { "device_presence-int-gpio", &shps_device_presence_int, 1 },
|
||||||
|
+ { "device_presence-gpio", &shps_device_presence, 1 },
|
||||||
|
+ { },
|
||||||
|
+};
|
||||||
|
+
|
||||||
|
+/* 5515a847-ed55-4b27-8352-cd320e10360a */
|
||||||
|
+static const guid_t shps_dsm_guid =
|
||||||
|
+ GUID_INIT(0x5515a847, 0xed55, 0x4b27, 0x83, 0x52, 0xcd,
|
||||||
|
+ 0x32, 0x0e, 0x10, 0x36, 0x0a);
|
||||||
|
+
|
||||||
|
+#define SHPS_DSM_REVISION 1
|
||||||
|
+
|
||||||
|
+enum shps_dsm_fn {
|
||||||
|
+ SHPS_DSM_FN_PCI_NUM_ENTRIES = 0x01,
|
||||||
|
+ SHPS_DSM_FN_PCI_GET_ENTRIES = 0x02,
|
||||||
|
+ SHPS_DSM_FN_IRQ_BASE_PRESENCE = 0x03,
|
||||||
|
+ SHPS_DSM_FN_IRQ_DEVICE_POWER = 0x04,
|
||||||
|
+ SHPS_DSM_FN_IRQ_DEVICE_PRESENCE = 0x05,
|
||||||
|
+};
|
||||||
|
+
|
||||||
|
+enum shps_irq_type {
|
||||||
|
+ /* NOTE: Must be in order of DSM function */
|
||||||
|
+ SHPS_IRQ_TYPE_BASE_PRESENCE = 0,
|
||||||
|
+ SHPS_IRQ_TYPE_DEVICE_POWER = 1,
|
||||||
|
+ SHPS_IRQ_TYPE_DEVICE_PRESENCE = 2,
|
||||||
|
+
|
||||||
|
+ SHPS_NUM_IRQS,
|
||||||
|
+};
|
||||||
|
+
|
||||||
|
+static const char *const shps_gpio_names[] = {
|
||||||
|
+ [SHPS_IRQ_TYPE_BASE_PRESENCE] = "base_presence",
|
||||||
|
+ [SHPS_IRQ_TYPE_DEVICE_POWER] = "device_power",
|
||||||
|
+ [SHPS_IRQ_TYPE_DEVICE_PRESENCE] = "device_presence",
|
||||||
|
+};
|
||||||
|
+
|
||||||
|
+struct shps_device {
|
||||||
|
+ struct mutex lock[SHPS_NUM_IRQS];
|
||||||
|
+ struct gpio_desc *gpio[SHPS_NUM_IRQS];
|
||||||
|
+ unsigned int irq[SHPS_NUM_IRQS];
|
||||||
|
+};
|
||||||
|
+
|
||||||
|
+#define SHPS_IRQ_NOT_PRESENT ((unsigned int)-1)
|
||||||
|
+
|
||||||
|
+static void shps_dsm_notify_irq(struct platform_device *pdev,
|
||||||
|
+ enum shps_irq_type type)
|
||||||
|
+{
|
||||||
|
+ struct shps_device *sdev = platform_get_drvdata(pdev);
|
||||||
|
+ acpi_handle handle = ACPI_HANDLE(&pdev->dev);
|
||||||
|
+ union acpi_object *result;
|
||||||
|
+ union acpi_object param;
|
||||||
|
+ int value;
|
||||||
|
+
|
||||||
|
+ mutex_lock(&sdev->lock[type]);
|
||||||
|
+
|
||||||
|
+ value = gpiod_get_value_cansleep(sdev->gpio[type]);
|
||||||
|
+ if (value < 0) {
|
||||||
|
+ mutex_unlock(&sdev->lock[type]);
|
||||||
|
+ dev_err(&pdev->dev, "failed to get gpio: %d (irq=%d)\n",
|
||||||
|
+ type, value);
|
||||||
|
+ return;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ dev_dbg(&pdev->dev, "IRQ notification via DSM (irq=%d, value=%d)\n",
|
||||||
|
+ type, value);
|
||||||
|
+
|
||||||
|
+ param.type = ACPI_TYPE_INTEGER;
|
||||||
|
+ param.integer.value = value;
|
||||||
|
+
|
||||||
|
+ result = acpi_evaluate_dsm(handle, &shps_dsm_guid, SHPS_DSM_REVISION,
|
||||||
|
+ SHPS_DSM_FN_IRQ_BASE_PRESENCE + type, ¶m);
|
||||||
|
+
|
||||||
|
+ if (!result) {
|
||||||
|
+ mutex_unlock(&sdev->lock[type]);
|
||||||
|
+ dev_err(&pdev->dev,
|
||||||
|
+ "IRQ notification via DSM failed (irq=%d, gpio=%d)\n",
|
||||||
|
+ type, value);
|
||||||
|
+ return;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ if (result->type != ACPI_TYPE_BUFFER) {
|
||||||
|
+ dev_err(&pdev->dev,
|
||||||
|
+ "IRQ notification via DSM failed: unexpected result type (irq=%d, gpio=%d)\n",
|
||||||
|
+ type, value);
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ if (result->buffer.length != 1 || result->buffer.pointer[0] != 0) {
|
||||||
|
+ dev_err(&pdev->dev,
|
||||||
|
+ "IRQ notification via DSM failed: unexpected result value (irq=%d, gpio=%d)\n",
|
||||||
|
+ type, value);
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ mutex_unlock(&sdev->lock[type]);
|
||||||
|
+ ACPI_FREE(result);
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+static irqreturn_t shps_handle_irq(int irq, void *data)
|
||||||
|
+{
|
||||||
|
+ struct platform_device *pdev = data;
|
||||||
|
+ struct shps_device *sdev = platform_get_drvdata(pdev);
|
||||||
|
+ int type;
|
||||||
|
+
|
||||||
|
+ /* Figure out which IRQ we're handling. */
|
||||||
|
+ for (type = 0; type < SHPS_NUM_IRQS; type++)
|
||||||
|
+ if (irq == sdev->irq[type])
|
||||||
|
+ break;
|
||||||
|
+
|
||||||
|
+ /* We should have found our interrupt, if not: this is a bug. */
|
||||||
|
+ if (WARN(type >= SHPS_NUM_IRQS, "invalid IRQ number: %d\n", irq))
|
||||||
|
+ return IRQ_HANDLED;
|
||||||
|
+
|
||||||
|
+ /* Forward interrupt to ACPI via DSM. */
|
||||||
|
+ shps_dsm_notify_irq(pdev, type);
|
||||||
|
+ return IRQ_HANDLED;
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+static int shps_setup_irq(struct platform_device *pdev, enum shps_irq_type type)
|
||||||
|
+{
|
||||||
|
+ unsigned long flags = IRQF_ONESHOT | IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING;
|
||||||
|
+ struct shps_device *sdev = platform_get_drvdata(pdev);
|
||||||
|
+ struct gpio_desc *gpiod;
|
||||||
|
+ acpi_handle handle = ACPI_HANDLE(&pdev->dev);
|
||||||
|
+ const char *irq_name;
|
||||||
|
+ const int dsm = SHPS_DSM_FN_IRQ_BASE_PRESENCE + type;
|
||||||
|
+ int status, irq;
|
||||||
|
+
|
||||||
|
+ /* Initialize as "not present". */
|
||||||
|
+ sdev->gpio[type] = NULL;
|
||||||
|
+ sdev->irq[type] = SHPS_IRQ_NOT_PRESENT;
|
||||||
|
+
|
||||||
|
+ /* Only set up interrupts that we actually need. */
|
||||||
|
+ if (!acpi_check_dsm(handle, &shps_dsm_guid, SHPS_DSM_REVISION, BIT(dsm))) {
|
||||||
|
+ dev_dbg(&pdev->dev, "IRQ notification via DSM not present (irq=%d)\n",
|
||||||
|
+ type);
|
||||||
|
+ return 0;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ gpiod = devm_gpiod_get(&pdev->dev, shps_gpio_names[type], GPIOD_ASIS);
|
||||||
|
+ if (IS_ERR(gpiod))
|
||||||
|
+ return PTR_ERR(gpiod);
|
||||||
|
+
|
||||||
|
+ irq = gpiod_to_irq(gpiod);
|
||||||
|
+ if (irq < 0)
|
||||||
|
+ return irq;
|
||||||
|
+
|
||||||
|
+ irq_name = devm_kasprintf(&pdev->dev, GFP_KERNEL, "shps-irq-%d", type);
|
||||||
|
+ if (!irq_name)
|
||||||
|
+ return -ENOMEM;
|
||||||
|
+
|
||||||
|
+ status = devm_request_threaded_irq(&pdev->dev, irq, NULL, shps_handle_irq,
|
||||||
|
+ flags, irq_name, pdev);
|
||||||
|
+ if (status)
|
||||||
|
+ return status;
|
||||||
|
+
|
||||||
|
+ dev_dbg(&pdev->dev, "set up irq %d as type %d\n", irq, type);
|
||||||
|
+
|
||||||
|
+ sdev->gpio[type] = gpiod;
|
||||||
|
+ sdev->irq[type] = irq;
|
||||||
|
+
|
||||||
|
+ return 0;
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+static int surface_hotplug_probe(struct platform_device *pdev)
|
||||||
|
+{
|
||||||
|
+ struct shps_device *sdev;
|
||||||
|
+ int status, i;
|
||||||
|
+
|
||||||
|
+ if (gpiod_count(&pdev->dev, NULL) < 0)
|
||||||
|
+ return -ENODEV;
|
||||||
|
+
|
||||||
|
+ status = devm_acpi_dev_add_driver_gpios(&pdev->dev, shps_acpi_gpios);
|
||||||
|
+ if (status)
|
||||||
|
+ return status;
|
||||||
|
+
|
||||||
|
+ sdev = devm_kzalloc(&pdev->dev, sizeof(*sdev), GFP_KERNEL);
|
||||||
|
+ if (!sdev)
|
||||||
|
+ return -ENOMEM;
|
||||||
|
+
|
||||||
|
+ platform_set_drvdata(pdev, sdev);
|
||||||
|
+
|
||||||
|
+ /* Set up IRQs. */
|
||||||
|
+ for (i = 0; i < SHPS_NUM_IRQS; i++) {
|
||||||
|
+ mutex_init(&sdev->lock[i]);
|
||||||
|
+
|
||||||
|
+ status = shps_setup_irq(pdev, i);
|
||||||
|
+ if (status) {
|
||||||
|
+ dev_err(&pdev->dev, "failed to set up IRQ %d: %d\n",
|
||||||
|
+ i, status);
|
||||||
|
+ return status;
|
||||||
|
+ }
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ /* Ensure everything is up-to-date. */
|
||||||
|
+ for (i = 0; i < SHPS_NUM_IRQS; i++)
|
||||||
|
+ if (sdev->irq[i] != SHPS_IRQ_NOT_PRESENT)
|
||||||
|
+ shps_dsm_notify_irq(pdev, i);
|
||||||
|
+
|
||||||
|
+ return 0;
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+static int surface_hotplug_remove(struct platform_device *pdev)
|
||||||
|
+{
|
||||||
|
+ struct shps_device *sdev = platform_get_drvdata(pdev);
|
||||||
|
+ int i;
|
||||||
|
+
|
||||||
|
+ /* Ensure that IRQs have been fully handled and won't trigger any more. */
|
||||||
|
+ for (i = 0; i < SHPS_NUM_IRQS; i++)
|
||||||
|
+ if (sdev->irq[i] != SHPS_IRQ_NOT_PRESENT)
|
||||||
|
+ disable_irq(sdev->irq[i]);
|
||||||
|
+
|
||||||
|
+ return 0;
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+static const struct acpi_device_id surface_hotplug_acpi_match[] = {
|
||||||
|
+ { "MSHW0153", 0 },
|
||||||
|
+ { },
|
||||||
|
+};
|
||||||
|
+MODULE_DEVICE_TABLE(acpi, surface_hotplug_acpi_match);
|
||||||
|
+
|
||||||
|
+static struct platform_driver surface_hotplug_driver = {
|
||||||
|
+ .probe = surface_hotplug_probe,
|
||||||
|
+ .remove = surface_hotplug_remove,
|
||||||
|
+ .driver = {
|
||||||
|
+ .name = "surface_hotplug",
|
||||||
|
+ .acpi_match_table = surface_hotplug_acpi_match,
|
||||||
|
+ .probe_type = PROBE_PREFER_ASYNCHRONOUS,
|
||||||
|
+ },
|
||||||
|
+};
|
||||||
|
+module_platform_driver(surface_hotplug_driver);
|
||||||
|
+
|
||||||
|
+MODULE_AUTHOR("Maximilian Luz <luzmaximilian@gmail.com>");
|
||||||
|
+MODULE_DESCRIPTION("Surface Hot-Plug Signaling Driver for Surface Book Devices");
|
||||||
|
+MODULE_LICENSE("GPL");
|
||||||
|
--
|
||||||
|
2.29.2
|
||||||
|
|
|
@ -1,4 +1,4 @@
|
||||||
From 3b1684d25ed312de993110c8191ce1ed633f70a1 Mon Sep 17 00:00:00 2001
|
From 158561783d0465c451fffc62d47dc611f8467017 Mon Sep 17 00:00:00 2001
|
||||||
From: =?UTF-8?q?Jonas=20Dre=C3=9Fler?= <verdre@v0yd.nl>
|
From: =?UTF-8?q?Jonas=20Dre=C3=9Fler?= <verdre@v0yd.nl>
|
||||||
Date: Thu, 5 Nov 2020 13:09:45 +0100
|
Date: Thu, 5 Nov 2020 13:09:45 +0100
|
||||||
Subject: [PATCH] hid/multitouch: Turn off Type Cover keyboard backlight when
|
Subject: [PATCH] hid/multitouch: Turn off Type Cover keyboard backlight when
|
Loading…
Reference in a new issue