Hi,
On Jul 20 2016 10:52, mengdong.lin@xxxxxxxxxxxxxxx wrote:
From: Mengdong Lin <mengdong.lin@xxxxxxxxxxxxxxx>
There is no ABI change in this series.
Remove support for control flag SNDRV_CTL_ELEM_ACCESS_USER as Takashi
Sakamoto suggested. Also fix 2 compiler warnings and add check before
string cpy.
History:
v2: Revise commit messages.
Mengdong Lin (3):
topology: Remove support for control flag SNDRV_CTL_ELEM_ACCESS_USER
topology: Add ATTRIBUTE_UNUSED for unused parameters to fix compiler
warning
topology: Check address and length before string copy
src/topology/ctl.c | 1 -
src/topology/pcm.c | 8 ++++----
src/topology/tplg_local.h | 3 +++
3 files changed, 7 insertions(+), 5 deletions(-)
Reviewed-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx>
But I believe that we can seek better ways for the purpose of third
patch. For example, changing prototype of the function to return error
code might help callers to handle errors appropriately.
Anyway, thanks for the first patch.
Regards
Takashi Sakamoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
