[alsa-devel] [PATCH] conf/ucm: rt5651: Add UCM config for bytcr-rt5651

Takashi Iwai tiwai at suse.de
Tue Oct 24 21:53:42 CEST 2017


On Tue, 24 Oct 2017 13:27:12 +0200,
Carlo Caione wrote:
> 
> From: Carlo Caione <carlo at endlessm.com>
> 
> Add a UCM configuration for the rt5651 codec on Intel's Cherry-Trail
> platform. Adapted from [0].
> 
> [0] https://github.com/plbossart/UCM/tree/master/bytcr-rt5651
> 
> Signed-off-by: Carlo Caione <carlo at endlessm.com>

Pierre, is this upstream-ready?
I'm a bit worried by the current status of UCM profiles for Intel SST
things.  I know splitting UCM into another repo is pending, but I
think it's OK to apply a single patch before moving to another repo.


thanks,

Takashi


> ---
>  configure.ac                                |   1 +
>  src/conf/ucm/Makefile.am                    |   2 +-
>  src/conf/ucm/bytcr-rt5651/HiFi.conf         | 126 +++++++++++++++++++++++++
>  src/conf/ucm/bytcr-rt5651/Makefile.am       |   4 +
>  src/conf/ucm/bytcr-rt5651/bytcr-rt5651.conf | 138 ++++++++++++++++++++++++++++
>  5 files changed, 270 insertions(+), 1 deletion(-)
>  create mode 100644 src/conf/ucm/bytcr-rt5651/HiFi.conf
>  create mode 100644 src/conf/ucm/bytcr-rt5651/Makefile.am
>  create mode 100644 src/conf/ucm/bytcr-rt5651/bytcr-rt5651.conf
> 
> diff --git a/configure.ac b/configure.ac
> index fbcfa829..c16f2461 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -703,6 +703,7 @@ AC_OUTPUT(Makefile doc/Makefile doc/pictures/Makefile doc/doxygen.cfg \
>  	  src/conf/ucm/VEYRON-I2S/Makefile \
>  	  src/conf/ucm/chtrt5645/Makefile \
>  	  src/conf/ucm/DB410c/Makefile \
> +	  src/conf/ucm/bytcr-rt5651/Makefile \
>  	  src/conf/topology/Makefile \
>  	  src/conf/topology/broadwell/Makefile \
>  	  modules/Makefile modules/mixer/Makefile modules/mixer/simple/Makefile \
> diff --git a/src/conf/ucm/Makefile.am b/src/conf/ucm/Makefile.am
> index ea9b3850..2bd4ebe7 100644
> --- a/src/conf/ucm/Makefile.am
> +++ b/src/conf/ucm/Makefile.am
> @@ -1 +1 @@
> -SUBDIRS=DAISY-I2S PandaBoard PandaBoardES SDP4430 tegraalc5632 PAZ00 GoogleNyan broadwell-rt286 skylake-rt286 VEYRON-I2S chtrt5645 DB410c broxton-rt298
> +SUBDIRS=DAISY-I2S PandaBoard PandaBoardES SDP4430 tegraalc5632 PAZ00 GoogleNyan broadwell-rt286 skylake-rt286 VEYRON-I2S chtrt5645 DB410c broxton-rt298 bytcr-rt5651
> diff --git a/src/conf/ucm/bytcr-rt5651/HiFi.conf b/src/conf/ucm/bytcr-rt5651/HiFi.conf
> new file mode 100644
> index 00000000..b9da30f7
> --- /dev/null
> +++ b/src/conf/ucm/bytcr-rt5651/HiFi.conf
> @@ -0,0 +1,126 @@
> +# Adapted from https://github.com/plbossart/UCM/tree/master/bytcr-rt5651
> +
> +SectionVerb {
> +	EnableSequence [
> +		cdev "hw:bytcrrt5651"
> +		exec "echo HiFi enable sequence"
> +	]
> +
> +	DisableSequence [
> +		exec "echo HiFi disable sequence"
> +	]
> +
> +	Value {
> +		PlaybackPCM "hw:bytcrrt5651"
> +		CapturePCM "hw:bytcrrt5651"
> +	}
> +}
> +
> +SectionDevice."Headphones" {
> +	Comment "Headphones playback"
> +
> +	ConflictingDevice [
> +		"Speaker"
> +	]
> +
> +	EnableSequence [
> +		cdev "hw:bytcrrt5651"
> +		exec "echo Headphones playback enable sequence"
> +		cset "name='Headphone Switch' on"
> +		cset "name='HPO L Playback Switch' on"
> +		cset "name='HPO R Playback Switch' on"
> +		cset "name='HP Playback Volume' 100"
> +	]
> +
> +	DisableSequence [
> +		cdev "hw:bytcrrt5651"
> +		exec "echo Headphones playback disable sequence"
> +		cset "name='Headphone Switch' off"
> +		cset "name='HPO L Playback Switch' off"
> +		cset "name='HPO R Playback Switch' off"
> +		cset "name='HP Playback Volume' 0"
> +	]
> +
> +	Value {
> +		PlaybackChannels "2"
> +		JackControl "Headphone Jack"
> +	}
> +}
> +
> +SectionDevice."Speaker" {
> +	Comment "Speaker playback"
> +
> +	ConflictingDevice [
> +		"Headphones"
> +	]
> +
> +	EnableSequence [
> +		cdev "hw:bytcrrt5651"
> +		exec "echo Speaker enable sequence"
> +		cset "name='Speaker Switch' on"
> +	]
> +
> +	DisableSequence [
> +		cdev "hw:bytcrrt5651"
> +		exec "echo Speaker disable sequence"
> +		cset "name='Speaker Switch' off"
> +	]
> +
> +	Value {
> +		PlaybackChannels "2"
> +	}
> +}
> +
> +SectionDevice."Handset" {
> +	Comment "Handset Microphone"
> +
> +	ConflictingDevice [
> +		"Mainmic"
> +	]
> +
> +	EnableSequence [
> +		cdev "hw:bytcrrt5651"
> +		exec "echo Handset Microphone enable sequence"
> +		cset "name='RECMIXL BST1 Switch' on"
> +		cset "name='RECMIXR BST1 Switch' on"
> +	]
> +
> +	DisableSequence [
> +		cdev "hw:bytcrrt5651"
> +		exec "echo Handset Microphone disable sequence"
> +		cset "name='RECMIXL BST1 Switch' off"
> +		cset "name='RECMIXR BST1 Switch' off"
> +	]
> +
> +	Value {
> +		CaptureChannels "2"
> +		JackControl "Headset Mic Jack"
> +	}
> +}
> +
> +
> +SectionDevice."Mainmic" {
> +	Comment "Main Microphone"
> +
> +	ConflictingDevice [
> +		"Handset"
> +	]
> +
> +	EnableSequence [
> +		cdev "hw:bytcrrt5651"
> +		exec "echo Main Microphone enable sequence"
> +		cset "name='RECMIXL BST2 Switch' on"
> +		cset "name='RECMIXR BST2 Switch' on"
> +	]
> +
> +	DisableSequence [
> +		cdev "hw:bytcrrt5651"
> +		exec "echo Main Microphone disable sequence"
> +		cset "name='RECMIXL BST2 Switch' off"
> +		cset "name='RECMIXR BST2 Switch' off"
> +	]
> +
> +	Value {
> +		CaptureChannels "2"
> +	}
> +}
> diff --git a/src/conf/ucm/bytcr-rt5651/Makefile.am b/src/conf/ucm/bytcr-rt5651/Makefile.am
> new file mode 100644
> index 00000000..95aec978
> --- /dev/null
> +++ b/src/conf/ucm/bytcr-rt5651/Makefile.am
> @@ -0,0 +1,4 @@
> +alsaconfigdir = @ALSA_CONFIG_DIR@
> +ucmdir = $(alsaconfigdir)/ucm/bytcr-rt5651
> +ucm_DATA = bytcr-rt5651.conf HiFi.conf
> +EXTRA_DIST = $(ucm_DATA)
> diff --git a/src/conf/ucm/bytcr-rt5651/bytcr-rt5651.conf b/src/conf/ucm/bytcr-rt5651/bytcr-rt5651.conf
> new file mode 100644
> index 00000000..2dcc72cc
> --- /dev/null
> +++ b/src/conf/ucm/bytcr-rt5651/bytcr-rt5651.conf
> @@ -0,0 +1,138 @@
> +# Adapted from https://github.com/plbossart/UCM/tree/master/bytcr-rt5651
> +
> +SectionUseCase."HiFi" {
> +	File "HiFi"
> +	Comment "Play HiFi quality Music"
> +}
> +
> +SectionDefaults [
> +	cdev "hw:bytcrrt5651"
> +
> +	# media mixer settings
> +	#   compress
> +	cset "name='media0_in Gain 0 Switch' on"
> +	cset "name='media0_in Gain 0 Volume' 0"
> +	#   normal
> +	cset "name='media1_in Gain 0 Switch' on"
> +	cset "name='media1_in Gain 0 Volume' 0"
> +	#   swm loopback
> +	cset "name='media2_in Gain 0 Switch' off"
> +	cset "name='media2_in Gain 0 Volume' 0%"
> +	#   deep buffer
> +	cset "name='media3_in Gain 0 Switch' on"
> +	cset "name='media3_in Gain 0 Volume' 0"
> +	cset "name='media0_out mix 0 media0_in Switch' on"
> +	cset "name='media0_out mix 0 media1_in Switch' on"
> +	cset "name='media0_out mix 0 media2_in Switch' off"
> +	cset "name='media0_out mix 0 media3_in Switch' on"
> +	cset "name='media1_out mix 0 media0_in Switch' off"
> +	cset "name='media1_out mix 0 media1_in Switch' off"
> +	cset "name='media1_out mix 0 media2_in Switch' off"
> +	cset "name='media1_out mix 0 media3_in Switch' off"
> +	cset "name='pcm0_in Gain 0 Switch' on"
> +	cset "name='pcm0_in Gain 0 Volume' 0"
> +	cset "name='pcm1_in Gain 0 Switch' off"
> +	cset "name='pcm1_in Gain 0 Volume' 0%"
> +
> +	# codec0_out settings
> +	cset "name='codec_out0 mix 0 codec_in0 Switch' off"
> +	cset "name='codec_out0 mix 0 codec_in1 Switch' off"
> +	cset "name='codec_out0 mix 0 media_loop1_in Switch' off"
> +	cset "name='codec_out0 mix 0 media_loop2_in Switch' off"
> +	cset "name='codec_out0 mix 0 pcm0_in Switch' on"
> +	cset "name='codec_out0 mix 0 pcm1_in Switch' off"
> +	cset "name='codec_out0 mix 0 sprot_loop_in Switch' off"
> +	cset "name='codec_out0 Gain 0 Switch' on"
> +	cset "name='codec_out0 Gain 0 Volume' 0"
> +
> +	# input settings
> +	# pcm1_out settings
> +	cset "name='codec_in0 Gain 0 Switch' on"
> +	cset "name='codec_in0 Gain 0 Volume' 0"
> +	cset "name='pcm1_out mix 0 codec_in0 Switch' on"
> +	cset "name='pcm1_out mix 0 codec_in1 Switch' off"
> +	cset "name='pcm1_out mix 0 media_loop1_in Switch' off"
> +	cset "name='pcm1_out mix 0 media_loop2_in Switch' off"
> +	cset "name='pcm1_out mix 0 pcm0_in Switch' off"
> +	cset "name='pcm1_out mix 0 pcm1_in Switch' off"
> +	cset "name='pcm1_out mix 0 sprot_loop_in Switch' off"
> +	cset "name='pcm1_out Gain 0 Switch' on"
> +	cset "name='pcm1_out Gain 0 Volume' 0"
> +
> +	# disable codec_out1
> +	cset "name='codec_out1 mix 0 codec_in0 Switch' off"
> +	cset "name='codec_out1 mix 0 codec_in1 Switch' off"
> +	cset "name='codec_out1 mix 0 media_loop1_in Switch' off"
> +	cset "name='codec_out1 mix 0 media_loop2_in Switch' off"
> +	cset "name='codec_out1 mix 0 pcm0_in Switch' off"
> +	cset "name='codec_out1 mix 0 pcm1_in Switch' off"
> +	cset "name='codec_out1 mix 0 sprot_loop_in Switch' off"
> +	cset "name='codec_out1 Gain 0 Switch' off"
> +	cset "name='codec_out1 Gain 0 Volume' 0%"
> +
> +	# disable codec_in1
> +	cset "name='codec_in1 Gain 0 Switch' off"
> +	cset "name='codec_in1 Gain 0 Volume' 0%"
> +
> +	# disable all loops
> +	cset "name='media_loop1_out mix 0 codec_in0 Switch' off"
> +	cset "name='media_loop1_out mix 0 codec_in1 Switch' off"
> +	cset "name='media_loop1_out mix 0 media_loop1_in Switch' off"
> +	cset "name='media_loop1_out mix 0 media_loop2_in Switch' off"
> +	cset "name='media_loop1_out mix 0 pcm0_in Switch' off"
> +	cset "name='media_loop1_out mix 0 pcm1_in Switch' off"
> +	cset "name='media_loop1_out mix 0 sprot_loop_in Switch' off"
> +	cset "name='media_loop2_out mix 0 codec_in0 Switch' off"
> +	cset "name='media_loop2_out mix 0 codec_in1 Switch' off"
> +	cset "name='media_loop2_out mix 0 media_loop1_in Switch' off"
> +	cset "name='media_loop2_out mix 0 media_loop2_in Switch' off"
> +	cset "name='media_loop2_out mix 0 pcm0_in Switch' off"
> +	cset "name='media_loop2_out mix 0 pcm1_in Switch' off"
> +	cset "name='media_loop2_out mix 0 sprot_loop_in Switch' off"
> +	cset "name='sprot_loop_out mix 0 codec_in0 Switch' off"
> +	cset "name='sprot_loop_out mix 0 codec_in1 Switch' off"
> +	cset "name='sprot_loop_out mix 0 media_loop1_in Switch' off"
> +	cset "name='sprot_loop_out mix 0 media_loop2_in Switch' off"
> +	cset "name='sprot_loop_out mix 0 pcm0_in Switch' off"
> +	cset "name='sprot_loop_out mix 0 pcm1_in Switch' off"
> +	cset "name='sprot_loop_out mix 0 sprot_loop_in Switch' off"
> +
> +	# default state
> +	cset "name='ADC Capture Volume' 126"
> +	cset "name='HP Playback Volume' 29"
> +	cset "name='HPO L Playback Switch' on"
> +	cset "name='HPO MIX DAC1 Switch' on"
> +	cset "name='HPO MIX HPVOL Switch' on"
> +	cset "name='HPO R Playback Switch' on"
> +	cset "name='HPOVOL L Switch' on"
> +	cset "name='HPOVOL R Switch' on"
> +	cset "name='IF1 ASRC Switch' on"
> +	cset "name='IN Capture Volume' 27"
> +	cset "name='IN2 Boost' 1"
> +	cset "name='IN1 Boost' 1"
> +	cset "name='LOUT L Playback Switch' on"
> +	cset "name='LOUT MIX DAC L1 Switch' on"
> +	cset "name='LOUT MIX DAC R1 Switch' on"
> +	cset "name='LOUT R Playback Switch' on"
> +	cset "name='OUT Playback Volume' 33"
> +	cset "name='Stereo DAC MIXL DAC L1 Switch' on"
> +	cset "name='Stereo DAC MIXR DAC R1 Switch' on"
> +	cset "name='Stereo1 ADC MIXR ADC1 Switch' on"
> +	cset "name='Stereo1 ADC MIXR ADC2 Switch' on"
> +	cset "name='Stereo2 ADC MIXL ADC1 Switch' on"
> +	cset "name='Stereo2 ADC MIXL ADC2 Switch' on"
> +	cset "name='Stereo2 ADC MIXR ADC1 Switch' on"
> +	cset "name='Stereo2 ADC MIXR ADC2 Switch' on"
> +
> +	# input
> +	cset "name='Stereo1 ADC L1 Mux' ADC"
> +	cset "name='Stereo1 ADC R1 Mux' ADC"
> +	cset "name='Stereo1 ADC MIXL ADC1 Switch' on"
> +	cset "name='Stereo1 ADC MIXR ADC1 Switch' on"
> +	cset "name='RECMIXL BST1 Switch' off"
> +	cset "name='RECMIXR BST1 Switch' off"
> +	cset "name='RECMIXL BST3 Switch' off"
> +	cset "name='RECMIXR BST3 Switch' off"
> +	cset "name='RECMIXL BST2 Switch' off"
> +	cset "name='RECMIXR BST2 Switch' off"
> +]
> -- 
> 2.14.2
> 
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 


More information about the Alsa-devel mailing list