On Sun, Jan 10, 2016 at 12:38:05AM +0100, Michael Trimarchi wrote: > + if (priv->codec_model == PCM1795) > + formats = PCM1795_FORMATS; This should be a switch statement to handle any future variations. > static const struct of_device_id pcm179x_of_match[] = { > { .compatible = "ti,pcm1792a", }, > + { .compatible = "ti,pcm1796", }, > + { .compatible = "ti,pcm1795", > + .data = &codec_model, > + }, Please keep tables sorted, it avoid conflicts and merge issues when adding new entries.
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel