On 06/11/2017 10:28 AM, Hans Verkuil wrote:
Hi Steve, Philipp, While preparing the pull request I noticed that the MAINTAINERS file wasn't updated. Steve, can you post a patch adding entries for the imx and ov5640 driver? Philipp, can you do the same for the video mux? I assume you're the maintainer for this? Thanks! I also made it possible to compile-test this driver with this patch: Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> --- diff --git a/drivers/staging/media/imx/Kconfig b/drivers/staging/media/imx/Kconfig index 7eff50bcea39..22f968cf32b1 100644 --- a/drivers/staging/media/imx/Kconfig +++ b/drivers/staging/media/imx/Kconfig @@ -1,6 +1,7 @@ config VIDEO_IMX_MEDIA tristate "i.MX5/6 V4L2 media core driver" - depends on MEDIA_CONTROLLER && VIDEO_V4L2 && ARCH_MXC && IMX_IPUV3_CORE + depends on MEDIA_CONTROLLER && VIDEO_V4L2 + depends on (ARCH_MXC && IMX_IPUV3_CORE) || COMPILE_TEST select V4L2_FWNODE ---help--- Say yes here to enable support for video4linux media controller
^^^^ This is too optimistic, it fails on IPU code. Ignore this chunk. Hans
diff --git a/drivers/staging/media/imx/imx-media-csi.c b/drivers/staging/media/imx/imx-media-csi.c index c306146a4247..a2d26693912e 100644 --- a/drivers/staging/media/imx/imx-media-csi.c +++ b/drivers/staging/media/imx/imx-media-csi.c @@ -13,6 +13,7 @@ #include <linux/gcd.h> #include <linux/interrupt.h> #include <linux/module.h> +#include <linux/pinctrl/consumer.h> #include <linux/platform_device.h> #include <media/v4l2-ctrls.h> #include <media/v4l2-device.h> Steve, if you're OK with that I was planning to just modify your original patch rather than adding another patch on top. Regards, Hans
