Messages in this thread | | | From | Arnd Bergmann <> | Subject | Re: [PATCH v2 8/9] [media] handle 64-bit time_t in v4l2_buffer | Date | Fri, 18 Sep 2015 12:14:41 +0200 |
| |
On Friday 18 September 2015 11:49:50 Hans Verkuil wrote: > *If* the conversion takes place only in v4l2-ioctl.c, then it makes sense > have these structs + ioctls moved to v4l2-ioctl.h.
Ok.
> I noticed that v4l2-compat-ioctl32.c wasn't changed. Is that right? I have > unfortunately no time to double-check that, but it surprised me.
Yes, I pointed that out in the cover letter as something that is still left to be done. If we end up taking the current approach (plus the change to the location of the conversion), the compat code would basically be duplicated.
Alternatively, the video_usercopy() function could just be changed to handle both 32-bit formats directly (including conversion of the multiplane array) and then we can remove that compat handler for all eight ioctls.
Arnd
| |