ASoC: wm_adsp: Allow compressed buffers in any memory region
authorAndrew Ford <aford@opensource.cirrus.com>
Tue, 19 Feb 2019 17:31:56 +0000 (17:31 +0000)
committerMark Brown <broonie@kernel.org>
Wed, 20 Feb 2019 12:02:37 +0000 (12:02 +0000)
Currently, compressed buffers can only be specified in the XM memory
region. There is no reason to have such a restriction with the newer
meta-data based way of specifying the buffers, so remove it.

Signed-off-by: Andrew Ford <aford@opensource.cirrus.com>
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/wm_adsp.c

index 1dd291c..12ef85e 100644 (file)
@@ -344,6 +344,7 @@ struct wm_adsp_compr_buf {
        u32 irq_count;
        int read_index;
        int avail;
+       int host_buf_mem_type;
 };
 
 struct wm_adsp_compr {
@@ -3219,14 +3220,14 @@ static int wm_adsp_write_data_word(struct wm_adsp *dsp, int mem_type,
 static inline int wm_adsp_buffer_read(struct wm_adsp_compr_buf *buf,
                                      unsigned int field_offset, u32 *data)
 {
-       return wm_adsp_read_data_word(buf->dsp, WMFW_ADSP2_XM,
+       return wm_adsp_read_data_word(buf->dsp, buf->host_buf_mem_type,
                                      buf->host_buf_ptr + field_offset, data);
 }
 
 static inline int wm_adsp_buffer_write(struct wm_adsp_compr_buf *buf,
                                       unsigned int field_offset, u32 data)
 {
-       return wm_adsp_write_data_word(buf->dsp, WMFW_ADSP2_XM,
+       return wm_adsp_write_data_word(buf->dsp, buf->host_buf_mem_type,
                                       buf->host_buf_ptr + field_offset, data);
 }
 
@@ -3264,6 +3265,8 @@ static int wm_adsp_legacy_host_buf_addr(struct wm_adsp_compr_buf *buf)
        if (!buf->host_buf_ptr)
                return -EIO;
 
+       buf->host_buf_mem_type = WMFW_ADSP2_XM;
+
        adsp_dbg(dsp, "host_buf_ptr=%x\n", buf->host_buf_ptr);
 
        return 0;
@@ -3282,6 +3285,7 @@ wm_adsp_find_host_buffer_ctrl(struct wm_adsp_compr_buf *buf)
                if (!ctl->enabled)
                        continue;
 
+               buf->host_buf_mem_type = ctl->alg_region.type;
                return ctl;
        }