Hi Hans, I'm not very much against it, but I'm not sure how option 2 is significantly simpler than option 1. It's just one cast away from being the same, unless you have some more rework in mind, e.g. making vb2 aware of devices somehow and skip driver's involvement for example? Thanks, Pawel On Sun, Sep 21, 2014 at 10:05 PM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > Hi Marek, Pawel, > > Currently for dma_config (and the dma_sg code that I posted before) drivers have > to allocate a alloc_ctx context, but in practice that just contains a device pointer. > > Is there any reason why we can't just change in struct vb2_queue: > > void *alloc_ctx[VIDEO_MAX_PLANES]; > > to: > > struct device *alloc_ctx[VIDEO_MAX_PLANES]; > > or possibly even just: > > struct device *alloc_ctx; > > That simplifies the code quite a bit and I don't see and need for anything > else. The last option would make it impossible to have different allocation > contexts for different planes, but that might be something that Samsumg needs. > > Regards, > > Hans -- Best regards, Pawel Osciak -- 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
