efi: Fix types in EFI calls to match EFI function definitions.
authorRoy Franz <roy.franz@linaro.org>
Sun, 22 Sep 2013 22:45:41 +0000 (15:45 -0700)
committerMatt Fleming <matt.fleming@intel.com>
Wed, 25 Sep 2013 11:34:44 +0000 (12:34 +0100)
EFI calls can made directly on ARM, so the function pointers
are directly invoked.  This allows types to be checked at
compile time, so here we ensure that the parameters match
the function signature. The wrappers used by x86 prevent
any type checking.
Correct the type of chunksize to be based on native
width as specified by the EFI_FILE_PROTOCOL read()
function.

Signed-off-by: Roy Franz <roy.franz@linaro.org>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>
drivers/firmware/efi/efi-stub-helper.c

index 5cea5d5a9a1ffeab376f8265a7c4a2426303e6ee..4252d01089b2e437dc5a08b1249512f4916282cb 100644 (file)
@@ -324,7 +324,7 @@ static efi_status_t handle_cmdline_files(efi_system_table_t *sys_table_arg,
        status = efi_call_phys3(sys_table_arg->boottime->allocate_pool,
                                EFI_LOADER_DATA,
                                nr_files * sizeof(*files),
-                               &files);
+                               (void **)&files);
        if (status != EFI_SUCCESS) {
                efi_printk(sys_table_arg, "Failed to alloc mem for file handle list\n");
                goto fail;
@@ -375,7 +375,8 @@ static efi_status_t handle_cmdline_files(efi_system_table_t *sys_table_arg,
                        boottime = sys_table_arg->boottime;
 
                        status = efi_call_phys3(boottime->handle_protocol,
-                                       image->device_handle, &fs_proto, &io);
+                                       image->device_handle, &fs_proto,
+                                               (void **)&io);
                        if (status != EFI_SUCCESS) {
                                efi_printk(sys_table_arg, "Failed to handle fs_proto\n");
                                goto free_files;
@@ -409,7 +410,8 @@ static efi_status_t handle_cmdline_files(efi_system_table_t *sys_table_arg,
 
 grow:
                status = efi_call_phys3(sys_table_arg->boottime->allocate_pool,
-                                       EFI_LOADER_DATA, info_sz, &info);
+                                       EFI_LOADER_DATA, info_sz,
+                                       (void **)&info);
                if (status != EFI_SUCCESS) {
                        efi_printk(sys_table_arg, "Failed to alloc mem for file info\n");
                        goto close_handles;
@@ -459,18 +461,19 @@ grow:
 
                addr = file_addr;
                for (j = 0; j < nr_files; j++) {
-                       u64 size;
+                       unsigned long size;
 
                        size = files[j].size;
                        while (size) {
-                               u64 chunksize;
+                               unsigned long chunksize;
                                if (size > EFI_READ_CHUNK_SIZE)
                                        chunksize = EFI_READ_CHUNK_SIZE;
                                else
                                        chunksize = size;
                                status = efi_call_phys3(fh->read,
                                                        files[j].handle,
-                                                       &chunksize, addr);
+                                                       &chunksize,
+                                                       (void *)addr);
                                if (status != EFI_SUCCESS) {
                                        efi_printk(sys_table_arg, "Failed to read file\n");
                                        goto free_file_total;