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

Add basic Microphone support on Windows. #13229

Merged
merged 8 commits into from
Aug 1, 2020
Merged

Conversation

shenweip
Copy link
Contributor

Helps #12336, #10970, tested TalkMan, Audio Mechanica V8d.
TalkMan works now.
1
In Audio Mechanica V8d, I can hear the playback of the recording, but I don't see the dots jumping on the screen like a real PSP, it may be a issue apart.
There are still a bunch of HLE functions about microphone not implemented, also need to support on other platforms.

Core/HLE/sceKernel.cpp Outdated Show resolved Hide resolved
@hrydgard hrydgard added the Audio label Aug 1, 2020
@hrydgard hrydgard added this to the v1.11.0 milestone Aug 1, 2020
… you exit to menu from ingame.

Also use memcpy instead of a loop.
@hrydgard
Copy link
Owner

hrydgard commented Aug 1, 2020

I had always thought we'd end up implementing this using the recording capabilities of the audio APIs (DSound, WASAPI) but I guess using the capture device API makes sense too, especially when we're recording video+audio like in Invizimals.

Has it been tested with that game, by the way? (of course not a requirement, just curious)

@shenweip
Copy link
Contributor Author

shenweip commented Aug 1, 2020

I have not tested Invizimals, I forget this game also use microphone:p.

@hrydgard hrydgard merged commit 45b05b3 into hrydgard:master Aug 1, 2020
@jordenlang32robloxwikia
Copy link

I'll merge this to my fork of ppsspp.

@Panderner Panderner mentioned this pull request Aug 1, 2020
@shenweip shenweip deleted the UsbMic branch August 2, 2020 02:15
@zackzick2
Copy link

Amazing bro. We will waiting improvement progress to playing invizimals with microphone perfectly ,👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants