Re: [PATCH v2 7/9] [media] v4l2: introduce v4l2_timeval

2015-09-18 Thread Hans Verkuil
On 09/17/15 23:19, Arnd Bergmann wrote: > The v4l2 API uses a 'struct timeval' to communicate time stamps to user > space. This is broken on 32-bit architectures as soon as we have a C library > that defines time_t as 64 bit, which then changes the structure layout of > struct v4l2_buffer. > >

Re: [PATCH v2 7/9] [media] v4l2: introduce v4l2_timeval

2015-09-18 Thread Hans Verkuil
On 09/18/15 11:09, Arnd Bergmann wrote: > On Friday 18 September 2015 10:05:06 Hans Verkuil wrote: >> On 09/17/15 23:19, Arnd Bergmann wrote: >>> The v4l2 API uses a 'struct timeval' to communicate time stamps to user >>> space. This is broken on 32-bit architectures as soon as we have a C library

Re: [PATCH v2 7/9] [media] v4l2: introduce v4l2_timeval

2015-09-18 Thread Hans Verkuil
On 09/18/15 12:08, Arnd Bergmann wrote: > On Friday 18 September 2015 11:52:28 Hans Verkuil wrote: >> On 09/18/15 11:43, Arnd Bergmann wrote: >>> On Friday 18 September 2015 11:27:40 Hans Verkuil wrote: Ah, OK. Got it. I think this is dependent on the upcoming media workshop next

Re: [PATCH v2 7/9] [media] v4l2: introduce v4l2_timeval

2015-09-18 Thread Arnd Bergmann
On Friday 18 September 2015 11:27:40 Hans Verkuil wrote: > Ah, OK. Got it. > > I think this is dependent on the upcoming media workshop next month. If we > decide to redesign v4l2_buffer anyway, then we can avoid timeval completely. > And the only place where we would need to convert it in the

Re: [PATCH v2 7/9] [media] v4l2: introduce v4l2_timeval

2015-09-18 Thread Hans Verkuil
On 09/18/15 11:43, Arnd Bergmann wrote: > On Friday 18 September 2015 11:27:40 Hans Verkuil wrote: >> Ah, OK. Got it. >> >> I think this is dependent on the upcoming media workshop next month. If we >> decide to redesign v4l2_buffer anyway, then we can avoid timeval completely. >> And the only

Re: [PATCH v2 7/9] [media] v4l2: introduce v4l2_timeval

2015-09-18 Thread Arnd Bergmann
On Friday 18 September 2015 11:52:28 Hans Verkuil wrote: > On 09/18/15 11:43, Arnd Bergmann wrote: > > On Friday 18 September 2015 11:27:40 Hans Verkuil wrote: > >> Ah, OK. Got it. > >> > >> I think this is dependent on the upcoming media workshop next month. If we > >> decide to redesign

Re: [PATCH v2 7/9] [media] v4l2: introduce v4l2_timeval

2015-09-18 Thread Arnd Bergmann
On Friday 18 September 2015 10:05:06 Hans Verkuil wrote: > On 09/17/15 23:19, Arnd Bergmann wrote: > > The v4l2 API uses a 'struct timeval' to communicate time stamps to user > > space. This is broken on 32-bit architectures as soon as we have a C library > > that defines time_t as 64 bit, which

[PATCH v2 7/9] [media] v4l2: introduce v4l2_timeval

2015-09-17 Thread Arnd Bergmann
The v4l2 API uses a 'struct timeval' to communicate time stamps to user space. This is broken on 32-bit architectures as soon as we have a C library that defines time_t as 64 bit, which then changes the structure layout of struct v4l2_buffer. Since existing user space source code relies on the