[alsa-devel] [PATCH v2 0/5] Stripe control functionality

[Pierre-Louis Bossart]

>>> Stripe control verb reports and controls the stripe capability of an
>>> Audio Output Converter. This verb needs to be implemented only for an
>>> audio output converter and only if the stripe bit of the Audio Widget
>>> Capabilities parameter is 1.
>>> Stripe control: get code(0xf24) and set code(0x724)
>>
>> the series look ok (one minor comment on operator precedence) and 
>> aligned with my understanding of the HDaudio 1.0a spec.
>>
>> That said you made no mention of the Stripe bit  (figure 86) and 
>> fields 22:20 (Stripe capability) in Figure 75, so it's not clear to 
>> me if the support added in this patchset is sufficient or if there is 
>> additional logic to be set.
>>
> Stripe bit is part of Audio Widget Capability parameter and is 
> mentioned above in the last section of background. Patch-5(v2) checks 
> for the stripe bit, before sending stripe control verb.

Ah yes, I thought this patchset added stripe control starting from 
scratch but some definitions were already present. Thanks for the precision.

> As far as stripe control register is concerned (22:20), I don't think 
> any SW programming is required for this.

Yes correct. this is on GET only so something the codec needs to worry 
about, not the driver.