Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-02-04 Thread Jasmin J.
Hi! > Strictly speaking, building the media subsystem on older kernels is a job of > the media build system. Yes I agree. > In general I would thus ask for the fix to be merged in media-build.git. Which I do mostly, but in this case it is a coding error in mainline. > In this specific case, as

Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-02-04 Thread Jasmin J.
Hi! > We're waiting for the end of the 4.16 merge window. Ahh, this is the reason for the delay. > For me that meant that I simply did (and still don't) have time to look at > fixing the media_build. No problem, I do it when I have time ;) > The media_build is also broken worse than usual AFAIK,

Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-02-04 Thread Laurent Pinchart
Hi Jasmin, On Sunday, 4 February 2018 12:37:08 EET Jasmin J. wrote: > Hi Laurent! > > > Reviewed-by: Laurent Pinchart > > THX! > Don't forget the "Acked-by: Arnd Bergmann " (see Patchwork: > https://patchwork.linuxtv.org/patch/46464 ). Sure. > > and taken into my tree for v4.17. > > When wil

Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-02-04 Thread Hans Verkuil
Hi Jasmin, On 02/04/2018 11:37 AM, Jasmin J. wrote: > Hi Laurent! > >> Reviewed-by: Laurent Pinchart > THX! > Don't forget the "Acked-by: Arnd Bergmann " (see Patchwork: > https://patchwork.linuxtv.org/patch/46464 ). > >> and taken into my tree for v4.17. > When will this merged to the media-tr

Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-02-04 Thread Jasmin J.
Hi Laurent! > Reviewed-by: Laurent Pinchart THX! Don't forget the "Acked-by: Arnd Bergmann " (see Patchwork: https://patchwork.linuxtv.org/patch/46464 ). > and taken into my tree for v4.17. When will this merged to the media-tree trunk? In another month or earlier? This issue was overlooked whe

Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-02-02 Thread Laurent Pinchart
Hi Jasmin, Thank you for the patch. On Sunday, 14 January 2018 12:21:43 EET Jasmin J. wrote: > From: Jasmin Jessich > > Commit 828ee8c71950 ("media: uvcvideo: Use ktime_t for timestamps") > changed to use ktime_t for timestamps. Older Kernels use a struct for > ktime_t, which requires the conve

Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-01-27 Thread Jasmin J.
Hello Mauro! > Ping! > It is required to compile for Kernels older 4.10. It would be nice to get this merged, so that we can see if older Kernels will compile again. BR, Jasmin

Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-01-18 Thread Jasmin J.
Ping! It is required to compile for Kernels older 4.10.

Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-01-14 Thread Arnd Bergmann
On Sun, Jan 14, 2018 at 11:21 AM, Jasmin J. wrote: > From: Jasmin Jessich > > Commit 828ee8c71950 ("media: uvcvideo: Use ktime_t for timestamps") > changed to use ktime_t for timestamps. Older Kernels use a struct for > ktime_t, which requires the conversion function ktime_to_ns to be used on > s

Re: [PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-01-14 Thread Jasmin J.
Hi! I tested this patch (compile only) on Kernel 4.4. For 3.13 there is something else not working, so I (or Hans) needs to have a look on that. BR, Jasmin

[PATCH] media: uvcvideo: Fixed ktime_t to ns conversion

2018-01-14 Thread Jasmin J.
From: Jasmin Jessich Commit 828ee8c71950 ("media: uvcvideo: Use ktime_t for timestamps") changed to use ktime_t for timestamps. Older Kernels use a struct for ktime_t, which requires the conversion function ktime_to_ns to be used on some places. With this patch it will compile now also for older