[alsa-devel] [PATCH 1/1] ALSA: virtuoso: add de-emphasis control

Clemens Ladisch clemens at ladisch.de
Sun Dec 16 17:37:45 CET 2018


tom.ty89 at gmail.com wrote:
> +	switch (data->current_rate) {
> +	case 32000:
> +	case 64000:
> +		reg |= PCM1796_DMF_32;
> +		break;
> +	case 44100:
> +	case 88200:
> +	case 176400:
> +		reg |= PCM1796_DMF_441;
> +		break;
> +	case 48000:
> +	case 96000:
> +	case 192000:
> +		reg |= PCM1796_DMF_48;
> +		break;

The DAC does not have filters for frequencies higher than 48 kHz.

> +static const struct snd_kcontrol_new deemph_control = {
> +	.iface = SNDRV_CTL_ELEM_IFACE_MIXER,
> +	.name = "De-emphasis Switch",

This should be called "De-emphasis Playback Switch".
(But I see that other drivers are quite inconsistent about this.)


Regards,
Clemens


More information about the Alsa-devel mailing list