[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <84b6f1f7-b9c1-fd0c-b684-1e07d35663ba@linaro.org>
Date: Fri, 6 Jul 2018 18:51:20 +0100
From: Srinivas Kandagatla <srinivas.kandagatla@...aro.org>
To: Rohit kumar <rohitkr@...eaurora.org>, lgirdwood@...il.com,
broonie@...nel.org, robh+dt@...nel.org, mark.rutland@....com,
plai@...eaurora.org, bgoswami@...eaurora.org, perex@...ex.cz,
tiwai@...e.com, alsa-devel@...a-project.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 2/5] ASoC: qcom: Add support to parse common audio
device nodes
On 06/07/18 10:43, Rohit kumar wrote:
> This adds support to parse cpu, platform and codec
> device nodes and add them in dai-links. Also, add
> API to add slave components associated with machine
> driver.
>
> Signed-off-by: Rohit kumar<rohitkr@...eaurora.org>
> ---
> sound/soc/qcom/Kconfig | 3 +
> sound/soc/qcom/Makefile | 1 +
> sound/soc/qcom/common.c | 150 ++++++++++++++++++++++++++++++++++++++++++++++++
> sound/soc/qcom/common.h | 14 +++++
> 4 files changed, 168 insertions(+)
> create mode 100644 sound/soc/qcom/common.c
common.c seems to miss
MODULE_LICENSE("GPL v2");
this will be an issue for snd card module auto-loading!
Also patch 5/5 can go with this.
Other than that it looks good.
Once these 2 fixed you can add my Ack
Acked-by: Srinivas Kandagatla <srinivas.kandagatla@...aro.org>
--srini
> create mode 100644 sound/soc/qcom/common.h
Powered by blists - more mailing lists