GNOME Bugzilla – Bug 783097
omx: add support for Zynq UltraScale+
Last modified: 2017-07-03 14:20:37 UTC
We are working on integrating gst-omx for the Zynq UltraScale+ board. This is the first commit to do so.
Created attachment 352579 [details] [review] add 'zynqultrascaleplus' as OMX target Adding support for the 'Zynq UltraScale+' as a new OMX target. See https://www.xilinx.com/products/silicon-devices/soc/zynq-ultrascale-mpsoc.html for details about the platform.
I'm guessing there will be more patches? :)
(In reply to Tim-Philipp Müller from comment #2) > I'm guessing there will be more patches? :) Yes, adding extra features. But I'll split them up. This one is already usable as it so can already be reviewed and merged.
That's enough to make it run? If not, please add all the other commits here too.
I think we should collect all the patches before starting to merge things to see if it's something we want to have upstream or not, which may depend on how intrusive the patches are.
(In reply to Sebastian Dröge (slomo) from comment #4) > That's enough to make it run? If not, please add all the other commits here > too. Yes it is. With just this patch we can already use the H264 encoder and decoder on the board. The other patches (which I still have to clean-up/rebase) are extra features such as new properties, DMA support, etc which, I think, should be discussed/merged separately as they are not required to have the existing code working.
Created attachment 352769 [details] [review] omxh264enc: set 'parsed=true' on src caps Allow to remove h264parse from pipeline and so remove the latency introduced by the parser.
Comment on attachment 352769 [details] [review] omxh264enc: set 'parsed=true' on src caps No longer needed once bug #784344 will have been fixed.
Thanks. I do think long term we'll need to re-org a bit if the number of target becomes large, but in the mean time this seems to be the way forward. Attachment 352579 [details] pushed as 2be1214 - add 'zynqultrascaleplus' as OMX target