On 05/20/2012 02:39 PM, Mauro Carvalho Chehab wrote: > Em 14-05-2012 18:39, Sylwester Nawrocki escreveu: >> On 05/10/2012 10:48 AM, Sylwester Nawrocki wrote: >>> On 05/04/2012 05:31 PM, Sylwester Nawrocki wrote: ... >>> The following changes since commit ae45d3e9aea0ab951dbbca2238fbfbf3993f1e7f: >>> >>> s5p-fimc: Correct memory allocation for VIDIOC_CREATE_BUFS (2012-05-09 16:07:49 +0200) >>> >>> are available in the git repository at: >>> >>> git://git.infradead.org/users/kmpark/linux-samsung v4l-fimc-exynos4x12 >>> >>> for you to fetch changes up to 5feefe6656583de6fd4ef1d53b19031dd5efeec1: >>> >>> s5p-fimc: Use selection API in place of crop operations (2012-05-09 16:11:29 +0200) >>> >>> ---------------------------------------------------------------- >>> Sylwester Nawrocki (14): >>> V4L: Extend V4L2_CID_COLORFX with more image effects >>> s5p-fimc: Avoid crash with null platform_data >>> s5p-fimc: Move m2m node driver into separate file >> >> It seems there is a conflict now with this patch: >> http://git.linuxtv.org/media_tree.git/commit/5126f2590bee412e3053de851cb07f531e4be36a >> >> Attached are updated versions of the two conflicting patches, the others >> don't need touching. >> >> I could provide rebased version of the whole change set tomorrow - if needed. > > Please do that, as this patch doesn't apply as-is. I guess there is no intervention from my side needed, since you already applied those updated patches to the media tree (since I pushed the rebased patch to git.infradead.org a few days ago already) ? However, there is going to be conflicts now with my patch from Sakari's pull request: http://patchwork.linuxtv.org/patch/11336. As we talked in #v4l IRC, even if the API is experimental, any changes to it must not cause build breaks. I didn't discuss that yet with Sakari. I have now reworked the renaming patch, so it now includes backward compatibility definitions like this: #define V4L2_SEL_TGT_CROP_ACTIVE V4L2_SEL_TGT_CROP #define V4L2_SEL_TGT_COMPOSE_ACTIVE V4L2_SEL_TGT_COMPOSE I would then make a patch for Documentation/feature-removal-schedule.txt to indicate those aliases will be removed after two kernel releases. Does it sound like a right thing to do ? Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html