[DO NOT MERGE] PR only for internal CI test - #5901
Draft
wszypelt wants to merge 37 commits into
Draft
Conversation
Some platforms do not have a SSP clock, so a dummy define for CLK_SSP and related macros shouldn't be required to exist in order for the firmware to build. Signed-off-by: Paul Olaru <paul.olaru@nxp.com>
Compiler options has to be changed in order to compute the literal block address correctly. Some literals must be declared explicitly and loaded indirectly to make sure that compiler does not optimize the out to another shared region. Signed-off-by: Marcin Maka <marcin.maka@linux.intel.com>
Fix errorneous parameters in DAI_ADD. Signed-off-by: Jaska Uimonen <jaska.uimonen@intel.com>
Correct log verbosity in SAI driver Signed-off-by: Guido Roncarolo <guido.roncarolo@nxp.com> Signed-off-by: Jerome Laclavere <jerome.laclavere@nxp.com>
xcc HIFI2 do not vec_memcpy, we can only use vec_memcpy on HIFI3 supported platforms. Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
Use SOF_MEM_VECBASE_TEXT_BASE to repalce unset variable. Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
xcc build will have bigger TEXT. Need to keep the total block size unchange as the block need to be 4k align. Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
The previous version changes behavior on the BYT platform. Fixes: ee9283c ("sof: lib: notifier: Do not break build on platforms without CLK_SSP"). Signed-off-by: Paul Olaru <paul.olaru@nxp.com>
This patch fixes DAI_ADD function call for speaker pipeline. The function call had incorrect number of agruments passed which was causing issues with playback. Signed-off-by: WojciechX Wittbrodt <wojciechx.wittbrodt@intel.com>
Invalidates memory regions, which are requested to be uncached. This way we can make sure we won't overwrite data by accident. Signed-off-by: Tomasz Lauda <tomasz.lauda@linux.intel.com>
Fixes modulo bug in get_ptr_from_heap function, where alignment can potentially be 0. Modulo zero is undefined operation. Signed-off-by: Tomasz Lauda <tomasz.lauda@linux.intel.com>
This patch fixes the issue of audio left muted after a cycle of mute and unmute commands. The additional improvements are applying of gain ramp for mute that helps to avoid annoying click sounds. Also since the driver issues both volume and switch commands back to back the duplicate scheduling of ramp update is prevented if there is already a ramp happening. The old ramp will operate based on new step and direction if it was changed. Some variable names were changed in function volume_set_chan() for better readability. The impacted trace print commands were also updated to not split strings unnecessarily. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
The firmware may have modified these values to execute features such as a boot beep. Therefore we cannot assume they are set to their defaults. Lets write the defaults just to be safe. Fixes #1847 Signed-off-by: Curtis Malainey <cujomalainey@google.com>
BDW DMAC0 can not be used with scheduler and no-one used it. The DMA chan is not initiated and may cause DSP hang when read null point. Skip the DMAC0 to avoid such issue. Signed-off-by: Tomasz Lauda <tomasz.lauda@linux.intel.com> Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
SSP clock on BDW is 24MHz. Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
If CONFIG_GDB_DEBUG is set, the build is currently failing because of these void pointer arithmentic operations. This patch fixes that. Signed-off-by: Dragos Tarcatu <dragos_tarcatu@mentor.com>
If CONFIG_GDB_DEBUG is set, the build fails at link time because of undefined references to arch_gdb_memory_load_and_store, arch_gdb_load_from_memory, gdb_debug_info, and several others. Fix that by replacing CONFIG_GDB with CONFIG_GDB_DEBUG. Signed-off-by: Dragos Tarcatu <dragos_tarcatu@mentor.com>
Sets iir_delay pointer to NULL when freed to prevent attempts to free the buffer multiple times. Signed-off-by: Tomasz Lauda <tomasz.lauda@linux.intel.com>
Sets fir_delay pointer to NULL when freed to prevent attempts to free the buffer multiple times. Signed-off-by: Tomasz Lauda <tomasz.lauda@linux.intel.com>
Fixes low latency task insertion into the priority list. Tasks should be inserted into the list from the highest to the lowest priority. Tasks with the same priority should be served on a first-come-first-serve basis. Note that the list_item_append method, which is used to insert task into the list is a bit misleading, since it normally should work on list's head instead of any element from the list. This piece of code should be revisited in the future and proper list_item_insert method should be implemented and used. Signed-off-by: Tomasz Lauda <tomasz.lauda@linux.intel.com>
That field is intended to point to the first free block of this particular block map (part of heap). As long as the heap doesn't fill up the old code manages it correctly. However once the block map becomes full (all blocks are occupied) this member will no longer point to a free block. Without this fix rfree may fail to update this pointer to point to the correct block when freeing a block from a full heap. Not updating this will cause subsequent memory allocations to hand out used blocks as if they were free. This fix ensures that will not happen in this edge case. Signed-off-by: Paul Olaru <paul.olaru@nxp.com>
Command git describe looks only for annotated tags, but we should get any tag reachable from master, that's why --tags flag is needed. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Update bug form to point to new location of the document. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Aligns sof structure to cache line size. It is needed in order to assure that no data will be randomly overwritten by cache eviction with multicore access. Signed-off-by: Tomasz Lauda <tomasz.lauda@linux.intel.com>
Add registers for checking clock status. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Add missing registers for requesting clock changes. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Add registers for requesting clocks. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Add registers for checking clock status. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Add registers for requesting clocks. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Add registers for checking clock status. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Add registers for requesting clocks. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Add registers for checking clock status. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
On some platforms clocks may not be switched instantly and f.e. there may be need to wait for clock availability before switching to it. That's why there is need for mask that is used for checking status. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Moves interrupt controller names to cavs platform header. They are only used by cAVS platforms and shouldn't be generic to xtensa architecture. Signed-off-by: Tomasz Lauda <tomasz.lauda@linux.intel.com>
Add freq indexes for LPRO and HPRO, to be able to request clock source by providing specific frequency index instead of frequency in Hz. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
On some platforms there is a need to perform tasks after coming back from waiti. There is no central interrupts processing function in SOF. Every interrupt is registered separaterly, so we need to execute 'wake up' functions in xtos interrupts handlers. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
Platforms with cAVS version 1.8 & 2.0 have hardware requirement that DSP should use LPRO as clock source in waiti. This patch adds config for that and enables it for platforms that need it. Signed-off-by: Janusz Jankowski <janusz.jankowski@linux.intel.com>
|
@copilot rebase this PR to newest commit main branch |
Merged the newest |
|
The last copilot action was reverted with git reset hard to 7605252 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[DO NOT MERGE] PR only for internal CI test