From: Felix Fietkau Date: Tue, 10 Jun 2014 09:05:28 +0000 (+0000) Subject: atheros: various comments fixes X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=cb50efa835322c34e27056d2afa23db850f9e3c8;p=openwrt%2Fstaging%2Flinusw.git atheros: various comments fixes Fix comment issues detected by checkpatch. Convert C99 // comments to ordinary /* ... */ comments. Correct block comments style or convert them to single line comments. Signed-off-by: Sergey Ryazanov SVN-Revision: 41081 --- diff --git a/target/linux/atheros/patches-3.10/100-board.patch b/target/linux/atheros/patches-3.10/100-board.patch index 4524f944c9..a0dd6185b4 100644 --- a/target/linux/atheros/patches-3.10/100-board.patch +++ b/target/linux/atheros/patches-3.10/100-board.patch @@ -1766,11 +1766,11 @@ + .set = ar5312_gpio_set_value, + .get = ar5312_gpio_get_value, + .base = 0, -+ .ngpio = AR531X_GPIO_IRQ_COUNT, // 22 ++ .ngpio = AR531X_GPIO_IRQ_COUNT, /* 22 */ + } +}; + -+// end of gpiolib ++/* end of gpiolib */ + +static struct physmap_flash_data ar5312_flash_data = { + .width = 2, @@ -2406,7 +2406,7 @@ + u32 rett; + if (!(gpch->valid_mask & mask)) + return 0; -+ rett = ar2315_gpio_get(gpch->valid_mask); // legacy code ++ rett = ar2315_gpio_get(gpch->valid_mask); /* legacy code */ + return !!(rett & mask); +} + @@ -2418,7 +2418,7 @@ + u32 mask = 1 << gpio; + if (!(gpch->valid_mask & mask)) + return; -+ ar2315_gpio_set(mask, (!!value) * mask); // legacy ++ ar2315_gpio_set(mask, (!!value) * mask); /* legacy */ +} + +static int @@ -2429,7 +2429,7 @@ + u32 mask = 1 << gpio; + if (!(gpch->valid_mask & mask)) + return -ENXIO; -+ ar2315_gpio_set_output(mask, 0); // legacy ++ ar2315_gpio_set_output(mask, 0); /* legacy */ + return 0; +} + @@ -2441,7 +2441,7 @@ + u32 mask = 1 << gpio; + if (!(gpch->valid_mask & mask)) + return -ENXIO; -+ ar2315_gpio_set_output(mask, mask); // both legacy ++ ar2315_gpio_set_output(mask, mask); /* both legacy */ + ar2315_gpio_set(mask, (!!value) * mask); + return 0; +} @@ -2455,11 +2455,11 @@ + .set = ar2315_gpio_set_value, + .get = ar2315_gpio_get_value, + .base = 0, -+ .ngpio = AR531X_GPIO_IRQ_COUNT, // 22 ++ .ngpio = AR531X_GPIO_IRQ_COUNT, /* 22 */ + } +}; + -+// end of gpiolib ++/* end of gpiolib */ + + +static struct ar231x_eth ar2315_eth_data = { diff --git a/target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch b/target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch index 4f712ac9cd..e3773e3ade 100644 --- a/target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch +++ b/target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch @@ -32,7 +32,7 @@ +obj-$(CONFIG_NET_VENDOR_AR231X) += ar231x.o --- /dev/null +++ b/drivers/net/ethernet/ar231x/ar231x.c -@@ -0,0 +1,1278 @@ +@@ -0,0 +1,1265 @@ +/* + * ar231x.c: Linux driver for the Atheros AR231x Ethernet device. + * @@ -103,7 +103,7 @@ + +#include "ar231x.h" + -+/* ++/** + * New interrupt handler strategy: + * + * An old interrupt handler worked using the traditional method of @@ -142,7 +142,7 @@ + * handler. + */ + -+/* ++/** + * Threshold values for RX buffer allocation - the low water marks for + * when to start refilling the rings are set to 75% of the ring + * sizes. It seems to make sense to refill the rings entirely from the @@ -173,7 +173,7 @@ + +#define virt_to_phys(x) ((u32)(x) & 0x1fffffff) + -+// prototypes ++/* prototypes */ +static void ar231x_halt(struct net_device *dev); +static void rx_tasklet_func(unsigned long data); +static void rx_tasklet_cleanup(struct net_device *dev); @@ -265,7 +265,7 @@ + return (-ENXIO); + } + -+ /* ++ /** + * When there's only one MAC, PHY regs are typically on ENET0, + * even though the MAC might be on ENET1. + * Needto remap PHY regs separately in this case @@ -302,9 +302,7 @@ + memcpy(dev->dev_addr, sp->cfg->macaddr, 6); + + if (ar231x_init(dev)) { -+ /* -+ * ar231x_init() calls ar231x_init_cleanup() on error. -+ */ ++ /* ar231x_init() calls ar231x_init_cleanup() on error */ + kfree(dev); + return -ENODEV; + } @@ -373,7 +371,7 @@ +{ + struct ar231x_private *sp = netdev_priv(dev); + -+ /* ++ /** + * Tasklet may be scheduled. Need to get it removed from the list + * since we're about to free the struct. + */ @@ -397,7 +395,7 @@ +} + + -+/* ++/** + * Restart the AR2313 ethernet controller. + */ +static int ar231x_restart(struct net_device *dev) @@ -483,7 +481,7 @@ +} + + -+/* ++/** + * Generic cleanup handling data allocated during init. Used when the + * module is unloaded or if an error occurs during initialization + */ @@ -545,13 +543,17 @@ + struct net_device *dev = (struct net_device *) data; + struct ar231x_private *sp = netdev_priv(dev); + -+ // see if the link status changed -+ // This was needed to make sure we set the PHY to the -+ // autonegotiated value of half or full duplex. ++ /** ++ * See if the link status changed. ++ * This was needed to make sure we set the PHY to the ++ * autonegotiated value of half or full duplex. ++ */ + ar231x_check_link(dev); + -+ // Loop faster when we don't have link. -+ // This was needed to speed up the AP bootstrap time. ++ /** ++ * Loop faster when we don't have link. ++ * This was needed to speed up the AP bootstrap time. ++ */ + if (sp->link == 0) { + mod_timer(&sp->link_timer, jiffies + HZ / 2); + } else { @@ -567,8 +569,10 @@ + phy_data = ar231x_mdiobus_read(sp->mii_bus, sp->phy, MII_BMSR); + if (sp->phy_data != phy_data) { + if (phy_data & BMSR_LSTATUS) { -+ /* link is present, ready link partner ability to deterine -+ duplexity */ ++ /** ++ * Link is present, ready link partner ability to ++ * deterine duplexity. ++ */ + int duplex = 0; + u16 reg; + @@ -672,9 +676,7 @@ + struct ar231x_private *sp = netdev_priv(dev); + int ecode = 0; + -+ /* -+ * Allocate descriptors -+ */ ++ /* Allocate descriptors */ + if (ar231x_allocate_descriptors(dev)) { + printk("%s: %s: ar231x_allocate_descriptors failed\n", + dev->name, __FUNCTION__); @@ -682,9 +684,7 @@ + goto init_error; + } + -+ /* -+ * Get the memory for the skb rings. -+ */ ++ /* Get the memory for the skb rings */ + if (sp->rx_skb == NULL) { + sp->rx_skb = + kmalloc(sizeof(struct sk_buff *) * AR2313_DESCR_ENTRIES, @@ -711,7 +711,7 @@ + } + memset(sp->tx_skb, 0, sizeof(struct sk_buff *) * AR2313_DESCR_ENTRIES); + -+ /* ++ /** + * Set tx_csm before we start receiving interrupts, otherwise + * the interrupt handler might think it is supposed to process + * tx ints before we are up and running, which may cause a null @@ -722,25 +722,19 @@ + sp->tx_prd = 0; + sp->tx_csm = 0; + -+ /* -+ * Zero the stats before starting the interface -+ */ ++ /* Zero the stats before starting the interface */ + memset(&dev->stats, 0, sizeof(dev->stats)); + -+ /* ++ /** + * We load the ring here as there seem to be no way to tell the + * firmware to wipe the ring without re-initializing it. + */ + ar231x_load_rx_ring(dev, RX_RING_SIZE); + -+ /* -+ * Init hardware -+ */ ++ /* Init hardware */ + ar231x_reset_reg(dev); + -+ /* -+ * Get the IRQ -+ */ ++ /* Get the IRQ */ + ecode = + request_irq(dev->irq, &ar231x_interrupt, + IRQF_DISABLED, @@ -761,7 +755,7 @@ + return ecode; +} + -+/* ++/** + * Load the rx ring. + * + * Loading rings is safe without holding the spin lock since this is @@ -790,9 +784,7 @@ + break; + } + -+ /* -+ * Make sure IP header starts on a fresh cache line. -+ */ ++ /* Make sure IP header starts on a fresh cache line */ + skb->dev = dev; + sp->rx_skb[idx] = skb; + @@ -830,8 +822,7 @@ + + idx = sp->cur_rx; + -+ /* process at most the entire ring and then wait for another interrupt -+ */ ++ /* process at most the entire ring and then wait for another int */ + while (1) { + + rxdesc = &sp->rx_ring[idx]; @@ -991,16 +982,14 @@ + unsigned int status, enabled; + + /* clear interrupt */ -+ /* -+ * Don't clear RI bit if currently disabled. -+ */ ++ /* Don't clear RI bit if currently disabled */ + status = sp->dma_regs->status; + enabled = sp->dma_regs->intr_ena; + sp->dma_regs->status = status & enabled; + + if (status & DMA_STATUS_NIS) { + /* normal status */ -+ /* ++ /** + * Don't schedule rx processing if interrupt + * is already disabled. + */ @@ -1092,7 +1081,7 @@ + } +} + -+/* ++/** + * close should do nothing. Here's why. It's called when + * 'ifconfig bond0 down' is run. If it calls free_irq then + * the irq is gone forever ! When bond0 is made 'up' again, @@ -1105,12 +1094,10 @@ +static int ar231x_close(struct net_device *dev) +{ +#if 0 -+ /* -+ * Disable interrupts -+ */ ++ /* Disable interrupts */ + disable_irq(dev->irq); + -+ /* ++ /** + * Without (or before) releasing irq and stopping hardware, this + * is an absolute non-sense, by the way. It will be reset instantly + * by the first irq. @@ -1313,7 +1300,7 @@ + --- /dev/null +++ b/drivers/net/ethernet/ar231x/ar231x.h -@@ -0,0 +1,303 @@ +@@ -0,0 +1,289 @@ +/* + * ar231x.h: Linux driver for the Atheros AR231x Ethernet device. + * @@ -1339,9 +1326,7 @@ +#include +#include + -+/* -+ * probe link timer - 5 secs -+ */ ++/* probe link timer - 5 secs */ +#define LINK_TIMER (5*HZ) + +#define IS_DMA_TX_INT(X) (((X) & (DMA_STATUS_TI)) != 0) @@ -1350,9 +1335,7 @@ + +#define AR2313_TX_TIMEOUT (HZ/4) + -+/* -+ * Rings -+ */ ++/* Rings */ +#define DSC_RING_ENTRIES_SIZE (AR2313_DESCR_ENTRIES * sizeof(struct desc)) +#define DSC_NEXT(idx) ((idx + 1) & (AR2313_DESCR_ENTRIES - 1)) + @@ -1470,17 +1453,17 @@ + + +typedef struct { -+ volatile unsigned int status; // OWN, Device control and status. -+ volatile unsigned int devcs; // pkt Control bits + Length -+ volatile unsigned int addr; // Current Address. -+ volatile unsigned int descr; // Next descriptor in chain. ++ volatile unsigned int status; /* OWN, Device control and status. */ ++ volatile unsigned int devcs; /* pkt Control bits + Length */ ++ volatile unsigned int addr; /* Current Address. */ ++ volatile unsigned int descr; /* Next descriptor in chain. */ +} ar231x_descr_t; + + + -+// -+// New Combo structure for Both Eth0 AND eth1 -+// ++/** ++ * New Combo structure for Both Eth0 AND eth1 ++ */ +typedef struct { + volatile unsigned int mac_control; /* 0x00 */ + volatile unsigned int mac_addr[2]; /* 0x04 - 0x08 */ @@ -1526,7 +1509,7 @@ + volatile unsigned int cur_rx_buf_addr; /* 0x50 (CSR21) */ +} DMA; + -+/* ++/** + * Struct private for the Sibyte. + * + * Elements are grouped so variables used by the tx handling goes @@ -1549,9 +1532,7 @@ + + spinlock_t lock; /* Serialise access to device */ + -+ /* -+ * RX and TX descriptors, must be adjacent -+ */ ++ /* RX and TX descriptors, must be adjacent */ + ar231x_descr_t *rx_ring; + ar231x_descr_t *tx_ring; + @@ -1559,21 +1540,15 @@ + struct sk_buff **rx_skb; + struct sk_buff **tx_skb; + -+ /* -+ * RX elements -+ */ ++ /* RX elements */ + u32 rx_skbprd; + u32 cur_rx; + -+ /* -+ * TX elements -+ */ ++ /* TX elements */ + u32 tx_prd; + u32 tx_csm; + -+ /* -+ * Misc elements -+ */ ++ /* Misc elements */ + char name[48]; + struct { + u32 address; @@ -1597,9 +1572,7 @@ +}; + + -+/* -+ * Prototypes -+ */ ++/* Prototypes */ +static int ar231x_init(struct net_device *dev); +#ifdef TX_TIMEOUT +static void ar231x_tx_timeout(struct net_device *dev); diff --git a/target/linux/atheros/patches-3.10/220-enet_micrel_workaround.patch b/target/linux/atheros/patches-3.10/220-enet_micrel_workaround.patch index da86857213..870b76c65b 100644 --- a/target/linux/atheros/patches-3.10/220-enet_micrel_workaround.patch +++ b/target/linux/atheros/patches-3.10/220-enet_micrel_workaround.patch @@ -8,7 +8,7 @@ #ifndef ERR #define ERR(fmt, args...) printk("%s: " fmt, __func__, ##args) -@@ -182,6 +183,30 @@ static const struct net_device_ops ar231 +@@ -182,6 +183,32 @@ static const struct net_device_ops ar231 #endif }; @@ -16,8 +16,10 @@ +{ + int phy_reg; + -+ /* Grab the bits from PHYIR1, and put them -+ * in the upper half */ ++ /** ++ * Grab the bits from PHYIR1, and put them ++ * in the upper half. ++ */ + phy_reg = mdiobus_read(bus, addr, MII_PHYSID1); + + if (phy_reg < 0) @@ -39,12 +41,14 @@ int ar231x_probe(struct platform_device *pdev) { struct net_device *dev; -@@ -299,6 +324,21 @@ int ar231x_probe(struct platform_device +@@ -297,6 +324,23 @@ int ar231x_probe(struct platform_device mdiobus_register(sp->mii_bus); -+ /* Workaround for Micrel switch, which is only available on -+ * one PHY and cannot be configured through MDIO */ ++ /** ++ * Workaround for Micrel switch, which is only available on ++ * one PHY and cannot be configured through MDIO. ++ */ + if (!no_phy) { + u32 phy_id = 0; + get_phy_id(sp->mii_bus, 1, &phy_id); @@ -61,7 +65,7 @@ if (ar231x_mdiobus_probe(dev) != 0) { printk(KERN_ERR "%s: mdiobus_probe failed\n", dev->name); rx_tasklet_cleanup(dev); -@@ -355,8 +395,10 @@ static int ar231x_remove(struct platform +@@ -353,8 +397,10 @@ static int ar231x_remove(struct platform rx_tasklet_cleanup(dev); ar231x_init_cleanup(dev); unregister_netdev(dev); @@ -74,7 +78,7 @@ kfree(dev); return 0; } -@@ -1132,6 +1174,9 @@ static int ar231x_ioctl(struct net_devic +@@ -1119,6 +1165,9 @@ static int ar231x_ioctl(struct net_devic struct ar231x_private *sp = netdev_priv(dev); int ret; diff --git a/target/linux/atheros/patches-3.10/230-3_10-updates.patch b/target/linux/atheros/patches-3.10/230-3_10-updates.patch index a83bdcda51..873296dbfc 100644 --- a/target/linux/atheros/patches-3.10/230-3_10-updates.patch +++ b/target/linux/atheros/patches-3.10/230-3_10-updates.patch @@ -1,6 +1,6 @@ --- a/drivers/net/ethernet/ar231x/ar231x.c +++ b/drivers/net/ethernet/ar231x/ar231x.c -@@ -1291,7 +1291,7 @@ static int ar231x_mdiobus_probe (struct +@@ -1282,7 +1282,7 @@ static int ar231x_mdiobus_probe (struct BUG_ON(!phydev); BUG_ON(phydev->attached_dev);