block: partitions: only define function mac_fix_string for CONFIG_PPC_PMAC

ColinIanKing · axboe · commit 5205a4aa8fc9 · 2024-03-09T07:31:42.000-07:00
The helper function mac_fix_string is only required with CONFIG_PPC_PMAC, add #if CONFIG_PPC_PMAC and #endif around the function. Cleans up clang scan build warning: block/partitions/mac.c:23:20: warning: unused function 'mac_fix_string' [-Wunused-function] Signed-off-by: Colin Ian King <colin.i.king@gmail.com> Link: https://lore.kernel.org/r/20240308133921.2058227-1-colin.i.king@gmail.com Signed-off-by: Jens Axboe <axboe@kernel.dk>
diff --git a/block/partitions/mac.c b/block/partitions/mac.c
@@ -20,13 +20,15 @@ extern void note_bootable_part(dev_t dev, int part, int goodness);
  * Code to understand MacOS partition tables.
  */
 
+#ifdef CONFIG_PPC_PMAC
 static inline void mac_fix_string(char *stg, int len)
 {
 	int i;
 
 	for (i = len - 1; i >= 0 && stg[i] == ' '; i--)
 		stg[i] = 0;
 }
+#endif
 
 int mac_partition(struct parsed_partitions *state)
 {

Original file line number	Diff line number	Diff line change
`@@ -20,13 +20,15 @@ extern void note_bootable_part(dev_t dev, int part, int goodness);`
`20`	`20`	`* Code to understand MacOS partition tables.`
`21`	`21`	`*/`
`22`	`22`
	`23`	`+#ifdef CONFIG_PPC_PMAC`
`23`	`24`	`static inline void mac_fix_string(char *stg, int len)`
`24`	`25`	`{`
`25`	`26`	`int i;`
`26`	`27`
`27`	`28`	`for (i = len - 1; i >= 0 && stg[i] == ' '; i--)`
`28`	`29`	`stg[i] = 0;`
`29`	`30`	`}`
	`31`	`+#endif`
`30`	`32`
`31`	`33`	`int mac_partition(struct parsed_partitions *state)`
`32`	`34`	`{`