Hi Sergei, Vladimir, > >> Subject: [PATCH v5] V4L2: soc_camera: Renesas R-Car VIN driver > > >> From: Vladimir Barinov <vladimir.barinov@xxxxxxxxxxxxxxxxxx> > > >> Add Renesas R-Car VIN (Video In) V4L2 driver. > > >> Based on the patch by Phil Edworthy <phil.edworthy@xxxxxxxxxxx>. > > > I've seen old patches that add VIN to the Marzen board, do you have an > > updated version? > > The last version of that patchset is 4, here it is archived: > > http://marc.info/?l=linux-sh&m=136865481429756 > http://marc.info/?l=linux-sh&m=136865499029807 > http://marc.info/?l=linux-sh&m=136865509129843 > http://marc.info/?l=linux-sh&m=136865520029900 First of all, thank you for your work on this driver. I have tried your patches on the Marzen board using an Expansion Board with an OmniVision 10635 camera (progressive BT656), using an out-of-tree driver. There appears to be an issue with the interrupt handling compared to my original driver. Using a simple test app I wrote, I get an unhandled irq if the app does some work after stopping the capture. In this case, the work after capture is storing a captured image to a file. As a dirty hack to see what's actually being captured, I just commented out the code that checks the interrupt status: // if (!int_status) // goto done; This allows me to save the captured image. However, this shows about 2/3rds valid picture data (though it looks vertically shifted), with the rest black. Also, a couple of other lines are black. I realise that you don't have the Marzen Expansion Board & don't have an ov10635 camera. However, unfortunately, I don't have much time that I can spend on this. Do any of the boards you have use a progressive camera? Regards Phil -- 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