[libvirt] [PATCH v4] qemu: sound: Support intel 'ich6' model

Cole Robinson crobinso at redhat.com
Mon Jan 24 17:25:19 UTC 2011


On 01/24/2011 11:22 AM, Eric Blake wrote:
> On 01/24/2011 09:15 AM, Cole Robinson wrote:
>> The important thing is to keep a consistent API here, we don't want some
>> <sound> devs require tweaking codecs but not others. Steps I see to
>> accomplishing this:
>>
>>     - every <sound> device has a <codec type='default'/> (unless codecs are
>>         manually specified)
>>     - <codec type='none'/> is required to specify 'no codecs'
>>     - new audio settings like mic=on|off could then be exposed in
>>         <sound> or <codec> in a consistent manner for all sound models
>>
>> v2:
>>     Use model='ich6'
>>
>> v3:
>>     Use feature detection, from eblake
>>     Set codec id, bus, and cad values
>>
>> v4:
>>     intel-hda isn't supported if -device isn't available
> 
> But still no typo fixes.
> 
>> -    /* Hack for 2 wierdly unusal devices name in QEMU */
>> +    /* Hack for wierdly unusal devices name in QEMU */
> 
> (see my review at
> https://www.redhat.com/archives/libvir-list/2011-January/msg00929.html)
> 

Argh, sorry. Not intentional, I just keep forgetting that piece. Resent with
that change.

Thanks,
Cole




More information about the libvir-list mailing list