[Spice-devel] [spice-protocol PATCH v3] add volume syncronizatin to protocol

[Victor Toso]

Hi,

On Tue, Apr 14, 2015 at 03:40:05PM +0200, Marc-André Lureau wrote:
> Could you describe the protocol change in the commit log?

Sure, I'll send the updated patch to spice-protocol.

> Don't forget to update the submodule in spice-gtk (and bump protocol dep in
> vdagent)

Okay

> On Fri, Mar 27, 2015 at 3:16 PM, Marc-André Lureau <mlureau at redhat.com>
> wrote:
>
> > Hi
> >
> > ----- Original Message -----
> > > ---
> > >  spice/vd_agent.h | 9 +++++++++
> > >  1 file changed, 9 insertions(+)
> > >
> > > diff --git a/spice/vd_agent.h b/spice/vd_agent.h
> > > index 7464661..7732eca 100644
> > > --- a/spice/vd_agent.h
> > > +++ b/spice/vd_agent.h
> > > @@ -77,6 +77,7 @@ enum {
> > >      VD_AGENT_FILE_XFER_DATA,
> > >      VD_AGENT_CLIENT_DISCONNECTED,
> > >      VD_AGENT_MAX_CLIPBOARD,
> > > +    VD_AGENT_AUDIO_VOLUME_SYNC,
> >
> > I think it could be SET_AUDIO_VOLUME instead, but that doesn't matter much.
> >
> > >      VD_AGENT_END_MESSAGE,
> > >  };
> > >
> > > @@ -206,6 +207,13 @@ typedef struct SPICE_ATTR_PACKED
> > VDAgentMaxClipboard {
> > >      int32_t max;
> > >  } VDAgentMaxClipboard;
> > >
> > > +typedef struct SPICE_ATTR_PACKED VDAgentAudioVolumeSync {
> > > +    uint8_t is_playback;
> > > +    uint8_t mute;
> > > +    uint8_t nchannels;
> > > +    uint16_t volume[0];
> > > +} VDAgentAudioVolumeSync;
> > > +
> > >  enum {
> > >      VD_AGENT_CAP_MOUSE_STATE = 0,
> > >      VD_AGENT_CAP_MONITORS_CONFIG,
> > > @@ -218,6 +226,7 @@ enum {
> > >      VD_AGENT_CAP_GUEST_LINEEND_LF,
> > >      VD_AGENT_CAP_GUEST_LINEEND_CRLF,
> > >      VD_AGENT_CAP_MAX_CLIPBOARD,
> > > +    VD_AGENT_CAP_AUDIO_VOLUME_SYNC,
> > >      VD_AGENT_END_CAP,
> > >  };
> > >
> > > --
> > > 2.1.0
> > >
> > > _______________________________________________
> > > Spice-devel mailing list
> > > Spice-devel at lists.freedesktop.org
> > > http://lists.freedesktop.org/mailman/listinfo/spice-devel
> > >
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/spice-devel
> >
> 
> 
> 
> -- 
> Marc-André Lureau