- Sound-open-firmware - mailman.alsa-project.org

[Sound-open-firmware] [PATCH] intel-ipc: fix host ring buffer size not page aligned issue
by Keyon Jie 15 Nov '17

15 Nov '17

The host ring buffer size may be not page aligned, but the last page was ulitlized by host component wrongly, which may introduce beating noise. Here change to correct size for the last element, which will fix the issue. Signed-off-by: Keyon Jie <yang.jie(a)linux.intel.com> --- src/ipc/intel-ipc.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/ipc/intel-ipc.c b/src/ipc/intel-ipc.c index 56bd8ba..2e503e1 100644 --- a/src/ipc/intel-ipc.c +++ b/src/ipc/intel-ipc.c @@ -192,6 +192,17 @@ static int parse_page_descriptors(struct intel_ipc_data *iipc, else elem.dest = phy_addr; + if (elem.size * (i + 1) >= ring->size) { + /* the last page may be not full used */ + if (i == (ring->pages - 1)) + elem.size = ring->size - elem.size * i; + else { + trace_ipc_error("eBs"); + /* should not happen*/ + return -EINVAL; + } + } + if (is_trace) err = dma_trace_host_buffer(d, &elem, ring->size); else -- 2.11.0

3 2

[Sound-open-firmware] [PATCH] pipeline: remove preloader.
by Liam Girdwood 11 Nov '17

11 Nov '17

preloader can cause some out of order pipeline periods and timestamps wrt to host so remove. Signed-off-by: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> --- src/audio/pipeline.c | 63 ---------------------------------------------------- 1 file changed, 63 deletions(-) diff --git a/src/audio/pipeline.c b/src/audio/pipeline.c index 61d80b4..2b6f59e 100644 --- a/src/audio/pipeline.c +++ b/src/audio/pipeline.c @@ -478,57 +478,11 @@ static int component_op_upstream(struct op_data *op_data, return err; } -/* preload all downstream components - locks held by caller */ -static int preload_downstream(struct comp_dev *start, struct comp_dev *current) -{ - struct sof_ipc_comp_config *config = COMP_GET_CONFIG(current); - struct list_item *clist; - int i; - int total = 0; - int count = 0; - - tracev_pipe("PR-"); - tracev_value(current->comp.id); - - /* reached endpoint ? */ - if (current != start && current->is_endpoint) - return 0; - - /* now preload the buffers */ - for (i = 0; i < config->preload_count; i++) { - count = comp_preload(current); - - if (count < 0) - return count; - total += count; - } - - /* now run this operation downstream */ - list_for_item(clist, &current->bsink_list) { - struct comp_buffer *buffer; - - buffer = container_of(clist, struct comp_buffer, source_list); - - /* don't go downstream if this component is not connected */ - if (!buffer->connected) - continue; - - count = preload_downstream(start, buffer->sink); - if (count < 0) - return count; - - total += count; - } - - return total; -} - /* prepare the pipeline for usage - preload host buffers here */ int pipeline_prepare(struct pipeline *p, struct comp_dev *dev) { struct op_data op_data; int ret = -1; - int i; trace_pipe("pre"); @@ -540,27 +494,10 @@ int pipeline_prepare(struct pipeline *p, struct comp_dev *dev) /* playback pipelines can be preloaded from host before trigger */ if (dev->params.direction == SOF_IPC_STREAM_PLAYBACK) { - /* first of all prepare the pipeline */ ret = component_op_downstream(&op_data, dev, dev, NULL); if (ret < 0) goto out; - /* then preload buffers - the buffers must be moved - * downstream so that every component has full buffers for - * trigger start */ - for (i = 0; i < MAX_PRELOAD_SIZE; i++) { - - ret = preload_downstream(dev, dev); - - /* errors or complete ? */ - if (ret <= 0) - break; - } - if (ret < 0) { - /* failed to preload */ - trace_pipe_error("epl"); - ret = -EIO; - } } else { ret = component_op_upstream(&op_data, dev, dev, NULL); } -- 2.11.0

1 2

[Sound-open-firmware] [PATCH] sof: fix spelling mistakes in comments
by Pierre-Louis Bossart 10 Nov '17

10 Nov '17

M-x ispell-comments-and-strings :set spell spelllang=en_us Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> --- src/audio/dai.c | 4 ++-- src/audio/eq_iir.c | 4 ++-- src/audio/host.c | 4 ++-- src/audio/mixer.c | 4 ++-- src/audio/mux.c | 2 +- src/audio/pipeline.c | 48 +++++++++++++++++++------------------- src/audio/pipeline_static.c | 2 +- src/audio/src.c | 8 +++---- src/audio/src_core.c | 4 ++-- src/audio/switch.c | 2 +- src/audio/tone.c | 4 ++-- src/audio/volume.c | 2 +- src/include/reef/audio/buffer.h | 2 +- src/include/reef/audio/component.h | 12 +++++----- src/include/reef/debug.h | 2 +- src/include/reef/dma.h | 2 +- src/include/reef/trace.h | 2 +- src/include/reef/wait.h | 2 +- src/ipc/byt-ipc.c | 2 +- src/ipc/dma-copy.c | 2 +- src/ipc/intel-ipc.c | 2 +- src/ipc/ipc.c | 2 +- src/lib/alloc.c | 8 +++---- src/lib/work.c | 2 +- src/platform/baytrail/clk.c | 4 ++-- src/platform/baytrail/platform.c | 2 +- src/platform/baytrail/timer.c | 12 +++++----- 27 files changed, 73 insertions(+), 73 deletions(-) diff --git a/src/audio/dai.c b/src/audio/dai.c index 36671fe..4d3f0b7 100644 --- a/src/audio/dai.c +++ b/src/audio/dai.c @@ -136,7 +136,7 @@ static void dai_dma_cb(void *data, uint32_t type, struct dma_sg_elem *next) } } - /* notify pipeline that DAI needs it's buffer processed */ + /* notify pipeline that DAI needs its buffer processed */ pipeline_schedule_copy(dev->pipeline, 0); return; @@ -455,7 +455,7 @@ static int dai_reset(struct comp_dev *dev) return 0; } -/* used to pass standard and bespoke commandd (with data) to component */ +/* used to pass standard and bespoke command (with data) to component */ static int dai_cmd(struct comp_dev *dev, int cmd, void *data) { struct dai_data *dd = comp_get_drvdata(dev); diff --git a/src/audio/eq_iir.c b/src/audio/eq_iir.c index 88382fe..1c2389c 100644 --- a/src/audio/eq_iir.c +++ b/src/audio/eq_iir.c @@ -175,7 +175,7 @@ static int eq_iir_setup(struct iir_state_df2t iir[], || (config->channels_in_config > PLATFORM_MAX_CHANNELS)) return -EINVAL; - /* Collect index of respose start positions in all_coefficients[] */ + /* Collect index of response start positions in all_coefficients[] */ j = 0; assign_response = &config->data[0]; coef_data = &config->data[config->channels_in_config]; @@ -234,7 +234,7 @@ static int eq_iir_switch_response(struct iir_state_df2t iir[], { int ret; - /* Copy assign response from update and re-initilize EQ */ + /* Copy assign response from update and re-initialize EQ */ if ((config == NULL) || (ch >= PLATFORM_MAX_CHANNELS)) return -EINVAL; diff --git a/src/audio/host.c b/src/audio/host.c index 3c77aca..aa608e7 100644 --- a/src/audio/host.c +++ b/src/audio/host.c @@ -70,10 +70,10 @@ struct host_data { struct hc_buf host; struct hc_buf local; uint32_t host_size; - /* host possition reporting related */ + /* host position reporting related */ volatile uint32_t *host_pos; /* read/write pos, update to mailbox for host side */ uint32_t report_pos; /* position in current report period */ - uint32_t local_pos; /* the host side buffer local read/write possition, in bytes */ + uint32_t local_pos; /* the host side buffer local read/write position, in bytes */ /* pointers set during params to host or local above */ struct hc_buf *source; struct hc_buf *sink; diff --git a/src/audio/mixer.c b/src/audio/mixer.c index 41f1d88..4d50bf1 100644 --- a/src/audio/mixer.c +++ b/src/audio/mixer.c @@ -239,7 +239,7 @@ static int mixer_copy(struct comp_dev *dev) return 0; } - /* dont have any work if all sources are inactive */ + /* don't have any work if all sources are inactive */ if (num_mix_sources == 0) return 0; @@ -255,7 +255,7 @@ static int mixer_copy(struct comp_dev *dev) if (xru) return 0; - /* make sure sink has no overuns */ + /* make sure sink has no overruns */ sink = list_first_item(&dev->bsink_list, struct comp_buffer, source_list); if (sink->free < md->period_bytes) { comp_overrun(dev, sink, sink->free, md->period_bytes); diff --git a/src/audio/mux.c b/src/audio/mux.c index f420bd3..ff6ad7c 100644 --- a/src/audio/mux.c +++ b/src/audio/mux.c @@ -52,7 +52,7 @@ static void mux_free(struct comp_dev *dev) } -/* set component audio stream paramters */ +/* set component audio stream parameters */ static int mux_params(struct comp_dev *dev) { diff --git a/src/audio/pipeline.c b/src/audio/pipeline.c index 0ef971e..61d80b4 100644 --- a/src/audio/pipeline.c +++ b/src/audio/pipeline.c @@ -85,7 +85,7 @@ static void connect_upstream(struct pipeline *p, struct comp_dev *start, buffer = container_of(clist, struct comp_buffer, sink_list); - /* dont go upstream if this source is from another pipeline */ + /* don't go upstream if this source is from another pipeline */ if (buffer->source->comp.pipeline_id != p->ipc_pipe.pipeline_id) continue; @@ -117,7 +117,7 @@ static void connect_downstream(struct pipeline *p, struct comp_dev *start, buffer = container_of(clist, struct comp_buffer, source_list); - /* dont go downstream if this sink is from another pipeline */ + /* don't go downstream if this sink is from another pipeline */ if (buffer->sink->comp.pipeline_id != p->ipc_pipe.pipeline_id) continue; @@ -142,14 +142,14 @@ static void disconnect_upstream(struct pipeline *p, struct comp_dev *start, buffer = container_of(clist, struct comp_buffer, sink_list); - /* dont go upstream if this source is from another pipeline */ + /* don't go upstream if this source is from another pipeline */ if (buffer->source->comp.pipeline_id != p->ipc_pipe.pipeline_id) continue; connect_upstream(p, start, buffer->source); } - /* diconnect source from buffer */ + /* disconnect source from buffer */ spin_lock(&current->lock); list_item_del(&current->bsource_list); spin_unlock(&current->lock); @@ -171,14 +171,14 @@ static void disconnect_downstream(struct pipeline *p, struct comp_dev *start, buffer = container_of(clist, struct comp_buffer, source_list); - /* dont go downstream if this sink is from another pipeline */ + /* don't go downstream if this sink is from another pipeline */ if (buffer->sink->comp.pipeline_id != p->ipc_pipe.pipeline_id) continue; connect_downstream(p, start, buffer->sink); } - /* diconnect source from buffer */ + /* disconnect source from buffer */ spin_lock(&current->lock); list_item_del(&current->bsink_list); spin_unlock(&current->lock); @@ -273,7 +273,7 @@ int pipeline_complete(struct pipeline *p) trace_pipe("com"); trace_value(p->ipc_pipe.pipeline_id); - /* cheeck whether pipeline is already complete */ + /* check whether pipeline is already complete */ if (p->status != COMP_STATE_INIT) { trace_pipe_error("epc"); return -EINVAL; @@ -342,7 +342,7 @@ static int component_op_downstream(struct op_data *op_data, switch (op_data->op) { case COMP_OPS_PARAMS: - /* dont do any params downstream if current is running */ + /* don't do any params downstream if current is running */ if (current->state == COMP_STATE_ACTIVE) return 0; @@ -373,7 +373,7 @@ static int component_op_downstream(struct op_data *op_data, return -EINVAL; } - /* dont walk the graph any further if this component fails */ + /* don't walk the graph any further if this component fails */ if (err < 0) { trace_pipe_error("eOp"); return err; @@ -391,7 +391,7 @@ static int component_op_downstream(struct op_data *op_data, buffer = container_of(clist, struct comp_buffer, source_list); - /* dont go downstream if this component is not connected */ + /* don't go downstream if this component is not connected */ if (!buffer->connected) continue; @@ -419,7 +419,7 @@ static int component_op_upstream(struct op_data *op_data, switch (op_data->op) { case COMP_OPS_PARAMS: - /* dont do any params upstream if current is running */ + /* don't do any params upstream if current is running */ if (current->state == COMP_STATE_ACTIVE) return 0; @@ -450,7 +450,7 @@ static int component_op_upstream(struct op_data *op_data, return -EINVAL; } - /* dont walk the graph any further if this component fails */ + /* don't walk the graph any further if this component fails */ if (err < 0) { trace_pipe_error("eOp"); return err; @@ -465,7 +465,7 @@ static int component_op_upstream(struct op_data *op_data, buffer = container_of(clist, struct comp_buffer, sink_list); - /* dont go upstream if this component is not connected */ + /* don't go upstream if this component is not connected */ if (!buffer->connected) continue; @@ -509,7 +509,7 @@ static int preload_downstream(struct comp_dev *start, struct comp_dev *current) buffer = container_of(clist, struct comp_buffer, source_list); - /* dont go downstream if this component is not connected */ + /* don't go downstream if this component is not connected */ if (!buffer->connected) continue; @@ -707,11 +707,11 @@ static int pipeline_copy_from_upstream(struct comp_dev *start, buffer = container_of(clist, struct comp_buffer, sink_list); - /* dont go upstream if this component is not connected */ + /* don't go upstream if this component is not connected */ if (!buffer->connected || buffer->source->state != COMP_STATE_ACTIVE) continue; - /* dont go upstream if this source is from another pipeline */ + /* don't go upstream if this source is from another pipeline */ if (buffer->source->pipeline != current->pipeline) continue; @@ -765,11 +765,11 @@ static int pipeline_copy_to_downstream(struct comp_dev *start, buffer = container_of(clist, struct comp_buffer, source_list); - /* dont go downstream if this component is not connected */ + /* don't go downstream if this component is not connected */ if (!buffer->connected || buffer->sink->state != COMP_STATE_ACTIVE) continue; - /* dont go downstream if this sink is from another pipeline */ + /* don't go downstream if this sink is from another pipeline */ if (buffer->sink->pipeline != current->pipeline) continue; @@ -813,7 +813,7 @@ downstream: buffer = container_of(clist, struct comp_buffer, source_list); - /* dont go downstream if this component is not connected */ + /* don't go downstream if this component is not connected */ if (!buffer->connected || buffer->sink->state != COMP_STATE_ACTIVE) continue; @@ -856,7 +856,7 @@ upstream: buffer = container_of(clist, struct comp_buffer, sink_list); - /* dont go downstream if this component is not connected */ + /* don't go downstream if this component is not connected */ if (!buffer->connected || buffer->source->state != COMP_STATE_ACTIVE) continue; @@ -913,7 +913,7 @@ static void pipeline_for_each_downstream(struct pipeline *p, buffer = container_of(clist, struct comp_buffer, source_list); - /* dont go downstream if this component is not connected */ + /* don't go downstream if this component is not connected */ if (!buffer->connected) continue; @@ -939,7 +939,7 @@ static void pipeline_for_each_upstream(struct pipeline *p, buffer = container_of(clist, struct comp_buffer, sink_list); - /* dont go downstream if this component is not connected */ + /* don't go downstream if this component is not connected */ if (!buffer->connected) continue; @@ -957,7 +957,7 @@ void pipeline_xrun(struct pipeline *p, struct comp_dev *dev, { struct sof_ipc_stream_posn posn; - /* dont flood host */ + /* don't flood host */ if (p->xrun_bytes) return; @@ -995,7 +995,7 @@ static void pipeline_task(void *arg) tracev_pipe("PWs"); - /* copy data from upstream source enpoints to downstream endpoints */ + /* copy data from upstream source endpoints to downstream endpoints */ pipeline_copy_from_upstream(dev, dev); pipeline_copy_to_downstream(dev, dev); diff --git a/src/audio/pipeline_static.c b/src/audio/pipeline_static.c index 88d52f2..7812dad 100644 --- a/src/audio/pipeline_static.c +++ b/src/audio/pipeline_static.c @@ -382,7 +382,7 @@ int init_static_pipeline(struct ipc *ipc) /* register components for this pipeline */ for (j = 0; j < spipe[i].num_scomps; j++) { - /* all pipeline componnets have same header */ + /* all pipeline components have same header */ c = sc[j].comps; for (k = 0; k < sc[j].num_comps; k++) { diff --git a/src/audio/src.c b/src/audio/src.c index 9fe81d3..d71a27a 100644 --- a/src/audio/src.c +++ b/src/audio/src.c @@ -382,7 +382,7 @@ static int src_params(struct comp_dev *dev) return -EINVAL; } - /* free any existing dalay lines. TODO reuse if same size */ + /* free any existing delay lines. TODO reuse if same size */ if (cd->delay_lines != NULL) rfree(cd->delay_lines); @@ -397,7 +397,7 @@ static int src_params(struct comp_dev *dev) memset(cd->delay_lines, 0, delay_lines_size); buffer_start = cd->delay_lines + cd->param.sbuf_length; - /* Initize SRC for actual sample rate */ + /* Initialize SRC for actual sample rate */ n = src_polyphase_init(&cd->src, &cd->param, buffer_start); /* Reset stage buffer */ @@ -511,8 +511,8 @@ static int src_copy(struct comp_dev *dev) * for one SRC run. The blk_in and blk are minimum condition to * call copy. Copy can consume or produce a slightly larger block * with the rates where block sizes are not constant. E.g. for - * 1 ms schduling the blocks can be under or above 1 ms when the - * SRC interal block size constraint prevents exact 1 ms blocks. + * 1 ms scheduling the blocks can be under or above 1 ms when the + * SRC interval block size constraint prevents exact 1 ms blocks. */ need_source = cd->param.blk_in * dev->frame_bytes; need_sink = cd->param.blk_out * dev->frame_bytes; diff --git a/src/audio/src_core.c b/src/audio/src_core.c index 080256c..dc22772 100644 --- a/src/audio/src_core.c +++ b/src/audio/src_core.c @@ -29,7 +29,7 @@ * */ -/* Non optimised default C implementation guaranteed to work on any +/* Non optimized default C implementation guaranteed to work on any * architecture. */ @@ -55,7 +55,7 @@ int sof_rates[SOF_RATES_LENGTH] = {8000, 11025, 12000, 16000, 18900, 22050, 24000, 32000, 44100, 48000, 64000, 88200, 96000, 176400, 192000}; -/* The FIR maximum lenghts are per channel so need to multiply them */ +/* The FIR maximum lengths are per channel so need to multiply them */ #define MAX_FIR_DELAY_SIZE_XNCH (PLATFORM_MAX_CHANNELS * MAX_FIR_DELAY_SIZE) #define MAX_OUT_DELAY_SIZE_XNCH (PLATFORM_MAX_CHANNELS * MAX_OUT_DELAY_SIZE) diff --git a/src/audio/switch.c b/src/audio/switch.c index 7149b61..5b4a8c8 100644 --- a/src/audio/switch.c +++ b/src/audio/switch.c @@ -52,7 +52,7 @@ static void switch_free(struct comp_dev *dev) } -/* set component audio stream paramters */ +/* set component audio stream parameters */ static int switch_params(struct comp_dev *dev) { diff --git a/src/audio/tone.c b/src/audio/tone.c index 2957270..ef5b7b1 100644 --- a/src/audio/tone.c +++ b/src/audio/tone.c @@ -190,7 +190,7 @@ static void tonegen_control(struct tone_state *sg) if (sg->block_count < INT32_MAX) sg->block_count++; - /* Fadein ramp during tone */ + /* Fade-in ramp during tone */ if (sg->block_count < sg->tone_length) { if (sg->a == 0) sg->w = 0; /* Reset phase to have less clicky ramp */ @@ -208,7 +208,7 @@ static void tonegen_control(struct tone_state *sg) sg->a = (int32_t) a; } - /* Fadeout ramp after tone*/ + /* Fade-out ramp after tone*/ if (sg->block_count > sg->tone_length) { a = (int64_t) sg->a - sg->ramp_step; if (a < 0) diff --git a/src/audio/volume.c b/src/audio/volume.c index e1cbda2..415a853 100644 --- a/src/audio/volume.c +++ b/src/audio/volume.c @@ -573,7 +573,7 @@ static int volume_copy(struct comp_dev *dev) } /* - * Volume componnet is usually first and last in pipelines so it makes sense + * Volume component is usually first and last in pipelines so it makes sense * to also do some type conversion here too. */ static int volume_prepare(struct comp_dev *dev) diff --git a/src/include/reef/audio/buffer.h b/src/include/reef/audio/buffer.h index ea9bd9c..d1d0869 100644 --- a/src/include/reef/audio/buffer.h +++ b/src/include/reef/audio/buffer.h @@ -77,7 +77,7 @@ struct comp_buffer { struct comp_buffer *buffer_new(struct sof_ipc_buffer *desc); void buffer_free(struct comp_buffer *buffer); -/* called by a component after pruducing data into this buffer */ +/* called by a component after producing data into this buffer */ static inline void comp_update_buffer_produce(struct comp_buffer *buffer, uint32_t bytes) { diff --git a/src/include/reef/audio/component.h b/src/include/reef/audio/component.h index 8f5a508..0b77e87 100644 --- a/src/include/reef/audio/component.h +++ b/src/include/reef/audio/component.h @@ -50,7 +50,7 @@ * States may transform as below:- * * 1) i.e. Initialisation to playback and pause/release - * init --> setup --> preprare --> active <-> paused --+ + * init --> setup --> prepare --> active <-> paused --+ * ^ | * +-----------------------------+ * @@ -125,13 +125,13 @@ struct comp_ops { struct comp_dev *(*new)(struct sof_ipc_comp *comp); void (*free)(struct comp_dev *dev); - /* set component audio stream paramters */ + /* set component audio stream parameters */ int (*params)(struct comp_dev *dev); /* preload buffers */ int (*preload)(struct comp_dev *dev); - /* set component audio stream paramters */ + /* set component audio stream parameters */ int (*dai_config)(struct comp_dev *dev, struct sof_ipc_dai_config *dai_config); @@ -325,7 +325,7 @@ static inline int comp_buffer_reset(struct comp_dev *dev) buffer = container_of(clist, struct comp_buffer, source_list); - /* dont reset buffer if the component is not connected */ + /* don't reset buffer if the component is not connected */ if (!buffer->connected) continue; @@ -341,11 +341,11 @@ static inline int comp_buffer_reset(struct comp_dev *dev) } /* - * Convenince functions to install upstream/downstream common params. Only + * Convenience functions to install upstream/downstream common params. Only * applicable to single upstream source. Components with > 1 source or sink * must do this manually. * - * This allows params to propagate from the host PCM component downstrean on + * This allows params to propagate from the host PCM component downstream on * playback and upstream on capture. */ static inline void comp_install_params(struct comp_dev *dev, diff --git a/src/include/reef/debug.h b/src/include/reef/debug.h index ef64419..70a7802 100644 --- a/src/include/reef/debug.h +++ b/src/include/reef/debug.h @@ -98,7 +98,7 @@ *(__m++) = *(__a++); \ } while (0); -/* dump data area at addr and size count at mailbox ofset or shared memory */ +/* dump data area at addr and size count at mailbox offset or shared memory */ #define dump_at(addr, count, offset) \ do { \ volatile uint32_t *__m = (uint32_t*)mailbox_get_debug_base() + offset; \ diff --git a/src/include/reef/dma.h b/src/include/reef/dma.h index 8dd4adf..697e2c6 100644 --- a/src/include/reef/dma.h +++ b/src/include/reef/dma.h @@ -52,7 +52,7 @@ /* We will use this macro in cb handler to inform dma that - * we need to stop the reload for specail purpose + * we need to stop the reload for special purpose */ #define DMA_RELOAD_END 0 #define DMA_RELOAD_LLI 0xFFFFFFFF diff --git a/src/include/reef/trace.h b/src/include/reef/trace.h index 17dff5c..ef82d75 100644 --- a/src/include/reef/trace.h +++ b/src/include/reef/trace.h @@ -41,7 +41,7 @@ #include <platform/platform.h> #include <platform/timer.h> -/* general trace init codes - only used at boot when main trace is not availble */ +/* general trace init codes - only used at boot when main trace is not available */ #define TRACE_BOOT_START 0x1000 #define TRACE_BOOT_ARCH 0x2000 #define TRACE_BOOT_SYS 0x3000 diff --git a/src/include/reef/wait.h b/src/include/reef/wait.h index c5b370e..3354eb4 100644 --- a/src/include/reef/wait.h +++ b/src/include/reef/wait.h @@ -27,7 +27,7 @@ * * Author: Liam Girdwood <liam.r.girdwood(a)linux.intel.com> * - * Simple wait for event completion and signalling with timeouts. + * Simple wait for event completion and signaling with timeouts. */ #ifndef __INCLUDE_WAIT__ diff --git a/src/ipc/byt-ipc.c b/src/ipc/byt-ipc.c index fd1ad1e..8897bb9 100644 --- a/src/ipc/byt-ipc.c +++ b/src/ipc/byt-ipc.c @@ -179,7 +179,7 @@ void ipc_platform_send_msg(struct ipc *ipc) goto out; } - /* can't send nofication when one is in progress */ + /* can't send notification when one is in progress */ if (shim_read(SHIM_IPCDH) & (SHIM_IPCDH_BUSY | SHIM_IPCDH_DONE)) goto out; diff --git a/src/ipc/dma-copy.c b/src/ipc/dma-copy.c index 5c9ff97..61a919e 100644 --- a/src/ipc/dma-copy.c +++ b/src/ipc/dma-copy.c @@ -81,7 +81,7 @@ static void dma_complete(void *data, uint32_t type, struct dma_sg_elem *next) /* Copy DSP memory to host memory. * copies DSP memory to host in PAGE_SIZE or smaller blocks and waits/sleeps - * between blocks. Cant be used in IRQ context. + * between blocks. Can't be used in IRQ context. */ int dma_copy_to_host(struct dma_copy *dc, struct dma_sg_config *host_sg, int32_t host_offset, void *local_ptr, int32_t size) diff --git a/src/ipc/intel-ipc.c b/src/ipc/intel-ipc.c index f6d88ef..ec91499 100644 --- a/src/ipc/intel-ipc.c +++ b/src/ipc/intel-ipc.c @@ -117,7 +117,7 @@ static int get_page_descriptors(struct intel_ipc_data *iipc, config.cyclic = 0; list_init(&config.elem_list); - /* set up DMA desciptor */ + /* set up DMA descriptor */ elem.dest = (uint32_t)iipc->page_table; elem.src = ring->phy_addr; diff --git a/src/ipc/ipc.c b/src/ipc/ipc.c index 6c40d10..61b0cf2 100644 --- a/src/ipc/ipc.c +++ b/src/ipc/ipc.c @@ -46,7 +46,7 @@ /* * Components, buffers and pipelines all use the same set of monotonic ID * numbers passed in by the host. They are stored in different lists, hence - * more than 1 list may need to be searched for the correspodning component. + * more than 1 list may need to be searched for the corresponding component. */ struct ipc_comp_dev *ipc_get_comp(struct ipc *ipc, uint32_t id) diff --git a/src/lib/alloc.c b/src/lib/alloc.c index 63f92de..847cb93 100644 --- a/src/lib/alloc.c +++ b/src/lib/alloc.c @@ -259,7 +259,7 @@ static void *alloc_block(struct mm_heap *heap, int level, int bflags) return ptr; } -/* allocates continious blocks */ +/* allocates continuous blocks */ static void *alloc_cont_blocks(struct mm_heap *heap, int level, int bflags, size_t bytes) { @@ -276,7 +276,7 @@ static void *alloc_cont_blocks(struct mm_heap *heap, int level, int bflags, if (bytes % map->block_size) count++; - /* check for continious blocks from "start" */ + /* check for continuous blocks from "start" */ for (start = map->first_free; start < remaining; start++) { /* check that we have enough free blocks from start pos */ @@ -387,7 +387,7 @@ found: block = ((uint32_t)ptr - block_map->base) / block_map->block_size; hdr = &block_map->block[block]; - /* free block header and continious blocks */ + /* free block header and continuous blocks */ for (i = block; i < block + hdr->size; i++) { hdr = &block_map->block[i]; hdr->size = 0; @@ -608,7 +608,7 @@ int mm_pm_context_save(struct dma_copy *dc, struct dma_sg_config *sg) } /* - * Restore the DSP memories to modules abd the system. This must be called immediately + * Restore the DSP memories to modules and the system. This must be called immediately * after booting before any pipeline work. */ int mm_pm_context_restore(struct dma_copy *dc, struct dma_sg_config *sg) diff --git a/src/lib/work.c b/src/lib/work.c index 587113e..18d743c 100644 --- a/src/lib/work.c +++ b/src/lib/work.c @@ -314,7 +314,7 @@ static void work_notify(int message, void *data, void *event_data) spin_lock_irq(&queue->lock, flags); - /* we need to re-caclulate timer when CPU frequency changes */ + /* we need to re-calculate timer when CPU frequency changes */ if (message == CLOCK_NOTIFY_POST) { /* CPU frequency update complete */ diff --git a/src/platform/baytrail/clk.c b/src/platform/baytrail/clk.c index 28cdb83..eac64c3 100644 --- a/src/platform/baytrail/clk.c +++ b/src/platform/baytrail/clk.c @@ -117,7 +117,7 @@ static inline uint32_t get_freq(const struct freq_table *table, int size, { uint32_t i; - /* find lowest available frequency that is >= requested hz */ + /* find lowest available frequency that is >= requested Hz */ for (i = 0; i < size; i++) { if (hz <= table[i].freq) return i; @@ -159,7 +159,7 @@ uint32_t clock_set_freq(int clock, uint32_t hz) notify_data.old_freq = clk_pdata->clk[clock].freq; notify_data.old_ticks_per_usec = clk_pdata->clk[clock].ticks_per_usec; - /* atomic context for chaning clocks */ + /* atomic context for chaining clocks */ spin_lock_irq(&clk_pdata->clk[clock].lock, flags); switch (clock) { diff --git a/src/platform/baytrail/platform.c b/src/platform/baytrail/platform.c index c175a1b..aba7645 100644 --- a/src/platform/baytrail/platform.c +++ b/src/platform/baytrail/platform.c @@ -56,7 +56,7 @@ static const struct sof_ipc_fw_ready ready = { .cmd = SOF_IPC_FW_READY, .size = sizeof(struct sof_ipc_fw_ready), }, - /* dspbox is for DSP initiated IPC, hostbox is for host iniiated IPC */ + /* dspbox is for DSP initiated IPC, hostbox is for host initiated IPC */ .dspbox_offset = MAILBOX_HOST_OFFSET + MAILBOX_DSPBOX_OFFSET, .hostbox_offset = MAILBOX_HOST_OFFSET + MAILBOX_HOSTBOX_OFFSET, .dspbox_size = MAILBOX_DSPBOX_SIZE, diff --git a/src/platform/baytrail/timer.c b/src/platform/baytrail/timer.c index 3397ef0..8c205bc 100644 --- a/src/platform/baytrail/timer.c +++ b/src/platform/baytrail/timer.c @@ -53,7 +53,7 @@ static void platform_timer_64_handler(void *arg) /* get timeout value - will tell us timeout reason */ timeout = shim_read(SHIM_EXT_TIMER_CNTLL); - /* we dont use the timer clear bit as we only need to clear the ISR */ + /* we don't use the timer clear bit as we only need to clear the ISR */ shim_write(SHIM_PISR, SHIM_PISR_EXT_TIMER); /* is this a 32 bit rollover ? */ @@ -130,7 +130,7 @@ int platform_timer_set(struct timer *timer, uint64_t ticks) void platform_timer_clear(struct timer *timer) { - /* we dont use the timer clear bit as we only need to clear the ISR */ + /* we don't use the timer clear bit as we only need to clear the ISR */ shim_write(SHIM_PISR, SHIM_PISR_EXT_TIMER); } @@ -149,7 +149,7 @@ uint64_t platform_timer_get(struct timer *timer) /* check and see whether 32bit IRQ is pending for timer */ if (arch_interrupt_get_status() & IRQ_MASK_EXT_TIMER && shim_read(SHIM_EXT_TIMER_CNTLL) == 1) { - /* yes, overflow has occured but handler has not run */ + /* yes, overflow has occurred but handler has not run */ high = timer->hitime + 1; } else { /* no overflow */ @@ -169,7 +169,7 @@ void platform_host_timestamp(struct comp_dev *host, { int err; - /* get host postion */ + /* get host position */ err = comp_position(host, posn); if (err == 0) posn->flags |= SOF_TIME_HOST_VALID | SOF_TIME_HOST_64; @@ -181,7 +181,7 @@ void platform_dai_timestamp(struct comp_dev *dai, { int err; - /* get DAI postion */ + /* get DAI position */ err = comp_position(dai, posn); if (err == 0) posn->flags |= SOF_TIME_DAI_VALID; @@ -191,7 +191,7 @@ void platform_dai_timestamp(struct comp_dev *dai, posn->flags |= SOF_TIME_WALL_VALID | SOF_TIME_WALL_64; } -/* get current wallclock for componnent */ +/* get current wallclock for component */ void platform_dai_wallclock(struct comp_dev *dai, uint64_t *wallclock) { /* only 1 wallclock on BYT */ -- 2.14.1

2 1

[Sound-open-firmware] [PATCH 0/2] macros and register cleanups
by Pierre-Louis Bossart 10 Nov '17

10 Nov '17

Found those issues while randomly looking at register configurations No functionality change, just better looking/safer code Pierre-Louis Bossart (2): sof: protect macro parameters dma: dw: reorder register definitions by sequential address src/drivers/dw-dma.c | 24 ++++++++++++------------ src/include/reef/ssp.h | 10 +++++----- src/include/uapi/intel-ipc.h | 20 ++++++++++---------- src/include/uapi/ipc.h | 4 ++-- src/platform/baytrail/include/platform/dma.h | 8 ++++---- 5 files changed, 33 insertions(+), 33 deletions(-) -- 2.11.0

2 3

[Sound-open-firmware] [PATCH v2 4/4] Fix DMA host offset calculation and wrapper condition checking.
by yan.wang＠linux.intel.com 10 Nov '17

10 Nov '17

From: Yan Wang <yan.wang(a)linux.intel.com> 1. "size" should not be considered twice. 2. Change host_offset to uint32_t type. 3. size = min(hsize, lsize), so it is unncessary to check ">" for wrap local and host buffer. Signed-off-by: Yan Wang <yan.wang(a)linux.intel.com> --- src/include/reef/dma-trace.h | 2 +- src/lib/dma-trace.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/include/reef/dma-trace.h b/src/include/reef/dma-trace.h index 12d0956..641a4dc 100644 --- a/src/include/reef/dma-trace.h +++ b/src/include/reef/dma-trace.h @@ -56,7 +56,7 @@ struct dma_trace_data { struct dma_sg_config config; struct dma_trace_buf dmatb; struct dma_copy dc; - int32_t host_offset; + uint32_t host_offset; uint32_t host_size; struct work dmat_work; uint32_t enabled; diff --git a/src/lib/dma-trace.c b/src/lib/dma-trace.c index 5885037..763e955 100644 --- a/src/lib/dma-trace.c +++ b/src/lib/dma-trace.c @@ -90,12 +90,12 @@ static uint64_t trace_work(void *data, uint64_t delay) /* update host pointer and check for wrap */ d->host_offset += size; - if (d->host_offset + size >= d->host_size) + if (d->host_offset == d->host_size) d->host_offset = 0; /* update local pointer and check for wrap */ buffer->r_ptr += size; - if (buffer->r_ptr >= buffer->end_addr) + if (buffer->r_ptr == buffer->end_addr) buffer->r_ptr = buffer->addr; out: -- 2.7.4

2 1

[Sound-open-firmware] [PATCH] dw-dma: fix element might be used uninitialized issue
by Keyon Jie 10 Nov '17

10 Nov '17

The dma_sg_elem might be used uninitialized if the callback function forget to do that. Here adding initialization to fix that. Signed-off-by: Keyon Jie <yang.jie(a)linux.intel.com> --- src/drivers/dw-dma.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/drivers/dw-dma.c b/src/drivers/dw-dma.c index 05791e6..9aeb612 100644 --- a/src/drivers/dw-dma.c +++ b/src/drivers/dw-dma.c @@ -735,6 +735,7 @@ static void dw_dma_irq_handler(void *data) if ((status_tfr & mask) && (p->chan[i].cb_type & DMA_IRQ_TYPE_LLIST)) { + next.src = next.dest = DMA_RELOAD_LLI; next.size = DMA_RELOAD_LLI; /* will reload lli by default */ if (p->chan[i].cb) p->chan[i].cb(p->chan[i].cb_data, -- 2.11.0

2 1

[Sound-open-firmware] [PATCH 4/4] Fix DMA host offset calculation.
by yan.wang＠linux.intel.com 10 Nov '17

10 Nov '17

From: Yan Wang <yan.wang(a)linux.intel.com> 1. "size" should not be considered twice. 2. Change host_offset to uint32_t type. Signed-off-by: Yan Wang <yan.wang(a)linux.intel.com> --- src/include/reef/dma-trace.h | 2 +- src/lib/dma-trace.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/include/reef/dma-trace.h b/src/include/reef/dma-trace.h index 12d0956..641a4dc 100644 --- a/src/include/reef/dma-trace.h +++ b/src/include/reef/dma-trace.h @@ -56,7 +56,7 @@ struct dma_trace_data { struct dma_sg_config config; struct dma_trace_buf dmatb; struct dma_copy dc; - int32_t host_offset; + uint32_t host_offset; uint32_t host_size; struct work dmat_work; uint32_t enabled; diff --git a/src/lib/dma-trace.c b/src/lib/dma-trace.c index 5885037..249e18b 100644 --- a/src/lib/dma-trace.c +++ b/src/lib/dma-trace.c @@ -90,7 +90,7 @@ static uint64_t trace_work(void *data, uint64_t delay) /* update host pointer and check for wrap */ d->host_offset += size; - if (d->host_offset + size >= d->host_size) + if (d->host_offset >= d->host_size) d->host_offset = 0; /* update local pointer and check for wrap */ -- 2.7.4

4 4

[Sound-open-firmware] [PATCH 3/4] Send IPC message of DMA trace host update in DMA complete callback.
by yan.wang＠linux.intel.com 09 Nov '17

09 Nov '17

From: Yan Wang <yan.wang(a)linux.intel.com> It is called every time DMA completes. Signed-off-by: Yan Wang <yan.wang(a)linux.intel.com> --- src/ipc/dma-copy.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/ipc/dma-copy.c b/src/ipc/dma-copy.c index 5c9ff97..6ba8f57 100644 --- a/src/ipc/dma-copy.c +++ b/src/ipc/dma-copy.c @@ -34,6 +34,7 @@ #include <reef/reef.h> #include <reef/debug.h> #include <reef/trace.h> +#include <reef/ipc.h> #include <reef/dma.h> #include <reef/wait.h> #include <platform/dma.h> @@ -76,6 +77,8 @@ static void dma_complete(void *data, uint32_t type, struct dma_sg_elem *next) if (type == DMA_IRQ_TYPE_LLIST) wait_completed(comp); + ipc_dma_trace_send_position(); + next->size = DMA_RELOAD_END; } -- 2.7.4

1 0

[Sound-open-firmware] [PATCH 2/4] Add API to send DMA trace host offset.
by yan.wang＠linux.intel.com 09 Nov '17

09 Nov '17

From: Yan Wang <yan.wang(a)linux.intel.com> Fill IPC strucure by current host offset and send it. Signed-off-by: Yan Wang <yan.wang(a)linux.intel.com> --- src/include/reef/ipc.h | 3 +++ src/ipc/intel-ipc.c | 13 +++++++++++++ 2 files changed, 16 insertions(+) diff --git a/src/include/reef/ipc.h b/src/include/reef/ipc.h index 472aca8..fcab45b 100644 --- a/src/include/reef/ipc.h +++ b/src/include/reef/ipc.h @@ -166,4 +166,7 @@ struct ipc_comp_dev *ipc_get_comp(struct ipc *ipc, uint32_t id); */ int ipc_comp_dai_config(struct ipc *ipc, struct sof_ipc_dai_config *config); +/* send DMA trace host buffer position to host */ +int ipc_dma_trace_send_position(void); + #endif diff --git a/src/ipc/intel-ipc.c b/src/ipc/intel-ipc.c index 902e5a6..1df88c4 100644 --- a/src/ipc/intel-ipc.c +++ b/src/ipc/intel-ipc.c @@ -619,6 +619,19 @@ error: return -EINVAL; } +/* send DMA trace host buffer position to host */ +int ipc_dma_trace_send_position(void) +{ + struct sof_ipc_dma_trace_posn posn; + + posn.rhdr.hdr.cmd = SOF_IPC_GLB_TRACE_MSG | SOF_IPC_TRACE_DMA_POSITION; + posn.host_offset = _ipc->dmat.host_offset; + posn.rhdr.hdr.size = sizeof(posn); + + return ipc_queue_host_message(_ipc, posn.rhdr.hdr.cmd, &posn, + sizeof(posn), NULL, 0, NULL, NULL); +} + static int ipc_glb_debug_message(uint32_t header) { uint32_t cmd = (header & SOF_CMD_TYPE_MASK) >> SOF_CMD_TYPE_SHIFT; -- 2.7.4

1 0

[Sound-open-firmware] [PATCH 1/4] Add IPC new message type and structure for sending DMA trace host offset.
by yan.wang＠linux.intel.com 09 Nov '17

09 Nov '17

From: Yan Wang <yan.wang(a)linux.intel.com> It is used to update DMA trace host offset to kernel for realtime output DMA trace. Signed-off-by: Yan Wang <yan.wang(a)linux.intel.com> --- src/include/uapi/ipc.h | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/include/uapi/ipc.h b/src/include/uapi/ipc.h index fc83721..07235e1 100644 --- a/src/include/uapi/ipc.h +++ b/src/include/uapi/ipc.h @@ -118,6 +118,7 @@ /* trace and debug */ #define SOF_IPC_TRACE_DMA_PARAMS SOF_CMD_TYPE(0x001) +#define SOF_IPC_TRACE_DMA_POSITION SOF_CMD_TYPE(0x002) /* Get message component id */ #define SOF_IPC_MESSAGE_ID(x) (x & 0xffff) @@ -810,4 +811,10 @@ struct sof_ipc_dma_trace_params { struct sof_ipc_host_buffer buffer; } __attribute__((packed)); +/* DMA for Trace params info - SOF_IPC_DEBUG_DMA_PARAMS */ +struct sof_ipc_dma_trace_posn { + struct sof_ipc_reply rhdr; + uint32_t host_offset; /* Offset of DMA host buffer */ +} __attribute__((packed)); + #endif -- 2.7.4

1 0