[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <0f6aa59a-5350-684d-c90c-b250e6d2977a@codeaurora.org>
Date: Wed, 8 Sep 2021 18:36:40 +0530
From: Srinivasa Rao Mandadapu <srivasam@...eaurora.org>
To: Mark Brown <broonie@...nel.org>
Cc: Stephen Boyd <swboyd@...omium.org>, agross@...nel.org,
alsa-devel@...a-project.org, bgoswami@...eaurora.org,
bjorn.andersson@...aro.org, devicetree@...r.kernel.org,
judyhsiao@...omium.org, lgirdwood@...il.com,
linux-arm-msm@...r.kernel.org, linux-kernel@...r.kernel.org,
perex@...ex.cz, plai@...eaurora.org, robh+dt@...nel.org,
rohitkr@...eaurora.org, srinivas.kandagatla@...aro.org,
tiwai@...e.com
Subject: Re: [PATCH] ASoC: dt-bindings: lpass: add binding headers for digital
codecs
Thanks for Your Time Mark Brown!!!
On 9/8/2021 3:53 PM, Mark Brown wrote:
> On Wed, Sep 08, 2021 at 10:08:33AM +0530, Srinivasa Rao Mandadapu wrote:
>> On 9/8/2021 1:54 AM, Stephen Boyd wrote:
>>> Quoting Srinivasa Rao Mandadapu (2021-09-06 06:27:34)
>>>> +#define LPASS_CDC_DMA_VA0 8
>>>> +#define LPASS_MAX_PORTS 9
>>> Do we need LPASS_MAX_PORTS in the binding?
>> Yes. based on this creating array of streams in machine driver. So to make
>> upper limit introduced this macro.
> That's saying it's useful to have it in the code, do we need it in the
> binding itself though?
Okay. Got it. will remove it and share new patch.
--
Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc.,
is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.
Powered by blists - more mailing lists