On Fri, 3 Apr 2009, Sascha Hauer wrote: > On Fri, Apr 03, 2009 at 02:15:34PM +0200, Guennadi Liakhovetski wrote: > > Wondering, if it still will work then... At least it compiles. BTW, should > > it really also work with IMX? Then you might want to change this > > > > depends on VIDEO_DEV && ARCH_MX1 && SOC_CAMERA > > > > to > > > > depends on VIDEO_DEV && (ARCH_MX1 || ARCH_IMX) && SOC_CAMERA > > This shouldn't be necessary. ARCH_IMX does not have the platform part to > make use of this driver and will never get it. Confused... Then why the whole that "IMX/MX1" in the driver? And why will it never get it - are they compatible or not? Or just there's no demand / chips are EOLed or something... > > but you can do this later, maybe, when you actually get a chance to test > > it on IMX (if you haven't done so yet). > > > > Sascha, we need your ack for the ARM part. > > I'm OK with this driver: I have never worked with FIQs though so I can't > say much to it. Ok, I take it as an "Acked-by" then:-) Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer -- 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