diff options
author | Pan Xiuli <xiuli.pan@linux.intel.com> | 2020-04-15 15:28:06 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2020-04-16 00:23:10 +0100 |
commit | a9a9cbf081414de0261279e3b11ada2f0a7f7e83 (patch) | |
tree | 9fc77d1f8893b462632f9b8c65668da29fcfa646 /include/sound/sof | |
parent | 60829341aa602b74f5f5b9d903e0b809557a54a4 (diff) | |
download | linux-a9a9cbf081414de0261279e3b11ada2f0a7f7e83.tar.gz linux-a9a9cbf081414de0261279e3b11ada2f0a7f7e83.tar.bz2 linux-a9a9cbf081414de0261279e3b11ada2f0a7f7e83.zip |
ASoC: SOF: make sof_ipc_cc_version to fixed length
Align struct sof_ipc_cc_version to firmware definition in SOF ABI 3.15.0.
The struct definition was changed due to errors in FW build.
The Cadence XCC compiler produces incorrect linkage section sizes, when a
variable length array is used in the compiler version struct. The firmware
definition was changed to a fixed 32 byte compiler description string.
This length covers all released firmware binaries and thus only a minor
ABI change is needed.
As the same structure is used in IPC messages between driver and firmware,
the kernel needs to be aligned to firmware change.
Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Link: https://lore.kernel.org/r/20200415202816.934-15-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'include/sound/sof')
-rw-r--r-- | include/sound/sof/info.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/sound/sof/info.h b/include/sound/sof/info.h index 2ef98b2fee1f..d5eff3179a39 100644 --- a/include/sound/sof/info.h +++ b/include/sound/sof/info.h @@ -113,7 +113,7 @@ struct sof_ipc_cc_version { uint8_t name[16]; /* null terminated compiler name */ uint8_t optim[4]; /* null terminated compiler -O flag value */ - uint8_t desc[]; /* null terminated compiler description */ + uint8_t desc[32]; /* null terminated compiler description */ } __packed; /* extended data: Probe setup */ |