drivers/net/vsc9953: Cleanup patch
authorCodrin Ciubotariu <codrin.ciubotariu@freescale.com>
Fri, 24 Jul 2015 13:52:45 +0000 (16:52 +0300)
committerYork Sun <yorksun@freescale.com>
Mon, 21 Sep 2015 15:29:46 +0000 (08:29 -0700)
This patch groups some macros defined for registers and
replaces some magic numbers from vsc9953 with macros. Also,
"port" and "port_nr" words are replaced with "port_no",
puts each variable declaration on a line and removes
unnecessary tabs.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
Reviewed-by: York Sun <yorksun@freescale.com>
drivers/net/vsc9953.c
include/vsc9953.h

index 068ab8695bc8f3776133fdb50b2baf3114a8f9e1..524b979d525567b766e30e8cd8e0823606473e00 100644 (file)
@@ -10,6 +10,8 @@
 #include <asm/fsl_serdes.h>
 #include <fm_eth.h>
 #include <fsl_memac.h>
+#include <errno.h>
+#include <malloc.h>
 #include <vsc9953.h>
 
 static struct vsc9953_info vsc9953_l2sw = {
@@ -25,50 +27,50 @@ static struct vsc9953_info vsc9953_l2sw = {
                .port[9] = VSC9953_PORT_INFO_INITIALIZER(9),
 };
 
-void vsc9953_port_info_set_mdio(int port, struct mii_dev *bus)
+void vsc9953_port_info_set_mdio(int port_no, struct mii_dev *bus)
 {
-       if (!VSC9953_PORT_CHECK(port))
+       if (!VSC9953_PORT_CHECK(port_no))
                return;
 
-       vsc9953_l2sw.port[port].bus = bus;
+       vsc9953_l2sw.port[port_no].bus = bus;
 }
 
-void vsc9953_port_info_set_phy_address(int port, int address)
+void vsc9953_port_info_set_phy_address(int port_no, int address)
 {
-       if (!VSC9953_PORT_CHECK(port))
+       if (!VSC9953_PORT_CHECK(port_no))
                return;
 
-       vsc9953_l2sw.port[port].phyaddr = address;
+       vsc9953_l2sw.port[port_no].phyaddr = address;
 }
 
-void vsc9953_port_info_set_phy_int(int port, phy_interface_t phy_int)
+void vsc9953_port_info_set_phy_int(int port_no, phy_interface_t phy_int)
 {
-       if (!VSC9953_PORT_CHECK(port))
+       if (!VSC9953_PORT_CHECK(port_no))
                return;
 
-       vsc9953_l2sw.port[port].enet_if = phy_int;
+       vsc9953_l2sw.port[port_no].enet_if = phy_int;
 }
 
-void vsc9953_port_enable(int port)
+void vsc9953_port_enable(int port_no)
 {
-       if (!VSC9953_PORT_CHECK(port))
+       if (!VSC9953_PORT_CHECK(port_no))
                return;
 
-       vsc9953_l2sw.port[port].enabled = 1;
+       vsc9953_l2sw.port[port_no].enabled = 1;
 }
 
-void vsc9953_port_disable(int port)
+void vsc9953_port_disable(int port_no)
 {
-       if (!VSC9953_PORT_CHECK(port))
+       if (!VSC9953_PORT_CHECK(port_no))
                return;
 
-       vsc9953_l2sw.port[port].enabled = 0;
+       vsc9953_l2sw.port[port_no].enabled = 0;
 }
 
 static void vsc9953_mdio_write(struct vsc9953_mii_mng *phyregs, int port_addr,
                int regnum, int value)
 {
-       int                     timeout = 50000;
+       int timeout = 50000;
 
        out_le32(&phyregs->miimcmd, (0x1 << 31) | ((port_addr & 0x1f) << 25) |
                        ((regnum & 0x1f) << 20) | ((value & 0xffff) << 4) |
@@ -85,8 +87,8 @@ static void vsc9953_mdio_write(struct vsc9953_mii_mng *phyregs, int port_addr,
 static int vsc9953_mdio_read(struct vsc9953_mii_mng *phyregs, int port_addr,
                int regnum)
 {
-       int                     value = 0xFFFF;
-       int                     timeout = 50000;
+       int value = 0xFFFF;
+       int timeout = 50000;
 
        while ((in_le32(&phyregs->miimstatus) & MIIMIND_OPR_PEND) && --timeout)
                udelay(1);
@@ -120,8 +122,8 @@ static int vsc9953_mdio_read(struct vsc9953_mii_mng *phyregs, int port_addr,
 
 static int init_phy(struct eth_device *dev)
 {
-       struct vsc9953_port_info        *l2sw_port = dev->priv;
-       struct phy_device               *phydev = NULL;
+       struct vsc9953_port_info *l2sw_port = dev->priv;
+       struct phy_device *phydev = NULL;
 
 #ifdef CONFIG_PHYLIB
        if (!l2sw_port->bus)
@@ -148,21 +150,21 @@ static int init_phy(struct eth_device *dev)
        return 0;
 }
 
-static int vsc9953_port_init(int port)
+static int vsc9953_port_init(int port_no)
 {
-       struct eth_device               *dev;
+       struct eth_device *dev;
 
        /* Internal ports never have a PHY */
-       if (VSC9953_INTERNAL_PORT_CHECK(port))
+       if (VSC9953_INTERNAL_PORT_CHECK(port_no))
                return 0;
 
        /* alloc eth device */
        dev = (struct eth_device *)calloc(1, sizeof(struct eth_device));
        if (!dev)
-               return 1;
+               return -ENOMEM;
 
-       sprintf(dev->name, "SW@PORT%d", port);
-       dev->priv = &vsc9953_l2sw.port[port];
+       sprintf(dev->name, "SW@PORT%d", port_no);
+       dev->priv = &vsc9953_l2sw.port[port_no];
        dev->init = NULL;
        dev->halt = NULL;
        dev->send = NULL;
@@ -170,7 +172,7 @@ static int vsc9953_port_init(int port)
 
        if (init_phy(dev)) {
                free(dev);
-               return 1;
+               return -ENODEV;
        }
 
        return 0;
@@ -178,13 +180,15 @@ static int vsc9953_port_init(int port)
 
 void vsc9953_init(bd_t *bis)
 {
-       u32                             i, hdx_cfg = 0, phy_addr = 0;
-       int                             timeout;
-       struct vsc9953_system_reg       *l2sys_reg;
-       struct vsc9953_qsys_reg         *l2qsys_reg;
-       struct vsc9953_dev_gmii         *l2dev_gmii_reg;
-       struct vsc9953_analyzer         *l2ana_reg;
-       struct vsc9953_devcpu_gcb       *l2dev_gcb;
+       u32 i;
+       u32 hdx_cfg = 0;
+       u32 phy_addr = 0;
+       int timeout;
+       struct vsc9953_system_reg *l2sys_reg;
+       struct vsc9953_qsys_reg *l2qsys_reg;
+       struct vsc9953_dev_gmii *l2dev_gmii_reg;
+       struct vsc9953_analyzer *l2ana_reg;
+       struct vsc9953_devcpu_gcb *l2dev_gcb;
 
        l2dev_gmii_reg = (struct vsc9953_dev_gmii *)(VSC9953_OFFSET +
                        VSC9953_DEV_GMII_OFFSET);
@@ -312,83 +316,81 @@ void vsc9953_init(bd_t *bis)
 
 #ifdef CONFIG_VSC9953_CMD
 /* Enable/disable status of a VSC9953 port */
-static void vsc9953_port_status_set(int port_nr, u8 enabled)
+static void vsc9953_port_status_set(int port_no, u8 enabled)
 {
-       u32                     val;
-       struct vsc9953_qsys_reg *l2qsys_reg;
+       struct vsc9953_qsys_reg *l2qsys_reg;
 
        /* Administrative down */
-       if (vsc9953_l2sw.port[port_nr].enabled == 0)
+       if (!vsc9953_l2sw.port[port_no].enabled)
                return;
 
        l2qsys_reg = (struct vsc9953_qsys_reg *)(VSC9953_OFFSET +
                        VSC9953_QSYS_OFFSET);
 
-       val = in_le32(&l2qsys_reg->sys.switch_port_mode[port_nr]);
-       if (enabled == 1)
-               val |= (1 << 13);
+       if (enabled)
+               setbits_le32(&l2qsys_reg->sys.switch_port_mode[port_no],
+                            VSC9953_PORT_ENA);
        else
-               val &= ~(1 << 13);
-
-       out_le32(&l2qsys_reg->sys.switch_port_mode[port_nr], val);
+               clrbits_le32(&l2qsys_reg->sys.switch_port_mode[port_no],
+                            VSC9953_PORT_ENA);
 }
 
 /* Set all VSC9953 ports' status */
 static void vsc9953_port_all_status_set(u8 enabled)
 {
-       int             i;
+       int i;
 
        for (i = 0; i < VSC9953_MAX_PORTS; i++)
                vsc9953_port_status_set(i, enabled);
 }
 
 /* Start autonegotiation for a VSC9953 PHY */
-static void vsc9953_phy_autoneg(int port_nr)
+static void vsc9953_phy_autoneg(int port_no)
 {
-       if (!vsc9953_l2sw.port[port_nr].phydev)
+       if (!vsc9953_l2sw.port[port_no].phydev)
                return;
 
-       if (vsc9953_l2sw.port[port_nr].phydev->drv->startup(
-                       vsc9953_l2sw.port[port_nr].phydev))
-               printf("Failed to start PHY for port %d\n", port_nr);
+       if (vsc9953_l2sw.port[port_no].phydev->drv->startup(
+                       vsc9953_l2sw.port[port_no].phydev))
+               printf("Failed to start PHY for port %d\n", port_no);
 }
 
 /* Start autonegotiation for all VSC9953 PHYs */
 static void vsc9953_phy_all_autoneg(void)
 {
-       int             i;
+       int i;
 
        for (i = 0; i < VSC9953_MAX_PORTS; i++)
                vsc9953_phy_autoneg(i);
 }
 
 /* Print a VSC9953 port's configuration */
-static void vsc9953_port_config_show(int port)
+static void vsc9953_port_config_show(int port_no)
 {
-       int                     speed;
-       int                     duplex;
-       int                     link;
-       u8                      enabled;
-       u32                     val;
+       int speed;
+       int duplex;
+       int link;
+       u8 enabled;
+       u32 val;
        struct vsc9953_qsys_reg *l2qsys_reg;
 
        l2qsys_reg = (struct vsc9953_qsys_reg *)(VSC9953_OFFSET +
                        VSC9953_QSYS_OFFSET);
 
-       val = in_le32(&l2qsys_reg->sys.switch_port_mode[port]);
-       enabled = vsc9953_l2sw.port[port].enabled &
-                       ((val & 0x00002000) >> 13);
+       val = in_le32(&l2qsys_reg->sys.switch_port_mode[port_no]);
+       enabled = vsc9953_l2sw.port[port_no].enabled &&
+                 (val & VSC9953_PORT_ENA);
 
        /* internal ports (8 and 9) are fixed */
-       if (VSC9953_INTERNAL_PORT_CHECK(port)) {
+       if (VSC9953_INTERNAL_PORT_CHECK(port_no)) {
                link = 1;
                speed = SPEED_2500;
                duplex = DUPLEX_FULL;
        } else {
-               if (vsc9953_l2sw.port[port].phydev) {
-                       link = vsc9953_l2sw.port[port].phydev->link;
-                       speed = vsc9953_l2sw.port[port].phydev->speed;
-                       duplex = vsc9953_l2sw.port[port].phydev->duplex;
+               if (vsc9953_l2sw.port[port_no].phydev) {
+                       link = vsc9953_l2sw.port[port_no].phydev->link;
+                       speed = vsc9953_l2sw.port[port_no].phydev->speed;
+                       duplex = vsc9953_l2sw.port[port_no].phydev->duplex;
                } else {
                        link = -1;
                        speed = -1;
@@ -396,7 +398,7 @@ static void vsc9953_port_config_show(int port)
                }
        }
 
-       printf("%8d ", port);
+       printf("%8d ", port_no);
        printf("%8s ", enabled == 1 ? "enabled" : "disabled");
        printf("%8s ", link == 1 ? "up" : "down");
 
@@ -426,7 +428,7 @@ static void vsc9953_port_config_show(int port)
 /* Print VSC9953 ports' configuration */
 static void vsc9953_port_all_config_show(void)
 {
-       int             i;
+       int i;
 
        for (i = 0; i < VSC9953_MAX_PORTS; i++)
                vsc9953_port_config_show(i);
@@ -487,11 +489,11 @@ static int do_ethsw(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 
 U_BOOT_CMD(ethsw, 5, 0, do_ethsw,
           "vsc9953 l2 switch commands",
-          "port <port_nr> enable|disable\n"
+          "port <port_no> enable|disable\n"
           "    - enable/disable an l2 switch port\n"
-          "      port_nr=0..9; use \"all\" for all ports\n"
-          "ethsw port <port_nr> show\n"
+          "      port_no=0..9; use \"all\" for all ports\n"
+          "ethsw port <port_no> show\n"
           "    - show an l2 switch port's configuration\n"
-          "      port_nr=0..9; use \"all\" for all ports\n"
+          "      port_no=0..9; use \"all\" for all ports\n"
 );
 #endif /* CONFIG_VSC9953_CMD */
index 3f14dad313690e4137b1ee0c285f996382e4906e..6ebe141d68dde444e70bb5fb70158a750c6b7af0 100644 (file)
@@ -17,7 +17,6 @@
 #include <config.h>
 #include <miiphy.h>
 #include <asm/types.h>
-#include <malloc.h>
 
 #define VSC9953_OFFSET                 (CONFIG_SYS_CCSRBAR_DEFAULT + 0x800000)
 
 #define T1040_SWITCH_GMII_DEV_OFFSET   0x010000
 #define VSC9953_PHY_REGS_OFFST         0x0000AC
 
+/* Macros for vsc9953_chip_regs.soft_rst register */
 #define VSC9953_SOFT_SWC_RST_ENA       0x00000001
+
+/* Macros for vsc9953_sys_sys.reset_cfg register */
 #define VSC9953_CORE_ENABLE            0x80
 #define VSC9953_MEM_ENABLE             0x40
 #define VSC9953_MEM_INIT               0x20
 
-#define VSC9953_PORT_ENA               0x00003a00
+/* Macros for vsc9953_dev_gmii_mac_cfg_status.mac_ena_cfg register */
 #define VSC9953_MAC_ENA_CFG            0x00000011
+
+/* Macros for vsc9953_dev_gmii_mac_cfg_status.mac_mode_cfg register */
 #define VSC9953_MAC_MODE_CFG           0x00000011
+
+/* Macros for vsc9953_dev_gmii_mac_cfg_status.mac_ifg_cfg register */
 #define VSC9953_MAC_IFG_CFG            0x00000515
+
+/* Macros for vsc9953_dev_gmii_mac_cfg_status.mac_hdx_cfg register */
 #define VSC9953_MAC_HDX_CFG            0x00001043
+
+/* Macros for vsc9953_dev_gmii_mac_cfg_status.mac_maxlen_cfg register */
+#define VSC9953_MAC_MAX_LEN            0x000005ee
+
+/* Macros for vsc9953_dev_gmii_port_mode.clock_cfg register */
 #define VSC9953_CLOCK_CFG              0x00000001
 #define VSC9953_CLOCK_CFG_1000M                0x00000001
+
+/* Macros for vsc9953_sys_sys.front_port_mode register */
+#define VSC9953_FRONT_PORT_MODE        0x00000000
+
+/* Macros for vsc9953_ana_pfc.pfc_cfg register */
 #define VSC9953_PFC_FC                 0x00000001
 #define VSC9953_PFC_FC_QSGMII          0x00000000
+
+/* Macros for vsc9953_sys_pause_cfg.mac_fc_cfg register */
 #define VSC9953_MAC_FC_CFG             0x04700000
 #define VSC9953_MAC_FC_CFG_QSGMII      0x00700000
+
+/* Macros for vsc9953_sys_pause_cfg.pause_cfg register */
 #define VSC9953_PAUSE_CFG              0x001ffffe
+
+/* Macros for vsc9953_sys_pause_cfgtot_tail_drop_lvl register */
 #define VSC9953_TOT_TAIL_DROP_LVL      0x000003ff
-#define VSC9953_FRONT_PORT_MODE                0x00000000
-#define VSC9953_MAC_MAX_LEN            0x000005ee
 
+/* Macros for vsc9953_vcap_core_cfg.vcap_mv_cfg register */
 #define VSC9953_VCAP_MV_CFG            0x0000ffff
 #define VSC9953_VCAP_UPDATE_CTRL       0x01000004
+
+/* Macros for vsc9953_qsys_sys.switch_port_mode register */
+#define VSC9953_PORT_ENA               0x00003a00
+
 #define VSC9953_MAX_PORTS              10
 #define VSC9953_PORT_CHECK(port)       \
        (((port) < 0 || (port) >= VSC9953_MAX_PORTS) ? 0 : 1)
@@ -74,9 +101,9 @@ struct vsc9953_mdio_info {
        char    *name;
 };
 
-/* VSC9953 ANA structure for T1040 U-boot*/
+/* VSC9953 ANA structure */
 
-struct vsc9953_ana_port {
+struct vsc9953_ana_port {
        u32     vlan_cfg;
        u32     drop_cfg;
        u32     qos_cfg;
@@ -138,7 +165,7 @@ struct vsc9953_ana_pgid {
        u32     port_grp_id[91];
 };
 
-struct vsc9953_ana_pfc {
+struct vsc9953_ana_pfc {
        u32     pfc_cfg;
        u32     reserved1[15];
 };
@@ -149,7 +176,7 @@ struct vsc9953_ana_pol_misc {
        u32     pol_hyst;
 };
 
-struct vsc9953_ana_common {
+struct vsc9953_ana_common {
        u32     aggr_cfg;
        u32     cpuq_cfg;
        u32     cpuq_8021_cfg;
@@ -176,18 +203,18 @@ struct vsc9953_analyzer {
        u32     reserved5[196];
        struct vsc9953_ana_common       common;
 };
-/* END VSC9953 ANA structure for T1040 U-boot*/
+/* END VSC9953 ANA structure t*/
 
-/* VSC9953 DEV_GMII structure for T1040 U-boot*/
+/* VSC9953 DEV_GMII structure */
 
-struct vsc9953_dev_gmii_port_mode {
+struct vsc9953_dev_gmii_port_mode {
        u32     clock_cfg;
        u32     port_misc;
        u32     reserved1;
        u32     eee_cfg;
 };
 
-struct vsc9953_dev_gmii_mac_cfg_status {
+struct vsc9953_dev_gmii_mac_cfg_status {
        u32     mac_ena_cfg;
        u32     mac_mode_cfg;
        u32     mac_maxlen_cfg;
@@ -205,11 +232,11 @@ struct vsc9953_dev_gmii {
        struct vsc9953_dev_gmii_mac_cfg_status  mac_cfg_status;
 };
 
-/* END VSC9953 DEV_GMII structure for T1040 U-boot*/
+/* END VSC9953 DEV_GMII structure */
 
-/* VSC9953 QSYS structure for T1040 U-boot*/
+/* VSC9953 QSYS structure */
 
-struct vsc9953_qsys_hsch {
+struct vsc9953_qsys_hsch {
        u32     cir_cfg;
        u32     reserved1;
        u32     se_cfg;
@@ -218,7 +245,7 @@ struct      vsc9953_qsys_hsch {
        u32     reserved2[20];
 };
 
-struct vsc9953_qsys_sys {
+struct vsc9953_qsys_sys {
        u32     port_mode[12];
        u32     switch_port_mode[11];
        u32     stat_cnt_cfg;
@@ -232,32 +259,32 @@ struct    vsc9953_qsys_sys {
        u32     reserved1[23];
 };
 
-struct vsc9953_qsys_qos_cfg {
+struct vsc9953_qsys_qos_cfg {
        u32     red_profile[16];
        u32     res_qos_mode;
 };
 
-struct vsc9953_qsys_drop_cfg {
+struct vsc9953_qsys_drop_cfg {
        u32     egr_drop_mode;
 };
 
-struct vsc9953_qsys_mmgt {
+struct vsc9953_qsys_mmgt {
        u32     eq_cntrl;
        u32     reserved1;
 };
 
-struct vsc9953_qsys_hsch_misc {
+struct vsc9953_qsys_hsch_misc {
        u32     hsch_misc_cfg;
        u32     reserved1[546];
 };
 
-struct vsc9953_qsys_res_ctrl {
+struct vsc9953_qsys_res_ctrl {
        u32     res_cfg;
        u32     res_stat;
 
 };
 
-struct vsc9953_qsys_reg {
+struct vsc9953_qsys_reg {
        struct vsc9953_qsys_hsch        hsch[108];
        struct vsc9953_qsys_sys sys;
        struct vsc9953_qsys_qos_cfg     qos_cfg;
@@ -267,18 +294,18 @@ struct    vsc9953_qsys_reg {
        struct vsc9953_qsys_res_ctrl    res_ctrl[1024];
 };
 
-/* END VSC9953 QSYS structure for T1040 U-boot*/
+/* END VSC9953 QSYS structure */
 
-/* VSC9953 SYS structure for T1040 U-boot*/
+/* VSC9953 SYS structure */
 
-struct vsc9953_sys_stat {
+struct vsc9953_sys_stat {
        u32     rx_cntrs[64];
        u32     tx_cntrs[64];
        u32     drop_cntrs[64];
        u32     reserved1[6];
 };
 
-struct vsc9953_sys_sys {
+struct vsc9953_sys_sys {
        u32     reset_cfg;
        u32     reserved1;
        u32     vlan_etype_cfg;
@@ -289,7 +316,7 @@ struct      vsc9953_sys_sys {
        u32     reserved2[50];
 };
 
-struct vsc9953_sys_pause_cfg {
+struct vsc9953_sys_pause_cfg {
        u32     pause_cfg[11];
        u32     pause_tot_cfg;
        u32     tail_drop_level[11];
@@ -297,29 +324,29 @@ struct    vsc9953_sys_pause_cfg {
        u32     mac_fc_cfg[10];
 };
 
-struct vsc9953_sys_mmgt {
+struct vsc9953_sys_mmgt {
        u16     free_cnt;
 };
 
-struct vsc9953_system_reg {
+struct vsc9953_system_reg {
        struct vsc9953_sys_stat stat;
        struct vsc9953_sys_sys  sys;
        struct vsc9953_sys_pause_cfg    pause_cfg;
        struct vsc9953_sys_mmgt mmgt;
 };
 
-/* END VSC9953 SYS structure for T1040 U-boot*/
+/* END VSC9953 SYS structure */
 
 
-/* VSC9953 DEVCPU_GCB structure for T1040 U-boot*/
+/* VSC9953 DEVCPU_GCB structure */
 
-struct vsc9953_chip_regs {
+struct vsc9953_chip_regs {
        u32     chipd_id;
        u32     gpr;
        u32     soft_rst;
 };
 
-struct vsc9953_gpio {
+struct vsc9953_gpio {
        u32     gpio_out_set[10];
        u32     gpio_out_clr[10];
        u32     gpio_out[10];
@@ -338,31 +365,31 @@ struct vsc9953_mii_mng {
        u32     miiscan_lst_rslts_valid;
 };
 
-struct vsc9953_mii_read_scan {
+struct vsc9953_mii_read_scan {
        u32     mii_scan_results_sticky[2];
 };
 
-struct vsc9953_devcpu_gcb {
+struct vsc9953_devcpu_gcb {
        struct vsc9953_chip_regs        chip_regs;
        struct vsc9953_gpio             gpio;
        struct vsc9953_mii_mng  mii_mng[2];
        struct vsc9953_mii_read_scan    mii_read_scan;
 };
 
-/* END VSC9953 DEVCPU_GCB structure for T1040 U-boot*/
+/* END VSC9953 DEVCPU_GCB structure */
 
-/* VSC9953 IS* structure for T1040 U-boot*/
+/* VSC9953 IS* structure */
 
-struct vsc9953_vcap_core_cfg   {
+struct vsc9953_vcap_core_cfg {
        u32     vcap_update_ctrl;
        u32     vcap_mv_cfg;
 };
 
-struct vsc9953_vcap {
-struct vsc9953_vcap_core_cfg   vcap_core_cfg;
+struct vsc9953_vcap {
+       struct vsc9953_vcap_core_cfg    vcap_core_cfg;
 };
 
-/* END VSC9953 IS* structure for T1040 U-boot*/
+/* END VSC9953 IS* structure */
 
 #define VSC9953_PORT_INFO_INITIALIZER(idx) \
 {                                                                      \
@@ -388,15 +415,15 @@ struct vsc9953_port_info {
 
 /* Structure to describe a VSC9953 switch */
 struct vsc9953_info {
-       struct  vsc9953_port_info       port[VSC9953_MAX_PORTS];
+       struct vsc9953_port_info        port[VSC9953_MAX_PORTS];
 };
 
 void vsc9953_init(bd_t *bis);
 
-void vsc9953_port_info_set_mdio(int port, struct mii_dev *bus);
-void vsc9953_port_info_set_phy_address(int port, int address);
-void vsc9953_port_enable(int port);
-void vsc9953_port_disable(int port);
-void vsc9953_port_info_set_phy_int(int port, phy_interface_t phy_int);
+void vsc9953_port_info_set_mdio(int port_no, struct mii_dev *bus);
+void vsc9953_port_info_set_phy_address(int port_no, int address);
+void vsc9953_port_enable(int port_no);
+void vsc9953_port_disable(int port_no);
+void vsc9953_port_info_set_phy_int(int port_no, phy_interface_t phy_int);
 
 #endif /* _VSC9953_H_ */