RE: [REVIEW PATCH 00/14] OMAP3 camera + ISP + MT9P012 sensor driver v2
Thanks, Vaibhav Hiremath -Original Message- From: video4linux-list-boun...@redhat.com [mailto:video4linux-list- boun...@redhat.com] On Behalf Of Sakari Ailus Sent: Monday, January 19, 2009 3:50 PM To: Aguirre Rodriguez, Sergio Alberto Cc: Nagalla, Hari; video4linux-l...@redhat.com; linux- o...@vger.kernel.org; Tuukka.O Toivonen; linux-media@vger.kernel.org Subject: Re: [REVIEW PATCH 00/14] OMAP3 camera + ISP + MT9P012 sensor driver v2 Aguirre Rodriguez, Sergio Alberto wrote: Hi, I'm sending the following patchset for review to the relevant lists (linux-omap, v4l, linux-media). Includes: - Omap3 camera core + ISP drivers. - MT9P012 sensor driver (adapted to 3430SDP) - DW9710 lens driver (adapted to work with MT9P012 for SDP) - Necessary v4l2-int-device changes to make above drivers work - Redefine OMAP3 ISP platform device. - Review comments fixed from: (Thanks a lot for their time and help) - Hans Verkuil - Tony Lindgreen - Felipe Balbi - Vaibhav Hiremath - David Brownell Hi Sergio, We should try to figure out how we could synchronise our version of the ISP and camera ASAP before making any more changes... I wouldn't want to start posting a competing version. ;-) [Hiremath, Vaibhav] Sakari, Can you share your version of code (either in form of patches or source) here onto the mailing list, so that everybody those who are interested will be aware of changes? Atleast for me I would get prior view of changes which might affect BT656 support (posted on top of Sergio's patch). Again one more thing I would like to bring to your notice is about sub-device framework. Also we need to plan for the migration from V4L2-int to sub-device framework, which is of equal important. I feel the more time we spend in merging and aligning offline, more we are carrying risk. According to me as I mentioned before, the plan should be to push ISP-Camera with V4L2-int interface as early as possible with whatever minimal sensor/decoder support we have today, which will make sure that our underneath ISP-library is in place. Once that is placed in, we can have additional patches on top of it to add more features. In this way, we can plan for migration to sub-device framework and also be easier and simpler. Even if any customers are interested, they can pick it up the ISP library and start development on top of it. -- Sakari Ailus sakari.ai...@nokia.com -- video4linux-list mailing list Unsubscribe mailto:video4linux-list- requ...@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [REVIEW PATCH 00/14] OMAP3 camera + ISP + MT9P012 sensor driver v2
Hiremath, Vaibhav wrote: [Hiremath, Vaibhav] Sakari, Can you share your version of code (either in form of patches or source) here onto the mailing list, so that everybody those who are interested will be aware of changes? I will synchronise soon with Sergio (as he noted). After that we'll send a new patchset. I was thinking of separating the ISP and camera driver patches from other hardware dependent patches. The resulting patchset wouldn't be that huge anymore but on the other hand it wouldn't be very usable as such. Atleast for me I would get prior view of changes which might affect BT656 support (posted on top of Sergio's patch). Again one more thing I would like to bring to your notice is about sub-device framework. Also we need to plan for the migration from V4L2-int to sub-device framework, which is of equal important. I feel the more time we spend in merging and aligning offline, more we are carrying risk. I agree that we should be moving to v4l2 sub-device at some point, preferrably sooner than later, but I think the ISP driver should get a much much better interface than it currently has. So that should be part of the task. According to me as I mentioned before, the plan should be to push ISP-Camera with V4L2-int interface as early as possible with whatever minimal sensor/decoder support we have today, which will make sure that our underneath ISP-library is in place. Once that is placed in, we can have additional patches on top of it to add more features. I agree. But I'd say that fixing the bugs and cleaning up the code is as least as important than adding new features. In this way, we can plan for migration to sub-device framework and also be easier and simpler. Even if any customers are interested, they can pick it up the ISP library and start development on top of it. There's still a long way to get there --- the ISP driver's current interface and internals don't mix well with either v4l2 sub-device or v4l2-int-if. For example, there's no ISP object, just function calls and then output frame size / frame interval enumeration doesn't work properly for YUV (maybe not even for RAW10). Enumerating frame size has side effects. The ISP driver doesn't have a standard interface, it's now specific to OMAP 3. And this is just an example... Regards, -- Sakari Ailus sakari.ai...@maxwell.research.nokia.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [REVIEW PATCH 00/14] OMAP3 camera + ISP + MT9P012 sensor driver v2
On Friday 06 February 2009 16:03:26 Sakari Ailus wrote: Hiremath, Vaibhav wrote: [Hiremath, Vaibhav] Sakari, Can you share your version of code (either in form of patches or source) here onto the mailing list, so that everybody those who are interested will be aware of changes? I will synchronise soon with Sergio (as he noted). After that we'll send a new patchset. I was thinking of separating the ISP and camera driver patches from other hardware dependent patches. The resulting patchset wouldn't be that huge anymore but on the other hand it wouldn't be very usable as such. Atleast for me I would get prior view of changes which might affect BT656 support (posted on top of Sergio's patch). Again one more thing I would like to bring to your notice is about sub-device framework. Also we need to plan for the migration from V4L2-int to sub-device framework, which is of equal important. I feel the more time we spend in merging and aligning offline, more we are carrying risk. I agree that we should be moving to v4l2 sub-device at some point, preferrably sooner than later, but I think the ISP driver should get a much much better interface than it currently has. So that should be part of the task. According to me as I mentioned before, the plan should be to push ISP-Camera with V4L2-int interface as early as possible with whatever minimal sensor/decoder support we have today, which will make sure that our underneath ISP-library is in place. Once that is placed in, we can have additional patches on top of it to add more features. I agree. But I'd say that fixing the bugs and cleaning up the code is as least as important than adding new features. In this way, we can plan for migration to sub-device framework and also be easier and simpler. Even if any customers are interested, they can pick it up the ISP library and start development on top of it. There's still a long way to get there --- the ISP driver's current interface and internals don't mix well with either v4l2 sub-device or v4l2-int-if. For example, there's no ISP object, just function calls and then output frame size / frame interval enumeration doesn't work properly for YUV (maybe not even for RAW10). Enumerating frame size has side effects. The ISP driver doesn't have a standard interface, it's now specific to OMAP 3. And this is just an example... Please note that it is easy to add a subdevice-specific set of ops if needed. It is not implemented at the moment, but if you need it please ask and I can show you how to do it. In general, if you think something is missing in v4l2_subdev or if something is awkward to use, please ask me! I'd be happy to help you with that. It's an internal API, so changes are possible and almost certainly necessary for omap anyway. Regards, Hans -- Hans Verkuil - video4linux developer - sponsored by TANDBERG -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [REVIEW PATCH 00/14] OMAP3 camera + ISP + MT9P012 sensor driver v2
Thanks, Vaibhav Hiremath -Original Message- From: Sakari Ailus [mailto:sakari.ai...@maxwell.research.nokia.com] Sent: Friday, February 06, 2009 8:33 PM To: Hiremath, Vaibhav Cc: Ailus Sakari (Nokia-D/Helsinki); Aguirre Rodriguez, Sergio Alberto; Nagalla, Hari; video4linux-l...@redhat.com; linux- o...@vger.kernel.org; Toivonen Tuukka.O (Nokia-D/Oulu); linux- me...@vger.kernel.org Subject: Re: [REVIEW PATCH 00/14] OMAP3 camera + ISP + MT9P012 sensor driver v2 Hiremath, Vaibhav wrote: [Hiremath, Vaibhav] Sakari, Can you share your version of code (either in form of patches or source) here onto the mailing list, so that everybody those who are interested will be aware of changes? I will synchronise soon with Sergio (as he noted). After that we'll send a new patchset. I was thinking of separating the ISP and camera driver patches from other hardware dependent patches. The resulting patchset wouldn't be that huge anymore but on the other hand it wouldn't be very usable as such. [Hiremath, Vaibhav] How much change can we expect as compared to the patches posted by Sergio on 14th Jan? Atleast for me I would get prior view of changes which might affect BT656 support (posted on top of Sergio's patch). Again one more thing I would like to bring to your notice is about sub-device framework. Also we need to plan for the migration from V4L2-int to sub-device framework, which is of equal important. I feel the more time we spend in merging and aligning offline, more we are carrying risk. I agree that we should be moving to v4l2 sub-device at some point, preferrably sooner than later, but I think the ISP driver should get a much much better interface than it currently has. So that should be part of the task. According to me as I mentioned before, the plan should be to push ISP-Camera with V4L2-int interface as early as possible with whatever minimal sensor/decoder support we have today, which will make sure that our underneath ISP-library is in place. Once that is placed in, we can have additional patches on top of it to add more features. I agree. But I'd say that fixing the bugs and cleaning up the code is as least as important than adding new features. [Hiremath, Vaibhav] It would be really great if you could share your plan of action for ISP-Camera driver that would really help to plan our milestone accordingly. In this way, we can plan for migration to sub-device framework and also be easier and simpler. Even if any customers are interested, they can pick it up the ISP library and start development on top of it. There's still a long way to get there --- the ISP driver's current interface and internals don't mix well with either v4l2 sub-device or v4l2-int-if. For example, there's no ISP object, just function calls and then output frame size / frame interval enumeration doesn't work properly for YUV (maybe not even for RAW10). Enumerating frame size has side effects. The ISP driver doesn't have a standard interface, it's now specific to OMAP 3. And this is just an example... Regards, -- Sakari Ailus sakari.ai...@maxwell.research.nokia.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [REVIEW PATCH 00/14] OMAP3 camera + ISP + MT9P012 sensor driver v2
Aguirre Rodriguez, Sergio Alberto wrote: Hi, I'm sending the following patchset for review to the relevant lists (linux-omap, v4l, linux-media). Includes: - Omap3 camera core + ISP drivers. - MT9P012 sensor driver (adapted to 3430SDP) - DW9710 lens driver (adapted to work with MT9P012 for SDP) - Necessary v4l2-int-device changes to make above drivers work - Redefine OMAP3 ISP platform device. - Review comments fixed from: (Thanks a lot for their time and help) - Hans Verkuil - Tony Lindgreen - Felipe Balbi - Vaibhav Hiremath - David Brownell Hi Sergio, We should try to figure out how we could synchronise our version of the ISP and camera ASAP before making any more changes... I wouldn't want to start posting a competing version. ;-) -- Sakari Ailus sakari.ai...@nokia.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html