-
Notifications
You must be signed in to change notification settings - Fork 64
WIP: Hantro G1 H.264 support #26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
This can be used with https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/merge_requests/171. |
In case that is interesting for you, here's two left over from my previous work: https://github.com/ndufresne/libva-v4l2-request/commits/rockchip-support It fixes the mplane detection, and improves performance for DeriveImage calls. DMABuf exportation was not working for me in GStreamer, I concluded best was to port GStreamer to the new surface export API instead of relying on a derived image. |
I tried this against linux-media master tree today and it didn't work. The driver still tries to set the format in arbitrary order which cause an EBUSY, a restriction that exist on Hantro driver but not Cedrus. How did you manage to get this to work on Hantro driver ? |
This can be used to reduce number of issued ioctls, by setting multiple controls at once. Signed-off-by: Philipp Zabel <[email protected]>
This can be used to query codec mode controls, such as decode mode and start code for h.264. Signed-off-by: Philipp Zabel <[email protected]>
Update to the merged stateless h.264 kernel interface, as of commit c3adb85745ca ("media: uapi: h264: Get rid of the p0/b0/b1 ref-lists"). Signed-off-by: Philipp Zabel <[email protected]>
Signed-off-by: Philipp Zabel <[email protected]>
Signed-off-by: Philipp Zabel <[email protected]>
If the driver reports that it expects H.264 Annex B start codes, provide them. Signed-off-by: Philipp Zabel <[email protected]>
Signed-off-by: Philipp Zabel <[email protected]>
Signed-off-by: Philipp Zabel <[email protected]>
Signed-off-by: Philipp Zabel <[email protected]>
Signed-off-by: Philipp Zabel <[email protected]>
Signed-off-by: Philipp Zabel <[email protected]>
This requires modifications in gst-plugins-bad, libva, and gstreamer-vaapi. Signed-off-by: Philipp Zabel <[email protected]>
This requires modifications in gst-plugins-bad, libva, and gstreamer-vaapi. Signed-off-by: Philipp Zabel <[email protected]>
Signed-off-by: Philipp Zabel <[email protected]>
At this point it is unclear whether to store the Inter Y scaling matrix at index 1 (h.264 standard) or 3 [1]. Store it at both indices for now. [1] https://lore.kernel.org/linux-media/HE1PR06MB40118B3C30939861DD91113CACBE0@HE1PR06MB4011.eurprd06.prod.outlook.com/T/#m60af013132990335d525e6e5600c5f5bd692cfbf Signed-off-by: Philipp Zabel <[email protected]>
This should allow to decode H.264 videos on Hantro G1 with the currently merged (experimental) kernel interface for V4L2 stateless H.264 decoders.
WIP because this depends on #25 and intel/libva#332. The kernel interface is not yet stable, and the scaling matrix interface still has a big FIXME.