Commit 622f2199 authored by Stefan Binding's avatar Stefan Binding Committed by Takashi Iwai
Browse files

ALSA: hda: cs35l41: Add module parameter to control firmware load 



By default, the driver will automatically load DSP firmware
for the amps, if available. Adding this option allows the
autoload to be optional, which allows for different configurations.

Signed-off-by: default avatarStefan Binding <sbinding@opensource.cirrus.com>
Signed-off-by: default avatarVitaly Rodionov <vitalyr@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20220630002335.366545-15-vitalyr@opensource.cirrus.com


Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 47ceabd9

sound/pci/hda/cs35l41_hda.c

+16 −5
Original line number Diff line number Diff line
@@ -8,6 +8,7 @@ 


#include <linux/acpi.h>

#include <linux/module.h>

#include <linux/moduleparam.h>

#include <sound/hda_codec.h>

#include <sound/soc.h>

#include <linux/pm_runtime.h>
@@ -32,6 +33,11 @@ 
#define CAL_DSP_CTL_TYPE		5

#define CAL_DSP_CTL_ALG			205



static bool firmware_autostart = 1;

module_param(firmware_autostart, bool, 0444);

MODULE_PARM_DESC(firmware_autostart, "Allow automatic firmware download on boot"

			     "(0=Disable, 1=Enable) (default=1); ");



static const struct reg_sequence cs35l41_hda_config[] = {

	{ CS35L41_PLL_CLK_CTRL,		0x00000430 }, // 3072000Hz, BCLK Input, PLL_REFCLK_EN = 1

	{ CS35L41_DSP_CLK_CTRL,		0x00000003 }, // DSP CLK EN
@@ -843,11 +849,16 @@ static int cs35l41_hda_bind(struct device *dev, struct device *master, void *mas 


	cs35l41->firmware_type = HDA_CS_DSP_FW_SPK_PROT;



	if (firmware_autostart) {

		dev_dbg(cs35l41->dev, "Firmware Autostart.\n");

		cs35l41->request_fw_load = true;

		mutex_lock(&cs35l41->fw_mutex);

		if (cs35l41_smart_amp(cs35l41) < 0)

			dev_warn(cs35l41->dev, "Cannot Run Firmware, reverting to dsp bypass...\n");

		mutex_unlock(&cs35l41->fw_mutex);

	} else {

		dev_dbg(cs35l41->dev, "Firmware Autostart is disabled.\n");

	}



	ret = cs35l41_create_controls(cs35l41);