diff mbox

acpi: pnp: add ACPI_RESOURCE_TYPE_SERIAL_BUS as a valid type

Message ID 1456860705-24935-1-git-send-email-timur@codeaurora.org (mailing list archive)
State Accepted, archived
Delegated to: Rafael Wysocki
Headers show

Commit Message

Timur Tabi March 1, 2016, 7:31 p.m. UTC 
From: Harb Abdulhamid <harba@codeaurora.org>

An error message is printed for resources of type 19, which is a valid
supported resource type.  The Firmware Test Suite tool (fwts) reports
this as a test failure.  This change fixes the false test failures
for ASL that use type 19 (ACPI_RESOURCE_TYPE_SERIAL_BUS) resources.

Signed-off-by: Harb Abdulhamid <harba@codeaurora.org>
Signed-off-by: Timur Tabi <timur@codeaurora.org>
---
 drivers/pnp/pnpacpi/rsparser.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Rafael J. Wysocki March 10, 2016, 11:36 p.m. UTC | #1 
On Tuesday, March 01, 2016 01:31:45 PM Timur Tabi wrote:
> From: Harb Abdulhamid <harba@codeaurora.org>
> 
> An error message is printed for resources of type 19, which is a valid
> supported resource type.  The Firmware Test Suite tool (fwts) reports
> this as a test failure.  This change fixes the false test failures
> for ASL that use type 19 (ACPI_RESOURCE_TYPE_SERIAL_BUS) resources.
> 
> Signed-off-by: Harb Abdulhamid <harba@codeaurora.org>
> Signed-off-by: Timur Tabi <timur@codeaurora.org>

Applied, thanks!

Rafael

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/pnp/pnpacpi/rsparser.c b/drivers/pnp/pnpacpi/rsparser.c
index 0579649..4b717c6 100644
--- a/drivers/pnp/pnpacpi/rsparser.c
+++ b/drivers/pnp/pnpacpi/rsparser.c
@@ -252,6 +252,10 @@  static acpi_status pnpacpi_allocated_resource(struct acpi_resource *res,
 	case ACPI_RESOURCE_TYPE_GENERIC_REGISTER:
 		break;
 
+	case ACPI_RESOURCE_TYPE_SERIAL_BUS:
+		/* serial bus connections (I2C/SPI/UART) are not pnp */
+		break;
+
 	default:
 		dev_warn(&dev->dev, "unknown resource type %d in _CRS\n",
 			 res->type);