diff options
| author | Mika Kukkonen <mika@osdl.org> | 2004-06-28 19:34:15 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@ppc970.osdl.org> | 2004-06-28 19:34:15 -0700 |
| commit | 0816a653a7ebdb43233b5a07042aecdb1f70e633 (patch) | |
| tree | 6d9ea822bb2dbf7e2517a4e23b48c92cad009024 /include | |
| parent | 7df8818da1e3493340c385d1dc61524941b55e38 (diff) | |
[PATCH] sparse: fix ugly include/linux/efi.h typedef
efi.h declares a function type, and then uses that as an argument to
another function, and expects the compiler to magically demote the
function to a function pointer.
Even a gcc person (rth) was surprised that this was legal, and it
doesn't match any other use of a function pointer in the kernel, and
sparse doesn't like the implicit type-conversion.
So make the type sane in the first place, instead of depending on
a very weird corner case of the C language.
Diffstat (limited to 'include')
| -rw-r--r-- | include/linux/efi.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/efi.h b/include/linux/efi.h index 49dcae3501df..0c5c94ab1112 100644 --- a/include/linux/efi.h +++ b/include/linux/efi.h @@ -108,7 +108,7 @@ typedef struct { #endif } efi_memory_desc_t; -typedef int efi_freemem_callback_t (unsigned long start, unsigned long end, void *arg); +typedef int (*efi_freemem_callback_t) (unsigned long start, unsigned long end, void *arg); /* * Types and defines for Time Services |