Em Sun, 27 Dec 2015 19:11:36 +0200 Laurent Pinchart <laurent.pinch...@ideasonboard.com> escreveu:
> Hi Mauro, > > On Wednesday 23 December 2015 10:32:42 Mauro Carvalho Chehab wrote: > > Em Wed, 16 Dec 2015 16:03:01 +0200 Sakari Ailus escreveu: > > > On Wed, Dec 16, 2015 at 03:32:15PM +0200, Sakari Ailus wrote: > > > > This is the third version of the unrestricted media entity ID range > > > > support set. I've taken Mauro's comments into account and fixed a number > > > > of bugs as well (omap3isp memory leak and omap4iss stream start). > > > > > > Javier: Mauro told me you might have OMAP4 hardware. Would you be able to > > > test the OMAP4 ISS with these patches? > > > > > > Thanks. > > > > Sakari, > > > > Testing with OMAP4 is not possible. The driver is broken: it doesn't > > support DT, and the required pdata definition is missing. > > What do you mean by missing ? struct iss_platform_data is defined in > include/media/omap4iss.h. > As this driver is not DT, the platform data has to be part of the Kernel tree for the driver to work. However, there are no board-specific data nor any documentation about how to do that inside the Kernel tree. That means that this driver won't work without some OOT patch. So, on its current state, it is broken. It should either be converted to DT and have the needed board definitions added to the existing dts files or be removed. Another possible approach would be to have a patch like the one Javier and I tried to craft by adding the needed platform data into arch/arm/mach-omap2/pdata-quirks.c, but I guess it is better to just use DT instead. Regards, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
