On 10/17/2016 08:49 PM, Takashi Iwai wrote:
On Mon, 17 Oct 2016 07:55:21 +0200,
mengdong.lin@xxxxxxxxxxxxxxx wrote:
From: Mengdong Lin <mengdong.lin@xxxxxxxxxxxxxxx>
This series doesn't require or cause any ABI updates. It doesn't need the
kernel patches that trying to udpate ABI from v4 or v5.
It just fix a missing feature and enhance text conf file to support
including files to control the increasing size of text conf files.
The inclusion of a file can be done in alsaconf syntax via <xxx>,
too. We may extend it if needed instead?
Thanks for your review and advice!
It seems etter to extend and reuse alsaconf syntax, instead of adding
topology specific code. This can also benefit UCM if users want to split
common codec or SOC settings to other files.
alsaconf already support using absolute path to include a file. But it
seems we cannot give a relative path (e.g. just a file name) and a
configuration directory by <confdir:xxx>. I'll try to extend alsaconf.
The other patch is a bug fixing, if it's okay, could it be merged first?
Thanks
Mengdong
The patches themselves look OK, so I'm not strongly against merging
them, but just to make sure our standpoint.
thanks,
Takashi
Mengdong Lin (2):
topology: Fix missing stream name of widgets in text conf file
topology: Add support for included files
include/topology.h | 38 +++++
src/topology/dapm.c | 10 ++
src/topology/parser.c | 374 +++++++++++++++++++++++++++++++++++++++++++++-
src/topology/tplg_local.h | 17 +++
4 files changed, 434 insertions(+), 5 deletions(-)
--
2.5.0
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
