Re: [PATCH osinfo-db v2 2/5] device: Add sound ich9

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 05, 2018 at 02:30:23PM -0400, Cole Robinson wrote:
> On 10/04/2018 03:31 AM, Fabiano Fidêncio wrote:
> > On Fri, 2018-09-28 at 18:55 -0400, Cole Robinson wrote:
> > > Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
> > > ---
> > >   data/device/pcisig.com/pci-8086-293e.d/class.xml.in | 8 ++++++++
> > >   1 file changed, 8 insertions(+)
> > >   create mode 100644 data/device/pcisig.com/pci-8086-
> > > 293e.d/class.xml.in
> > > 
> > > diff --git a/data/device/pcisig.com/pci-8086-293e.d/class.xml.in
> > > b/data/device/pcisig.com/pci-8086-293e.d/class.xml.in
> > > new file mode 100644
> > > index 0000000..6326a4d
> > > --- /dev/null
> > > +++ b/data/device/pcisig.com/pci-8086-293e.d/class.xml.in
> > > @@ -0,0 +1,8 @@
> > > +<libosinfo version="0.0.1">
> > > +<!-- Licensed under the GNU General Public License version 2 or
> > > later.
> > > +     See http://www.gnu.org/licenses/ for a copy of the license text
> > > -->
> > > +  <device id="http://pcisig.com/pci/8086/293e";>
> > > +    <name>ich9</name>
> > > +    <class>audio</class>
> > > +  </device>
> > > +</libosinfo>
> > 
> > Reviewed-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>
> > 
> 
> Thanks, I pushed these first 2

In v1 I had asked for this to be changed to 'ich9-hda' since 'ich9'
refers to a chipset which provides a number of features, not just
audio. Could you post a followup to rename it.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux