Audio: MFCC: Fix sparse and 64 bit build issues

The comp_err() trace in mfcc_init() causes an error: sof/src/audio/mfcc/mfcc.c: In function 'mfcc_init': sof/src/include/sof/audio/component.h:160:20: error: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'size_t' {aka 'long unsigned int'} [-Werror=format=] The sparse error is sof/src/include/sof/audio/mfcc/mfcc_comp.h:128:27: error: dubious one-bit signed bitfield Signed-off-by: Seppo Ingalsuo <[email protected]>
thesofproject · Aug 22, 2024 · 33709f1 · 33709f1
1 parent 1009ba7
commit 33709f1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/audio/mfcc/mfcc.c b/src/audio/mfcc/mfcc.c
@@ -84,7 +84,7 @@ static int mfcc_init(struct processing_module *mod)
 
 	/* Check first that configuration blob size is sane */
 	if (bs > SOF_MFCC_CONFIG_MAX_SIZE) {
-		comp_err(dev, "mfcc_init() error: configuration blob size %u exceeds %d",
+		comp_err(dev, "mfcc_init() error: configuration blob size %lu exceeds %d",
 			 bs, SOF_MFCC_CONFIG_MAX_SIZE);
 		return -EINVAL;
 	}

diff --git a/src/include/sof/audio/mfcc/mfcc_comp.h b/src/include/sof/audio/mfcc/mfcc_comp.h
@@ -125,8 +125,8 @@ struct mfcc_state {
 	int low_freq;
 	int high_freq;
 	int sample_rate;
-	int waiting_fill:1; /**< booleans */
-	int prev_samples_valid:1;
+	bool waiting_fill:1; /**< booleans */
+	bool prev_samples_valid:1;
 	size_t sample_buffers_size; /**< bytes */
 };