OSDN Git Service

lib8390: Replace panic() call with BUILD_BUG_ON
authorArmin Wolf <W_Armin@gmx.de>
Tue, 29 Sep 2020 17:13:26 +0000 (19:13 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 29 Sep 2020 21:04:09 +0000 (14:04 -0700)
Replace panic() call in lib8390.c with BUILD_BUG_ON()
since checking the size of struct e8390_pkt_hdr should
happen at compile-time.

Signed-off-by: Armin Wolf <W_Armin@gmx.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/8390/lib8390.c

index 1f48d7f..deba94d 100644 (file)
@@ -50,6 +50,7 @@
 
   */
 
+#include <linux/build_bug.h>
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/jiffies.h>
@@ -1018,8 +1019,7 @@ static void __NS8390_init(struct net_device *dev, int startp)
            ? (0x48 | ENDCFG_WTS | (ei_local->bigendian ? ENDCFG_BOS : 0))
            : 0x48;
 
-       if (sizeof(struct e8390_pkt_hdr) != 4)
-               panic("8390.c: header struct mispacked\n");
+       BUILD_BUG_ON(sizeof(struct e8390_pkt_hdr) != 4);
        /* Follow National Semi's recommendations for initing the DP83902. */
        ei_outb_p(E8390_NODMA+E8390_PAGE0+E8390_STOP, e8390_base+E8390_CMD); /* 0x21 */
        ei_outb_p(endcfg, e8390_base + EN0_DCFG);       /* 0x48 or 0x49 */