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

2020-09-22 Thread Rohit Kumar
   res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "lpass-hdmiif"); Thanks, Rohit -- Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member of the Code Aurora Forum, hosted by the Linux Foundation.

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 2/4] staging: media: atomisp: fix style of open brace

2020-07-26 Thread Rohit K Bharadwaj
On 26/07/20 3:12 pm, Mauro Carvalho Chehab wrote: > Em Sun, 26 Jul 2020 14:35:12 +0530 > Rohit K Bharadwaj escreveu: > >> this patch fixes style of open brace after functions and if statements >> >> Signed-off-by: Rohit K Bharadwaj > >> @@ -119,8 +121,9 @@

Re: [PATCH v3 1/4] staging: media: atomisp: fix style of block comments

2020-07-26 Thread Rohit K Bharadwaj
On 26/07/20 2:42 pm, Greg KH wrote: > On Sun, Jul 26, 2020 at 02:35:10PM +0530, Rohit K Bharadwaj wrote: >> this patch fixes the coding style of block comments. >> >> Signed-off-by: Rohit K Bharadwaj >> --- >> v3: change patch subject prefix >> v2: split pat

[PATCH v3 4/4] staging: media: atomisp: fix line length exceeds

2020-07-26 Thread Rohit K Bharadwaj
this patch fixes the line length exceeded error from checkpatch.pl Signed-off-by: Rohit K Bharadwaj --- v3: change patch subject prefix v2: split patch into sequence of patches v1: fix all coding style issues in single patch drivers/staging/media/atomisp/pci/sh_css_firmware.c | 3 ++- 1 file

[PATCH v3 3/4] staging: media: atomisp: fix trailing statement of if

2020-07-26 Thread Rohit K Bharadwaj
this patch fixes the error from checkpatch.pl which says that trailing statements after if keyword to be on next line Signed-off-by: Rohit K Bharadwaj --- v3: change patch subject prefix v2: split patch into sequence of patches v1: fix all coding style issues in single patch drivers/staging

[PATCH v3 2/4] staging: media: atomisp: fix style of open brace

2020-07-26 Thread Rohit K Bharadwaj
this patch fixes style of open brace after functions and if statements Signed-off-by: Rohit K Bharadwaj --- v3: change patch subject prefix v2: split patch into sequence of patches v1: fix all coding style issues in single patch .../media/atomisp/pci/sh_css_firmware.c | 29

[PATCH v3 1/4] staging: media: atomisp: fix style of block comments

2020-07-26 Thread Rohit K Bharadwaj
this patch fixes the coding style of block comments. Signed-off-by: Rohit K Bharadwaj --- v3: change patch subject prefix v2: split patch into sequence of patches v1: fix all coding style issues in single patch .../media/atomisp/pci/sh_css_firmware.c | 28 +-- 1 file

Re: [PATCH v2 1/4] fix style of block comments

2020-07-26 Thread Rohit K Bharadwaj
On 26/07/20 2:18 pm, Rohit K Bharadwaj wrote: > this patch fixes the coding style of block comments. > > Signed-off-by: Rohit K Bharadwaj > --- > v2: split patch into sequence of patches > v1: fix all coding style issues in single patch > > .../media/atomisp/pci/sh_c

[PATCH v2 1/4] fix style of block comments

2020-07-26 Thread Rohit K Bharadwaj
this patch fixes the coding style of block comments. Signed-off-by: Rohit K Bharadwaj --- v2: split patch into sequence of patches v1: fix all coding style issues in single patch .../media/atomisp/pci/sh_css_firmware.c | 28 +-- 1 file changed, 19 insertions(+), 9

[PATCH] staging: media: atomisp: fix code style issues

2020-07-26 Thread Rohit K Bharadwaj
this patch fixes the warnings and errors generated after running checkpatch.pl on pci/sh_css_firmware.c Signed-off-by: Rohit K Bharadwaj --- .../media/atomisp/pci/sh_css_firmware.c | 62 +++ 1 file changed, 36 insertions(+), 26 deletions(-) diff --git a/drivers/staging

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
, the driver change will have checkpatch errors. Can you please suggest if should make first Documentation change in text file itself before driver change and finally have a patch to convert it to yaml at the end? Thanks, Rohit -- Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is

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

2020-06-27 Thread Rohit Kumar
pq8016_init_i2sctl_bitfields Please make sure that you compile the code before sending it out! Will take care in next patchset. --srini Thanks, Rohit -- Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member of the Code Aurora Forum, hosted by the Linux Foundation.

[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] staging: iio: ADIS16240: Remove unused include

2019-09-15 Thread Rohit Sarkar
On Sun, Sep 15, 2019 at 10:53:00AM +0100, Jonathan Cameron wrote: > On Sat, 14 Sep 2019 02:06:27 +0530 > Rohit Sarkar wrote: > > > Bcc: > > Subject: [PATCH] staging: iio: adis16240: remove unused include > > Reply-To: > Something odd happened here with pa

[PATCH] staging: iio: ADIS16240: Remove unused include

2019-09-13 Thread Rohit Sarkar
Bcc: Subject: [PATCH] staging: iio: adis16240: remove unused include Reply-To: '#include' isn't being used anywhere. Remove it. Signed-off-by: Rohit Sarkar --- drivers/staging/iio/accel/adis16240.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/iio/accel/adis16240.c b

[PATCH] staging: rtl8192u: ieee80211: Replace snprintf with scnprintf

2019-09-10 Thread Rohit Sarkar
When the number of bytes to be printed exceeds the limit snprintf returns the number of bytes that would have been printed (if there was no truncation). This might cause issues, hence use scnprintf which returns the actual number of bytes printed to buffer always. Signed-off-by: Rohit Sarkar

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
{ + pr_err("plb: returning -EPROBE_DEFER 2\n"); + return -EPROBE_DEFER; + + } /* * At least one of CPU DAI name or CPU device name/node must be Thanks, Rohit -- Qualcomm INDIA, on behalf of Qual

[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

  1   2   3   4   5   6   7   8   >