CHROMIUM: platform/x86: chromeos_acpi: cast sizeof(array) to u32 before use
authorSimon Que <sque@chromium.org>
Thu, 17 Jan 2013 00:37:55 +0000 (16:37 -0800)
committerChromeBot <chrome-bot@google.com>
Thu, 17 Jan 2013 19:05:12 +0000 (11:05 -0800)
Removes a compiler warning:
"comparison of distinct pointer types lacks a cast"

BUG=chromium-os:5542
TEST=emerge chromeos-kernel, should not see warnings in chromeos_acpi.o

Change-Id: Ia4779db9d30bd0a1206fe339d682d56411f6105b
Signed-off-by: Simon Que <sque@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/41483
Reviewed-by: Mandeep Singh Baines <msb@chromium.org>
drivers/platform/x86/chromeos_acpi.c

index 2ecd094..1af5ed7 100644 (file)
@@ -442,7 +442,7 @@ static void handle_nested_acpi_package(union acpi_object *po, char *pm,
 
                case ACPI_TYPE_STRING:
                        copy_size = min(element->string.length,
-                                       sizeof(attr_value) - 1);
+                                       (u32)(sizeof(attr_value)) - 1);
                        memcpy(attr_value, element->string.pointer, copy_size);
                        attr_value[copy_size] = '\0';
                        paa = create_sysfs_attribute(attr_value, pm, count, i);
@@ -621,7 +621,7 @@ static void handle_acpi_package(union acpi_object *po, char *pm)
 
                case ACPI_TYPE_STRING:
                        copy_size = min(element->string.length,
-                                       sizeof(attr_value) - 1);
+                                       (u32)(sizeof(attr_value)) - 1);
                        memcpy(attr_value, element->string.pointer, copy_size);
                        attr_value[copy_size] = '\0';
                        add_sysfs_attribute(attr_value, pm, count, j);