Re: [protocol v10 01/27] protocol: Add support for the VP8 and h264 video codecs

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

 



> Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> 

Agree!

Should we bump the spice-protocol version too?

Frediano

> On Tue, Mar 01, 2016 at 04:50:16PM +0100, Francois Gouget wrote:
> > Clients that support multiple codecs must advertise the
> > SPICE_DISPLAY_CAP_MULTI_CODEC capability and one
> > SPICE_DISPLAY_CAP_CODEC_XXX per supported codec.
> > 
> > Signed-off-by: Francois Gouget <fgouget@xxxxxxxxxxxxxxx>
> > ---
> >  spice.proto      | 2 ++
> >  spice/enums.h    | 2 ++
> >  spice/protocol.h | 4 ++++
> >  3 files changed, 8 insertions(+)
> > 
> > diff --git a/spice.proto b/spice.proto
> > index af970f2..4a0778d 100644
> > --- a/spice.proto
> > +++ b/spice.proto
> > @@ -329,6 +329,8 @@ flags8 path_flags { /* TODO: C enum names changes */
> >  
> >  enum8 video_codec_type {
> >      MJPEG = 1,
> > +    VP8,
> > +    H264,
> >  };
> >  
> >  flags8 stream_flags {
> > diff --git a/spice/enums.h b/spice/enums.h
> > index 613db52..c6e9840 100644
> > --- a/spice/enums.h
> > +++ b/spice/enums.h
> > @@ -139,6 +139,8 @@ typedef enum SpicePathFlags {
> >  
> >  typedef enum SpiceVideoCodecType {
> >      SPICE_VIDEO_CODEC_TYPE_MJPEG = 1,
> > +    SPICE_VIDEO_CODEC_TYPE_VP8,
> > +    SPICE_VIDEO_CODEC_TYPE_H264,
> >  
> >      SPICE_VIDEO_CODEC_TYPE_ENUM_END
> >  } SpiceVideoCodecType;
> > diff --git a/spice/protocol.h b/spice/protocol.h
> > index 3e6c624..f4a2259 100644
> > --- a/spice/protocol.h
> > +++ b/spice/protocol.h
> > @@ -137,6 +137,10 @@ enum {
> >      SPICE_DISPLAY_CAP_LZ4_COMPRESSION,
> >      SPICE_DISPLAY_CAP_PREF_COMPRESSION,
> >      SPICE_DISPLAY_CAP_GL_SCANOUT,
> > +    SPICE_DISPLAY_CAP_MULTI_CODEC,
> > +    SPICE_DISPLAY_CAP_CODEC_MJPEG,
> > +    SPICE_DISPLAY_CAP_CODEC_VP8,
> > +    SPICE_DISPLAY_CAP_CODEC_H264,
> >  };
> >  
> >  enum {
> > --
> > 2.7.0
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]