CI ONLY #9521
Draft
CI ONLY #9521
Conversation
This file contains 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
marcinszkudlinski
force-pushed
the
test1
branch
5 times, most recently
from
2110a40 to
37936d9
Compare
marcinszkudlinski
force-pushed
the
test1
branch
from
37936d9 to
d5a0785
Compare
marcinszkudlinski
force-pushed
the
test1
branch
3 times, most recently
from
ad1c8dd to
7af9547
Compare
module_source_status_count in fact counts "state" field of all components. Name changed to module_source_state_count Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>
comp_buffer::sink_list field used to have 2 meanings:
- a list connector of all data receivers from the module
in this case list is based on
comp_dev::bsource_list
- a list connector for data receivers for extra modules
allocated when raw data processing is in use
in this case list is based on
processing_module::sink_buffer_list
This commit separate both cases
case 1 - no changes
case 2 -
list is now based on processing_module::raw_data_buffers_list
list is connected by a new field comp_buffer::buffers_list
Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>
All control structures of buffers shared between cores are now stored in a non-cached memory alias. Cache writeback_invalidate is no longer needed here Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>
All componetns had been modified to use buffer's API some time ago, those 2 were either missed or have been modified later Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>
this commit adds 4 methods to comp_buffer API and refactors one macro to a static inline function in .c file the reason - macro was used once only and provides no type control Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>
add comp_dev_for_each_consumer_safe and comp_dev_for_each_producer_safe Those macros allow iteration through all buffers allowing buffers deletion Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>
Unit test should also - if possible - use buffer API There are, however, some parts of testing that still use direct access to comp_buffer internals. Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>
This commit make all pipeline management code using comp_buffer API. Since now, no access to comp_buffer internals should be performed by the SOF code (with exception of uint test) Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>
marcinszkudlinski
force-pushed
the
test1
branch
from
7af9547 to
0e7bb6d
Compare
marcinszkudlinski
force-pushed
the
test1
branch
from
ab31184 to
b96a987
Compare
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.
No description provided.