[alsa-devel] [PATCH 2/8] ASoC: alc5632: Use params_width() rather than memory format

Leon Romanovsky leon at leon.nu
Fri Jan 10 14:44:43 CET 2014


On Thu, Jan 9, 2014 at 12:33 PM, Mark Brown <broonie at kernel.org> wrote:
> On Thu, Jan 09, 2014 at 06:55:17AM +0200, Leon Romanovsky wrote:
>> On Wed, Jan 8, 2014 at 9:51 PM, Mark Brown <broonie at kernel.org> wrote:
>
>> > +       switch (params_width(params)) {
>> > +       case 16:
>
>> May be the better way will be to use define here and not hard coded value?
>
> No, it really is actually a number of bits.
Thanks, I didn't know about it.
In such case it is really unimportant to introduce define here.



-- 
Leon Romanovsky | Independent Linux Consultant
        www.leon.nu | leon at leon.nu


More information about the Alsa-devel mailing list