Hi Lucas, On Thu, May 24, 2012 at 7:50 PM, Lucas De Marchi <lucas.demarchi@xxxxxxxxxxxxxx> wrote: > On Thu, May 24, 2012 at 10:22 AM, Luiz Augusto von Dentz > <luiz.dentz@xxxxxxxxx> wrote: >> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx> >> >> Signature is now uint16 instead of byte >> --- >> audio/transport.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/audio/transport.c b/audio/transport.c >> index 4ad8608..6ed5d21 100644 >> --- a/audio/transport.c >> +++ b/audio/transport.c >> @@ -1076,5 +1076,5 @@ void media_transport_update_volume(struct media_transport *transport, >> >> emit_property_changed(transport->conn, transport->path, >> MEDIA_TRANSPORT_INTERFACE, "Volume", >> - DBUS_TYPE_BYTE, &transport->volume); >> + DBUS_TYPE_UINT16, &transport->volume); > > If you are changing it to uint16_t, why are you saying it's a "fix"? > > You should change transport->volume type together with this patch as well. Its a fix because this was first introduced and byte type, but you are right about uint16_t should be done in this patch. -- Luiz Augusto von Dentz -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
