-
Notifications
You must be signed in to change notification settings - Fork 350
[DRAFT] userspace LL/audio test PR #10558
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
cfa025c
1d68786
32b1cdf
1b74e9c
79423d3
5a5db66
d73e71e
7925ddd
9339704
d9ed8e3
eae7037
9428877
1895cdd
8096e55
715a0b9
99a529e
0143bd4
98f261a
6938fb5
81d7248
6a18d1a
61c897c
d3f39bc
c5cb17c
5a0041c
0a74bd9
2fbe798
5668d97
3c5be40
49d81e6
16cee1a
43a7781
59adf19
428502e
623ea7b
843c9f0
30b293f
86469a9
10ee826
faa53f3
8e80373
ff4e2b3
abf1ae5
9603f85
e9da9ac
3ff8323
345bec9
dcdd76d
7e36ef5
2c21495
0007cef
a2966fb
74844da
693a742
fe1f3ed
36c96cc
20a7918
38c2b13
e3439bf
3dd5ee3
6cbf702
1c7576f
fed1761
ea2b7a5
011c23b
a91a26a
76c6561
f42df88
67c3a92
271d7dc
d574f22
279b652
aa2d776
a00b0ef
47a8b8a
bd4789c
f7db230
090096d
12ae283
c029d49
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -36,7 +36,7 @@ | |
|
|
||
| LOG_MODULE_REGISTER(component, CONFIG_SOF_LOG_LEVEL); | ||
|
|
||
| static SHARED_DATA struct comp_driver_list cd; | ||
| static APP_SYSUSER_BSS SHARED_DATA struct comp_driver_list cd; | ||
|
|
||
| SOF_DEFINE_REG_UUID(component); | ||
|
|
||
|
|
@@ -699,3 +699,11 @@ void comp_update_ibs_obs_cpc(struct comp_dev *dev) | |
| #endif | ||
| } | ||
|
|
||
| #ifdef CONFIG_SOF_USERSPACE_LL | ||
| void comp_grant_access_to_thread(const struct comp_dev *dev, struct k_thread *th) | ||
| { | ||
| assert(dev->list_mutex); | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. description a bit confusing - this is only granting access to a mutex. Also |
||
| tr_dbg(&ipc_tr, "grant access to mutex %p for thread %p", dev->list_mutex, th); | ||
| k_thread_access_grant(th, dev->list_mutex); | ||
| } | ||
| #endif | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -50,10 +50,11 @@ struct host_data { | |
| /* local DMA config */ | ||
| #if CONFIG_ZEPHYR_NATIVE_DRIVERS | ||
| struct sof_dma *dma; | ||
| int chan_index; | ||
| #else | ||
| struct dma *dma; | ||
| #endif | ||
| struct dma_chan_data *chan; | ||
| #endif | ||
| struct dma_sg_config config; | ||
| #ifdef __ZEPHYR__ | ||
| struct dma_config z_config; | ||
|
|
@@ -112,6 +113,7 @@ struct host_data { | |
| uint64_t next_sync; | ||
| uint64_t period_in_cycles; | ||
| #endif | ||
| struct k_heap *heap; | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. wasn't this already referenced in the previous commit? |
||
| }; | ||
|
|
||
| int host_common_new(struct host_data *hd, struct comp_dev *dev, | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the commit, that is mentioned in the commit message, only moved buffer context objects to particular heaps. This commit moves actual data buffers to them too, which is different and (arguably) more risky