On 09/09/2014 12:52 AM, Hans Verkuil wrote: > On 09/09/14 09:49, Jean-Michel Hautbois wrote: >> 2014-08-27 16:23 GMT+02:00 Steve Longerbeam <steve_longerbeam@xxxxxxxxxx>: >> >>> Hi Jean-Michel, Phillip, >> Hi Steve, >> >>> I've done some work on Philipp's June 12 patchset, converting >>> the CSI driver to a CSI subdev entity, and fixing some issues here >>> and there. This June 12 patchset doesn't appear to be a fully working >>> driver, Phillip correct me if I am wrong. I can post this work as it >>> exists, it is incomplete but compiles. >> Dos it compile against a 3.17-rc3 kernel :) ? >> >>> I've also worked out what I think is a workable video pipeline graph for i.MX, >>> suitable for defining the entities, pads, and links. Unfortunately I haven't >>> been able to spend as much time as I'd like on it. >> This is very interesting, do you have written this somewhere ? >> >>> The complete driver I posted to the list does have some minor issues >>> mostly suggested by Hans Verkuil (switch to new selection API instead >>> of cropping API for example). It is a full featured driver but it does not >>> implement the media device framework, i.e. user does not have direct >>> control of the video pipeline, rather the driver chooses the pipeline based >>> on the traditional inputs from user (video format and controls). >>> >>> If there is interest I can submit another version of the traditional driver >>> to resolve the issues. But media device is a major rework, so I don't >>> know whether it would make sense to start from the traditional driver >>> and then implement media device on top later, since media device >>> is almost a complete rewrite. >> I, at least, am interested by this driver, even in its "traditionnal" >> form :). If you don't want to submit it directly because this is not >> using media controller, this is ok, you can provide me a git repo in >> order to get it, or send a patchset. > Is it possible to create a staging driver? Even if there are bits missing, > having the code in the kernel as a staging driver would help a lot. Hi Hans, that's a good idea. I can post it as a staging driver. The capture driver does require more support in the i.MX IPU driver which is not yet merged or proposed. Phillip has forwarded most of them to drm-next, but there are a few more required. I need to post those patches to drm-next as a first step. I can start working on converting the driver to staging and addressing the earlier issues, but posting it to media-tree will need to wait until the IPU patches are merged, unless I include the IPU patches along with the capture driver patchset. Steve -- 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