Re: [PATCH] avrcp: Handle of GetPlayerApplicationSettingAttributeText pdu

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

 



Hi,

--------------------------------------------------
From: "Luiz Augusto von Dentz" <luiz.dentz@xxxxxxxxx>
Sent: Wednesday, March 21, 2012 10:53 PM
To: <chethan.tn@xxxxxxxxxxx>
Cc: <linux-bluetooth@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH] avrcp: Handle of GetPlayerApplicationSettingAttributeText pdu

Hi,

On Wed, Mar 21, 2012 at 2:56 AM, CHETHAN TN <chethan.tn@xxxxxxxxxxx> wrote:

In the above patch the function "attr_to_str" have been taken from audio/media.c file. Since there migth be few functions that will be used commonly in audio/avrcp.c and audio/media.c why can't
such funtions moved to a common file?.

Actually we might want to handle this in media.c, so perhaps we can
reuse the callback to gets an attribute or create a new one to fetch
the text in case of this being dynamically, otherwise just have it
inline in avrcp.h might be fine.


--
Luiz Augusto von Dentz

Thanks for you valuable information.
I agree on your suggestions and implement it as inline functions in avrcp.h.

Regards
Chethan
--
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


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux