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

Android StageFright and MediaCodec HEVC hw accel #5374

Closed
wants to merge 3 commits into from
Closed

Android StageFright and MediaCodec HEVC hw accel #5374

wants to merge 3 commits into from

Conversation

jstebbins
Copy link

Adds support for HW HEVC decoding on Android

StageFright decoding works for me on an RK3288 based Android player.

MediaCodec fails with an IllegalStateException during MediaCodec::configure. I have not been able to track down the root cause of this. It could just be a problem with MediaCodec on my platform, but any pointers for debugging would be appreciated.

@MartijnKaijser
Copy link
Member

jenkins build this please
@davilla @koying
any concerns/remarks or whatever on the code/implementation?

@davilla
Copy link
Contributor

davilla commented Sep 17, 2014

looks ok, I don't have RK3288 based Android players.

@koying koying self-assigned this Sep 28, 2014
@MartijnKaijser
Copy link
Member

@topfs2 @koying ping

@MartijnKaijser
Copy link
Member

@jstebbins can you combine the cosmetics commit into the other commits?

@koying
Copy link
Contributor

koying commented Oct 17, 2014

I should get a rk3288 from Rockchip next week, so I'll be able to test

On 17 October 2014 08:19, Martijn Kaijser notifications@github.com wrote:

@jstebbins https://github.com/jstebbins can you combine the cosmetics
commit into the other commits?


Reply to this email directly or view it on GitHub
#5374 (comment).

// valid hvcC data (bitstream) always starts with the value 1 (version)
if(m_to_annexb)
{
if ( in_extradata[0] == 1 )

This comment was marked as spam.

@jstebbins
Copy link
Author

@MartijnKaijser, sure will squash cosmetics today sometime, been unplugged for a few days :)

@MartijnKaijser
Copy link
Member

small nudge

@jstebbins
Copy link
Author

cosmetics combined

@MartijnKaijser MartijnKaijser added v15 Isengard Type: Feature non-breaking change which adds functionality labels Nov 25, 2014
@koying
Copy link
Contributor

koying commented Dec 27, 2014

@jstebbins Could you rebase for inclusion, please

@koying
Copy link
Contributor

koying commented Dec 27, 2014

Nevermind. Did it myself in #6034.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature non-breaking change which adds functionality v15 Isengard
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants