Re: [PATCH alsa-ucm-conf v2 2/2] sof-soundwire: Add support for cs42l43/cs35l56 bridge configuration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 24, 2024 at 04:37:19PM +0100, Charles Keepax wrote:
> The cs42l43 has both a SPI master and an I2S interface, in some
> configurations 2 cs35l56 amplifiers are connected to these to provide
> bass speakers whilst the cs42l43's internal speaker drivers are used for
> the tweeters. Add UCM configuration for this type of system.
> 
> Signed-off-by: Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx>
> ---
> 
> Changes since v1:
>  - Update control names to just "Speaker Switch/Volume"
> 
> Thanks,
> Charles
> 
>  ucm2/sof-soundwire/cs35l56-bridge.conf | 61 ++++++++++++++++++++++++++
>  ucm2/sof-soundwire/sof-soundwire.conf  |  2 +-
>  2 files changed, 62 insertions(+), 1 deletion(-)
>  create mode 100644 ucm2/sof-soundwire/cs35l56-bridge.conf
> 
> diff --git a/ucm2/sof-soundwire/cs35l56-bridge.conf b/ucm2/sof-soundwire/cs35l56-bridge.conf
> new file mode 100644
> index 0000000..dcf5e5c
> --- /dev/null
> +++ b/ucm2/sof-soundwire/cs35l56-bridge.conf
> @@ -0,0 +1,61 @@
> +# Use case Configuration for sof-soundwire card
> +
> +LibraryConfig.remap.Config {
> +	ctl.default.map {
> +		"name='Speaker Switch'" {
> +			"name='AMPL Speaker Switch'".vindex.0 0
> +			"name='AMPR Speaker Switch'".vindex.1 0
> +		}
> +		"name='Speaker Volume'" {
> +			"name='AMPL Speaker Volume'".vindex.0 0
> +			"name='AMPR Speaker Volume'".vindex.1 0
> +		}
> +	}

Turns out there is an issue here. The machine driver already
creates a PIN_SWITCH control called Speaker Switch, so creating
these controls will fail. How would you feel about renaming them
back to something less generic?

Thanks,
Charles



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux