On Fri, Feb 17, 2012 at 10:24:13AM +0100, Guennadi Liakhovetski wrote: > Hi Sascha > > Thanks for the patch. Just one question: > > On Fri, 17 Feb 2012, Sascha Hauer wrote: > > > The i.MX27 dma support was introduced with the initial commit of > > this driver and originally created by me. However, I never got > > this stable due to the racy dma engine and used the EMMA engine > > instead. As the DMA support is most probably unused and broken in > > its current state, remove it. This also helps us to get rid of > > another user of the legacy i.MX DMA support, > > Also, remove the dependency on ARCH_MX* macros as these are scheduled > > for removal. > > > > This patch only removes the use_emma variable and assumes it's > > hardcoded '1'. The resulting dead code is removed in the next patch. > > > > Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > > --- > > drivers/media/video/mx2_camera.c | 21 ++++++++------------- > > 1 files changed, 8 insertions(+), 13 deletions(-) > > > > diff --git a/drivers/media/video/mx2_camera.c b/drivers/media/video/mx2_camera.c > > index 04aab0c..65709e4 100644 > > --- a/drivers/media/video/mx2_camera.c > > +++ b/drivers/media/video/mx2_camera.c > > [snip] > > > @@ -1620,7 +1616,6 @@ static int __devinit mx2_camera_probe(struct platform_device *pdev) > > > > if (res_emma && irq_emma >= 0) { > > dev_info(&pdev->dev, "Using EMMA\n"); > > - pcdev->use_emma = 1; > > pcdev->res_emma = res_emma; > > pcdev->irq_emma = irq_emma; > > if (mx27_camera_emma_init(pcdev)) > > If emma is becoming the only way to use this driver on i.MX27, shouldn't > the EMMA memory and IRQ resources become compulsory? I.e., if any of them > is missing we should error out? Yes, done in 2/2. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- 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