Re: [PATCH v6 3/5] Asoc:qcom:lpass-cpu:Update dts property read API

2020-09-22 Thread Rohit Kumar
On 9/22/2020 5:13 PM, Mark Brown wrote: On Tue, Sep 22, 2020 at 12:22:38PM +0100, Srinivas Kandagatla wrote: On 22/09/2020 12:08, Mark Brown wrote: I agree with you on this and I see the point, but Rob had a very different opinion about the reg-names bindings to start with. This topic been

Re: [PATCH][next] ASoC: qcom: add missing out of memory check on drvdata->clks allocation

2020-08-19 Thread Rohit Kumar
nce null return value") Fixes: 1220f6a76e77 ("ASoC: qcom: Add common array to initialize soc based core clocks") Signed-off-by: Colin Ian King --- Thanks for the fix. Reviewed-by: Rohit kumar sound/soc/qcom/lpass-apq8016.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH v6 12/12] dt-bindings: sound: lpass-cpu: Move to yaml format

2020-08-14 Thread Rohit kumar
Update lpass-cpu binding with yaml formats. Signed-off-by: Rohit kumar --- .../devicetree/bindings/sound/qcom,lpass-cpu.txt | 130 -- .../devicetree/bindings/sound/qcom,lpass-cpu.yaml | 189 + 2 files changed, 189 insertions(+), 130 deletions(-) delete mode

[PATCH v6 09/12] ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio

2020-08-14 Thread Rohit kumar
From: Ajit Pandey Add platform driver for configuring sc7180 lpass core I2S and DMA configuration to support playback & capture to external codecs connected over primary & secondary MI2S interfaces. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Reviewed-by: Srinivas Ka

[PATCH v6 06/12] ASoC: qcom: lpass-cpu: fix concurrency issue

2020-08-14 Thread Rohit kumar
river") Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-cpu.c | 16 1 file changed, 16 deletions(-) diff --git a/sound/soc/qcom/lpass-cpu.c b/sound/soc/qcom/lpass-cpu.c index 6b86f16..5d84f63 100644 --- a/sound/soc/qcom/lpass-cpu.c +++ b

[PATCH v6 03/12] ASoC: qcom: lpass-platform: Replace card->dev with component->dev

2020-08-14 Thread Rohit kumar
From: Ajit Pandey We are allocating dma memory for component->dev but trying to mmap such memory for substream->pcm->card->dev. Replace device argument in mmap with component->dev to fix this. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandaga

[PATCH v6 11/12] ASoC: qcom: lpass-platform: Use platform_get_irq

2020-08-14 Thread Rohit kumar
platform_get_irq_byname() is used when there is list of interrupts in the device node. As lpass-platform has only one interrupt entry, use platform_get_irq() instead. Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass-platform.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v6 10/12] ASoC: qcom: lpass-cpu: Use platform_get_resource

2020-08-14 Thread Rohit kumar
platform_get_resource_byname() is used when there is list of reg entries. As lpass-cpu node has only one reg entry, use platform_get_resource() instead. Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v6 05/12] ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers

2020-08-14 Thread Rohit kumar
I2SCTL and DMACTL registers has different bits alignment for newer LPASS variants of SC7180 soc. Use REG_FIELD_ID() to define the reg_fields in platform specific file and removed shifts and mask macros for such registers from header file. Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass

[PATCH v6 04/12] ASoC: qcom: lpass-platform: fix memory leak

2020-08-14 Thread Rohit kumar
lpass_pcm_data is never freed. Free it in close ops to avoid memory leak. Fixes: 022d00ee0b55 ("ASoC: lpass-platform: Fix broken pcm data usage") Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-platform.c | 3 ++- 1 file changed, 2 insertions(+),

[PATCH v6 08/12] include: dt-bindings: sound: Add sc7180-lpass bindings header

2020-08-14 Thread Rohit kumar
From: Ajit Pandey Add header defining dai-id and mclk id for SC7180 lpass soc. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Acked-by: Rob Herring --- include/dt-bindings/sound/sc7180-lpass.h | 10 ++ 1 file changed, 10 insertions(+) create mode 100644 include/dt-bindings

[PATCH v6 07/12] dt-bindings: sound: lpass-cpu: Add sc7180 lpass cpu node

2020-08-14 Thread Rohit kumar
Add dt-bindings to support "qcom,lpass-cpu-sc7180" node. Signed-off-by: Rohit kumar --- .../devicetree/bindings/sound/qcom,lpass-cpu.txt | 55 +- 1 file changed, 53 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/sound/qcom,lpass-

[PATCH v6 02/12] ASoC: qcom: lpass-cpu: Move ahbix clk to platform specific function

2020-08-14 Thread Rohit kumar
Ahbix clock is optional clock and not needed for all platforms. Move it to lpass-apq8016/ipq806x as it is not needed for sc7180. Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-apq8016.c | 27 ++ sound/soc/qcom/lpass-cpu.c | 40

[PATCH v6 00/12] ASoC: qcom: Add support for SC7180 lpass variant

2020-08-14 Thread Rohit kumar
ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio Rohit kumar (8): ASoC: qcom: lpass-cpu: Move ahbix clk to platform specific function ASoC: qcom: lpass-platform: fix memory leak ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers ASoC: qcom: lpass-cpu: fix concu

[PATCH v6 01/12] ASoC: qcom: Add common array to initialize soc based core clocks

2020-08-14 Thread Rohit kumar
-off-by: Ajit Pandey Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-apq8016.c | 39 +++ sound/soc/qcom/lpass.h | 10 +++--- 2 files changed, 26 insertions(+), 23 deletions(-) diff --git a/sound/soc/qcom/lpass

Re: [PATCH v5 01/12] ASoC: qcom: Add common array to initialize soc based core clocks

2020-08-11 Thread Rohit Kumar
Hello Mark, Do you see any concern with patches (1-11). As of now, there is comment only in patch 12 from Rob which I am planning to update once other patches are merged. Can you please review and let me know if anything is missing. Thanks, Rohit On 8/4/2020 1:37 PM, Rohit kumar wrote: From

Re: [PATCH v5 00/12] ASoC: qcom: Add support for SC7180 lpass variant

2020-08-06 Thread Rohit Kumar
. Thanks, Rohit On 8/4/2020 1:37 PM, Rohit kumar wrote: This patch chain add audio support for SC7180 soc by doing the required modification in existing common lpass-cpu/lpass-platform driver. This also fixes some concurrency issue. This patch series is already tested by Srinivas on Dragon Board

Re: [PATCH v4 01/12] ASoC: qcom: Add common array to initialize soc based core clocks

2020-08-05 Thread Rohit Kumar
Thanks Stephen for reviewing. On 8/6/2020 6:01 AM, Stephen Boyd wrote: Quoting Rohit kumar (2020-07-22 03:31:44) From: Ajit Pandey LPASS variants have their own soc specific clocks that needs to be enabled for MI2S audio support. Added a common variable in drvdata to initialize such clocks

Re: [PATCH v4 12/12] dt-bindings: sound: lpass-cpu: Move to yaml format

2020-08-04 Thread Rohit Kumar
On 8/3/2020 11:52 PM, Rob Herring wrote: On Mon, Aug 3, 2020 at 2:28 AM Rohit Kumar wrote: Thanks Rob for reviewing On 7/23/2020 10:46 PM, Rob Herring wrote: On Wed, Jul 22, 2020 at 04:01:55PM +0530, Rohit kumar wrote: Update lpass-cpu binding with yaml formats. Signed-off-by: Rohit

[PATCH v5 10/12] ASoC: qcom: lpass-cpu: Use platform_get_resource

2020-08-04 Thread Rohit kumar
platform_get_resource_byname() is used when there is list of reg entries. As lpass-cpu node has only one reg entry, use platform_get_resource() instead. Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v5 09/12] ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio

2020-08-04 Thread Rohit kumar
From: Ajit Pandey Add platform driver for configuring sc7180 lpass core I2S and DMA configuration to support playback & capture to external codecs connected over primary & secondary MI2S interfaces. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Reviewed-by: Srinivas Ka

[PATCH v5 08/12] include: dt-bindings: sound: Add sc7180-lpass bindings header

2020-08-04 Thread Rohit kumar
From: Ajit Pandey Add header defining dai-id and mclk id for SC7180 lpass soc. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Acked-by: Rob Herring --- include/dt-bindings/sound/sc7180-lpass.h | 10 ++ 1 file changed, 10 insertions(+) create mode 100644 include/dt-bindings

[PATCH v5 12/12] dt-bindings: sound: lpass-cpu: Move to yaml format

2020-08-04 Thread Rohit kumar
Update lpass-cpu binding with yaml formats. Signed-off-by: Rohit kumar --- .../devicetree/bindings/sound/qcom,lpass-cpu.txt | 130 --- .../devicetree/bindings/sound/qcom,lpass-cpu.yaml | 179 + 2 files changed, 179 insertions(+), 130 deletions(-) delete mode

[PATCH v5 06/12] ASoC: qcom: lpass-cpu: fix concurrency issue

2020-08-04 Thread Rohit kumar
river") Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-cpu.c | 16 1 file changed, 16 deletions(-) diff --git a/sound/soc/qcom/lpass-cpu.c b/sound/soc/qcom/lpass-cpu.c index 6b86f16..5d84f63 100644 --- a/sound/soc/qcom/lpass-cpu.c +++ b

[PATCH v5 11/12] ASoC: qcom: lpass-platform: Use platform_get_irq

2020-08-04 Thread Rohit kumar
platform_get_irq_byname() is used when there is list of interrupts in the device node. As lpass-platform has only one interrupt entry, use platform_get_irq() instead. Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass-platform.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v5 04/12] ASoC: qcom: lpass-platform: fix memory leak

2020-08-04 Thread Rohit kumar
lpass_pcm_data is never freed. Free it in close ops to avoid memory leak. Fixes: 022d00ee0b55 ("ASoC: lpass-platform: Fix broken pcm data usage") Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-platform.c | 3 ++- 1 file changed, 2 insertions(+),

[PATCH v5 00/12] ASoC: qcom: Add support for SC7180 lpass variant

2020-08-04 Thread Rohit kumar
sc7180-lpass bindings header ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio Rohit kumar (8): ASoC: qcom: lpass-cpu: Move ahbix clk to platform specific function ASoC: qcom: lpass-platform: fix memory leak ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers

[PATCH v5 07/12] dt-bindings: sound: lpass-cpu: Add sc7180 lpass cpu node

2020-08-04 Thread Rohit kumar
Add dt-bindings to support "qcom,lpass-cpu-sc7180" node. Signed-off-by: Rohit kumar --- .../devicetree/bindings/sound/qcom,lpass-cpu.txt | 55 +- 1 file changed, 53 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/sound/qcom,lpass-

[PATCH v5 05/12] ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers

2020-08-04 Thread Rohit kumar
I2SCTL and DMACTL registers has different bits alignment for newer LPASS variants of SC7180 soc. Use REG_FIELD_ID() to define the reg_fields in platform specific file and removed shifts and mask macros for such registers from header file. Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass

[PATCH v5 01/12] ASoC: qcom: Add common array to initialize soc based core clocks

2020-08-04 Thread Rohit kumar
-off-by: Ajit Pandey Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-apq8016.c | 39 +++ sound/soc/qcom/lpass.h | 10 +++--- 2 files changed, 26 insertions(+), 23 deletions(-) diff --git a/sound/soc/qcom/lpass

[PATCH v5 02/12] ASoC: qcom: lpass-cpu: Move ahbix clk to platform specific function

2020-08-04 Thread Rohit kumar
Ahbix clock is optional clock and not needed for all platforms. Move it to lpass-apq8016/ipq806x as it is not needed for sc7180. Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-apq8016.c | 27 ++ sound/soc/qcom/lpass-cpu.c | 40

[PATCH v5 03/12] ASoC: qcom: lpass-platform: Replace card->dev with component->dev

2020-08-04 Thread Rohit kumar
From: Ajit Pandey We are allocating dma memory for component->dev but trying to mmap such memory for substream->pcm->card->dev. Replace device argument in mmap with component->dev to fix this. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandaga

Re: [PATCH v4 12/12] dt-bindings: sound: lpass-cpu: Move to yaml format

2020-08-03 Thread Rohit Kumar
Thanks Rob for reviewing On 7/23/2020 10:46 PM, Rob Herring wrote: On Wed, Jul 22, 2020 at 04:01:55PM +0530, Rohit kumar wrote: Update lpass-cpu binding with yaml formats. Signed-off-by: Rohit kumar --- .../devicetree/bindings/sound/qcom,lpass-cpu.txt | 130

Re: [PATCH v4 00/12] ASoC: qcom: Add support for SC7180 lpass variant

2020-07-28 Thread Rohit Kumar
On 7/27/2020 3:20 PM, Rohit Kumar wrote: On 7/24/2020 4:52 PM, Srinivas Kandagatla wrote: On 22/07/2020 11:31, Rohit kumar wrote: This patch chain add audio support for SC7180 soc by doing the required modification in existing common lpass-cpu/lpass-platform driver. This also fixes some

Re: [PATCH v4 00/12] ASoC: qcom: Add support for SC7180 lpass variant

2020-07-27 Thread Rohit Kumar
On 7/24/2020 4:52 PM, Srinivas Kandagatla wrote: On 22/07/2020 11:31, Rohit kumar wrote: This patch chain add audio support for SC7180 soc by doing the required modification in existing common lpass-cpu/lpass-platform driver. This also fixes some concurrency issue. Changes since v3

Re: [PATCH v3 8/8] dt-bindings: sound: lpass-cpu: Move to yaml format

2020-07-22 Thread Rohit Kumar
Thanks Rob for the review. Posted v4 with fixes. On 7/14/2020 4:23 AM, Rob Herring wrote: On Wed, Jul 08, 2020 at 10:38:16AM +0530, Rohit kumar wrote: From: Ajit Pandey Update lpass-cpu binding with yaml formats. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar --- .../devicetree

[PATCH v4 12/12] dt-bindings: sound: lpass-cpu: Move to yaml format

2020-07-22 Thread Rohit kumar
Update lpass-cpu binding with yaml formats. Signed-off-by: Rohit kumar --- .../devicetree/bindings/sound/qcom,lpass-cpu.txt | 130 --- .../devicetree/bindings/sound/qcom,lpass-cpu.yaml | 185 + 2 files changed, 185 insertions(+), 130 deletions(-) delete mode

[PATCH v4 10/12] ASoC: qcom: lpass-cpu: Use platform_get_resource

2020-07-22 Thread Rohit kumar
platform_get_resource_byname() is used when there is list of reg entries. As lpass-cpu node has only one reg entry, use platform_get_resource() instead. Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass-cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/qcom

[PATCH v4 11/12] ASoC: qcom: lpass-platform: Use platform_get_irq

2020-07-22 Thread Rohit kumar
platform_get_irq_byname() is used when there is list of interrupts in the device node. As lpass-platform has only one interrupt entry, use platform_get_irq() instead. Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass-platform.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v4 06/12] ASoC: qcom: lpass-cpu: fix concurrency issue

2020-07-22 Thread Rohit kumar
river") Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass-cpu.c | 16 1 file changed, 16 deletions(-) diff --git a/sound/soc/qcom/lpass-cpu.c b/sound/soc/qcom/lpass-cpu.c index 6b86f16..5d84f63 100644 --- a/sound/soc/qcom/lpass-cpu.c +++ b/sound/soc/qcom/lpass-cpu.c @@ -266

[PATCH v4 03/12] ASoC: qcom: lpass-platform: Replace card->dev with component->dev

2020-07-22 Thread Rohit kumar
From: Ajit Pandey We are allocating dma memory for component->dev but trying to mmap such memory for substream->pcm->card->dev. Replace device argument in mmap with component->dev to fix this. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandaga

[PATCH v4 07/12] dt-bindings: sound: lpass-cpu: Add sc7180 lpass cpu node

2020-07-22 Thread Rohit kumar
Add dt-bindings to support "qcom,lpass-cpu-sc7180" node. Signed-off-by: Rohit kumar --- .../devicetree/bindings/sound/qcom,lpass-cpu.txt | 55 +- 1 file changed, 53 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/sound/qcom,lpass-

[PATCH v4 08/12] include: dt-bindings: sound: Add sc7180-lpass bindings header

2020-07-22 Thread Rohit kumar
From: Ajit Pandey Add header defining dai-id and mclk id for SC7180 lpass soc. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar --- include/dt-bindings/sound/sc7180-lpass.h | 10 ++ 1 file changed, 10 insertions(+) create mode 100644 include/dt-bindings/sound/sc7180-lpass.h

[PATCH v4 05/12] ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers

2020-07-22 Thread Rohit kumar
I2SCTL and DMACTL registers has different bits alignment for newer LPASS variants of SC7180 soc. Use REG_FIELD_ID() to define the reg_fields in platform specific file and removed shifts and mask macros for such registers from header file. Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass

[PATCH v4 09/12] ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio

2020-07-22 Thread Rohit kumar
From: Ajit Pandey Add platform driver for configuring sc7180 lpass core I2S and DMA configuration to support playback & capture to external codecs connected over primary & secondary MI2S interfaces. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Reviewed-by: Srinivas Ka

[PATCH v4 04/12] ASoC: qcom: lpass-platform: fix memory leak

2020-07-22 Thread Rohit kumar
lpass_pcm_data is never freed. Free it in close ops to avoid memory leak. Fixes: 022d00ee0b55 ("ASoC: lpass-platform: Fix broken pcm data usage") Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass-platform.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/soun

[PATCH v4 02/12] ASoC: qcom: lpass-cpu: Move ahbix clk to platform specific function

2020-07-22 Thread Rohit kumar
Ahbix clock is optional clock and not needed for all platforms. Move it to lpass-apq8016/ipq806x as it is not needed for sc7180. Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-apq8016.c | 27 ++ sound/soc/qcom/lpass-cpu.c | 40

[PATCH v4 00/12] ASoC: qcom: Add support for SC7180 lpass variant

2020-07-22 Thread Rohit kumar
de: dt-bindings: sound: Add sc7180-lpass bindings header ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio Rohit kumar (8): ASoC: qcom: lpass-cpu: Move ahbix clk to platform specific function ASoC: qcom: lpass-platform: fix memory leak ASoC: qcom: lpass: Use regmap_field for

[PATCH v4 01/12] ASoC: qcom: Add common array to initialize soc based core clocks

2020-07-22 Thread Rohit kumar
-off-by: Ajit Pandey Signed-off-by: Rohit kumar Reviewed-by: Srinivas Kandagatla --- sound/soc/qcom/lpass-apq8016.c | 39 +++ sound/soc/qcom/lpass.h | 10 +++--- 2 files changed, 26 insertions(+), 23 deletions(-) diff --git a/sound/soc/qcom/lpass

Re: [PATCH v3 3/8] ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers

2020-07-09 Thread Rohit Kumar
On 7/9/2020 3:36 PM, Srinivas Kandagatla wrote: On 09/07/2020 10:57, Rohit Kumar wrote: Thanks Srini for reviewing. On 7/9/2020 2:56 PM, Srinivas Kandagatla wrote: On 08/07/2020 06:08, Rohit kumar wrote: I2SCTL and DMACTL registers has different bits alignment for newer LPASS variants

Re: [PATCH v3 6/8] dt-bindings: sound: lpass-cpu: Add sc7180 lpass cpu node

2020-07-09 Thread Rohit Kumar
On 7/9/2020 3:38 PM, Srinivas Kandagatla wrote: On 09/07/2020 11:01, Rohit Kumar wrote: On 7/9/2020 2:57 PM, Srinivas Kandagatla wrote: On 08/07/2020 06:08, Rohit kumar wrote: Add dt-bindings to support "qcom,lpass-cpu-sc7180" node. Signed-off-by: Rohit kumar ---   Doc

Re: [PATCH v3 4/8] include: dt-bindings: sound: Add sc7180-lpass bindings header

2020-07-09 Thread Rohit Kumar
On 7/8/2020 8:15 PM, Mark Brown wrote: On Wed, Jul 08, 2020 at 10:38:12AM +0530, Rohit kumar wrote: From: Ajit Pandey Add header defining dai-id and mclk id for SC7180 lpass soc. Signed-off-by: Ajit Pandey --- This one is missing a signoff as well, and I can't seem to see any reference

Re: [PATCH v3 6/8] dt-bindings: sound: lpass-cpu: Add sc7180 lpass cpu node

2020-07-09 Thread Rohit Kumar
On 7/9/2020 2:57 PM, Srinivas Kandagatla wrote: On 08/07/2020 06:08, Rohit kumar wrote: Add dt-bindings to support "qcom,lpass-cpu-sc7180" node. Signed-off-by: Rohit kumar ---   Documentation/devicetree/bindings/sound/qcom,lpass-cpu.txt | 3 ++-   1 file changed, 2 insert

Re: [PATCH v3 3/8] ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers

2020-07-09 Thread Rohit Kumar
Thanks Srini for reviewing. On 7/9/2020 2:56 PM, Srinivas Kandagatla wrote: On 08/07/2020 06:08, Rohit kumar wrote: I2SCTL and DMACTL registers has different bits alignment for newer LPASS variants of SC7180 soc. Use REG_FIELD_ID() to define the reg_fields in platform specific file

Re: [PATCH v3 5/8] ASoC: qcom: lpass-platform: Replace card->dev with component->dev

2020-07-08 Thread Rohit Kumar
On 7/8/2020 10:20 PM, Mark Brown wrote: On Wed, Jul 08, 2020 at 10:38:13AM +0530, Rohit kumar wrote: From: Ajit Pandey We are allocating dma memory for component->dev but trying to mmap such memory for substream->pcm->card->dev. Replace device argument in mmap with component

Re: [RESEND][PATCH v3 7/8] ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio

2020-07-08 Thread Rohit Kumar
On 7/8/2020 4:03 PM, Mark Brown wrote: On Wed, Jul 08, 2020 at 10:44:46AM +0530, Rohit kumar wrote: From: Ajit Pandey Add platform driver for configuring sc7180 lpass core I2S and DMA configuration to support playback & capture to external codecs connected over primary & second

[RESEND][PATCH v3 7/8] ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio

2020-07-07 Thread Rohit kumar
From: Ajit Pandey Add platform driver for configuring sc7180 lpass core I2S and DMA configuration to support playback & capture to external codecs connected over primary & secondary MI2S interfaces. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar --- Resending to update Signed-

[PATCH v3 8/8] dt-bindings: sound: lpass-cpu: Move to yaml format

2020-07-07 Thread Rohit kumar
From: Ajit Pandey Update lpass-cpu binding with yaml formats. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar --- .../devicetree/bindings/sound/qcom,lpass-cpu.txt | 80 --- .../devicetree/bindings/sound/qcom,lpass-cpu.yaml | 154 + 2 files changed, 154

[PATCH v3 6/8] dt-bindings: sound: lpass-cpu: Add sc7180 lpass cpu node

2020-07-07 Thread Rohit kumar
Add dt-bindings to support "qcom,lpass-cpu-sc7180" node. Signed-off-by: Rohit kumar --- Documentation/devicetree/bindings/sound/qcom,lpass-cpu.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/sound/qcom,lpass-cpu.txt b/Doc

[PATCH v3 2/8] ASoC: qcom: lpass-cpu: Move ahbix clk to platform specific function

2020-07-07 Thread Rohit kumar
Ahbix clock is optional clock and not needed for all platforms. Move it to lpass-apq8016/ipq806x as it is not needed for sc7180. Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass-apq8016.c | 27 ++ sound/soc/qcom/lpass-cpu.c | 40

[PATCH v3 4/8] include: dt-bindings: sound: Add sc7180-lpass bindings header

2020-07-07 Thread Rohit kumar
From: Ajit Pandey Add header defining dai-id and mclk id for SC7180 lpass soc. Signed-off-by: Ajit Pandey --- include/dt-bindings/sound/sc7180-lpass.h | 10 ++ 1 file changed, 10 insertions(+) create mode 100644 include/dt-bindings/sound/sc7180-lpass.h diff --git

[PATCH v3 5/8] ASoC: qcom: lpass-platform: Replace card->dev with component->dev

2020-07-07 Thread Rohit kumar
From: Ajit Pandey We are allocating dma memory for component->dev but trying to mmap such memory for substream->pcm->card->dev. Replace device argument in mmap with component->dev to fix this. Signed-off-by: Ajit Pandey --- sound/soc/qcom/lpass-platform.c | 5 ++--- 1 file changed, 2

[PATCH v3 7/8] ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio

2020-07-07 Thread Rohit kumar
From: Ajit Pandey Add platform driver for configuring sc7180 lpass core I2S and DMA configuration to support playback & capture to external codecs connected over primary & secondary MI2S interfaces. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar --- sound/soc/qcom/Kconfig

[PATCH v3 3/8] ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers

2020-07-07 Thread Rohit kumar
I2SCTL and DMACTL registers has different bits alignment for newer LPASS variants of SC7180 soc. Use REG_FIELD_ID() to define the reg_fields in platform specific file and removed shifts and mask macros for such registers from header file. Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass

[PATCH v3 1/8] ASoC: qcom: Add common array to initialize soc based core clocks

2020-07-07 Thread Rohit kumar
-off-by: Ajit Pandey Signed-off-by: Rohit kumar --- sound/soc/qcom/lpass-apq8016.c | 39 +++ sound/soc/qcom/lpass.h | 10 +++--- 2 files changed, 26 insertions(+), 23 deletions(-) diff --git a/sound/soc/qcom/lpass-apq8016.c b/sound/soc/qcom/lpass

[PATCH v3 0/8] ASoC: qcom: Add support for SC7180 lpass variant

2020-07-07 Thread Rohit kumar
: Replace card->dev with component->dev ASoC: qcom: lpass-sc7180: Add platform driver for lpass audio dt-bindings: sound: lpass-cpu: Move to yaml format Rohit kumar (3): ASoC: qcom: lpass-cpu: Move ahbix clk to platform specific function ASoC: qcom: lpass: Use regmap_field for

Re: [PATCH v2 1/7] Documentation: device-tree: sound: Update lpass-cpu driver binding

2020-07-06 Thread Rohit Kumar
On 5/16/2020 11:07 PM, Ajit Pandey wrote: On 5/14/2020 10:14 PM, Mark Brown wrote: On Thu, May 14, 2020 at 10:08:12PM +0530, Ajit Pandey wrote: Done the required cleanups to update lpass-cpu binding with newer yaml formats. Please do YAML conversions as the last thing in any patch series,

Re: [PATCH v2 4/7] ASoC: qcom: lpass: Use regmap_field for i2sctl and dmactl registers

2020-06-27 Thread Rohit Kumar
Thanks Srini for reviewing the change. On 5/18/2020 3:19 PM, Srinivas Kandagatla wrote: On 14/05/2020 17:38, Ajit Pandey wrote: I2SCTL and DMACTL registers has different bits alignment for newer LPASS variants of SC7180 soc. Instead of adding extra overhead for calculating masks and shifts

[PATCH] asoc: Update supported rate and format for dummy dai

2020-06-27 Thread Rohit kumar
Add support for 384KHz sample rate and S24_3LE bitwidth for dummy dai. Signed-off-by: Rohit kumar --- sound/soc/soc-utils.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/soc/soc-utils.c b/sound/soc/soc-utils.c index 922eac9..364b248 100644 --- a/sound/soc/soc

Re: [PATCH 2/2] remoteproc: q6v5_adsp: Remove voting for lpass_aon clock

2019-02-17 Thread Rohit Kumar
Hello Bjorn, On 1/30/2019 10:38 AM, Rohit Kumar wrote: Hello Bjorn, On 1/14/2019 4:14 PM, Rohit Kumar wrote: On 1/7/2019 12:43 PM, Bjorn Andersson wrote: On Thu 03 Jan 20:36 PST 2019, Rohit Kumar wrote: Hello Bjorn, Can you please review this patch series too. LPASS_AON clock support

[PATCH v3] arm64: dts: sdm845: Add Q6V5 ADSP node

2019-02-05 Thread Rohit kumar
This patch adds Q6V5 ADSP pil remoteproc node for SDM845 SoC. Signed-off-by: Rohit kumar --- Changes since v2: Updated reg property values from <0x1730 0x410> to <0 0x1730 0 0x410> This depends on below upstream patches: Add adsp_mem dt node: https://lkml.org/lkml/2019/1/29

[PATCH v2] arm64: dts: sdm845: Add Q6V5 ADSP node

2019-01-30 Thread Rohit kumar
This patch adds Q6V5 ADSP pil remoteproc node for SDM845 SoC. Signed-off-by: Rohit kumar --- Changes since v1: Addressed comments given by Bjorn. This depends on below upstream patch which defines adsp_mem node: https://lkml.org/lkml/2019/1/29/1392 arch/arm64/boot/dts/qcom/sdm845.dtsi | 89

Re: [PATCH 2/2] remoteproc: q6v5_adsp: Remove voting for lpass_aon clock

2019-01-29 Thread Rohit Kumar
Hello Bjorn, On 1/14/2019 4:14 PM, Rohit Kumar wrote: On 1/7/2019 12:43 PM, Bjorn Andersson wrote: On Thu 03 Jan 20:36 PST 2019, Rohit Kumar wrote: Hello Bjorn, Can you please review this patch series too. LPASS_AON clock support is already removed from lpass clock driver. Applied

Re: next/master boot bisection: next-20190114 on sun8i-h2-plus-libretech-all-h3-cc

2019-01-14 Thread Rohit Kumar
is not called if it is not present in component list during sound card registration. Check if component is available in component list for platform and cpu dai before soundcard registration. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Signed-off-by: Mark

Re: [PATCH 2/2] remoteproc: q6v5_adsp: Remove voting for lpass_aon clock

2019-01-14 Thread Rohit Kumar
On 1/7/2019 12:43 PM, Bjorn Andersson wrote: On Thu 03 Jan 20:36 PST 2019, Rohit Kumar wrote: Hello Bjorn, Can you please review this patch series too. LPASS_AON clock support is already removed from lpass clock driver. Applied the two patches. Hello Bjorn, Can you please point me

Re: [alsa-devel] [PATCH] ASoC: soc-core: Fix null pointer dereference in soc_find_component

2019-01-11 Thread Rohit kumar
On 1/12/2019 3:19 AM, Pierre-Louis Bossart wrote: Thanks for the overnight fix. This update fixes the issue on my Skylake XPS13 test device (blind testing since I don't understand what the code does). Tested-by: Pierre-Louis Bossart I need to take this back, this set of changes

[PATCH] ASoC: soc-core: Fix null pointer dereference in soc_find_component

2019-01-11 Thread Rohit kumar
list") Reported-by: Pierre-Louis Bossart Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar --- sound/soc/soc-core.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index 0934b36..df05fb8 100644 --- a/sound/soc/

Re: [alsa-devel] [PATCH v2] ASoC: soc-core: defer card probe until all component is added to list

2019-01-11 Thread Rohit Kumar
sound card registration.     Check if component is available in component list for     platform and cpu dai before soundcard registration.     Signed-off-by: Ajit Pandey     Signed-off-by: Rohit kumar     Signed-off-by: Mark Brown :04 04 98da59b0a73551030a0c9030b8cd581140

[PATCH] ASoC: soc-core: Hold client_mutex around soc_init_dai_link()

2019-01-10 Thread Rohit kumar
) Reported-by: Kuninori Morimoto Signed-off-by: Rohit kumar Signed-off-by: Ajit Pandey --- sound/soc/soc-core.c | 4 1 file changed, 4 insertions(+) diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index eec92f1..0934b36 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c

Re: [alsa-devel] Applied "ASoC: soc-core: defer card probe until all component is added to list" to the asoc tree

2019-01-09 Thread Rohit Kumar
if it is not present in component list during sound card registration. Check if component is available in component list for platform and cpu dai before soundcard registration. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar Signed-off-by: Mark Brown --- I got below WARNING by this patch. I guess we

[PATCH v2] ASoC: soc-core: defer card probe until all component is added to list

2019-01-09 Thread Rohit kumar
From: Ajit Pandey DAI component probe is not called if it is not present in component list during sound card registration. Check if component is available in component list for platform and cpu dai before soundcard registration. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar

Re: [PATCH 2/2] ASoC: qcom: defer probe if platform dai is not registered

2019-01-09 Thread Rohit Kumar
Thanks Mark for review. On 1/7/2019 11:57 PM, Mark Brown wrote: On Mon, Dec 24, 2018 at 02:32:13PM +0530, Rohit kumar wrote: From: Ajit Pandey Platform DAI component probe is not called if it is not present in component list during sound card registration. Check if component is registered

Re: [PATCH 2/2] arm64: dts: qcom: sdm845: Add Q6V5 ADSP node

2019-01-08 Thread Rohit Kumar
Thanks Bjorn for review. On 1/4/2019 5:20 AM, Bjorn Andersson wrote: On Thu 20 Dec 05:39 PST 2018, Rohit kumar wrote: This patch adds Q6V5 ADSP remoteproc node for SDM845 SoCs. Thanks Rohit, nice to see these things on the list! Signed-off-by: Rohit kumar --- arch/arm64/boot/dts/qcom

Re: [PATCH 1/2] arm64: dts: qcom: sdm845: Add ADSP reserve-memory nodes

2019-01-06 Thread Rohit Kumar
Thanks Bjorn for review. On 1/4/2019 5:12 AM, Bjorn Andersson wrote: On Thu 20 Dec 05:39 PST 2018, Rohit kumar wrote: Add memory nodes required for remoteproc q6v5_adsp pil. This range doesn't match the documented memory map. I would prefer to see a "Specify all PIL regions as de

Re: [PATCH 2/2] remoteproc: q6v5_adsp: Remove voting for lpass_aon clock

2019-01-03 Thread Rohit Kumar
Hello Bjorn, Can you please review this patch series too. LPASS_AON clock support is already removed from lpass clock driver. Thanks, Rohit On 11/30/2018 12:59 PM, Rohit kumar wrote: Lpass_aon clock is on by default. Remove it from lpass clock list to avoid voting for it. Signed-off

[PATCH 1/2] ASoC: soc-core: add snd_soc_of_lookup_component()

2018-12-24 Thread Rohit kumar
-by: Ajit Pandey Signed-off-by: Rohit kumar --- include/sound/soc.h | 1 + sound/soc/soc-core.c | 28 2 files changed, 29 insertions(+) diff --git a/include/sound/soc.h b/include/sound/soc.h index 8ec1de8..dc251fc 100644 --- a/include/sound/soc.h +++ b/include/sound

[PATCH 2/2] ASoC: qcom: defer probe if platform dai is not registered

2018-12-24 Thread Rohit kumar
From: Ajit Pandey Platform DAI component probe is not called if it is not present in component list during sound card registration. Check if component is registered for platform dai before soundcard registration. Signed-off-by: Ajit Pandey Signed-off-by: Rohit kumar --- sound/soc/qcom

[PATCH 0/2] ASoC: soc-core: add snd_soc_of_lookup_component()

2018-12-24 Thread Rohit kumar
Platform DAI component probe is not called if it is not present in component list during sound card registration. Add snd_soc_of_lookup_component() to get component info from device_node. This can be used by machine driver to make sure that card components are registered before calling

[PATCH 2/2] arm64: dts: qcom: sdm845: Add Q6V5 ADSP node

2018-12-20 Thread Rohit kumar
This patch adds Q6V5 ADSP remoteproc node for SDM845 SoCs. Signed-off-by: Rohit kumar --- arch/arm64/boot/dts/qcom/sdm845.dtsi | 96 +++- 1 file changed, 95 insertions(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts

[PATCH 1/2] arm64: dts: qcom: sdm845: Add ADSP reserve-memory nodes

2018-12-20 Thread Rohit kumar
Add memory nodes required for remoteproc q6v5_adsp pil. Signed-off-by: Rohit kumar --- arch/arm64/boot/dts/qcom/sdm845.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi index 23a253b..c0a012f 100644

[PATCH 0/2] arm64: dts: qcom: sdm845: add support for ADSP PIL

2018-12-20 Thread Rohit kumar
clk list: https://lore.kernel.org/patchwork/patch/1019609/ Rohit kumar (2): arm64: dts: qcom: sdm845: Add ADSP reserve-memory nodes arm64: dts: qcom: sdm845: Add Q6V5 ADSP node arch/arm64/boot/dts/qcom/sdm845.dtsi | 101 ++- 1 file changed, 100 insertions

[PATCH 4/4] ASoC: qdsp6: q6routing: add display_port_rx port routing

2018-12-14 Thread Rohit kumar
This patch add support to Display_port_rx mixers required to select path between ASM stream and AFE ports. Signed-off-by: Rohit kumar --- sound/soc/qcom/qdsp6/q6routing.c | 9 + 1 file changed, 9 insertions(+) diff --git a/sound/soc/qcom/qdsp6/q6routing.c b/sound/soc/qcom/qdsp6

[PATCH 3/4] ASoC: qdsp6: q6afe-dai: add support to Display port RX dais

2018-12-14 Thread Rohit kumar
This patch adds support of AFE DAI for Display_port_rx port. Signed-off-by: Rohit kumar --- sound/soc/qcom/qdsp6/q6afe-dai.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/sound/soc/qcom/qdsp6/q6afe-dai.c b/sound/soc/qcom/qdsp6/q6afe-dai.c index bfc3bbf..dc645ba

[PATCH 2/4] ASoC: qdsp6: qdafe: add support for display_port_rx

2018-12-14 Thread Rohit kumar
This patch adds support for Display_Port_Rx port in AFE. Signed-off-by: Rohit kumar --- sound/soc/qcom/qdsp6/q6afe.c | 4 1 file changed, 4 insertions(+) diff --git a/sound/soc/qcom/qdsp6/q6afe.c b/sound/soc/qcom/qdsp6/q6afe.c index 829b5e987..e0945f7a 100644 --- a/sound/soc/qcom/qdsp6

[PATCH 1/4] ASoC: qdsp6: dt-bindings: Add q6afe display_port dt binding

2018-12-14 Thread Rohit kumar
This patch adds bindings required for DISPLAY_PORT_RX port on AFE. Signed-off-by: Rohit kumar --- include/dt-bindings/sound/qcom,q6afe.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/dt-bindings/sound/qcom,q6afe.h b/include/dt-bindings/sound/qcom,q6afe.h index e2d3892..1df06f8

[PATCH 0/4] ASoC: qdsp6: Add support for audio over DP

2018-12-14 Thread Rohit kumar
Add support for Audio over display port AFE port Rohit kumar (4): ASoC: qdsp6: dt-bindings: Add q6afe display_port dt binding ASoC: qdsp6: qdafe: add support for display_port_rx ASoC: qdsp6: q6afe-dai: add support to Display port RX dais ASoC: qdsp6: q6routing: add display_port_rx port

[PATCH] ASoC: sdm845: set jack only for a specific backend

2018-12-14 Thread Rohit kumar
Headset codec is connected over PRIMARY_MI2S interface. Call set_jack for codec associated with Primary Mi2s interface. Also, set_jack to NULL when jack is freed. Signed-off-by: Rohit kumar --- sound/soc/qcom/sdm845.c | 31 ++- 1 file changed, 22 insertions(+), 9

Re: [PATCH v2 2/5] ASoC: qcom: sdm845: Create and setup jack in init callback

2018-12-10 Thread Rohit Kumar
Hello Mark, On 12/7/2018 1:51 AM, Mark Brown wrote: On Wed, Nov 28, 2018 at 05:00:34PM +0800, Cheng-Yi Chiang wrote: Add a callback for init ops on dai_link to create and setup jack. Signed-off-by: Rohit kumar Signed-off-by: Cheng-Yi Chiang This looks like you're forwarding a patch from

[PATCH 1/2] dt-binding: remoteproc: Remove lpass_aon clock from adsp pil clock list

2018-11-29 Thread Rohit kumar
LPASS_Audio_Wrapper_AON clock is on by default. Remove it from lpass clock list to avoid voting for it. Signed-off-by: Rohit kumar --- Documentation/devicetree/bindings/remoteproc/qcom,adsp-pil.txt | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree

[PATCH 2/2] remoteproc: q6v5_adsp: Remove voting for lpass_aon clock

2018-11-29 Thread Rohit kumar
Lpass_aon clock is on by default. Remove it from lpass clock list to avoid voting for it. Signed-off-by: Rohit kumar --- drivers/remoteproc/qcom_q6v5_adsp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/remoteproc/qcom_q6v5_adsp.c b/drivers/remoteproc

  1   2   3   >