Commit 9e2d3cd3 authored by Al Viro's avatar Al Viro Committed by Linus Torvalds

[PATCH] mod_devicetable.h fixes

 * ieee1394_device_id has kernel_ulong_t field after an odd number of
   __u32 ones.  Since mod_devicetable.h is included both from kernel and
   from host build helper, we may be in trouble if we are building on
   32bit host for 64bit target - userland sees unsigned long long,
   kernel sees unsigned long and while their sizes match, alignments
   might not.  Fixed by forcing alignment.  Fortunately, almost nobody
   else needs that - the rest of such fields is naturally aligned as it
   is.

 * of_device_id has void * in it.  Host userland helpers need
   kernel_ulong_t instead, since their void * might have nothing to do
   with the kernel one.  Fixed in the same way it's done for similar
   problems in pcmcia_device_id (ifdef __KERNEL__).

 * pcmcia_device_id has the same problem as ieee1394_device_id.  Fixed
   the same way.
Signed-off-by: default avatarAl Viro <viro@parcelfarce.linux.theplanet.co.uk>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 5bbe6ab9
...@@ -33,7 +33,8 @@ struct ieee1394_device_id { ...@@ -33,7 +33,8 @@ struct ieee1394_device_id {
__u32 model_id; __u32 model_id;
__u32 specifier_id; __u32 specifier_id;
__u32 version; __u32 version;
kernel_ulong_t driver_data; kernel_ulong_t driver_data
__attribute__((aligned(sizeof(kernel_ulong_t))));
}; };
...@@ -182,7 +183,11 @@ struct of_device_id ...@@ -182,7 +183,11 @@ struct of_device_id
char name[32]; char name[32];
char type[32]; char type[32];
char compatible[128]; char compatible[128];
#if __KERNEL__
void *data; void *data;
#else
kernel_ulong_t data;
#endif
}; };
...@@ -208,7 +213,8 @@ struct pcmcia_device_id { ...@@ -208,7 +213,8 @@ struct pcmcia_device_id {
#ifdef __KERNEL__ #ifdef __KERNEL__
const char * prod_id[4]; const char * prod_id[4];
#else #else
kernel_ulong_t prod_id[4]; kernel_ulong_t prod_id[4]
__attribute__((aligned(sizeof(kernel_ulong_t))));
#endif #endif
/* not matched against */ /* not matched against */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment