• Arnd Bergmann's avatar
    net: bcm63xx: avoid referencing uninitialized variable · 4b75ca5a
    Arnd Bergmann authored
    gcc found a reference to an uninitialized variable in the error handling
    of bcm_enet_open, introduced by a recent cleanup:
    
    drivers/net/ethernet/broadcom/bcm63xx_enet.c: In function 'bcm_enet_open'
    drivers/net/ethernet/broadcom/bcm63xx_enet.c:1129:2: warning: 'phydev' may be used uninitialized in this function [-Wmaybe-uninitialized]
    
    This makes the use of that variable conditional, so we only reference it
    here after it has been used before. Unlike my normal patches, I have not
    build-tested this one, as I don't currently have mips test in my
    randconfig setup.
    
    Fixes: 625eb866 ("net: ethernet: broadcom: bcm63xx: use phydev from struct net_device")
    Cc: Philippe Reynes <tremyfr@gmail.com>
    Reported-by: default avatarkbuild test robot <fengguang.wu@intel.com>
    Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    4b75ca5a
bcm63xx_enet.c 72.2 KB