[kernel: 77/85] adjust patch set for 3.18: rediffs and drops

Adam Williamson adamwill at fedoraproject.org
Wed Nov 26 20:55:14 UTC 2014


commit 54fd44e88b6ac3a9bf79ff07a96df87900b437bd
Author: Adam Williamson <awilliam at redhat.com>
Date:   Sat Nov 8 15:30:23 2014 -0800

    adjust patch set for 3.18: rediffs and drops
    
    Once again try a 'real' fix for ACPI _DEP (the one currently
    in upstream review). Re-diff the ath6kl SDIO ID patch. Drop
    the UEFI 'big arse revert', should be merged upstream now. For
    now do not apply the mic fix patch as I *think* a correct fix
    has gone upstream:
    http://mailman.alsa-project.org/pipermail/alsa-devel/2014-October/081780.html
    Update config-x86-generic with a bunch more ASoC drivers to
    disable.

 ...t-to-fix-battery-issue-on-the-Asus-T100TA.patch |  225 ++++++++++++++++++++
 UEFI-big-arse-revert.patch                         |  166 --------------
 config-x86-generic                                 |    5 +
 kernel.spec                                        |   20 +--
 ...l-OEM-chipset-found-in-Venue-8-Pro-SDIO-I.patch |    4 +-
 5 files changed, 239 insertions(+), 181 deletions(-)
---
diff --git a/RFC-V2-ACPI-Add-_DEP-Operation-Region-Dependencies-support-to-fix-battery-issue-on-the-Asus-T100TA.patch b/RFC-V2-ACPI-Add-_DEP-Operation-Region-Dependencies-support-to-fix-battery-issue-on-the-Asus-T100TA.patch
new file mode 100644
index 0000000..55524f9
--- /dev/null
+++ b/RFC-V2-ACPI-Add-_DEP-Operation-Region-Dependencies-support-to-fix-battery-issue-on-the-Asus-T100TA.patch
@@ -0,0 +1,225 @@
+From patchwork Mon Oct 27 15:09:44 2014
+Content-Type: text/plain; charset="utf-8"
+MIME-Version: 1.0
+Content-Transfer-Encoding: 7bit
+Subject: [RFC, V2] ACPI: Add _DEP(Operation Region Dependencies) support to fix
+	battery issue on the Asus T100TA
+From: "lan,Tianyu" <tianyu.lan at intel.com>
+X-Patchwork-Id: 5161621
+Message-Id: <1414422584-2638-1-git-send-email-tianyu.lan at intel.com>
+To: rjw at rjwysocki.net, lenb at kernel.org, wsa at the-dreams.de,
+	robert.moore at intel.com, lv.zheng at intel.com
+Cc: Lan Tianyu <tianyu.lan at intel.com>, linux-acpi at vger.kernel.org,
+	linux-kernel at vger.kernel.org, linux-i2c at vger.kernel.org, devel at acpica.org
+Date: Mon, 27 Oct 2014 23:09:44 +0800
+
+ACPI 5.0 introduces _DEP to designate device objects that OSPM should
+assign a higher priority in start ordering due to future operation region
+accesses.
+
+On Asus T100TA, ACPI battery info are read from a I2C slave device via
+I2C operation region. Before I2C operation region handler is installed,
+battery _STA always returns 0. There is a _DEP method of designating
+start order under battery device node.
+
+This patch is to implement _DEP feature to fix battery issue on the Asus T100TA.
+Introducing acpi_dep_list and adding dep_unmet count in the struct
+acpi_device. During ACPI namespace scan, create struct acpi_dep_data for a
+valid pair of master (device pointed to by _DEP)/slave(device with _DEP), record
+master's and slave's ACPI handle in it and put it into acpi_dep_list. The dep_unmet
+count will increase by one if there is a device under its _DEP. Driver's probe() should
+return EPROBE_DEFER when find dep_unmet larger than 0. When I2C operation
+region handler is installed, remove all struct acpi_dep_data on the acpi_dep_list
+whose master is pointed to I2C host controller and decrease slave's dep_unmet.
+When dep_unmet decreases to 0, all _DEP conditions are met and then do acpi_bus_attach()
+for the device in order to resolve battery _STA issue on the Asus T100TA.
+
+Signed-off-by: Lan Tianyu <tianyu.lan at intel.com>
+
+---
+Change since V1:
+	Rework struct acpi_dep_data, record master and slave's ACPI handle.
+Rename dep_present with dep_unmet and make it as a count. Increase dep_unmet
+during bootup and decrease it when associated operation region is installed.
+
+ drivers/acpi/scan.c     | 91 +++++++++++++++++++++++++++++++++++++++++++++++++
+ drivers/i2c/i2c-core.c  |  1 +
+ include/acpi/acpi_bus.h |  1 +
+ include/linux/acpi.h    |  3 ++
+ 4 files changed, 96 insertions(+)
+
+diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
+index 4da55d8..bfa9c77 100644
+--- a/drivers/acpi/scan.c
++++ b/drivers/acpi/scan.c
+@@ -36,6 +36,8 @@ bool acpi_force_hot_remove;
+ 
+ static const char *dummy_hid = "device";
+ 
++static LIST_HEAD(acpi_dep_list);
++static DEFINE_MUTEX(acpi_dep_list_lock);
+ static LIST_HEAD(acpi_bus_id_list);
+ static DEFINE_MUTEX(acpi_scan_lock);
+ static LIST_HEAD(acpi_scan_handlers_list);
+@@ -43,6 +45,12 @@ DEFINE_MUTEX(acpi_device_lock);
+ LIST_HEAD(acpi_wakeup_device_list);
+ static DEFINE_MUTEX(acpi_hp_context_lock);
+ 
++struct acpi_dep_data {
++	struct list_head node;
++	acpi_handle master;
++	acpi_handle slave;
++};
++
+ struct acpi_device_bus_id{
+ 	char bus_id[15];
+ 	unsigned int instance_no;
+@@ -2121,6 +2129,63 @@ static void acpi_scan_init_hotplug(struct acpi_device *adev)
+ 	}
+ }
+ 
++static void acpi_device_dep_initialize(struct acpi_device *adev)
++{
++	struct acpi_dep_data *dep;
++	struct acpi_handle_list dep_devices;
++	struct acpi_device_info *info;
++	acpi_status status;
++	int i, skip;
++
++	if (!acpi_has_method(adev->handle, "_DEP"))
++		return;
++
++	status = acpi_evaluate_reference(adev->handle, "_DEP", NULL,
++					&dep_devices);
++	if (ACPI_FAILURE(status)) {
++		dev_err(&adev->dev, "Failed to evaluate _DEP.\n");
++		return;
++	}
++
++	for (i = 0; i < dep_devices.count; i++) {
++
++		status = acpi_get_object_info(dep_devices.handles[i], &info);
++		if (ACPI_FAILURE(status)) {
++			dev_err(&adev->dev, "Error reading device info\n");
++			continue;
++		}
++
++		/*
++		 * Skip the dependency of Windows System Power
++		 * Management Controller
++		 */
++		if (info->valid & ACPI_VALID_HID
++		    && !strcmp(info->hardware_id.string, "INT3396"))
++			skip = 1;
++		else
++			skip = 0;
++
++		kfree(info);
++
++		if (skip)
++			continue;
++
++		dep = kzalloc(sizeof(struct acpi_dep_data), GFP_KERNEL);
++		if (!dep) {
++			dev_err(&adev->dev, "Not enough memory for _DEP list entry.\n");
++			return;
++		}
++
++		dep->master = dep_devices.handles[i];
++		dep->slave  = adev->handle;
++		adev->dep_unmet++;
++
++		mutex_lock(&acpi_dep_list_lock);
++		list_add_tail(&dep->node , &acpi_dep_list);
++		mutex_unlock(&acpi_dep_list_lock);
++	}
++}
++
+ static acpi_status acpi_bus_check_add(acpi_handle handle, u32 lvl_not_used,
+ 				      void *not_used, void **return_value)
+ {
+@@ -2147,6 +2212,7 @@ static acpi_status acpi_bus_check_add(acpi_handle handle, u32 lvl_not_used,
+ 		return AE_CTRL_DEPTH;
+ 
+ 	acpi_scan_init_hotplug(device);
++	acpi_device_dep_initialize(device);
+ 
+  out:
+ 	if (!*return_value)
+@@ -2267,6 +2333,31 @@ static void acpi_bus_attach(struct acpi_device *device)
+ 		device->handler->hotplug.notify_online(device);
+ }
+ 
++int acpi_walk_dep_device_list(acpi_handle handle)
++{
++	struct acpi_dep_data *dep, *tmp;
++	struct acpi_device *adev;
++
++	mutex_lock(&acpi_dep_list_lock);
++	list_for_each_entry_safe(dep, tmp, &acpi_dep_list, node) {
++		if (dep->master == handle) {
++			acpi_bus_get_device(dep->slave, &adev);
++			if (!adev)
++				continue;
++
++			adev->dep_unmet--;
++			if (!adev->dep_unmet)
++				acpi_bus_attach(adev);
++			list_del(&dep->node);
++			kfree(dep);
++		}
++	}
++	mutex_unlock(&acpi_dep_list_lock);
++
++	return 0;
++}
++EXPORT_SYMBOL_GPL(acpi_walk_dep_device_list);
++
+ /**
+  * acpi_bus_scan - Add ACPI device node objects in a given namespace scope.
+  * @handle: Root of the namespace scope to scan.
+diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c
+index 126e182..d5495ca 100644
+--- a/drivers/i2c/i2c-core.c
++++ b/drivers/i2c/i2c-core.c
+@@ -402,6 +402,7 @@ static int acpi_i2c_install_space_handler(struct i2c_adapter *adapter)
+ 		return -ENOMEM;
+ 	}
+ 
++	acpi_walk_dep_device_list(handle);
+ 	return 0;
+ }
+ 
+diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
+index 98cd723..269f068 100644
+--- a/include/acpi/acpi_bus.h
++++ b/include/acpi/acpi_bus.h
+@@ -367,6 +367,7 @@ struct acpi_device {
+ 	void *driver_data;
+ 	struct device dev;
+ 	unsigned int physical_node_count;
++	unsigned int dep_unmet;
+ 	struct list_head physical_node_list;
+ 	struct mutex physical_node_lock;
+ 	void (*remove)(struct acpi_device *);
+diff --git a/include/linux/acpi.h b/include/linux/acpi.h
+index 2b9c235..cec5f55 100644
+--- a/include/linux/acpi.h
++++ b/include/linux/acpi.h
+@@ -428,6 +428,7 @@ extern bool acpi_driver_match_device(struct device *dev,
+ 
+ int acpi_device_uevent_modalias(struct device *, struct kobj_uevent_env *);
+ int acpi_device_modalias(struct device *, char *, int);
++int acpi_walk_dep_device_list(acpi_handle handle);
+ 
+ struct platform_device *acpi_create_platform_device(struct acpi_device *);
+ #define ACPI_PTR(_ptr)	(_ptr)
+@@ -446,6 +447,8 @@ static inline const char *acpi_dev_name(struct acpi_device *adev)
+ 
+ static inline void acpi_early_init(void) { }
+ 
++static inline int acpi_walk_dep_device_list(acpi_handle handle) { }
++
+ static inline int early_acpi_boot_init(void)
+ {
+ 	return 0;
diff --git a/config-x86-generic b/config-x86-generic
index 917b20d..e1da139 100644
--- a/config-x86-generic
+++ b/config-x86-generic
@@ -258,6 +258,11 @@ CONFIG_SND_SOC_INTEL_BYT_MAX98090_MACH=m
 # CONFIG_SND_SOC_TAS2552 is not set
 # CONFIG_SND_SOC_TLV320AIC31XX is not set
 # CONFIG_SND_SOC_WM8770 is not set
+# CONFIG_SND_SOC_CS35L32 is not set
+# CONFIG_SND_SOC_ES8328 is not set
+# CONFIG_SND_SOC_SSM2602_I2C is not set
+# CONFIG_SND_SOC_SSM4567 is not set
+# CONFIG_SND_SOC_WM8978 is not set
 
 
 # Baytrail other (from pbrobinson)
diff --git a/kernel.spec b/kernel.spec
index 7ad3bcb..ce6c395 100644
--- a/kernel.spec
+++ b/kernel.spec
@@ -628,26 +628,21 @@ Patch30000: kernel-arm64.patch
 
 # AWB PATCH DEFINITIONS (BAYTRAIL)
 
-# https://lkml.org/lkml/2014/4/16/418
+# https://patchwork.kernel.org/patch/5161621/(minor rediff required)
 # https://bugzilla.kernel.org/show_bug.cgi?id=69011
-# The 'permanent' fix for this is under development but current patch doesn't
-# work, see #69011
-Patch31010: 0001-ACPI-temporary-dep-solution-for-battery-support.patch
+# Fixes battery status
+Patch31010: RFC-V2-ACPI-Add-_DEP-Operation-Region-Dependencies-support-to-fix-battery-issue-on-the-Asus-T100TA.patch
 
 # Add SDIO ID for the V8P wireless adapter to ath6kl driver
 Patch31011: support-Dell-OEM-chipset-found-in-Venue-8-Pro-SDIO-I.patch
 
-# UEFI boot fixes from Matt Fleming, try to fix
-# https://bugzilla.kernel.org/show_bug.cgi?id=84241
-Patch31012: UEFI-big-arse-revert.patch
-
 # Map 'Home' button on Venue 8 Pro to left meta key (i.e. "Start")
 # from Jan-Michael Brummer
-Patch31013: soc_button_use_leftmeta.patch
+Patch31012: soc_button_use_leftmeta.patch
 
 # Enable mic on RT5640 (i.e. lots of Baytrail hardware, we hope)
 # from Jan-Michael Brummer
-Patch31014: rt5640_enable_mic.patch
+Patch31013: rt5640_enable_mic.patch
 
 # END OF AWB PATCH DEFINITIONS
 
@@ -1387,10 +1382,9 @@ ApplyPatch kernel-arm64.patch -R
 # END OF FEDORA PATCH APPLICATIONS
 
 # AWB (BAYTRAIL) PATCH APPLICATIONS
-ApplyPatch 0001-ACPI-temporary-dep-solution-for-battery-support.patch
+ApplyPatch RFC-V2-ACPI-Add-_DEP-Operation-Region-Dependencies-support-to-fix-battery-issue-on-the-Asus-T100TA.patch
 ApplyPatch support-Dell-OEM-chipset-found-in-Venue-8-Pro-SDIO-I.patch
-ApplyPatch UEFI-big-arse-revert.patch
-ApplyPatch rt5640_enable_mic.patch
+#ApplyPatch rt5640_enable_mic.patch
 ApplyPatch soc_button_use_leftmeta.patch
 
 # END OF AWB (BAYTRAIL) PATCH APPLICATIONS
diff --git a/support-Dell-OEM-chipset-found-in-Venue-8-Pro-SDIO-I.patch b/support-Dell-OEM-chipset-found-in-Venue-8-Pro-SDIO-I.patch
index dbcd736..a422a2c 100644
--- a/support-Dell-OEM-chipset-found-in-Venue-8-Pro-SDIO-I.patch
+++ b/support-Dell-OEM-chipset-found-in-Venue-8-Pro-SDIO-I.patch
@@ -12,10 +12,10 @@ diff --git a/drivers/net/wireless/ath/ath6kl/sdio.c b/drivers/net/wireless/ath/a
 index 339d89f..5384f14 100644
 --- a/drivers/net/wireless/ath/ath6kl/sdio.c
 +++ b/drivers/net/wireless/ath/ath6kl/sdio.c
-@@ -1400,6 +1400,7 @@ static const struct sdio_device_id ath6kl_sdio_devices[] = {
- 	{SDIO_DEVICE(MANUFACTURER_CODE, (MANUFACTURER_ID_AR6003_BASE | 0x1))},
+@@ -1401,6 +1401,7 @@ static const struct sdio_device_id ath6kl_sdio_devices[] = {
  	{SDIO_DEVICE(MANUFACTURER_CODE, (MANUFACTURER_ID_AR6004_BASE | 0x0))},
  	{SDIO_DEVICE(MANUFACTURER_CODE, (MANUFACTURER_ID_AR6004_BASE | 0x1))},
+ 	{SDIO_DEVICE(MANUFACTURER_CODE, (MANUFACTURER_ID_AR6004_BASE | 0x2))},
 +	{SDIO_DEVICE(MANUFACTURER_CODE, (MANUFACTURER_ID_AR6004_BASE | 0x18))},
  	{},
  };


More information about the scm-commits mailing list