Re: [PATCH] virtio: Add ID for virtio media

Brian Daniels <[email protected]>
Newsgroups dev.linux.lists.virtualization,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
> On Fri, Apr 24, 2026 at 03:37:14PM +0200, Albert Esteve wrote:
> > On Fri, Apr 24, 2026 at 3:07 PM Michael S. Tsirkin <[email protected]> wrote:
> > >
> > > On Thu, Apr 09, 2026 at 06:53:41PM +0200, Albert Esteve wrote:
> > > > Hi,
> > > >
> > > >
> > > > On Tue, Mar 10, 2026 at 9:41 AM Albert Esteve <[email protected]> wrote:
> > > > >
> > > > > Add VIRTIO_ID_MEDIA definition for virtio-media.
> > > > >
> > > > > Signed-off-by: Albert Esteve <[email protected]>
> > > >
> > > > Are there any thoughts on this patch, or any actions I need to take?
> > > >
> > > > Thanks,
> > > > Albert
> > >
> > > Create a github issue and request a vote as documented in README.md
> > 
> > The inclusion of this ID was already voted and approved. The device
> > already appears in the VirtIO v1.4 specification, with the same ID (as
> > linked below). This patch is only meant to translate the decision to
> > the kernel header.
> > 
> > >
> > > > > ---
> > > > > While the Linux driver for virtio media is currently under active
> > > > > development and review [1], the VirtIO v1.4 specification has
> > > > > already been published and oficially defines the media device type [2].
> > > > >
> > > > > Adding the device ID now allows other components and tools to reference it
> > > > > before the driver lands, and ensures the kernel headers stay aligned
> > > > > with the specification.
> > > > >
> > > > > [1] https://lore.kernel.org/linux-media/[email protected]/
> > > > > [2] https://docs.oasis-open.org/virtio/virtio/v1.4/virtio-v1.4.html#x1-82200022
> > > > > ---
> > > > >  include/uapi/linux/virtio_ids.h | 1 +
> > > > >  1 file changed, 1 insertion(+)
> > > > >
> > > > > diff --git a/include/uapi/linux/virtio_ids.h b/include/uapi/linux/virtio_ids.h
> > > > > index 6c12db16faa3a..f9056af0c6223 100644
> > > > > --- a/include/uapi/linux/virtio_ids.h
> > > > > +++ b/include/uapi/linux/virtio_ids.h
> > > > > @@ -69,6 +69,7 @@
> > > > >  #define VIRTIO_ID_BT                   40 /* virtio bluetooth */
> > > > >  #define VIRTIO_ID_GPIO                 41 /* virtio gpio */
> > > > >  #define VIRTIO_ID_SPI                  45 /* virtio spi */
> > > > > +#define VIRTIO_ID_MEDIA                        48 /* virtio media */
> > > > >
> > > > >  /*
> > > > >   * Virtio Transitional IDs
> > > > >
> > > > > ---
> > > > > base-commit: 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f
> > > > > change-id: 20260310-virtio-media-id-b31179fcdc44
> > > > >
> > > > > Best regards,
> > > > > --
> > > > > Albert Esteve <[email protected]>
> > > > >
> > >
> 
> 
> Oh my bad. I misfiled it into the spec folder. I will take care of this.
> Thanks!

Any plans to merge this in soon? This is a dependency of the virtio-media driver.

Thanks!
Brian
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.