diff options
author | Solomon Peachy <pizza@shaftnet.org> | 2024-06-27 20:27:58 -0400 |
---|---|---|
committer | Solomon Peachy <pizza@shaftnet.org> | 2024-06-27 20:42:32 -0400 |
commit | f3de4729ce7bd5c36478c3c30784f4f1dc6a80ed (patch) | |
tree | 8a767a3b663345a1f4b6cab07e05f6700f977823 | |
parent | dca5fb9514cb88abfcdd4c0237f5742f07b68ba8 (diff) | |
download | rockbox-f3de4729ce7bd5c36478c3c30784f4f1dc6a80ed.tar.gz rockbox-f3de4729ce7bd5c36478c3c30784f4f1dc6a80ed.zip |
Fix an issue building the fft plugin on greyscale targets on some simulator toolchains
The fft plugin uses pluginlib osd stuff _and_ greylib together,
which results in two conflicting declarations of the _grey_info struct.
Normally we treat the one in pluginlib as __weak but we disable that on
windows simulator builds, because the osd code ends up requiring it.
due to not using-function-sections and -fdata-sections on simulator builds.
Simply disabling all of that crap doesn't seem to help.
So instead, for simulator builds, mark the fft's struct as extern, only
using the one in pluginlib.
Change-Id: I1adf9acc265a60f91b8ac0fcad9d792e3240a9d9
-rw-r--r-- | apps/plugins/fft/fft.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/apps/plugins/fft/fft.c b/apps/plugins/fft/fft.c index 4d36302ddf..99d9e033fc 100644 --- a/apps/plugins/fft/fft.c +++ b/apps/plugins/fft/fft.c | |||
@@ -33,8 +33,13 @@ | |||
33 | #include "lib/osd.h" | 33 | #include "lib/osd.h" |
34 | 34 | ||
35 | #ifndef HAVE_LCD_COLOR | 35 | #ifndef HAVE_LCD_COLOR |
36 | #if defined(SIMULATOR) | ||
37 | // Use the one in pluginlib/osd | ||
38 | extern GREY_INFO_STRUCT | ||
39 | #else | ||
36 | GREY_INFO_STRUCT | 40 | GREY_INFO_STRUCT |
37 | #endif | 41 | #endif |
42 | #endif | ||
38 | 43 | ||
39 | #include "lib/pluginlib_actions.h" | 44 | #include "lib/pluginlib_actions.h" |
40 | 45 | ||