Blob Blame History Raw
From: Bob Moore <robert.moore@intel.com>
Date: Thu, 3 Aug 2017 14:26:38 +0800
Subject: ACPICA: Add a comment, no functional change
Patch-mainline: v4.14-rc1
Git-commit: 583abc9bd585ff0d5fb7411e5601053b21fdd9d9
References: bsc#1117419

ACPICA commit d586c29a026a6172c1113df4d75fd6d764196e77

Describe 2nd byte of the end_tag resource descriptor.

Link: https://github.com/acpica/acpica/commit/d586c29a
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: Lee, Chun-Yi <jlee@suse.com>
---
 drivers/acpi/acpica/utresrc.c |    7 +++++++
 1 file changed, 7 insertions(+)

--- a/drivers/acpi/acpica/utresrc.c
+++ b/drivers/acpi/acpica/utresrc.c
@@ -239,6 +239,13 @@ acpi_ut_walk_aml_resources(struct acpi_w
 				return_ACPI_STATUS(AE_AML_NO_RESOURCE_END_TAG);
 			}
 
+			/*
+			 * Don't attempt to perform any validation on the 2nd byte.
+			 * Although all known ASL compilers insert a zero for the 2nd
+			 * byte, it can also be a checksum (as per the ACPI spec),
+			 * and this is occasionally seen in the field. July 2017.
+			 */
+
 			/* Return the pointer to the end_tag if requested */
 
 			if (!user_function) {