Re: [PATCH 6/7] [RFC] [media]: v4l2: introduce v4l2_timeval

2015-09-16 Thread Arnd Bergmann
On Wednesday 16 September 2015 10:12:00 Hans Verkuil wrote: > > Are you also attending the ELCE in Dublin? We could have a quick talk there. > I think the discussion whether to switch to a new v4l2_buffer struct isn't > really > dependent on anything y2038. No, unfortunately I won't be there.

Re: [PATCH 6/7] [RFC] [media]: v4l2: introduce v4l2_timeval

2015-09-16 Thread Arnd Bergmann
On Wednesday 16 September 2015 08:51:14 Hans Verkuil wrote: > > a) Similar to my first attempt, define a new struct v4l2_timeval, but > >only use it when building with a y2038-aware libc, so we don't break > >existing environments: > > > > /* some compile-time conditional that we

Re: [PATCH 6/7] [RFC] [media]: v4l2: introduce v4l2_timeval

2015-09-16 Thread Hans Verkuil
On 09/16/2015 09:56 AM, Arnd Bergmann wrote: > On Wednesday 16 September 2015 08:51:14 Hans Verkuil wrote: > >>> a) Similar to my first attempt, define a new struct v4l2_timeval, but >>>only use it when building with a y2038-aware libc, so we don't break >>>existing environments: >>> >>>

Re: [PATCH 6/7] [RFC] [media]: v4l2: introduce v4l2_timeval

2015-09-16 Thread Hans Verkuil
On 09/15/2015 10:26 PM, Arnd Bergmann wrote: > On Tuesday 15 September 2015 18:27:19 Hans Verkuil wrote: >> On 09/15/2015 05:49 PM, 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

Re: [PATCH 6/7] [RFC] [media]: v4l2: introduce v4l2_timeval

2015-09-15 Thread Hans Verkuil
On 09/15/2015 05:49 PM, 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. >

[PATCH 6/7] [RFC] [media]: v4l2: introduce v4l2_timeval

2015-09-15 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. Fortunately, almost all v4l2 drivers use monotonic

Re: [PATCH 6/7] [RFC] [media]: v4l2: introduce v4l2_timeval

2015-09-15 Thread Arnd Bergmann
On Tuesday 15 September 2015 18:27:19 Hans Verkuil wrote: > On 09/15/2015 05:49 PM, 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,