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

Headphones do not work properly #40

Open
alpgarcia opened this issue Jun 5, 2021 · 2 comments · Fixed by #42
Open

Headphones do not work properly #40

alpgarcia opened this issue Jun 5, 2021 · 2 comments · Fixed by #42
Assignees
Labels
bug Something isn't working
Milestone

Comments

@alpgarcia
Copy link
Contributor

Describe the bug
The internal speaker does not turn off when headphones are plugged in.

Custom Firmware version: v3.0 Alpha 1

Console (please complete the following information):

  • Brand: Powkiddy
  • Model: a12, a13, PS5000
  • Version: -
  • Mods: -

To Reproduce
Steps to reproduce the behavior:

  1. Plug your headphones in.
  2. Start a game.

Expected behavior
The internal speaker is turned off when the headphones are plugged in.

Screenshots

Additional context
It seems the hotplug script is missing.

@alpgarcia alpgarcia added the bug Something isn't working label Jun 5, 2021
@alpgarcia alpgarcia self-assigned this Jun 5, 2021
@alpgarcia alpgarcia added this to the 3.0 milestone Jun 5, 2021
@alpgarcia alpgarcia linked a pull request Jun 13, 2021 that will close this issue
@alpgarcia
Copy link
Contributor Author

  • In the case of a12/13 it seems to be a problem of the build that is already solved when building the rootfs from scratch.
  • For the PS5000 the issue is a bit more complicated as the volume controls didn't work and after changing the alsa configuration aliases (we need to add rockchip_rk312x cards.rk817 in the file buildroot/board/rockchip/ps5000/fs-overlay/usr/share/alsa/cards/rockchip_rk312x.conf) the headphones stopped working. The fix seems to be to remove the lock true line from the hook args for the playback.pcm entry in the configuration for RK817 driver (see the change in the PR).

Side note: for the a12/13 the alias is not needed and everything worked as it is, so I didn't explore it following a simple approach: if it ain't broke, don't fix it. Nevertheless, it would be a good idea in the future to have a look and understand what is actually happening and find out whether we have part of the driver config that is actually never used.

@alpgarcia
Copy link
Contributor Author

Reopening as it seems there is a regression here on the PS5000 at least. As the system didn't change at all, it is probably related to the init process of the external rootfs.

@alpgarcia alpgarcia reopened this Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant