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

Takashi Iwai tiwai at suse.de
Wed Oct 25 10:51:08 CEST 2017


On Wed, 25 Oct 2017 10:23:15 +0200,
Pierre-Louis Bossart wrote:
> 
> On 10/25/17 1:23 AM, Takashi Iwai wrote:
> > 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.
> 
> I was planning to clean this up once we are done with the v1.0 release
> of SOF which currently keeps us busy. If you want to take this one
> directly in alsa-lib that's fine with me.

Well, it's up to you.  But, in general, if applying this doesn't make
things too troublesome, I'd rather pick up sooner than later.


thanks,

Takashi

> 
> >
> >
> > 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