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

Fix Alsa softvol linear mapping #950

Merged
merged 3 commits into from
Jan 27, 2022

Conversation

roderickvd
Copy link
Member

When --volume-range was set, that propagated to the mapped volume control, but not to the Alsa mixer itself. This caused incorrect mapped_to_linear calculations.

Fixes: #948

@roderickvd roderickvd merged commit c8971dc into librespot-org:dev Jan 27, 2022
michaelherger added a commit to michaelherger/librespot that referenced this pull request Feb 17, 2022
* spotty-dev: (76 commits)
  Quantum-realm level normalisation optimization (librespot-org#965)
  Only log runtime argument if it starts with a dash "-"
  Improved error handling when fetching tokens or track data.
  update changelog
  Prevent shuffle crash
  Remove basic normalisation deprecation warning
  Fix Alsa softvol linear mapping (librespot-org#950)
  simplify get_factor (librespot-org#942)
  Update LMS integration to
  Remove unsafe code (librespot-org#940)
  Save some more CPU cycles in the limiter (librespot-org#939)
  New dynamic limiter for very wide dynamic ranges (librespot-org#935)
  Fix `--device` argument to various backends (librespot-org#938)
  examples/playlist_tracks: Use normal URI parser
  Fix clippy lint warning
  Clean up list_compatible_devices
  Sink: pass ownership of the packet on write()
  Restore giving feedback about auth success. The plugin is checking for this.
  Remove that last couple unwraps from main
  Fix auto fallback for --alsa-mixer-device and --alsa-mixer-index
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Alsa ALSA backend/Mixer audio bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Alsa mixer sets raw alsa volume to 0 for very high volume ranges
1 participant