• Dan McGee's avatar
    alsa: fix option parsing and restore default period_time · 27baf691
    Dan McGee authored
    Two bugs here led to a large number of interrupts being generated on the
    sound card when ALSA output is being used. Because we specify no default
    period_time, the sound card gives us 3000 interrupts/sec rather than a more
    sane 20 or 30. This completes the revert of dd7711 already started by
    4ca24f.
    
    The larger bug was in the change to config_get_block_unsigned() and using 0
    as the default value for both 'buffer_time' and 'period_time'. This means
    any pre-setting of these options in newAlsaData() gets wiped out. Add a new
    default for period_time, and ensure default values for buffer_time and
    period_time are used if none are provided by the user.
    Signed-off-by: 's avatarDan McGee <dan@archlinux.org>
    [mk: set defaults in newAlsaData() to fix auto-configuration; renamed
    "_MS" back to "_US" because ALSA expects microseconds, not milliseconds]
    Signed-off-by: 's avatarMax Kellermann <max@duempel.org>
    27baf691
alsa_plugin.c 11.6 KB