Hi Lucas, On Fri, Sep 9, 2011 at 2:32 AM, Lucas De Marchi <lucas.demarchi@xxxxxxxxxxxxxx> wrote: > Metadata field number 0x7 should be the track duration and not the > progress of the track playback. Thus rename it to a better description. > --- > parser/avrcp.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/parser/avrcp.c b/parser/avrcp.c > index f8e4443..0b98a3e 100644 > --- a/parser/avrcp.c > +++ b/parser/avrcp.c > @@ -166,7 +166,7 @@ > #define AVRCP_MEDIA_ATTRIBUTE_TRACK 0x4 > #define AVRCP_MEDIA_ATTRIBUTE_TOTAL 0x5 > #define AVRCP_MEDIA_ATTRIBUTE_GENRE 0x6 > -#define AVRCP_MEDIA_ATTRIBUTE_PROGRESS 0x7 > +#define AVRCP_MEDIA_ATTRIBUTE_DURATION 0x7 > > /* play status */ > #define AVRCP_PLAY_STATUS_STOPPED 0x00 > @@ -909,8 +909,8 @@ static const char *mediattr2str(uint32_t attr) > return "Track Total"; > case AVRCP_MEDIA_ATTRIBUTE_GENRE: > return "Genre"; > - case AVRCP_MEDIA_ATTRIBUTE_PROGRESS: > - return "Progress"; > + case AVRCP_MEDIA_ATTRIBUTE_DURATION: > + return "Track duration"; > default: > return "Reserved"; > } > -- > 1.7.6.1 Ack @David: Lets keep hcidump simple, if we start doing enums for everything in hcidump we would just introduce more code for almost nothing back for a tracing tool. -- 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
