Re: [Y2038] [PATCH] input_event: Provide override for sparc64

2019-01-15 Thread Deepa Dinamani
> On Jan 15, 2019, at 2:32 PM, Dmitry Torokhov > wrote: > >> On Tue, Jan 15, 2019 at 1:29 PM David Miller wrote: >> >> From: Arnd Bergmann >> Date: Tue, 15 Jan 2019 22:19:27 +0100 >> >>> The correct check appears to be >>> >>> #if defined(__sparc__) && defined(__arch64__) >> >> That is

Re: [Y2038] [PATCH] input_event: Provide override for sparc64

2019-01-15 Thread Dmitry Torokhov
On Tue, Jan 15, 2019 at 1:29 PM David Miller wrote: > > From: Arnd Bergmann > Date: Tue, 15 Jan 2019 22:19:27 +0100 > > > The correct check appears to be > > > > #if defined(__sparc__) && defined(__arch64__) > > That is correct. OK. Deepa, could you please send me a fixup as I already pushed

Re: [Y2038] [PATCH] input_event: Provide override for sparc64

2019-01-15 Thread David Miller
From: Arnd Bergmann Date: Tue, 15 Jan 2019 22:19:27 +0100 > The correct check appears to be > > #if defined(__sparc__) && defined(__arch64__) That is correct. ___ Y2038 mailing list Y2038@lists.linaro.org

Re: [Y2038] [PATCH] input_event: Provide override for sparc64

2019-01-15 Thread Arnd Bergmann
On Sat, Dec 29, 2018 at 7:35 PM Deepa Dinamani wrote: > struct timeval time; > #define input_event_sec time.tv_sec > #define input_event_usec time.tv_usec > #else > __kernel_ulong_t __sec; > +#ifdef CONFIG_SPARC64 > + unsigned int __usec; > +#else >

Re: [Y2038] [PATCH] input_event: Provide override for sparc64

2019-01-13 Thread Dmitry Torokhov
On Sat, Dec 29, 2018 at 10:35:14AM -0800, Deepa Dinamani wrote: > The usec part of the timeval is defined as > __kernel_suseconds_t tv_usec; /* microseconds */ > > Arnd noticed that sparc64 is the only architecture > that defines __kernel_suseconds_t as int rather than long. > > This breaks the

Re: [Y2038] [PATCH] input_event: Provide override for sparc64

2019-01-13 Thread Deepa Dinamani
> On Dec 29, 2018, at 10:35 AM, Deepa Dinamani wrote: > > The usec part of the timeval is defined as > __kernel_suseconds_ttv_usec; /* microseconds */ > > Arnd noticed that sparc64 is the only architecture > that defines __kernel_suseconds_t as int rather than long. > > This breaks the