Remove redeclaration found by gcc build

[lwhsu]

Summary

Remove redeclaration found by gcc build

i40e_blink_phy_link_led is declared twice in this file.

Test Plan

Build with gcc and clang.

[lwhsu]

ping?

[jacob.e.keller_intel.com]

This looks correct to me. One of the folks actively working on the Intel networking drivers should take a look to make sure they're aware of it.

sys/dev/ixl/i40e_prototype.h

609–610 ↗

(On Diff #75408)

Here's the other declaration.

[krzysztof.galazka_intel.com]

@lwhsu: Could you, please, remove the declaration in line 610 instead? The first occurrence is in a group of other LED functions.

[lwhsu]

In D25954#577171, @krzysztof.galazka_intel.com wrote:

@lwhsu: Could you, please, remove the declaration in line 610 instead? The first occurrence is in a group of other LED functions.

Of course, I was also thinking about that but ended with choosing to remove the first one because of their locations of the definitions in i40e_common.c.

head/sys/dev/ixl/i40e_prototype.h

struct devsw bioshd = {
         .dv_type = DEVT_DISK,
         .dv_init = bd_init,
         .dv_strategy = bd_strategy,
         .dv_open = bd_open,
         .dv_close = bd_close,
         .dv_ioctl = bd_ioctl,
         .dv_print = bd_print,
         .dv_cleanup = nullsys,
+        .dv_fmtdev = disk_fmtdev,
 };
 
 static bdinfo_list_t *
 bd_get_bdinfo_list(struct devsw *dev)
 {
         if (dev->dv_type == DEVT_DISK)
                 return (&hdinfo);
         if (dev->dv_type == DEVT_CD)