Skip to content
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

ffmpeg: add v4l2 backports and pending patches #3927

Closed
wants to merge 2 commits into from

Conversation

lrusak
Copy link
Member

@lrusak lrusak commented Oct 21, 2019

All of these patches are either merged upstream or are awaiting approval on the mailing list.

This moves from my original patches to the ones recently submitted by @tmm1

I have yet to runtime test this yet which I will do shortly. @chewitt if you have time you could also test this out (if possible).

@chewitt
Copy link
Member

chewitt commented Oct 25, 2019

These changes currently break as much as they solve for Amlogic, but it's fine to merge them if we don't want to hold up other platforms. Once the next batch of Amlogic vdec/firmware changes from @Elyotna land on kernel mailling lists @tmm1 will need to revalidate the ffmpeg side anyway as the updated vdec drivers include some V4L2 compliance changes. Amlogic reached a maturity point where some deliberate breakage and rework is essential to keep the stack aligned with the kernel stateful APIs. And since Python3 is breaking stuff on the Kodi side anyway.. now's a good time.

@lrusak
Copy link
Member Author

lrusak commented Oct 25, 2019

Unfortunately this patch set doesn't work properly at the moment and I'm still trying to find some time to debug it properly. Hopefully I'll get to it this weekend and we can have a nicer patch set that properly tracks upstream.

@150balbes
Copy link
Contributor

Are there recommendations with which kernel and patches to check these patches ?

@chewitt
Copy link
Member

chewitt commented Jan 31, 2020

Closing as will be superseded by #4166

@chewitt chewitt closed this Jan 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants