net: stmmac: make chain_mode a boolean

priv->chain_mode is only tested for non-zero, so it can be a boolean.
Change its type to boolean, and add a comment describing this member.

Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Link: https://patch.msgid.link/E1vvuXi-0000000AvnR-3btC@rmk-PC.armlinux.org.uk
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac.h b/drivers/net/ethernet/stmicro/stmmac/stmmac.h
index 57ded8e5177fc4988e9bef29fd8d1b5d9a407f14..e98cb9f3a44c6f908de6692e8a62e6108d36c640 100644 (file)
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac.h
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac.h
@@ -307,8 +307,9 @@ struct stmmac_priv {
         *  when set, struct dma_extended_desc
         */
        bool extend_desc;
+       /* chain_mode: requested descriptor mode */
+       bool chain_mode;
        unsigned int mode;
-       unsigned int chain_mode;
        struct kernel_hwtstamp_config tstamp_config;
        struct ptp_clock *ptp_clock;
        struct ptp_clock_info ptp_clock_ops;

diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
index 4e788f54bbbc845f2dd872d64a0c8b390738e8d6..1ab958769b96b2db2552100dceb6fed7fe4384c4 100644 (file)
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
@@ -7339,7 +7339,7 @@ static int stmmac_hw_init(struct stmmac_priv *priv)
        /* dwmac-sun8i only work in chain mode */
        if (priv->plat->flags & STMMAC_FLAG_HAS_SUN8I)
                chain_mode = 1;
-       priv->chain_mode = chain_mode;
+       priv->chain_mode = !!chain_mode;
 
        /* Initialize HW Interface */
        ret = stmmac_hwif_init(priv);