[PATCH] ASoC: qcom: lpass-platform: Update memremap flag to MEMREMAP_WC

[PATCH] ASoC: qcom: lpass-platform: Update memremap flag to MEMREMAP_WC

[Srinivasa Rao Mandadapu]

Update memremap flag from MEMREMAP_WT to MEMREMAP_WC for better
performance.

Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
Co-developed-by: Venkata Prasad Potturu <quic_potturu@quicinc.com>
Signed-off-by: Venkata Prasad Potturu <quic_potturu@quicinc.com>
---
 sound/soc/qcom/lpass-platform.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/qcom/lpass-platform.c b/sound/soc/qcom/lpass-platform.c
index a7c677f..2590d65 100644
--- a/sound/soc/qcom/lpass-platform.c
+++ b/sound/soc/qcom/lpass-platform.c
@@ -1160,7 +1160,7 @@ static int lpass_platform_prealloc_cdc_dma_buffer(struct snd_soc_component *comp
 		break;
 	}
 
-	buf->area = (unsigned char * __force)memremap(buf->addr, buf->bytes, MEMREMAP_WT);
+	buf->area = (unsigned char * __force)memremap(buf->addr, buf->bytes, MEMREMAP_WC);
 
 	return 0;
 }
-- 
2.7.4

[PATCH] ASoC: qcom: lpass-platform: Update memremap flag to MEMREMAP_WC

[Srinivasa Rao Mandadapu]

Update memremap flag from MEMREMAP_WT to MEMREMAP_WC for better
performance.

Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
Co-developed-by: Venkata Prasad Potturu <quic_potturu@quicinc.com>
Signed-off-by: Venkata Prasad Potturu <quic_potturu@quicinc.com>
---
 sound/soc/qcom/lpass-platform.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/qcom/lpass-platform.c b/sound/soc/qcom/lpass-platform.c
index a7c677f..2590d65 100644
--- a/sound/soc/qcom/lpass-platform.c
+++ b/sound/soc/qcom/lpass-platform.c
@@ -1160,7 +1160,7 @@ static int lpass_platform_prealloc_cdc_dma_buffer(struct snd_soc_component *comp
 		break;
 	}
 
-	buf->area = (unsigned char * __force)memremap(buf->addr, buf->bytes, MEMREMAP_WT);
+	buf->area = (unsigned char * __force)memremap(buf->addr, buf->bytes, MEMREMAP_WC);
 
 	return 0;
 }
-- 
2.7.4

Re: [PATCH] ASoC: qcom: lpass-platform: Update memremap flag to MEMREMAP_WC

[Judy Hsiao]

> -       buf->area = (unsigned char * __force)memremap(buf->addr, buf->bytes, MEMREMAP_WT);
> +       buf->area = (unsigned char * __force)memremap(buf->addr, buf->bytes, MEMREMAP_WC);
Should we replace the memremap() with other standard DMA buffer
allocation API like: snd_pcm_set_managed_buffer() ?
Thanks!
>
>         return 0;
>  }
> --
> 2.7.4
>

Re: [PATCH] ASoC: qcom: lpass-platform: Update memremap flag to MEMREMAP_WC

[Judy Hsiao]

> -       buf->area = (unsigned char * __force)memremap(buf->addr, buf->bytes, MEMREMAP_WT);
> +       buf->area = (unsigned char * __force)memremap(buf->addr, buf->bytes, MEMREMAP_WC);
Should we replace the memremap() with other standard DMA buffer
allocation API like: snd_pcm_set_managed_buffer() ?
Thanks!
>
>         return 0;
>  }
> --
> 2.7.4
>

Re: [PATCH] ASoC: qcom: lpass-platform: Update memremap flag to MEMREMAP_WC

[Mark Brown]

On Wed, 13 Apr 2022 15:39:56 +0530, Srinivasa Rao Mandadapu wrote:
> Update memremap flag from MEMREMAP_WT to MEMREMAP_WC for better
> performance.
> 
> 

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/1] ASoC: qcom: lpass-platform: Update memremap flag to MEMREMAP_WC
      commit: 0a8ff26dea6e9aeb11db3f1af9fc1848b7042661

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark