ACPI: property: Support strings in Apple _DSM props
authorHector Martin <marcan@marcan.st>
Mon, 24 Apr 2023 06:46:57 +0000 (15:46 +0900)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 27 Apr 2023 16:50:26 +0000 (18:50 +0200)
The Wi-Fi module in x86 Apple machines has a "module-instance" device
property that specifies the platform type and is used for firmware
selection. Its value is a string, so add support for string values in
acpi_extract_apple_properties().

Reviewed-by: Lukas Wunner <lukas@wunner.de>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Hector Martin <marcan@marcan.st>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/x86/apple.c

index 8812ecd03d5525cb9880b4eaa0ca3b0f104d7466..45d0f16f374f44e0de121a975d4ced127a18ec49 100644 (file)
@@ -71,13 +71,16 @@ void acpi_extract_apple_properties(struct acpi_device *adev)
 
                if ( key->type != ACPI_TYPE_STRING ||
                    (val->type != ACPI_TYPE_INTEGER &&
-                    val->type != ACPI_TYPE_BUFFER))
+                    val->type != ACPI_TYPE_BUFFER &&
+                    val->type != ACPI_TYPE_STRING))
                        continue; /* skip invalid properties */
 
                __set_bit(i, valid);
                newsize += key->string.length + 1;
                if ( val->type == ACPI_TYPE_BUFFER)
                        newsize += val->buffer.length;
+               else if (val->type == ACPI_TYPE_STRING)
+                       newsize += val->string.length + 1;
        }
 
        numvalid = bitmap_weight(valid, numprops);
@@ -119,6 +122,12 @@ void acpi_extract_apple_properties(struct acpi_device *adev)
                newprops[v].type = val->type;
                if (val->type == ACPI_TYPE_INTEGER) {
                        newprops[v].integer.value = val->integer.value;
+               } else if (val->type == ACPI_TYPE_STRING) {
+                       newprops[v].string.length = val->string.length;
+                       newprops[v].string.pointer = free_space;
+                       memcpy(free_space, val->string.pointer,
+                              val->string.length);
+                       free_space += val->string.length + 1;
                } else {
                        newprops[v].buffer.length = val->buffer.length;
                        newprops[v].buffer.pointer = free_space;