Hi Guennadi, On Wed, Jun 30, 2010 at 09:20:36AM +0200, Guennadi Liakhovetski wrote: > > > --- a/arch/arm/plat-mxc/include/mach/memory.h > > > +++ b/arch/arm/plat-mxc/include/mach/memory.h > > > @@ -44,12 +44,12 @@ > > > */ > > > #define CONSISTENT_DMA_SIZE SZ_8M > > > > > > -#elif defined(CONFIG_MX1_VIDEO) > > > +#elif defined(CONFIG_MX1_VIDEO) || defined(CONFIG_MX2_VIDEO) > > > /* > > > * Increase size of DMA-consistent memory region. > > > * This is required for i.MX camera driver to capture at least four VGA frames. > > > */ > > > #define CONSISTENT_DMA_SIZE SZ_4M > > > -#endif /* CONFIG_MX1_VIDEO */ > > > +#endif /* CONFIG_MX1_VIDEO || CONFIG_MX2_VIDEO */ > > Why not use CONFIG_VIDEO_MX2 here and get rid of CONFIG_MX2_VIDEO? > > Well, firstly for uniformity with MX1 and MX3, Using a common scheme for names on all platforms is fine, but if the existing names are bad better establish a nicer scheme. > secondly not to have to use > (CONFIG_VIDEO_MX2 || CONFIG_VIDEO_MX2_MODULE), ah, didn't notice that MX?_VIDEO is bool while VIDEO_MX? is tristate. That's fine. Still I would prefer a better naming that doesn't force having to look up which variable is for the driver and which is for the arch stuff. > also note, that > CONFIG_MX1_VIDEO is also used for linking of the FIQ handler for the camera. This is just a matter of fixing the corresponding Makefile. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | -- 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
