[Spice-devel] [PATCH spice-protocol] Add support for VP9 video codec

[Frediano Ziglio]

ping

> 
> On Tue, Jan 03, 2017 at 06:13:55PM +0100, Victor Toso wrote:
> > Hi,
> > 
> > On Tue, Jan 03, 2017 at 01:42:33PM +0000, Frediano Ziglio wrote:
> > > Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> > > ---
> > >  spice/enums.h    | 1 +
> > >  spice/protocol.h | 1 +
> > >  2 files changed, 2 insertions(+)
> > >
> > > diff --git a/spice/enums.h b/spice/enums.h
> > > index 743a4a8..86d02d7 100644
> > > --- a/spice/enums.h
> > > +++ b/spice/enums.h
> > > @@ -148,6 +148,7 @@ typedef enum SpiceVideoCodecType {
> > >      SPICE_VIDEO_CODEC_TYPE_MJPEG = 1,
> > >      SPICE_VIDEO_CODEC_TYPE_VP8,
> > >      SPICE_VIDEO_CODEC_TYPE_H264,
> > > +    SPICE_VIDEO_CODEC_TYPE_VP9,
> > >
> > 
> > I don't see a reason to not have it.
> > 
> > In spice-gtk we will need to update gst_opts[] array to include the
> > protocol + vp9dec bits. (channel-display-gst.c)
> > 
> > In server, although not mandatory, I would include the gstreamer:vp9 in
> > default_renderer string (reds.c).
> 
> Yes, I would wait until we have server/client patches using this before
> getting it in.
> 
> Christophe
>