[coreboot-gerrit] Change in coreboot[master]: acpi/acpigen_dsm: fix I2C HID DSM to report correct function support

[Josie Nordrum (Code Review)]

Josie Nordrum has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/46429 )

Change subject: acpi/acpigen_dsm: fix I2C HID DSM to report correct function support
......................................................................

acpi/acpigen_dsm: fix I2C HID DSM to report correct function support

Fix DSM function 0 (query function) to correctly report function support
for its revision.  Revision 1 should return 0x3, all higher revisions
should return 0.

BUG=b:170862147
BRANCH=Zork
TEST=check for dmesg error

Signed-off-by: Josie Nordrum 
Change-Id: Iee082ef5cf44c4cf7ab304345af56f3b5173ca56
---
M src/acpi/acpigen_dsm.c
1 file changed, 3 insertions(+), 11 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/46429/1

diff --git a/src/acpi/acpigen_dsm.c b/src/acpi/acpigen_dsm.c
index 2336537..6a89c73 100644
--- a/src/acpi/acpigen_dsm.c
+++ b/src/acpi/acpigen_dsm.c
@@ -12,23 +12,15 @@
    /* ToInteger (Arg1, Local2) */
    acpigen_write_to_integer(ARG1_OP, LOCAL2_OP);
    /* If (LEqual (Local2, 0x0)) */
-	acpigen_write_if_lequal_op_int(LOCAL2_OP, 0x0);
-	/*   Return (Buffer (One) { 0x1f }) */
-	acpigen_write_return_singleton_buffer(0x1f);
+	acpigen_write_if_lequal_op_int(LOCAL2_OP, 0x1);
+	/*   Return (Buffer (One) { 0x3 }) */
+	acpigen_write_return_singleton_buffer(0x3);
    acpigen_pop_len();	/* Pop : If */
    /* Else */
    acpigen_write_else();
-	/*   If (LEqual (Local2, 0x1)) */
-	acpigen_write_if_lequal_op_int(LOCAL2_OP, 0x1);
-	/*     Return (Buffer (One) { 0x3f }) */
-	acpigen_write_return_singleton_buffer(0x3f);
-	acpigen_pop_len();	/* Pop : If */
-	/*   Else */
-	acpigen_write_else();
    /*     Return (Buffer (One) { 0x0 }) */
    acpigen_write_return_singleton_buffer(0x0);
    acpigen_pop_len();	/* Pop : Else */
-	acpigen_pop_len();	/* Pop : Else */
 }

static void i2c_hid_func1_cb(void *arg)