[PATCH] media i.MX27 camera: remove legacy dma support

Sascha Hauer s.hauer at pengutronix.de
Wed Aug 24 03:36:14 EDT 2011


Hi Guennadi,

On Wed, Aug 24, 2011 at 09:19:24AM +0200, Guennadi Liakhovetski wrote:
> Sure, if it's broken, let's remove it. But there are a couple of points, 
> that we have to fix in this patch. Sorry, a stupid question: has this been 
> tested on i.MX27?

Nope, I currently do not have mainline board support for this driver.
Could be a good opportunity to add some...

Your other points are totally valid and I will fix them in the next
round. Let's first see if someone proves me wrong and says this dma
support is indeed working.

> > -	return IRQ_HANDLED;
> > -}
> > -#else
> >  static irqreturn_t mx27_camera_irq(int irq_csi, void *data)
> >  {
> >  	return IRQ_NONE;
> >  }
> 
> If this is really all, what's needed for i.MX27 ISR, let's remove it 
> completely. But maybe you could explain to me, how it is now supposed to 
> work on i.MX27. In probe() we have
> 
> 	irq_handler_t mx2_cam_irq_handler = cpu_is_mx25() ? mx25_camera_irq
> 		: mx27_camera_irq;
> 
> 	...
> 
> 	err = request_irq(pcdev->irq_csi, mx2_cam_irq_handler, 0,
> 			MX2_CAM_DRV_NAME, pcdev);
> 
> So, after this patch i.MX27 will always have a dummy camera ISR and just 
> use EMMA, right?

Yes, only the EMMA irq is used, we can remove this one.

> Then maybe we have to make EMMA resource availability 
> compulsory on those SoCs, and not optional, as now? You'll have to make 
> emma the only possibility on i.MX27, then pcdev->use_emma will disappear, 
> locations like

ok.

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 |



More information about the linux-arm-kernel mailing list