mfiutil(8): Fix building with -fno-common.
authorSascha Wildner <saw@online.de>
Sun, 28 Jun 2020 09:22:42 +0000 (11:22 +0200)
committerSascha Wildner <saw@online.de>
Sun, 28 Jun 2020 09:22:42 +0000 (11:22 +0200)
-fno-common will be default in future compilers (GCC 10, for example).

This removes some redundant code.

Taken-from: FreeBSD

usr.sbin/mfiutil/mfi_flash.c
usr.sbin/mfiutil/mfi_show.c
usr.sbin/mfiutil/mfiutil.c
usr.sbin/mfiutil/mfiutil.h

index 1940b10..8b7e272 100644 (file)
 
 #define        FLASH_BUF_SIZE  (64 * 1024)
 
 
 #define        FLASH_BUF_SIZE  (64 * 1024)
 
-int fw_name_width, fw_version_width, fw_date_width, fw_time_width;
-
-static void
-scan_firmware(struct mfi_info_component *comp)
-{
-       int len;
-
-       len = strlen(comp->name);
-       if (fw_name_width < len)
-               fw_name_width = len;
-       len = strlen(comp->version);
-       if (fw_version_width < len)
-               fw_version_width = len;
-       len = strlen(comp->build_date);
-       if (fw_date_width < len)
-               fw_date_width = len;
-       len = strlen(comp->build_time);
-       if (fw_time_width < len)
-               fw_time_width = len;
-}
-
-static void
-display_firmware(struct mfi_info_component *comp)
-{
-
-       printf("%-*s  %-*s  %-*s  %-*s\n", fw_name_width, comp->name,
-           fw_version_width, comp->version, fw_date_width, comp->build_date,
-           fw_time_width, comp->build_time);
-}
-
 static int
 display_pending_firmware(int fd)
 {
 static int
 display_pending_firmware(int fd)
 {
@@ -96,9 +66,9 @@ display_pending_firmware(int fd)
                info.pending_image_component_count = 8;
        for (i = 0; i < info.pending_image_component_count; i++)
                scan_firmware(&info.pending_image_component[i]);
                info.pending_image_component_count = 8;
        for (i = 0; i < info.pending_image_component_count; i++)
                scan_firmware(&info.pending_image_component[i]);
-       display_firmware(&header);
+       display_firmware(&header, "");
        for (i = 0; i < info.pending_image_component_count; i++)
        for (i = 0; i < info.pending_image_component_count; i++)
-               display_firmware(&info.pending_image_component[i]);
+               display_firmware(&info.pending_image_component[i], "");
 
        return (0);
 }
 
        return (0);
 }
index 0c1b481..5c57b7c 100644 (file)
@@ -556,36 +556,6 @@ error:
 }
 MFI_COMMAND(show, drives, show_drives);
 
 }
 MFI_COMMAND(show, drives, show_drives);
 
-int fw_name_width, fw_version_width, fw_date_width, fw_time_width;
-
-static void
-scan_firmware(struct mfi_info_component *comp)
-{
-       int len;
-
-       len = strlen(comp->name);
-       if (fw_name_width < len)
-               fw_name_width = len;
-       len = strlen(comp->version);
-       if (fw_version_width < len)
-               fw_version_width = len;
-       len = strlen(comp->build_date);
-       if (fw_date_width < len)
-               fw_date_width = len;
-       len = strlen(comp->build_time);
-       if (fw_time_width < len)
-               fw_time_width = len;
-}
-
-static void
-display_firmware(struct mfi_info_component *comp, const char *tag)
-{
-
-       printf("%-*s  %-*s  %-*s  %-*s  %s\n", fw_name_width, comp->name,
-           fw_version_width, comp->version, fw_date_width, comp->build_date,
-           fw_time_width, comp->build_time, tag);
-}
-
 static int
 show_firmware(int ac, __unused char **av)
 {
 static int
 show_firmware(int ac, __unused char **av)
 {
index 55ed3e1..91d024b 100644 (file)
@@ -44,8 +44,8 @@ MFI_TABLE(top, stop);
 MFI_TABLE(top, abort);
 
 int mfi_unit;
 MFI_TABLE(top, abort);
 
 int mfi_unit;
-
 u_int mfi_opts;
 u_int mfi_opts;
+static int fw_name_width, fw_version_width, fw_date_width, fw_time_width;
 
 static void
 usage(void)
 
 static void
 usage(void)
@@ -150,3 +150,31 @@ main(int ac, char **av)
        warnx("Unknown command %s.", av[0]);
        return (1);
 }
        warnx("Unknown command %s.", av[0]);
        return (1);
 }
+
+void
+scan_firmware(struct mfi_info_component *comp)
+{
+       int len;
+
+       len = strlen(comp->name);
+       if (fw_name_width < len)
+               fw_name_width = len;
+       len = strlen(comp->version);
+       if (fw_version_width < len)
+               fw_version_width = len;
+       len = strlen(comp->build_date);
+       if (fw_date_width < len)
+               fw_date_width = len;
+       len = strlen(comp->build_time);
+       if (fw_time_width < len)
+               fw_time_width = len;
+}
+
+void
+display_firmware(struct mfi_info_component *comp, const char *tag)
+{
+
+       printf("%-*s  %-*s  %-*s  %-*s  %s\n", fw_name_width, comp->name,
+           fw_version_width, comp->version, fw_date_width, comp->build_date,
+           fw_time_width, comp->build_time, tag);
+}
index fc84935..606f36f 100644 (file)
@@ -114,4 +114,7 @@ void        print_ld(struct mfi_ld_info *info, int state_len);
 void   print_pd(struct mfi_pd_info *info, int state_len);
 void   dump_config(int fd, struct mfi_config_data *config, const char* msg_prefix);
 
 void   print_pd(struct mfi_pd_info *info, int state_len);
 void   dump_config(int fd, struct mfi_config_data *config, const char* msg_prefix);
 
+void   scan_firmware(struct mfi_info_component *comp);
+void   display_firmware(struct mfi_info_component *comp, const char *tag);
+
 #endif /* !__MFIUTIL_H__ */
 #endif /* !__MFIUTIL_H__ */