[alsa-devel] [PATCH 6/8] ASoC: Intel: add function to set parameter to sound effect module waves

Mark Brown broonie at kernel.org
Fri Mar 6 12:33:26 CET 2015


On Mon, Mar 02, 2015 at 03:36:16PM +0800, han.lu at intel.com wrote:

> +	if (ret < 0)
> +		dev_err(dev, "ipc: module set parameter failed - %d\n", ret);
> +	else
> +		dev_info(dev, "ipc: module set parameter success\n");

Logging on error is good but logging every time someone sets a control
is going to get too noisy, please remove the print there.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20150306/3b79222d/attachment-0001.sig>


More information about the Alsa-devel mailing list