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 documentationAs 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 sensorSame 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. -- Thanks, Sylwester -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
