Skip to content

Conversation

@popcornmix
Copy link
Member

The Pi's GPU supports high quality resampling, format conversion and up/down mixing.
This PR creates a factory for different resample implementations, moves the current one into ResampleFFMPEG, and adds a ResamplePi GPU accelerated implementation.

More likely for Helix+1 (although it should be low risk), but submitting for visibility.

@popcornmix popcornmix force-pushed the resample_factory branch 2 times, most recently from a3bf3a8 to ce0cafb Compare October 9, 2014 21:09
@FernetMenta
Copy link
Contributor

Looks fine and I can confirm the low risk. Apart from some renaming it does not touch existing behavior.

@MartijnKaijser
Copy link
Member

jenkins build this please

@Memphiz
Copy link
Member

Memphiz commented Oct 11, 2014

@popcornmix i made a pull request to your branch with sync of xcode and visual studio projects (should cure the build errors).

@popcornmix
Copy link
Member Author

Thanks @Memphiz. jenkins build this please

@MartijnKaijser
Copy link
Member

@FernetMenta any comments? (win32 fail unrelated)

@MartijnKaijser
Copy link
Member

jenkins build this please

@FernetMenta
Copy link
Contributor

@MartijnKaijser comment referring to what? Anything changed after my first comment here?

@MartijnKaijser
Copy link
Member

ah missed that you already commented. sorry.

popcornmix and others added 7 commits October 17, 2014 11:10
This replaces the format conversion, up/down mixing and resampling code from ActiveAE with a GPU accelerated version.
Should significantly reduce CPU when using paplayer or dvdplayer.

Requires updated firmware
@topfs2
Copy link
Contributor

topfs2 commented Oct 17, 2014

@FernetMenta @popcornmix Helix or Helix+1?

@popcornmix
Copy link
Member Author

I'd prefer Helix if that's acceptable.

@topfs2
Copy link
Contributor

topfs2 commented Oct 17, 2014

I'm fine with it :) @FernetMenta you too I reckon from your first response?

@topfs2 topfs2 added the Helix label Oct 17, 2014
@FernetMenta
Copy link
Contributor

Yes, I am fine with it too

@topfs2
Copy link
Contributor

topfs2 commented Oct 18, 2014

Jenkins build this please

@MartijnKaijser
Copy link
Member

jenkins build this please (lowercase ;) )

MartijnKaijser added a commit that referenced this pull request Oct 18, 2014
[AE] Add resample factory and Pi accelerated resampling
@MartijnKaijser MartijnKaijser merged commit 31ce987 into xbmc:master Oct 18, 2014
@MartijnKaijser MartijnKaijser added this to the Helix 14.0-alpha5 milestone Oct 18, 2014
@popcornmix popcornmix deleted the resample_factory branch October 18, 2014 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants