On Tue, 7 Jun 2011 09:44:45 +0200 (CEST) Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote: > > +obj-$(CONFIG_VIDEO_MMP_CAMERA) += marvell-ccic/ > > Wouldn't it be better to have only one symbol, selecting the marvell-ccic > directory in the Makefile and have all CAFE implementations select that > symbol? Except there's two drivers in that directory and you'll almost never want to build them both. I guess I could replace one Makefile line with two Kconfig lines, but I'm not sure it would improve things. > > +config VIDEO_MMP_CAMERA > > + tristate "Marvell Armada 610 integrated camera controller support" > > + depends on ARCH_MMP && I2C && VIDEO_V4L2 > > + select VIDEO_OV7670 > > Is ov7670 really _integrated_ with the camera controller? Can it not be > used with any other sensor? It should work with other sensors, yes, modulo the format information that now has to be kept in the controller driver. I'd be more than happy to see it work with something other than the ov7670, I just don't have the hardware to test it. Thanks, jon -- 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
