[alsa-devel] [PATCH] ASoC: max98927: Adding support for TDM mode and Envelop tracking control

Mark Brown broonie at kernel.org
Sun May 14 12:11:15 CEST 2017


On Fri, May 05, 2017 at 10:42:15AM -0700, Ryan Lee wrote:
> Signed-off-by: Ryan Lee <ryans.lee at maximintegrated.com>
> ---

The subject line suggests that this is adding two new features.  Unless
they are very closely related in the code they should be submitted as
separate patches as covered in SubmittingPatches.

>  	default:
> -		dev_err(codec->dev, "DAI clock mode unsupported");
> +		dev_err(codec->dev, "DAI clock mode unsupported\n");
>  		return -EINVAL;
>  	}
>  

It also seems like there are a number of unrelated stylistic changes in
this patch.  While it's fine to make such changes they too should be in
separate patches.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20170514/d39ef292/attachment.sig>


More information about the Alsa-devel mailing list