atheros: various comments fixes
authorFelix Fietkau <nbd@openwrt.org>
Tue, 10 Jun 2014 09:05:28 +0000 (09:05 +0000)
committerFelix Fietkau <nbd@openwrt.org>
Tue, 10 Jun 2014 09:05:28 +0000 (09:05 +0000)
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 <ryazanov.s.a@gmail.com>
SVN-Revision: 41081

target/linux/atheros/patches-3.10/100-board.patch
target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch
target/linux/atheros/patches-3.10/220-enet_micrel_workaround.patch
target/linux/atheros/patches-3.10/230-3_10-updates.patch

index 4524f944c91777c39ca0a72c5bb3de140bcdf27f..a0dd6185b44bc9d7906480c37786c2ccd86664f9 100644 (file)
 +              .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,
 +      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);
 +}
 +
 +      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
 +      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;
 +}
 +
 +      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;
 +}
 +              .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 = {
index 4f712ac9cdbe4aa229941db8aa191ad587ed96d9..e3773e3ade6bbd7b484591f0d7d61d2d7835f6dd 100644 (file)
@@ -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.
 + *
 +
 +#include "ar231x.h"
 +
-+/*
++/**
 + * New interrupt handler strategy:
 + *
 + * An old interrupt handler worked using the traditional method of
 + * 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
 +
 +#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);
 +              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
 +      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;
 +      }
 +{
 +      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.
 +       */
 +}
 +
 +
-+/*
++/**
 + * Restart the AR2313 ethernet controller.
 + */
 +static int ar231x_restart(struct net_device *dev)
 +}
 +
 +
-+/*
++/**
 + * Generic cleanup handling data allocated during init. Used when the
 + * module is unloaded or if an error occurs during initialization
 + */
 +      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 {
 +      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;
 +
 +      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__);
 +              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,
 +      }
 +      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
 +      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,
 +      return ecode;
 +}
 +
-+/*
++/**
 + * Load the rx ring.
 + *
 + * Loading rings is safe without holding the spin lock since this is
 +                      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;
 +
 +
 +      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];
 +      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.
 +               */
 +      }
 +}
 +
-+/*
++/**
 + * 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,
 +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.
 +
 --- /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.
 + *
 +#include <asm/bootinfo.h>
 +#include <ar231x_platform.h>
 +
-+/*
-+ * 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)
 +
 +#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))
 +
 +
 +
 +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 */
 +      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
 +
 +      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;
 +
 +      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;
 +};
 +
 +
-+/*
-+ * Prototypes
-+ */
++/* Prototypes */
 +static int ar231x_init(struct net_device *dev);
 +#ifdef TX_TIMEOUT
 +static void ar231x_tx_timeout(struct net_device *dev);
index da86857213a6c37202f648f0f2e891ae45f5375c..870b76c65b0cb0db6efffc683615cd378b5da4ba 100644 (file)
@@ -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
  };
  
 +{
 +      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)
  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;
  
index a83bdcda517f8fba50d4c0ee5b194c9999ca1ad4..873296dbfce46c902e05fedd186e1bba85dcbe71 100644 (file)
@@ -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);