Skip to content

checkpatch: add __sparse_cache and __sparse_force attributes#5806

Merged
lgirdwood merged 1 commit intothesofproject:mainfrom
marc-hb:checkpatch-sparse
May 13, 2022
Merged

checkpatch: add __sparse_cache and __sparse_force attributes#5806
lgirdwood merged 1 commit intothesofproject:mainfrom
marc-hb:checkpatch-sparse

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented May 12, 2022

These attributes were added to Zephyr's include/zephyr/debug/sparse.h by
commit 17eb313a1b15 ("sparse: add an address space and a __sparse_force
annotation")

This gets rid of dozens of warnings in
https://github.com/thesofproject/sof/runs/6387526478

ERROR: need consistent spacing around '*' (ctx:WxV)
+	struct comp_buffer __sparse_cache *buffer_c, *sink_c;
 	                                  ^

ERROR: need consistent spacing around '*' (ctx:WxV)
+	struct comp_buffer __sparse_cache *source_c;
 	                                  ^

etc.

Signed-off-by: Marc Herbert marc.herbert@intel.com

These attributes were added to Zephyr's include/zephyr/debug/sparse.h by
commit 17eb313a1b15 ("sparse: add an address space and a __sparse_force
annotation")

This gets rid of dozens of warnings in
https://github.com/thesofproject/sof/runs/6387526478

```
ERROR: need consistent spacing around '*' (ctx:WxV)
+	struct comp_buffer __sparse_cache *buffer_c, *sink_c;
 	                                  ^

ERROR: need consistent spacing around '*' (ctx:WxV)
+	struct comp_buffer __sparse_cache *source_c;
 	                                  ^
```

etc.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb marc-hb marked this pull request as ready for review May 12, 2022 00:49
@marc-hb marc-hb requested a review from aborisovich as a code owner May 12, 2022 00:49
@marc-hb marc-hb requested review from lgirdwood and lyakh May 12, 2022 00:50
@marc-hb
Copy link
Collaborator Author

marc-hb commented May 12, 2022

The test failures in https://sof-ci.01.org/sofpr/PR5806/build332/devicetest/ are obviously unrelated to this pure checkpatch change.

@lgirdwood lgirdwood merged commit 7c4a53f into thesofproject:main May 13, 2022
@marc-hb marc-hb deleted the checkpatch-sparse branch May 13, 2022 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants