On Wed, Feb 03, 2016 at 09:24:25AM +0100, Lars-Peter Clausen wrote: > On 02/02/2016 08:30 PM, Mark Brown wrote: > > On Tue, Feb 02, 2016 at 04:38:04PM +0000, Charles Keepax wrote: > > > >> +#define ARIZONA_48K_RATES 12000, 24000, 48000, 96000, 192000, 4000, 8000, \ > >> + 16000, 32000 > > > > This just looks wrong. It's valid code but with the combination of the > > split line and the list without any sort of parenthesis it sets off > > alarm bells. Duplicating might not be the worst thing ever... > > For list constraints it is possible to specify a mask for which of the > entries in the rates array should be considered. So you could use the same > rate array, but use different masks. The adau1977 drive for example does this. > > Thanks wasn't aware of that I will have a look and respin the patch. Thanks, Charles _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
