[alsa-devel] [PATCH] ALSA: add support for disabling period irq

Jaroslav Kysela perex at perex.cz
Tue Nov 2 08:14:53 CET 2010

On Mon, 1 Nov 2010, pl bossart wrote:

> Thanks Jaroslav for this quick review.
>> Use one 'if' and a goto behind the last condition to simplify things.
> will do
>>> struct snd_interval {
>>>        unsigned int min, max;
>> The PCM protocol version should be increased.
> Humm, this one I don't understand. I didn't change anything, just used
> a bit in an existing flag? What are you specifically referring to?

Increase subminor in SNDRV_PCM_VERSION . It allows alsa-lib to check if 
the feature is implemented in the driver (although I admit that this 
change does not require special checks against the protocol version, but 
it's standard procedure to change the subminor when the API is extended).

>>> @@ -297,6 +297,7 @@ struct snd_pcm_runtime {
>>>        unsigned int info;
>>>        unsigned int rate_num;
>>>        unsigned int rate_den;
>>> +        bool no_period_irq;
>> I would use an unsigned int bit field rather than bool in this case to make
>> addition in-line with other structures in this file.
> will do.



Jaroslav Kysela <perex at perex.cz>
Linux Kernel Sound Maintainer
ALSA Project, Red Hat, Inc.

More information about the Alsa-devel mailing list