On Fri July 27 2012 17:46:45 Eduard Vaks wrote: > it may be an old compiler I don't know > also i can't seem to get a diff so i have included the whole file I > haven't tested it (well it works on my system but I don' even know if I > use that part of the code) but it is realativly minor. > > I have just named the union that func and offset are contained in > because the initalizer did not work + other resulting changes: > struct v4l2_ioctl_info { > unsigned int ioctl; > u32 flags; > const char * const name; > union { > u32 offset; > int (*func)(const struct v4l2_ioctl_ops *ops, > struct file *file, void *fh, void *p); > } u; > void (*debug)(const void *arg, bool write_only); > }; A patch for this (similar to yours) is waiting to be merged: http://patchwork.linuxtv.org/patch/13336/ Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html