[PATCH] ASoC: topology: KUnit: Convert from cpu to data format

Amadeusz Sławiński amadeuszx.slawinski at linux.intel.com
Tue Feb 2 17:31:23 CET 2021


When creating topology templates and overriding data in specific test
cases it should be done with cpu_to_le32 macro, so we operate on correct
data on all architectures, as topology parser use le32_to_cpu to parse
data from structures.

Reported-by: kernel test robot <lkp at intel.com>
Signed-off-by: Amadeusz Sławiński <amadeuszx.slawinski at linux.intel.com>
---
 sound/soc/soc-topology-test.c | 66 +++++++++++++++++------------------
 1 file changed, 33 insertions(+), 33 deletions(-)

diff --git a/sound/soc/soc-topology-test.c b/sound/soc/soc-topology-test.c
index 36e2a3486dbf..ae3968161509 100644
--- a/sound/soc/soc-topology-test.c
+++ b/sound/soc/soc-topology-test.c
@@ -121,19 +121,19 @@ struct tplg_tmpl_001 {
 
 static struct tplg_tmpl_001 tplg_tmpl_empty = {
 	.header = {
-		.magic = SND_SOC_TPLG_MAGIC,
-		.abi = 5,
+		.magic = cpu_to_le32(SND_SOC_TPLG_MAGIC),
+		.abi = cpu_to_le32(5),
 		.version = 0,
-		.type = SND_SOC_TPLG_TYPE_MANIFEST,
-		.size = sizeof(struct snd_soc_tplg_hdr),
+		.type = cpu_to_le32(SND_SOC_TPLG_TYPE_MANIFEST),
+		.size = cpu_to_le32(sizeof(struct snd_soc_tplg_hdr)),
 		.vendor_type = 0,
-		.payload_size = sizeof(struct snd_soc_tplg_manifest),
+		.payload_size = cpu_to_le32(sizeof(struct snd_soc_tplg_manifest)),
 		.index = 0,
-		.count = 1,
+		.count = cpu_to_le32(1),
 	},
 
 	.manifest = {
-		.size = sizeof(struct snd_soc_tplg_manifest),
+		.size = cpu_to_le32(sizeof(struct snd_soc_tplg_manifest)),
 		/* rest of fields is 0 */
 	},
 };
@@ -149,60 +149,60 @@ struct tplg_tmpl_002 {
 
 static struct tplg_tmpl_002 tplg_tmpl_with_pcm = {
 	.header = {
-		.magic = SND_SOC_TPLG_MAGIC,
-		.abi = 5,
+		.magic = cpu_to_le32(SND_SOC_TPLG_MAGIC),
+		.abi = cpu_to_le32(5),
 		.version = 0,
-		.type = SND_SOC_TPLG_TYPE_MANIFEST,
-		.size = sizeof(struct snd_soc_tplg_hdr),
+		.type = cpu_to_le32(SND_SOC_TPLG_TYPE_MANIFEST),
+		.size = cpu_to_le32(sizeof(struct snd_soc_tplg_hdr)),
 		.vendor_type = 0,
-		.payload_size = sizeof(struct snd_soc_tplg_manifest),
+		.payload_size = cpu_to_le32(sizeof(struct snd_soc_tplg_manifest)),
 		.index = 0,
-		.count = 1,
+		.count = cpu_to_le32(1),
 	},
 	.manifest = {
-		.size = sizeof(struct snd_soc_tplg_manifest),
-		.pcm_elems = 1,
+		.size = cpu_to_le32(sizeof(struct snd_soc_tplg_manifest)),
+		.pcm_elems = cpu_to_le32(1),
 		/* rest of fields is 0 */
 	},
 	.pcm_header = {
-		.magic = SND_SOC_TPLG_MAGIC,
-		.abi = 5,
+		.magic = cpu_to_le32(SND_SOC_TPLG_MAGIC),
+		.abi = cpu_to_le32(5),
 		.version = 0,
-		.type = SND_SOC_TPLG_TYPE_PCM,
-		.size = sizeof(struct snd_soc_tplg_hdr),
+		.type = cpu_to_le32(SND_SOC_TPLG_TYPE_PCM),
+		.size = cpu_to_le32(sizeof(struct snd_soc_tplg_hdr)),
 		.vendor_type = 0,
-		.payload_size = sizeof(struct snd_soc_tplg_pcm),
+		.payload_size = cpu_to_le32(sizeof(struct snd_soc_tplg_pcm)),
 		.index = 0,
-		.count = 1,
+		.count = cpu_to_le32(1),
 	},
 	.pcm = {
-		.size = sizeof(struct snd_soc_tplg_pcm),
+		.size = cpu_to_le32(sizeof(struct snd_soc_tplg_pcm)),
 		.pcm_name = "KUNIT Audio",
 		.dai_name = "kunit-audio-dai",
 		.pcm_id = 0,
 		.dai_id = 0,
-		.playback = 1,
-		.capture = 1,
+		.playback = cpu_to_le32(1),
+		.capture = cpu_to_le32(1),
 		.compress = 0,
 		.stream = {
 			[0] = {
-				.channels = 2,
+				.channels = cpu_to_le32(2),
 			},
 			[1] = {
-				.channels = 2,
+				.channels = cpu_to_le32(2),
 			},
 		},
 		.num_streams = 0,
 		.caps = {
 			[0] = {
 				.name = "kunit-audio-playback",
-				.channels_min = 2,
-				.channels_max = 2,
+				.channels_min = cpu_to_le32(2),
+				.channels_max = cpu_to_le32(2),
 			},
 			[1] = {
 				.name = "kunit-audio-capture",
-				.channels_min = 2,
-				.channels_max = 2,
+				.channels_min = cpu_to_le32(2),
+				.channels_max = cpu_to_le32(2),
 			},
 		},
 		.flag_mask = 0,
@@ -460,7 +460,7 @@ static void snd_soc_tplg_test_load_empty_tplg_bad_magic(struct kunit *test)
 	 * override abi
 	 * any value != magic number is wrong
 	 */
-	data->header.magic = SND_SOC_TPLG_MAGIC + 1;
+	data->header.magic = cpu_to_le32(SND_SOC_TPLG_MAGIC + 1);
 
 	kunit_comp->fw.data = (u8 *)data;
 	kunit_comp->fw.size = size;
@@ -516,7 +516,7 @@ static void snd_soc_tplg_test_load_empty_tplg_bad_abi(struct kunit *test)
 	 * override abi
 	 * any value != accepted range is wrong
 	 */
-	data->header.abi = SND_SOC_TPLG_ABI_VERSION + 1;
+	data->header.abi = cpu_to_le32(SND_SOC_TPLG_ABI_VERSION + 1);
 
 	kunit_comp->fw.data = (u8 *)data;
 	kunit_comp->fw.size = size;
@@ -572,7 +572,7 @@ static void snd_soc_tplg_test_load_empty_tplg_bad_size(struct kunit *test)
 	 * override size
 	 * any value != struct size is wrong
 	 */
-	data->header.size = sizeof(struct snd_soc_tplg_hdr) + 1;
+	data->header.size = cpu_to_le32(sizeof(struct snd_soc_tplg_hdr) + 1);
 
 	kunit_comp->fw.data = (u8 *)data;
 	kunit_comp->fw.size = size;
-- 
2.25.1



More information about the Alsa-devel mailing list