On Sunday, 4 February 2018 12:37:08 EET Jasmin J. wrote:
> Hi Laurent!
> > Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Don't forget the "Acked-by: Arnd Bergmann <a...@arndb.de>" (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?
As Hans explained, the v4.16 merge window is open. It should close in a week,
and I'll then send a pull request to Mauro.
> This issue was overlooked when merging the change from Arnd in the first
> place. This broke the Kernel build for older Kernels more than two months
> ago! I fixed that in my holidays expecting this gets merged soon and now
> the build is still broken because of this problem.
I had missed this patch as I wasn't CC'ed, until you pinged me directly.
Please try to CC me when submitting uvcvideo patches in the future, otherwise
there's a high chance I won't see them.
> In the past Mauro merged those simple fixes soon and now it seems nobody
> cares about building for older Kernels (it's broken for more than two months
> now!). I mostly try to fix such issues in a short time frame (even on
> vacation), but then it gets lost ... . Sorry, but this is frustrating!
> We don't talk about a nice to have fix but a essential fix to get the media
> build system working again. Such patches need to get merged as early as
> possible in my opinion, especially when someone else sent already an
> "Acked-by" (THX to Arnd).
> I could have made this as a patch in the Build system also, but this would
> be the wrong place, but then Hans would have merged it already and I could
> look into the other build problems.
Strictly speaking, building the media subsystem on older kernels is a job of
the media build system. In general I would thus ask for the fix to be merged
in media-build.git. In this specific case, as the mainline code uses both u64
and ktime_t types, I'm fine with merging your patch to use explicit conversion
functions in mainline even if the two types are now equivalent. However, as
this doesn't fix a bug in the mainline kernel, I don't think this patch is a
candidate for stable releases, and should thus get merged in v4.17. It can
also be included in media-build.git in order to build kernels that currently