Em Tue, 29 Oct 2013 01:06:30 +0100 Sylwester Nawrocki <sylvester.nawrocki@xxxxxxxxx> escreveu: > Hi Mauro, > > On 10/28/2013 11:11 PM, Mauro Carvalho Chehab wrote: > >> The following changes since commit 8ca5d2d8e58df7235b77ed435e63c484e123fede: > >> > > >> > [media] uvcvideo: Fix data type for pan/tilt control (2013-10-17 06:55:29 -0300) > >> > > >> > are available in the git repository at: > >> > > >> > git://linuxtv.org/snawrocki/samsung.git for-v3.13-2 > >> > > >> > for you to fetch changes up to 6eb89d71b27e6731755ab5722f3cdc0f6e8273f2: > >> > > >> > V4L: Add s5k4e5 sensor driver (2013-10-18 21:36:42 +0200) > >> > > >> > ---------------------------------------------------------------- > >> > Arun Kumar K (12): > >> > exynos5-fimc-is: Add Exynos5 FIMC-IS device tree bindings documentation > > > > As agreed during KS, the subsystem maintainers should wait for a documentation > > review on DT by the DT maintainers, at least for a while. > > > > So, I'd like to see either their reviews on this patch: > > https://patchwork.linuxtv.org/patch/20439/ > > > > Or their ack for us to apply it. > > I agree with you on that. Just please note the first version of this patch > has been posted *5 months* ago https://patchwork.linuxtv.org/patch/18684 > > Stephen has reviewed subsequent version about 3 months ago: > https://patchwork.linuxtv.org/patch/19521 > > Then we got no more comments from DT maintainers, I have reviewed this patch > multiple times on the mailing lists: > https://patchwork.linuxtv.org/patch/19715 > https://patchwork.linuxtv.org/patch/19749 > > And explicitly asked for an Ack: > https://patchwork.linuxtv.org/patch/19832 > > Then those 2 versions passed silently: > https://patchwork.linuxtv.org/patch/20055 > https://patchwork.linuxtv.org/patch/20225 > > And...huh...we got another review, I didn't notice it until now: > https://patchwork.linuxtv.org/patch/20439 Thanks Mark. > > Arun, care to address those review comments and send us an updated > binding documentation patch ? > > Hence I think we have waited for a while. ;) > > >> > exynos5-fimc-is: Add driver core files > >> > exynos5-fimc-is: Add common driver header files > >> > exynos5-fimc-is: Add register definition and context header > >> > exynos5-fimc-is: Add isp subdev > >> > exynos5-fimc-is: Add scaler subdev > >> > exynos5-fimc-is: Add sensor interface > >> > exynos5-fimc-is: Add the hardware pipeline control > >> > exynos5-fimc-is: Add the hardware interface module > >> > exynos5-is: Add Kconfig and Makefile > >> > V4L: Add DT binding doc for s5k4e5 image sensor > > > > Same applies to this patch: > > https://patchwork.linuxtv.org/patch/20448/ > > This one also have been on the mailing list for quite some time and it > uses already standard bindings, so I assumed it is OK to merge it. > > https://patchwork.linuxtv.org/project/linux-media/list/?state=*&q=s5k4e5 > > But if there must be an Ack then we shall wait, it will probably won't > make a big difference now, if this patch is postponed by 3 more months. Yeah, it seems that we've waited for a long time to get an ack there. So, let's do this: Please send a new version with Mark's comments. Also, please split Doc changes from the code changes on the new series. I'll wait for a couple days for DT people to review it. If we don't have any reply, I'll review and apply it for Kernel 3.13, if I don't see anything really weird on it. Regards, Mauro -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
