* [PATCH 1/5] drivers: atm: checkpatch.pl fixed coding style errors in eni.c
@ 2013-01-20 23:12 Patrik Karlin
2013-01-20 23:12 ` [PATCH 2/5] drivers: atm: checkpatch.pl fixes removed all trailing whitespaces " Patrik Karlin
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Patrik Karlin @ 2013-01-20 23:12 UTC (permalink / raw)
To: chas; +Cc: linux-atm-general, netdev, linux-kernel, Patrik Karlin
This patch removes all space after , errors in eni.c
Signed-off-by: Patrik Kårlin <patrik.karlin@gmail.com>
---
drivers/atm/eni.c | 632 ++++++++++++++++++++++++++---------------------------
1 file changed, 316 insertions(+), 316 deletions(-)
diff --git a/drivers/atm/eni.c b/drivers/atm/eni.c
index c1eb6fa..6ae0d89 100644
--- a/drivers/atm/eni.c
+++ b/drivers/atm/eni.c
@@ -32,7 +32,7 @@
#if !defined(__i386__) && !defined(__x86_64__)
#ifndef ioremap_nocache
-#define ioremap_nocache(X,Y) ioremap(X,Y)
+#define ioremap_nocache(X, Y) ioremap(X, Y)
#endif
#endif
@@ -62,9 +62,9 @@
#if 0
-#define DPRINTK(format,args...) printk(KERN_DEBUG format,##args)
+#define DPRINTK(format, args...) printk(KERN_DEBUG format, ##args)
#else
-#define DPRINTK(format,args...)
+#define DPRINTK(format, args...)
#endif
@@ -79,7 +79,7 @@
#define NULLCHECK(x)
-#define EVENT(s,a,b)
+#define EVENT(s, a, b)
static void event_dump(void)
@@ -96,7 +96,7 @@ static void event_dump(void)
#define NULLCHECK(x) \
if ((unsigned long) (x) < 0x30) \
- printk(KERN_CRIT #x "==0x%lx\n",(unsigned long) (x))
+ printk(KERN_CRIT #x "==0x%lx\n", (unsigned long) (x))
/*
* Very extensive activity logging. Greatly improves bug detection speed but
@@ -106,11 +106,11 @@ static void event_dump(void)
#define EV 64
static const char *ev[EV];
-static unsigned long ev_a[EV],ev_b[EV];
+static unsigned long ev_a[EV], ev_b[EV];
static int ec = 0;
-static void EVENT(const char *s,unsigned long a,unsigned long b)
+static void EVENT(const char *s, unsigned long a, unsigned long b)
{
ev[ec] = s;
ev_a[ec] = a;
@@ -121,12 +121,12 @@ static void EVENT(const char *s,unsigned long a,unsigned long b)
static void event_dump(void)
{
- int n,i;
+ int n, i;
for (n = 0; n < EV; n++) {
i = (ec+n) % EV;
printk(KERN_NOTICE);
- printk(ev[i] ? ev[i] : "(null)",ev_a[i],ev_b[i]);
+ printk(ev[i] ? ev[i] : "(null)", ev_a[i], ev_b[i]);
}
}
@@ -141,23 +141,23 @@ static void event_dump(void)
* xxPMOK operating on a circular buffer of "c" words
*/
-#define NEPJOK(a0,a1,b) \
+#define NEPJOK(a0, a1, b) \
((a0) < (a1) ? (b) <= (a0) || (b) > (a1) : (b) <= (a0) && (b) > (a1))
-#define EEPJOK(a0,a1,b) \
+#define EEPJOK(a0, a1, b) \
((a0) < (a1) ? (b) < (a0) || (b) >= (a1) : (b) < (a0) && (b) >= (a1))
-#define NEPMOK(a0,d,b,c) NEPJOK(a0,(a0+d) & (c-1),b)
-#define EEPMOK(a0,d,b,c) EEPJOK(a0,(a0+d) & (c-1),b)
+#define NEPMOK(a0, d, b, c) NEPJOK(a0, (a0+d) & (c-1), b)
+#define EEPMOK(a0, d, b, c) EEPJOK(a0, (a0+d) & (c-1), b)
-static int tx_complete = 0,dma_complete = 0,queued = 0,requeued = 0,
- backlogged = 0,rx_enqueued = 0,rx_dequeued = 0,pushed = 0,submitted = 0,
+static int tx_complete = 0, dma_complete = 0, queued = 0, requeued = 0,
+ backlogged = 0, rx_enqueued = 0, rx_dequeued = 0, pushed = 0, submitted = 0,
putting = 0;
static struct atm_dev *eni_boards = NULL;
/* Read/write registers on card */
#define eni_in(r) readl(eni_dev->reg+(r)*4)
-#define eni_out(v,r) writel((v),eni_dev->reg+(r)*4)
+#define eni_out(v, r) writel((v), eni_dev->reg+(r)*4)
/*-------------------------------- utilities --------------------------------*/
@@ -168,7 +168,7 @@ static void dump_mem(struct eni_dev *eni_dev)
int i;
for (i = 0; i < eni_dev->free_len; i++)
- printk(KERN_DEBUG " %d: %p %d\n",i,
+ printk(KERN_DEBUG " %d: %p %d\n", i,
eni_dev->free_list[i].start,
1 << eni_dev->free_list[i].order);
}
@@ -186,12 +186,12 @@ static void dump(struct atm_dev *dev)
printk(KERN_NOTICE "TX buffers\n");
for (i = 0; i < NR_CHAN; i++)
if (eni_dev->tx[i].send)
- printk(KERN_NOTICE " TX %d @ %p: %ld\n",i,
- eni_dev->tx[i].send,eni_dev->tx[i].words*4);
+ printk(KERN_NOTICE " TX %d @ %p: %ld\n", i,
+ eni_dev->tx[i].send, eni_dev->tx[i].words*4);
printk(KERN_NOTICE "RX buffers\n");
for (i = 0; i < 1024; i++)
if (eni_dev->rx_map[i] && ENI_VCC(eni_dev->rx_map[i])->rx)
- printk(KERN_NOTICE " RX %d @ %p: %ld\n",i,
+ printk(KERN_NOTICE " RX %d @ %p: %ld\n", i,
ENI_VCC(eni_dev->rx_map[i])->recv,
ENI_VCC(eni_dev->rx_map[i])->words*4);
printk(KERN_NOTICE "----\n");
@@ -202,16 +202,16 @@ static void eni_put_free(struct eni_dev *eni_dev, void __iomem *start,
unsigned long size)
{
struct eni_free *list;
- int len,order;
+ int len, order;
- DPRINTK("init 0x%lx+%ld(0x%lx)\n",start,size,size);
+ DPRINTK("init 0x%lx+%ld(0x%lx)\n", start, size, size);
start += eni_dev->base_diff;
list = eni_dev->free_list;
len = eni_dev->free_len;
while (size) {
if (len >= eni_dev->free_list_size) {
printk(KERN_CRIT "eni_put_free overflow (%p,%ld)\n",
- start,size);
+ start, size);
break;
}
for (order = 0; !(((unsigned long)start | size) & (1 << order)); order++);
@@ -235,14 +235,14 @@ static void __iomem *eni_alloc_mem(struct eni_dev *eni_dev, unsigned long *size)
{
struct eni_free *list;
void __iomem *start;
- int len,i,order,best_order,index;
+ int len, i, order, best_order, index;
list = eni_dev->free_list;
len = eni_dev->free_len;
if (*size < MID_MIN_BUF_SIZE) *size = MID_MIN_BUF_SIZE;
if (*size > MID_MAX_BUF_SIZE) return NULL;
for (order = 0; (1 << order) < *size; order++);
- DPRINTK("trying: %ld->%d\n",*size,order);
+ DPRINTK("trying: %ld->%d\n", *size, order);
best_order = 65; /* we don't have more than 2^64 of anything ... */
index = 0; /* silence GCC */
for (i = 0; i < len; i++)
@@ -260,9 +260,9 @@ static void __iomem *eni_alloc_mem(struct eni_dev *eni_dev, unsigned long *size)
list[index] = list[--len];
eni_dev->free_len = len;
*size = 1 << order;
- eni_put_free(eni_dev,start+*size,(1 << best_order)-*size);
- DPRINTK("%ld bytes (order %d) at 0x%lx\n",*size,order,start);
- memset_io(start,0,*size); /* never leak data */
+ eni_put_free(eni_dev, start+*size, (1 << best_order)-*size);
+ DPRINTK("%ld bytes (order %d) at 0x%lx\n", *size, order, start);
+ memset_io(start, 0, *size); /* never leak data */
/*dump_mem(eni_dev);*/
return start;
}
@@ -272,18 +272,18 @@ static void eni_free_mem(struct eni_dev *eni_dev, void __iomem *start,
unsigned long size)
{
struct eni_free *list;
- int len,i,order;
+ int len, i, order;
start += eni_dev->base_diff;
list = eni_dev->free_list;
len = eni_dev->free_len;
for (order = -1; size; order++) size >>= 1;
- DPRINTK("eni_free_mem: %p+0x%lx (order %d)\n",start,size,order);
+ DPRINTK("eni_free_mem: %p+0x%lx (order %d)\n", start, size, order);
for (i = 0; i < len; i++)
if (((unsigned long) list[i].start) == ((unsigned long)start^(1 << order)) &&
list[i].order == order) {
- DPRINTK("match[%d]: 0x%lx/0x%lx(0x%x), %d/%d\n",i,
- list[i].start,start,1 << order,list[i].order,order);
+ DPRINTK("match[%d]: 0x%lx/0x%lx(0x%x), %d/%d\n", i,
+ list[i].start, start, 1 << order, list[i].order, order);
list[i] = list[--len];
start = (void __iomem *) ((unsigned long) start & ~(unsigned long) (1 << order));
order++;
@@ -291,7 +291,7 @@ static void eni_free_mem(struct eni_dev *eni_dev, void __iomem *start,
continue;
}
if (len >= eni_dev->free_list_size) {
- printk(KERN_ALERT "eni_free_mem overflow (%p,%d)\n",start,
+ printk(KERN_ALERT "eni_free_mem overflow (%p,%d)\n", start,
order);
return;
}
@@ -318,19 +318,19 @@ static void rx_ident_err(struct atm_vcc *vcc)
eni_dev = ENI_DEV(dev);
/* immediately halt adapter */
eni_out(eni_in(MID_MC_S) &
- ~(MID_DMA_ENABLE | MID_TX_ENABLE | MID_RX_ENABLE),MID_MC_S);
+ ~(MID_DMA_ENABLE | MID_TX_ENABLE | MID_RX_ENABLE), MID_MC_S);
/* dump useful information */
eni_vcc = ENI_VCC(vcc);
printk(KERN_ALERT DEV_LABEL "(itf %d): driver error - RX ident "
- "mismatch\n",dev->number);
- printk(KERN_ALERT " VCI %d, rxing %d, words %ld\n",vcc->vci,
- eni_vcc->rxing,eni_vcc->words);
+ "mismatch\n", dev->number);
+ printk(KERN_ALERT " VCI %d, rxing %d, words %ld\n", vcc->vci,
+ eni_vcc->rxing, eni_vcc->words);
printk(KERN_ALERT " host descr 0x%lx, rx pos 0x%lx, descr value "
- "0x%x\n",eni_vcc->descr,eni_vcc->rx_pos,
+ "0x%x\n", eni_vcc->descr, eni_vcc->rx_pos,
(unsigned) readl(eni_vcc->recv+eni_vcc->descr*4));
- printk(KERN_ALERT " last %p, servicing %d\n",eni_vcc->last,
+ printk(KERN_ALERT " last %p, servicing %d\n", eni_vcc->last,
eni_vcc->servicing);
- EVENT("---dump ends here---\n",0,0);
+ EVENT("---dump ends here---\n", 0, 0);
printk(KERN_NOTICE "---recent events---\n");
event_dump();
ENI_DEV(dev)->fast = NULL; /* really stop it */
@@ -339,28 +339,28 @@ static void rx_ident_err(struct atm_vcc *vcc)
}
-static int do_rx_dma(struct atm_vcc *vcc,struct sk_buff *skb,
- unsigned long skip,unsigned long size,unsigned long eff)
+static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
+ unsigned long skip, unsigned long size, unsigned long eff)
{
struct eni_dev *eni_dev;
struct eni_vcc *eni_vcc;
- u32 dma_rd,dma_wr;
+ u32 dma_rd, dma_wr;
u32 dma[RX_DMA_BUF*2];
dma_addr_t paddr;
unsigned long here;
- int i,j;
+ int i, j;
eni_dev = ENI_DEV(vcc->dev);
eni_vcc = ENI_VCC(vcc);
paddr = 0; /* GCC, shut up */
if (skb) {
- paddr = pci_map_single(eni_dev->pci_dev,skb->data,skb->len,
+ paddr = pci_map_single(eni_dev->pci_dev, skb->data, skb->len,
PCI_DMA_FROMDEVICE);
ENI_PRV_PADDR(skb) = paddr;
if (paddr & 3)
printk(KERN_CRIT DEV_LABEL "(itf %d): VCI %d has "
- "mis-aligned RX data (0x%lx)\n",vcc->dev->number,
- vcc->vci,(unsigned long) paddr);
+ "mis-aligned RX data (0x%lx)\n", vcc->dev->number,
+ vcc->vci, (unsigned long) paddr);
ENI_PRV_SIZE(skb) = size+skip;
/* PDU plus descriptor */
ATM_SKB(skb)->vcc = vcc;
@@ -380,7 +380,7 @@ static int do_rx_dma(struct atm_vcc *vcc,struct sk_buff *skb,
if (!size) {
DPRINTK("strange things happen ...\n");
EVENT("strange things happen ... (skip=%ld,eff=%ld)\n",
- size,eff);
+ size, eff);
}
words = eff;
if (paddr & 15) {
@@ -457,42 +457,42 @@ static int do_rx_dma(struct atm_vcc *vcc,struct sk_buff *skb,
* Can I move the dma_wr pointer by 2j+1 positions without overwriting
* data that hasn't been read (position of dma_rd) yet ?
*/
- if (!NEPMOK(dma_wr,j+j+1,dma_rd,NR_DMA_RX)) { /* @@@ +1 is ugly */
+ if (!NEPMOK(dma_wr, j+j+1, dma_rd, NR_DMA_RX)) { /* @@@ +1 is ugly */
printk(KERN_WARNING DEV_LABEL "(itf %d): RX DMA full\n",
vcc->dev->number);
goto trouble;
}
for (i = 0; i < j; i++) {
- writel(dma[i*2],eni_dev->rx_dma+dma_wr*8);
- writel(dma[i*2+1],eni_dev->rx_dma+dma_wr*8+4);
+ writel(dma[i*2], eni_dev->rx_dma+dma_wr*8);
+ writel(dma[i*2+1], eni_dev->rx_dma+dma_wr*8+4);
dma_wr = (dma_wr+1) & (NR_DMA_RX-1);
}
if (skb) {
ENI_PRV_POS(skb) = eni_vcc->descr+size+1;
- skb_queue_tail(&eni_dev->rx_queue,skb);
+ skb_queue_tail(&eni_dev->rx_queue, skb);
eni_vcc->last = skb;
rx_enqueued++;
}
eni_vcc->descr = here;
- eni_out(dma_wr,MID_DMA_WR_RX);
+ eni_out(dma_wr, MID_DMA_WR_RX);
return 0;
trouble:
if (paddr)
- pci_unmap_single(eni_dev->pci_dev,paddr,skb->len,
+ pci_unmap_single(eni_dev->pci_dev, paddr, skb->len,
PCI_DMA_FROMDEVICE);
if (skb) dev_kfree_skb_irq(skb);
return -1;
}
-static void discard(struct atm_vcc *vcc,unsigned long size)
+static void discard(struct atm_vcc *vcc, unsigned long size)
{
struct eni_vcc *eni_vcc;
eni_vcc = ENI_VCC(vcc);
- EVENT("discard (size=%ld)\n",size,0);
- while (do_rx_dma(vcc,NULL,1,size,0)) EVENT("BUSY LOOP",0,0);
+ EVENT("discard (size=%ld)\n", size, 0);
+ while (do_rx_dma(vcc, NULL, 1, size, 0)) EVENT("BUSY LOOP", 0, 0);
/* could do a full fallback, but that might be more expensive */
if (eni_vcc->rxing) ENI_PRV_POS(eni_vcc->last) += size+1;
else eni_vcc->rx_pos = (eni_vcc->rx_pos+size+1) & (eni_vcc->words-1);
@@ -527,15 +527,15 @@ static int rx_aal0(struct atm_vcc *vcc)
else {
length = ATM_CELL_SIZE-1; /* no HEC */
}
- skb = length ? atm_alloc_charge(vcc,length,GFP_ATOMIC) : NULL;
+ skb = length ? atm_alloc_charge(vcc, length, GFP_ATOMIC) : NULL;
if (!skb) {
- discard(vcc,length >> 2);
+ discard(vcc, length >> 2);
return 0;
}
- skb_put(skb,length);
+ skb_put(skb, length);
skb->tstamp = eni_vcc->timestamp;
- DPRINTK("got len %ld\n",length);
- if (do_rx_dma(vcc,skb,1,length >> 2,length >> 2)) return 1;
+ DPRINTK("got len %ld\n", length);
+ if (do_rx_dma(vcc, skb, 1, length >> 2, length >> 2)) return 1;
eni_vcc->rxing++;
return 0;
}
@@ -545,10 +545,10 @@ static int rx_aal5(struct atm_vcc *vcc)
{
struct eni_vcc *eni_vcc;
unsigned long descr;
- unsigned long size,eff,length;
+ unsigned long size, eff, length;
struct sk_buff *skb;
- EVENT("rx_aal5\n",0,0);
+ EVENT("rx_aal5\n", 0, 0);
DPRINTK(">rx_aal5\n");
eni_vcc = ENI_VCC(vcc);
descr = readl(eni_vcc->recv+eni_vcc->descr*4);
@@ -558,7 +558,7 @@ static int rx_aal5(struct atm_vcc *vcc)
}
if (descr & (MID_RED_T | MID_RED_CRC_ERR)) {
if (descr & MID_RED_T) {
- EVENT("empty cell (descr=0x%lx)\n",descr,0);
+ EVENT("empty cell (descr=0x%lx)\n", descr, 0);
DPRINTK(DEV_LABEL "(itf %d): trashing empty cell\n",
vcc->dev->number);
size = 0;
@@ -573,7 +573,7 @@ static int rx_aal5(struct atm_vcc *vcc)
silence = (jiffies+2*HZ)|1;
}
size = (descr & MID_RED_COUNT)*(ATM_CELL_PAYLOAD >> 2);
- EVENT("CRC error (descr=0x%lx,size=%ld)\n",descr,
+ EVENT("CRC error (descr=0x%lx,size=%ld)\n", descr,
size);
}
eff = length = 0;
@@ -581,30 +581,30 @@ static int rx_aal5(struct atm_vcc *vcc)
}
else {
size = (descr & MID_RED_COUNT)*(ATM_CELL_PAYLOAD >> 2);
- DPRINTK("size=%ld\n",size);
+ DPRINTK("size=%ld\n", size);
length = readl(eni_vcc->recv+(((eni_vcc->descr+size-1) &
(eni_vcc->words-1)))*4) & 0xffff;
/* -trailer(2)+header(1) */
if (length && length <= (size << 2)-8 && length <=
ATM_MAX_AAL5_PDU) eff = (length+3) >> 2;
else { /* ^ trailer length (8) */
- EVENT("bad PDU (descr=0x08%lx,length=%ld)\n",descr,
- length);
+ EVENT("bad PDU (descr=0x08%lx,length=%ld)\n", descr,
+ length);
printk(KERN_ERR DEV_LABEL "(itf %d): bad AAL5 PDU "
"(VCI=%d,length=%ld,size=%ld (descr 0x%lx))\n",
- vcc->dev->number,vcc->vci,length,size << 2,descr);
+ vcc->dev->number, vcc->vci, length, size << 2, descr);
length = eff = 0;
atomic_inc(&vcc->stats->rx_err);
}
}
- skb = eff ? atm_alloc_charge(vcc,eff << 2,GFP_ATOMIC) : NULL;
+ skb = eff ? atm_alloc_charge(vcc, eff << 2, GFP_ATOMIC) : NULL;
if (!skb) {
- discard(vcc,size);
+ discard(vcc, size);
return 0;
}
- skb_put(skb,length);
- DPRINTK("got len %ld\n",length);
- if (do_rx_dma(vcc,skb,1,size,eff)) return 1;
+ skb_put(skb, length);
+ DPRINTK("got len %ld\n", length);
+ if (do_rx_dma(vcc, skb, 1, size, eff)) return 1;
eni_vcc->rxing++;
return 0;
}
@@ -618,29 +618,29 @@ static inline int rx_vcc(struct atm_vcc *vcc)
eni_vcc = ENI_VCC(vcc);
vci_dsc = ENI_DEV(vcc->dev)->vci+vcc->vci*16;
- EVENT("rx_vcc(1)\n",0,0);
+ EVENT("rx_vcc(1)\n", 0, 0);
while (eni_vcc->descr != (tmp = (readl(vci_dsc+4) & MID_VCI_DESCR) >>
MID_VCI_DESCR_SHIFT)) {
EVENT("rx_vcc(2: host dsc=0x%lx, nic dsc=0x%lx)\n",
- eni_vcc->descr,tmp);
- DPRINTK("CB_DESCR %ld REG_DESCR %d\n",ENI_VCC(vcc)->descr,
+ eni_vcc->descr, tmp);
+ DPRINTK("CB_DESCR %ld REG_DESCR %d\n", ENI_VCC(vcc)->descr,
(((unsigned) readl(vci_dsc+4) & MID_VCI_DESCR) >>
MID_VCI_DESCR_SHIFT));
if (ENI_VCC(vcc)->rx(vcc)) return 1;
}
/* clear IN_SERVICE flag */
- writel(readl(vci_dsc) & ~MID_VCI_IN_SERVICE,vci_dsc);
+ writel(readl(vci_dsc) & ~MID_VCI_IN_SERVICE, vci_dsc);
/*
* If new data has arrived between evaluating the while condition and
* clearing IN_SERVICE, we wouldn't be notified until additional data
* follows. So we have to loop again to be sure.
*/
- EVENT("rx_vcc(3)\n",0,0);
+ EVENT("rx_vcc(3)\n", 0, 0);
while (ENI_VCC(vcc)->descr != (tmp = (readl(vci_dsc+4) & MID_VCI_DESCR)
>> MID_VCI_DESCR_SHIFT)) {
EVENT("rx_vcc(4: host dsc=0x%lx, nic dsc=0x%lx)\n",
- eni_vcc->descr,tmp);
- DPRINTK("CB_DESCR %ld REG_DESCR %d\n",ENI_VCC(vcc)->descr,
+ eni_vcc->descr, tmp);
+ DPRINTK("CB_DESCR %ld REG_DESCR %d\n", ENI_VCC(vcc)->descr,
(((unsigned) readl(vci_dsc+4) & MID_VCI_DESCR) >>
MID_VCI_DESCR_SHIFT));
if (ENI_VCC(vcc)->rx(vcc)) return 1;
@@ -656,7 +656,7 @@ static void poll_rx(struct atm_dev *dev)
eni_dev = ENI_DEV(dev);
while ((curr = eni_dev->fast)) {
- EVENT("poll_rx.fast\n",0,0);
+ EVENT("poll_rx.fast\n", 0, 0);
if (rx_vcc(curr)) return;
eni_dev->fast = ENI_VCC(curr)->next;
ENI_VCC(curr)->next = ENI_VCC_NOS;
@@ -664,7 +664,7 @@ static void poll_rx(struct atm_dev *dev)
ENI_VCC(curr)->servicing--;
}
while ((curr = eni_dev->slow)) {
- EVENT("poll_rx.slow\n",0,0);
+ EVENT("poll_rx.slow\n", 0, 0);
if (rx_vcc(curr)) return;
eni_dev->slow = ENI_VCC(curr)->next;
ENI_VCC(curr)->next = ENI_VCC_NOS;
@@ -688,14 +688,14 @@ static void get_service(struct atm_dev *dev)
vcc = eni_dev->rx_map[vci & 1023];
if (!vcc) {
printk(KERN_CRIT DEV_LABEL "(itf %d): VCI %ld not "
- "found\n",dev->number,vci);
+ "found\n", dev->number, vci);
continue; /* nasty but we try to go on anyway */
/* @@@ nope, doesn't work */
}
- EVENT("getting from service\n",0,0);
+ EVENT("getting from service\n", 0, 0);
if (ENI_VCC(vcc)->next != ENI_VCC_NOS) {
- EVENT("double service\n",0,0);
- DPRINTK("Grr, servicing VCC %ld twice\n",vci);
+ EVENT("double service\n", 0, 0);
+ DPRINTK("Grr, servicing VCC %ld twice\n", vci);
continue;
}
ENI_VCC(vcc)->timestamp = ktime_get_real();
@@ -734,37 +734,37 @@ static void dequeue_rx(struct atm_dev *dev)
if (!skb) {
if (first) {
DPRINTK(DEV_LABEL "(itf %d): RX but not "
- "rxing\n",dev->number);
- EVENT("nothing to dequeue\n",0,0);
+ "rxing\n", dev->number);
+ EVENT("nothing to dequeue\n", 0, 0);
}
break;
}
- EVENT("dequeued (size=%ld,pos=0x%lx)\n",ENI_PRV_SIZE(skb),
- ENI_PRV_POS(skb));
+ EVENT("dequeued (size=%ld,pos=0x%lx)\n", ENI_PRV_SIZE(skb),
+ ENI_PRV_POS(skb));
rx_dequeued++;
vcc = ATM_SKB(skb)->vcc;
eni_vcc = ENI_VCC(vcc);
first = 0;
vci_dsc = eni_dev->vci+vcc->vci*16;
- if (!EEPMOK(eni_vcc->rx_pos,ENI_PRV_SIZE(skb),
+ if (!EEPMOK(eni_vcc->rx_pos, ENI_PRV_SIZE(skb),
(readl(vci_dsc+4) & MID_VCI_READ) >> MID_VCI_READ_SHIFT,
eni_vcc->words)) {
- EVENT("requeuing\n",0,0);
- skb_queue_head(&eni_dev->rx_queue,skb);
+ EVENT("requeuing\n", 0, 0);
+ skb_queue_head(&eni_dev->rx_queue, skb);
break;
}
eni_vcc->rxing--;
eni_vcc->rx_pos = ENI_PRV_POS(skb) & (eni_vcc->words-1);
- pci_unmap_single(eni_dev->pci_dev,ENI_PRV_PADDR(skb),skb->len,
- PCI_DMA_TODEVICE);
+ pci_unmap_single(eni_dev->pci_dev, ENI_PRV_PADDR(skb), skb->len,
+ PCI_DMA_TODEVICE);
if (!skb->len) dev_kfree_skb_irq(skb);
else {
- EVENT("pushing (len=%ld)\n",skb->len,0);
+ EVENT("pushing (len=%ld)\n", skb->len, 0);
if (vcc->qos.aal == ATM_AAL0)
*(unsigned long *) skb->data =
ntohl(*(unsigned long *) skb->data);
- memset(skb->cb,0,sizeof(struct eni_skb_prv));
- vcc->push(vcc,skb);
+ memset(skb->cb, 0, sizeof(struct eni_skb_prv));
+ vcc->push(vcc, skb);
pushed++;
}
atomic_inc(&vcc->stats->rx);
@@ -788,8 +788,8 @@ static int open_rx_first(struct atm_vcc *vcc)
if (size > MID_MAX_BUF_SIZE && vcc->qos.rxtp.max_sdu <=
MID_MAX_BUF_SIZE)
size = MID_MAX_BUF_SIZE;
- eni_vcc->recv = eni_alloc_mem(eni_dev,&size);
- DPRINTK("rx at 0x%lx\n",eni_vcc->recv);
+ eni_vcc->recv = eni_alloc_mem(eni_dev, &size);
+ DPRINTK("rx at 0x%lx\n", eni_vcc->recv);
eni_vcc->words = size >> 2;
if (!eni_vcc->recv) return -ENOBUFS;
eni_vcc->rx = vcc->qos.aal == ATM_AAL5 ? rx_aal5 : rx_aal0;
@@ -816,26 +816,26 @@ static int open_rx_second(struct atm_vcc *vcc)
if (!eni_vcc->rx) return 0;
/* set up VCI descriptor */
here = eni_dev->vci+vcc->vci*16;
- DPRINTK("loc 0x%x\n",(unsigned) (eni_vcc->recv-eni_dev->ram)/4);
+ DPRINTK("loc 0x%x\n", (unsigned) (eni_vcc->recv-eni_dev->ram)/4);
size = eni_vcc->words >> 8;
for (order = -1; size; order++) size >>= 1;
- writel(0,here+4); /* descr, read = 0 */
- writel(0,here+8); /* write, state, count = 0 */
+ writel(0, here+4); /* descr, read = 0 */
+ writel(0, here+8); /* write, state, count = 0 */
if (eni_dev->rx_map[vcc->vci])
printk(KERN_CRIT DEV_LABEL "(itf %d): BUG - VCI %d already "
- "in use\n",vcc->dev->number,vcc->vci);
+ "in use\n", vcc->dev->number, vcc->vci);
eni_dev->rx_map[vcc->vci] = vcc; /* now it counts */
writel(((vcc->qos.aal != ATM_AAL5 ? MID_MODE_RAW : MID_MODE_AAL5) <<
MID_VCI_MODE_SHIFT) | MID_VCI_PTI_MODE |
(((eni_vcc->recv-eni_dev->ram) >> (MID_LOC_SKIP+2)) <<
- MID_VCI_LOCATION_SHIFT) | (order << MID_VCI_SIZE_SHIFT),here);
+ MID_VCI_LOCATION_SHIFT) | (order << MID_VCI_SIZE_SHIFT), here);
return 0;
}
static void close_rx(struct atm_vcc *vcc)
{
- DECLARE_WAITQUEUE(wait,current);
+ DECLARE_WAITQUEUE(wait, current);
void __iomem *here;
struct eni_dev *eni_dev;
struct eni_vcc *eni_vcc;
@@ -847,17 +847,17 @@ static void close_rx(struct atm_vcc *vcc)
here = eni_dev->vci+vcc->vci*16;
/* block receiver */
writel((readl(here) & ~MID_VCI_MODE) | (MID_MODE_TRASH <<
- MID_VCI_MODE_SHIFT),here);
+ MID_VCI_MODE_SHIFT), here);
/* wait for receiver to become idle */
udelay(27);
/* discard pending cell */
- writel(readl(here) & ~MID_VCI_IN_SERVICE,here);
+ writel(readl(here) & ~MID_VCI_IN_SERVICE, here);
/* don't accept any new ones */
eni_dev->rx_map[vcc->vci] = NULL;
/* wait for RX queue to drain */
DPRINTK("eni_close: waiting for RX ...\n");
- EVENT("RX closing\n",0,0);
- add_wait_queue(&eni_dev->rx_wait,&wait);
+ EVENT("RX closing\n", 0, 0);
+ add_wait_queue(&eni_dev->rx_wait, &wait);
set_current_state(TASK_UNINTERRUPTIBLE);
barrier();
for (;;) {
@@ -866,9 +866,9 @@ static void close_rx(struct atm_vcc *vcc)
barrier();
if (!eni_vcc->rxing) break;
}
- EVENT("drain PDUs (rx %ld, serv %ld)\n",eni_vcc->rxing,
+ EVENT("drain PDUs (rx %ld, serv %ld)\n", eni_vcc->rxing,
eni_vcc->servicing);
- printk(KERN_INFO "%d+%d RX left\n",eni_vcc->servicing,
+ printk(KERN_INFO "%d+%d RX left\n", eni_vcc->servicing,
eni_vcc->rxing);
schedule();
set_current_state(TASK_UNINTERRUPTIBLE);
@@ -883,16 +883,16 @@ static void close_rx(struct atm_vcc *vcc)
tasklet_enable(&eni_dev->task);
if (at_end) break;
EVENT("drain discard (host 0x%lx, nic 0x%lx)\n",
- eni_vcc->rx_pos,tmp);
+ eni_vcc->rx_pos, tmp);
printk(KERN_INFO "draining RX: host 0x%lx, nic 0x%x\n",
- eni_vcc->rx_pos,tmp);
+ eni_vcc->rx_pos, tmp);
schedule();
set_current_state(TASK_UNINTERRUPTIBLE);
}
set_current_state(TASK_RUNNING);
- remove_wait_queue(&eni_dev->rx_wait,&wait);
+ remove_wait_queue(&eni_dev->rx_wait, &wait);
}
- eni_free_mem(eni_dev,eni_vcc->recv,eni_vcc->words << 2);
+ eni_free_mem(eni_dev, eni_vcc->recv, eni_vcc->words << 2);
eni_vcc->rx = NULL;
}
@@ -915,7 +915,7 @@ static int start_rx(struct atm_dev *dev)
init_waitqueue_head(&eni_dev->rx_wait);
skb_queue_head_init(&eni_dev->rx_queue);
eni_dev->serv_read = eni_in(MID_SERV_WRITE);
- eni_out(0,MID_DMA_WR_RX);
+ eni_out(0, MID_DMA_WR_RX);
return 0;
}
@@ -923,27 +923,27 @@ static int start_rx(struct atm_dev *dev)
/*----------------------------------- TX ------------------------------------*/
-enum enq_res { enq_ok,enq_next,enq_jam };
+enum enq_res { enq_ok, enq_next, enq_jam };
-static inline void put_dma(int chan,u32 *dma,int *j,dma_addr_t paddr,
- u32 size)
+static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
+ u32 size)
{
- u32 init,words;
+ u32 init, words;
- DPRINTK("put_dma: 0x%lx+0x%x\n",(unsigned long) paddr,size);
- EVENT("put_dma: 0x%lx+0x%lx\n",(unsigned long) paddr,size);
+ DPRINTK("put_dma: 0x%lx+0x%x\n", (unsigned long) paddr, size);
+ EVENT("put_dma: 0x%lx+0x%lx\n", (unsigned long) paddr, size);
#if 0 /* don't complain anymore */
if (paddr & 3)
- printk(KERN_ERR "put_dma: unaligned addr (0x%lx)\n",paddr);
+ printk(KERN_ERR "put_dma: unaligned addr (0x%lx)\n", paddr);
if (size & 3)
- printk(KERN_ERR "put_dma: unaligned size (0x%lx)\n",size);
+ printk(KERN_ERR "put_dma: unaligned size (0x%lx)\n", size);
#endif
if (paddr & 3) {
init = 4-(paddr & 3);
if (init > size || size < 7) init = size;
DPRINTK("put_dma: %lx DMA: %d/%d bytes\n",
- (unsigned long) paddr,init,size);
+ (unsigned long) paddr, init, size);
dma[(*j)++] = MID_DT_BYTE | (init << MID_DMA_COUNT_SHIFT) |
(chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
@@ -956,7 +956,7 @@ static inline void put_dma(int chan,u32 *dma,int *j,dma_addr_t paddr,
init = 8-((paddr & 31) >> 2);
if (init > words) init = words;
DPRINTK("put_dma: %lx DMA: %d/%d words\n",
- (unsigned long) paddr,init,words);
+ (unsigned long) paddr, init, words);
dma[(*j)++] = MID_DT_WORD | (init << MID_DMA_COUNT_SHIFT) |
(chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
@@ -966,7 +966,7 @@ static inline void put_dma(int chan,u32 *dma,int *j,dma_addr_t paddr,
#ifdef CONFIG_ATM_ENI_BURST_TX_16W /* may work with some PCI chipsets ... */
if (words & ~15) {
DPRINTK("put_dma: %lx DMA: %d*16/%d words\n",
- (unsigned long) paddr,words >> 4,words);
+ (unsigned long) paddr, words >> 4, words);
dma[(*j)++] = MID_DT_16W | ((words >> 4) << MID_DMA_COUNT_SHIFT)
| (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
@@ -977,7 +977,7 @@ static inline void put_dma(int chan,u32 *dma,int *j,dma_addr_t paddr,
#ifdef CONFIG_ATM_ENI_BURST_TX_8W /* recommended */
if (words & ~7) {
DPRINTK("put_dma: %lx DMA: %d*8/%d words\n",
- (unsigned long) paddr,words >> 3,words);
+ (unsigned long) paddr, words >> 3, words);
dma[(*j)++] = MID_DT_8W | ((words >> 3) << MID_DMA_COUNT_SHIFT)
| (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
@@ -988,7 +988,7 @@ static inline void put_dma(int chan,u32 *dma,int *j,dma_addr_t paddr,
#ifdef CONFIG_ATM_ENI_BURST_TX_4W /* probably useless if TX_8W or TX_16W */
if (words & ~3) {
DPRINTK("put_dma: %lx DMA: %d*4/%d words\n",
- (unsigned long) paddr,words >> 2,words);
+ (unsigned long) paddr, words >> 2, words);
dma[(*j)++] = MID_DT_4W | ((words >> 2) << MID_DMA_COUNT_SHIFT)
| (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
@@ -999,7 +999,7 @@ static inline void put_dma(int chan,u32 *dma,int *j,dma_addr_t paddr,
#ifdef CONFIG_ATM_ENI_BURST_TX_2W /* probably useless if TX_4W, TX_8W, ... */
if (words & ~1) {
DPRINTK("put_dma: %lx DMA: %d*2/%d words\n",
- (unsigned long) paddr,words >> 1,words);
+ (unsigned long) paddr, words >> 1, words);
dma[(*j)++] = MID_DT_2W | ((words >> 1) << MID_DMA_COUNT_SHIFT)
| (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
@@ -1008,16 +1008,16 @@ static inline void put_dma(int chan,u32 *dma,int *j,dma_addr_t paddr,
}
#endif
if (words) {
- DPRINTK("put_dma: %lx DMA: %d words\n",(unsigned long) paddr,
- words);
+ DPRINTK("put_dma: %lx DMA: %d words\n", (unsigned long) paddr,
+ words);
dma[(*j)++] = MID_DT_WORD | (words << MID_DMA_COUNT_SHIFT) |
(chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
paddr += words << 2;
}
if (size) {
- DPRINTK("put_dma: %lx DMA: %d bytes\n",(unsigned long) paddr,
- size);
+ DPRINTK("put_dma: %lx DMA: %d bytes\n", (unsigned long) paddr,
+ size);
dma[(*j)++] = MID_DT_BYTE | (size << MID_DMA_COUNT_SHIFT) |
(chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
@@ -1032,13 +1032,13 @@ static enum enq_res do_tx(struct sk_buff *skb)
struct eni_vcc *eni_vcc;
struct eni_tx *tx;
dma_addr_t paddr;
- u32 dma_rd,dma_wr;
+ u32 dma_rd, dma_wr;
u32 size; /* in words */
- int aal5,dma_size,i,j;
+ int aal5, dma_size, i, j;
DPRINTK(">do_tx\n");
NULLCHECK(skb);
- EVENT("do_tx: skb=0x%lx, %ld bytes\n",(unsigned long) skb,skb->len);
+ EVENT("do_tx: skb=0x%lx, %ld bytes\n", (unsigned long) skb, skb->len);
vcc = ATM_SKB(skb)->vcc;
NULLCHECK(vcc);
eni_dev = ENI_DEV(vcc->dev);
@@ -1059,7 +1059,7 @@ static enum enq_res do_tx(struct sk_buff *skb)
#if 0 /* should work now */
if ((unsigned long) skb->data & 3)
printk(KERN_ERR DEV_LABEL "(itf %d): VCI %d has mis-aligned "
- "TX data\n",vcc->dev->number,vcc->vci);
+ "TX data\n", vcc->dev->number, vcc->vci);
#endif
/*
* Potential future IP speedup: make hard_header big enough to put
@@ -1084,10 +1084,10 @@ static enum enq_res do_tx(struct sk_buff *skb)
* to the read pointer ? TX_GAP means to leave some space for what
* the manual calls "too close".
*/
- if (!NEPMOK(tx->tx_pos,size+TX_GAP,
- eni_in(MID_TX_RDPTR(tx->index)),tx->words)) {
+ if (!NEPMOK(tx->tx_pos, size+TX_GAP,
+ eni_in(MID_TX_RDPTR(tx->index)), tx->words)) {
DPRINTK(DEV_LABEL "(itf %d): TX full (size %d)\n",
- vcc->dev->number,size);
+ vcc->dev->number, size);
return enq_next;
}
/* check DMA */
@@ -1095,22 +1095,22 @@ static enum enq_res do_tx(struct sk_buff *skb)
dma_rd = eni_in(MID_DMA_RD_TX);
dma_size = 3; /* JK for descriptor and final fill, plus final size
mis-alignment fix */
-DPRINTK("iovcnt = %d\n",skb_shinfo(skb)->nr_frags);
+DPRINTK("iovcnt = %d\n", skb_shinfo(skb)->nr_frags);
if (!skb_shinfo(skb)->nr_frags) dma_size += 5;
else dma_size += 5*(skb_shinfo(skb)->nr_frags+1);
if (dma_size > TX_DMA_BUF) {
printk(KERN_CRIT DEV_LABEL "(itf %d): needs %d DMA entries "
- "(got only %d)\n",vcc->dev->number,dma_size,TX_DMA_BUF);
+ "(got only %d)\n", vcc->dev->number, dma_size, TX_DMA_BUF);
}
- DPRINTK("dma_wr is %d, tx_pos is %ld\n",dma_wr,tx->tx_pos);
+ DPRINTK("dma_wr is %d, tx_pos is %ld\n", dma_wr, tx->tx_pos);
if (dma_wr != dma_rd && ((dma_rd+NR_DMA_TX-dma_wr) & (NR_DMA_TX-1)) <
dma_size) {
printk(KERN_WARNING DEV_LABEL "(itf %d): TX DMA full\n",
- vcc->dev->number);
+ vcc->dev->number);
return enq_jam;
}
- paddr = pci_map_single(eni_dev->pci_dev,skb->data,skb->len,
- PCI_DMA_TODEVICE);
+ paddr = pci_map_single(eni_dev->pci_dev, skb->data, skb->len,
+ PCI_DMA_TODEVICE);
ENI_PRV_PADDR(skb) = paddr;
/* prepare DMA queue entries */
j = 0;
@@ -1119,20 +1119,20 @@ DPRINTK("iovcnt = %d\n",skb_shinfo(skb)->nr_frags);
MID_DT_JK;
j++;
if (!skb_shinfo(skb)->nr_frags)
- if (aal5) put_dma(tx->index,eni_dev->dma,&j,paddr,skb->len);
- else put_dma(tx->index,eni_dev->dma,&j,paddr+4,skb->len-4);
+ if (aal5) put_dma(tx->index, eni_dev->dma, &j, paddr, skb->len);
+ else put_dma(tx->index, eni_dev->dma, &j, paddr+4, skb->len-4);
else {
-DPRINTK("doing direct send\n"); /* @@@ well, this doesn't work anyway */
+ DPRINTK("doing direct send\n"); /* @@@ well, this doesn't work anyway */
for (i = -1; i < skb_shinfo(skb)->nr_frags; i++)
if (i == -1)
- put_dma(tx->index,eni_dev->dma,&j,(unsigned long)
- skb->data,
- skb_headlen(skb));
+ put_dma(tx->index, eni_dev->dma, &j, (unsigned long)
+ skb->data,
+ skb_headlen(skb));
else
- put_dma(tx->index,eni_dev->dma,&j,(unsigned long)
- skb_frag_page(&skb_shinfo(skb)->frags[i]) +
+ put_dma(tx->index, eni_dev->dma, &j, (unsigned long)
+ skb_frag_page(&skb_shinfo(skb)->frags[i]) +
skb_shinfo(skb)->frags[i].page_offset,
- skb_frag_size(&skb_shinfo(skb)->frags[i]));
+ skb_frag_size(&skb_shinfo(skb)->frags[i]));
}
if (skb->len & 3) {
put_dma(tx->index, eni_dev->dma, &j, eni_dev->zero.dma,
@@ -1143,34 +1143,34 @@ DPRINTK("doing direct send\n"); /* @@@ well, this doesn't work anyway */
MID_DMA_COUNT_SHIFT) | (tx->index << MID_DMA_CHAN_SHIFT) |
MID_DMA_END | MID_DT_JK;
j++;
- DPRINTK("DMA at end: %d\n",j);
+ DPRINTK("DMA at end: %d\n", j);
/* store frame */
writel((MID_SEG_TX_ID << MID_SEG_ID_SHIFT) |
(aal5 ? MID_SEG_AAL5 : 0) | (tx->prescaler << MID_SEG_PR_SHIFT) |
(tx->resolution << MID_SEG_RATE_SHIFT) |
- (size/(ATM_CELL_PAYLOAD/4)),tx->send+tx->tx_pos*4);
+ (size/(ATM_CELL_PAYLOAD/4)), tx->send+tx->tx_pos*4);
/*printk("dsc = 0x%08lx\n",(unsigned long) readl(tx->send+tx->tx_pos*4));*/
writel((vcc->vci << MID_SEG_VCI_SHIFT) |
(aal5 ? 0 : (skb->data[3] & 0xf)) |
(ATM_SKB(skb)->atm_options & ATM_ATMOPT_CLP ? MID_SEG_CLP : 0),
tx->send+((tx->tx_pos+1) & (tx->words-1))*4);
- DPRINTK("size: %d, len:%d\n",size,skb->len);
+ DPRINTK("size: %d, len:%d\n", size, skb->len);
if (aal5)
- writel(skb->len,tx->send+
+ writel(skb->len, tx->send+
((tx->tx_pos+size-AAL5_TRAILER) & (tx->words-1))*4);
j = j >> 1;
for (i = 0; i < j; i++) {
- writel(eni_dev->dma[i*2],eni_dev->tx_dma+dma_wr*8);
- writel(eni_dev->dma[i*2+1],eni_dev->tx_dma+dma_wr*8+4);
+ writel(eni_dev->dma[i*2], eni_dev->tx_dma+dma_wr*8);
+ writel(eni_dev->dma[i*2+1], eni_dev->tx_dma+dma_wr*8+4);
dma_wr = (dma_wr+1) & (NR_DMA_TX-1);
}
ENI_PRV_POS(skb) = tx->tx_pos;
ENI_PRV_SIZE(skb) = size;
ENI_VCC(vcc)->txing += size;
tx->tx_pos = (tx->tx_pos+size) & (tx->words-1);
- DPRINTK("dma_wr set to %d, tx_pos is now %ld\n",dma_wr,tx->tx_pos);
- eni_out(dma_wr,MID_DMA_WR_TX);
- skb_queue_tail(&eni_dev->tx_queue,skb);
+ DPRINTK("dma_wr set to %d, tx_pos is now %ld\n", dma_wr, tx->tx_pos);
+ eni_out(dma_wr, MID_DMA_WR_TX);
+ skb_queue_tail(&eni_dev->tx_queue, skb);
queued++;
return enq_ok;
}
@@ -1191,7 +1191,7 @@ static void poll_tx(struct atm_dev *dev)
res = do_tx(skb);
if (res == enq_ok) continue;
DPRINTK("re-queuing TX PDU\n");
- skb_queue_head(&tx->backlog,skb);
+ skb_queue_head(&tx->backlog, skb);
requeued++;
if (res == enq_jam) return;
break;
@@ -1215,17 +1215,17 @@ static void dequeue_tx(struct atm_dev *dev)
NULLCHECK(vcc);
tx = ENI_VCC(vcc)->tx;
NULLCHECK(ENI_VCC(vcc)->tx);
- DPRINTK("dequeue_tx: next 0x%lx curr 0x%x\n",ENI_PRV_POS(skb),
+ DPRINTK("dequeue_tx: next 0x%lx curr 0x%x\n", ENI_PRV_POS(skb),
(unsigned) eni_in(MID_TX_DESCRSTART(tx->index)));
if (ENI_VCC(vcc)->txing < tx->words && ENI_PRV_POS(skb) ==
eni_in(MID_TX_DESCRSTART(tx->index))) {
- skb_queue_head(&eni_dev->tx_queue,skb);
+ skb_queue_head(&eni_dev->tx_queue, skb);
break;
}
ENI_VCC(vcc)->txing -= ENI_PRV_SIZE(skb);
- pci_unmap_single(eni_dev->pci_dev,ENI_PRV_PADDR(skb),skb->len,
- PCI_DMA_TODEVICE);
- if (vcc->pop) vcc->pop(vcc,skb);
+ pci_unmap_single(eni_dev->pci_dev, ENI_PRV_PADDR(skb), skb->len,
+ PCI_DMA_TODEVICE);
+ if (vcc->pop) vcc->pop(vcc, skb);
else dev_kfree_skb_irq(skb);
atomic_inc(&vcc->stats->tx);
wake_up(&eni_dev->tx_wait);
@@ -1234,7 +1234,7 @@ dma_complete++;
}
-static struct eni_tx *alloc_tx(struct eni_dev *eni_dev,int ubr)
+static struct eni_tx *alloc_tx(struct eni_dev *eni_dev, int ubr)
{
int i;
@@ -1244,10 +1244,10 @@ static struct eni_tx *alloc_tx(struct eni_dev *eni_dev,int ubr)
}
-static int comp_tx(struct eni_dev *eni_dev,int *pcr,int reserved,int *pre,
- int *res,int unlimited)
+static int comp_tx(struct eni_dev *eni_dev, int *pcr, int reserved, int *pre,
+ int *res, int unlimited)
{
- static const int pre_div[] = { 4,16,128,2048 };
+ static const int pre_div[] = { 4, 16, 128, 2048 };
/* 2^(((x+2)^2-(x+2))/2+1) */
if (unlimited) *pre = *res = 0;
@@ -1258,7 +1258,7 @@ static int comp_tx(struct eni_dev *eni_dev,int *pcr,int reserved,int *pre,
for (*pre = 0; *pre < 3; (*pre)++)
if (TS_CLOCK/pre_div[*pre]/64 <= *pcr) break;
div = pre_div[*pre]**pcr;
- DPRINTK("min div %d\n",div);
+ DPRINTK("min div %d\n", div);
*res = TS_CLOCK/div-1;
}
else {
@@ -1269,28 +1269,28 @@ static int comp_tx(struct eni_dev *eni_dev,int *pcr,int reserved,int *pre,
if (TS_CLOCK/pre_div[*pre]/64 > -*pcr) break;
if (*pre < 3) (*pre)++; /* else fail later */
div = pre_div[*pre]*-*pcr;
- DPRINTK("max div %d\n",div);
+ DPRINTK("max div %d\n", div);
*res = DIV_ROUND_UP(TS_CLOCK, div)-1;
}
if (*res < 0) *res = 0;
if (*res > MID_SEG_MAX_RATE) *res = MID_SEG_MAX_RATE;
}
*pcr = TS_CLOCK/pre_div[*pre]/(*res+1);
- DPRINTK("out pcr: %d (%d:%d)\n",*pcr,*pre,*res);
+ DPRINTK("out pcr: %d (%d:%d)\n", *pcr, *pre, *res);
return 0;
}
-static int reserve_or_set_tx(struct atm_vcc *vcc,struct atm_trafprm *txtp,
- int set_rsv,int set_shp)
+static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
+ int set_rsv, int set_shp)
{
struct eni_dev *eni_dev = ENI_DEV(vcc->dev);
struct eni_vcc *eni_vcc = ENI_VCC(vcc);
struct eni_tx *tx;
unsigned long size;
void __iomem *mem;
- int rate,ubr,unlimited,new_tx;
- int pre,res,order;
+ int rate, ubr, unlimited, new_tx;
+ int pre, res, order;
int error;
rate = atm_pcr_goal(txtp);
@@ -1315,26 +1315,26 @@ static int reserve_or_set_tx(struct atm_vcc *vcc,struct atm_trafprm *txtp,
mem = NULL; /* for gcc */
if (!new_tx) tx = eni_vcc->tx;
else {
- mem = eni_alloc_mem(eni_dev,&size);
+ mem = eni_alloc_mem(eni_dev, &size);
if (!mem) return -ENOBUFS;
- tx = alloc_tx(eni_dev,unlimited);
+ tx = alloc_tx(eni_dev, unlimited);
if (!tx) {
- eni_free_mem(eni_dev,mem,size);
+ eni_free_mem(eni_dev, mem, size);
return -EBUSY;
}
- DPRINTK("got chan %d\n",tx->index);
+ DPRINTK("got chan %d\n", tx->index);
tx->reserved = tx->shaping = 0;
tx->send = mem;
tx->words = size >> 2;
skb_queue_head_init(&tx->backlog);
for (order = 0; size > (1 << (order+10)); order++);
eni_out((order << MID_SIZE_SHIFT) |
- ((tx->send-eni_dev->ram) >> (MID_LOC_SKIP+2)),
- MID_TX_PLACE(tx->index));
+ ((tx->send-eni_dev->ram) >> (MID_LOC_SKIP+2)),
+ MID_TX_PLACE(tx->index));
tx->tx_pos = eni_in(MID_TX_DESCRSTART(tx->index)) &
MID_DESCR_START;
}
- error = comp_tx(eni_dev,&rate,tx->reserved,&pre,&res,unlimited);
+ error = comp_tx(eni_dev, &rate, tx->reserved, &pre, &res, unlimited);
if (!error && txtp->min_pcr > rate) error = -EINVAL;
if (!error && txtp->max_pcr && txtp->max_pcr != ATM_MAX_PCR &&
txtp->max_pcr < rate) error = -EINVAL;
@@ -1347,7 +1347,7 @@ static int reserve_or_set_tx(struct atm_vcc *vcc,struct atm_trafprm *txtp,
if (error) {
if (new_tx) {
tx->send = NULL;
- eni_free_mem(eni_dev,mem,size);
+ eni_free_mem(eni_dev, mem, size);
}
return error;
}
@@ -1364,7 +1364,7 @@ static int reserve_or_set_tx(struct atm_vcc *vcc,struct atm_trafprm *txtp,
tx->shaping = rate;
}
if (set_shp) eni_vcc->tx = tx;
- DPRINTK("rsv %d shp %d\n",tx->reserved,tx->shaping);
+ DPRINTK("rsv %d shp %d\n", tx->reserved, tx->shaping);
return 0;
}
@@ -1374,7 +1374,7 @@ static int open_tx_first(struct atm_vcc *vcc)
ENI_VCC(vcc)->tx = NULL;
if (vcc->qos.txtp.traffic_class == ATM_NONE) return 0;
ENI_VCC(vcc)->txing = 0;
- return reserve_or_set_tx(vcc,&vcc->qos.txtp,1,1);
+ return reserve_or_set_tx(vcc, &vcc->qos.txtp, 1, 1);
}
@@ -1386,7 +1386,7 @@ static int open_tx_second(struct atm_vcc *vcc)
static void close_tx(struct atm_vcc *vcc)
{
- DECLARE_WAITQUEUE(wait,current);
+ DECLARE_WAITQUEUE(wait, current);
struct eni_dev *eni_dev;
struct eni_vcc *eni_vcc;
@@ -1395,7 +1395,7 @@ static void close_tx(struct atm_vcc *vcc)
eni_dev = ENI_DEV(vcc->dev);
/* wait for TX queue to drain */
DPRINTK("eni_close: waiting for TX ...\n");
- add_wait_queue(&eni_dev->tx_wait,&wait);
+ add_wait_queue(&eni_dev->tx_wait, &wait);
set_current_state(TASK_UNINTERRUPTIBLE);
for (;;) {
int txing;
@@ -1404,12 +1404,12 @@ static void close_tx(struct atm_vcc *vcc)
txing = skb_peek(&eni_vcc->tx->backlog) || eni_vcc->txing;
tasklet_enable(&eni_dev->task);
if (!txing) break;
- DPRINTK("%d TX left\n",eni_vcc->txing);
+ DPRINTK("%d TX left\n", eni_vcc->txing);
schedule();
set_current_state(TASK_UNINTERRUPTIBLE);
}
set_current_state(TASK_RUNNING);
- remove_wait_queue(&eni_dev->tx_wait,&wait);
+ remove_wait_queue(&eni_dev->tx_wait, &wait);
if (eni_vcc->tx != eni_dev->ubr) {
/*
* Looping a few times in here is probably far cheaper than
@@ -1419,7 +1419,7 @@ static void close_tx(struct atm_vcc *vcc)
while (eni_in(MID_TX_RDPTR(eni_vcc->tx->index)) !=
eni_in(MID_TX_DESCRSTART(eni_vcc->tx->index)))
schedule();
- eni_free_mem(eni_dev,eni_vcc->tx->send,eni_vcc->tx->words << 2);
+ eni_free_mem(eni_dev, eni_vcc->tx->send, eni_vcc->tx->words << 2);
eni_vcc->tx->send = NULL;
eni_dev->tx_bw += eni_vcc->tx->reserved;
}
@@ -1439,7 +1439,7 @@ static int start_tx(struct atm_dev *dev)
init_waitqueue_head(&eni_dev->tx_wait);
eni_dev->ubr = NULL;
skb_queue_head_init(&eni_dev->tx_queue);
- eni_out(0,MID_DMA_WR_TX);
+ eni_out(0, MID_DMA_WR_TX);
for (i = 0; i < NR_CHAN; i++) {
eni_dev->tx[i].send = NULL;
eni_dev->tx[i].index = i;
@@ -1458,33 +1458,33 @@ static void foo(void)
printk(KERN_INFO
"tx_complete=%d,dma_complete=%d,queued=%d,requeued=%d,sub=%d,\n"
"backlogged=%d,rx_enqueued=%d,rx_dequeued=%d,putting=%d,pushed=%d\n",
- tx_complete,dma_complete,queued,requeued,submitted,backlogged,
- rx_enqueued,rx_dequeued,putting,pushed);
-if (eni_boards) printk(KERN_INFO "loss: %ld\n",ENI_DEV(eni_boards)->lost);
+ tx_complete, dma_complete, queued, requeued, submitted, backlogged,
+ rx_enqueued, rx_dequeued, putting, pushed);
+if (eni_boards) printk(KERN_INFO "loss: %ld\n", ENI_DEV(eni_boards)->lost);
}
#endif
-static void bug_int(struct atm_dev *dev,unsigned long reason)
+static void bug_int(struct atm_dev *dev, unsigned long reason)
{
DPRINTK(">bug_int\n");
if (reason & MID_DMA_ERR_ACK)
printk(KERN_CRIT DEV_LABEL "(itf %d): driver error - DMA "
- "error\n",dev->number);
+ "error\n", dev->number);
if (reason & MID_TX_IDENT_MISM)
printk(KERN_CRIT DEV_LABEL "(itf %d): driver error - ident "
- "mismatch\n",dev->number);
+ "mismatch\n", dev->number);
if (reason & MID_TX_DMA_OVFL)
printk(KERN_CRIT DEV_LABEL "(itf %d): driver error - DMA "
- "overflow\n",dev->number);
- EVENT("---dump ends here---\n",0,0);
+ "overflow\n", dev->number);
+ EVENT("---dump ends here---\n", 0, 0);
printk(KERN_NOTICE "---recent events---\n");
event_dump();
}
-static irqreturn_t eni_int(int irq,void *dev_id)
+static irqreturn_t eni_int(int irq, void *dev_id)
{
struct atm_dev *dev;
struct eni_dev *eni_dev;
@@ -1494,7 +1494,7 @@ static irqreturn_t eni_int(int irq,void *dev_id)
dev = dev_id;
eni_dev = ENI_DEV(dev);
reason = eni_in(MID_ISA);
- DPRINTK(DEV_LABEL ": int 0x%lx\n",(unsigned long) reason);
+ DPRINTK(DEV_LABEL ": int 0x%lx\n", (unsigned long) reason);
/*
* Must handle these two right now, because reading ISA doesn't clear
* them, so they re-occur and we never make it to the tasklet. Since
@@ -1502,11 +1502,11 @@ static irqreturn_t eni_int(int irq,void *dev_id)
* with eni_dev->events == 0.
*/
if (reason & MID_STAT_OVFL) {
- EVENT("stat overflow\n",0,0);
+ EVENT("stat overflow\n", 0, 0);
eni_dev->lost += eni_in(MID_STAT) & MID_OVFL_TRASH;
}
if (reason & MID_SUNI_INT) {
- EVENT("SUNI int\n",0,0);
+ EVENT("SUNI int\n", 0, 0);
dev->phy->interrupt(dev);
#if 0
foo();
@@ -1527,38 +1527,38 @@ static void eni_tasklet(unsigned long data)
unsigned long flags;
u32 events;
- DPRINTK("eni_tasklet (dev %p)\n",dev);
- spin_lock_irqsave(&eni_dev->lock,flags);
- events = xchg(&eni_dev->events,0);
- spin_unlock_irqrestore(&eni_dev->lock,flags);
+ DPRINTK("eni_tasklet (dev %p)\n", dev);
+ spin_lock_irqsave(&eni_dev->lock, flags);
+ events = xchg(&eni_dev->events, 0);
+ spin_unlock_irqrestore(&eni_dev->lock, flags);
if (events & MID_RX_DMA_COMPLETE) {
- EVENT("INT: RX DMA complete, starting dequeue_rx\n",0,0);
+ EVENT("INT: RX DMA complete, starting dequeue_rx\n", 0, 0);
dequeue_rx(dev);
- EVENT("dequeue_rx done, starting poll_rx\n",0,0);
+ EVENT("dequeue_rx done, starting poll_rx\n", 0, 0);
poll_rx(dev);
- EVENT("poll_rx done\n",0,0);
+ EVENT("poll_rx done\n", 0, 0);
/* poll_tx ? */
}
if (events & MID_SERVICE) {
- EVENT("INT: service, starting get_service\n",0,0);
+ EVENT("INT: service, starting get_service\n", 0, 0);
get_service(dev);
- EVENT("get_service done, starting poll_rx\n",0,0);
+ EVENT("get_service done, starting poll_rx\n", 0, 0);
poll_rx(dev);
- EVENT("poll_rx done\n",0,0);
+ EVENT("poll_rx done\n", 0, 0);
}
if (events & MID_TX_DMA_COMPLETE) {
- EVENT("INT: TX DMA COMPLETE\n",0,0);
+ EVENT("INT: TX DMA COMPLETE\n", 0, 0);
dequeue_tx(dev);
}
if (events & MID_TX_COMPLETE) {
- EVENT("INT: TX COMPLETE\n",0,0);
+ EVENT("INT: TX COMPLETE\n", 0, 0);
tx_complete++;
wake_up(&eni_dev->tx_wait);
/* poll_rx ? */
}
if (events & (MID_DMA_ERR_ACK | MID_TX_IDENT_MISM | MID_TX_DMA_OVFL)) {
- EVENT("bug interrupt\n",0,0);
- bug_int(dev,events);
+ EVENT("bug interrupt\n", 0, 0);
+ bug_int(dev, events);
}
poll_tx(dev);
}
@@ -1570,7 +1570,7 @@ tx_complete++;
static char * const media_name[] = {
"MMF", "SMF", "MMF", "03?", /* 0- 3 */
"UTP", "05?", "06?", "07?", /* 4- 7 */
- "TAXI","09?", "10?", "11?", /* 8-11 */
+ "TAXI", "09?", "10?", "11?", /* 8-11 */
"12?", "13?", "14?", "15?", /* 12-15 */
"MMF", "SMF", "18?", "19?", /* 16-19 */
"UTP", "21?", "22?", "23?", /* 20-23 */
@@ -1581,12 +1581,12 @@ static char * const media_name[] = {
#define SET_SEPROM \
({ if (!error && !pci_error) { \
- pci_error = pci_write_config_byte(eni_dev->pci_dev,PCI_TONGA_CTRL,tonga); \
+ pci_error = pci_write_config_byte(eni_dev->pci_dev, PCI_TONGA_CTRL, tonga); \
udelay(10); /* 10 usecs */ \
} })
#define GET_SEPROM \
({ if (!error && !pci_error) { \
- pci_error = pci_read_config_byte(eni_dev->pci_dev,PCI_TONGA_CTRL,&tonga); \
+ pci_error = pci_read_config_byte(eni_dev->pci_dev, PCI_TONGA_CTRL, &tonga); \
udelay(10); /* 10 usecs */ \
} })
@@ -1595,8 +1595,8 @@ static int get_esi_asic(struct atm_dev *dev)
{
struct eni_dev *eni_dev;
unsigned char tonga;
- int error,failed,pci_error;
- int address,i,j;
+ int error, failed, pci_error;
+ int address, i, j;
eni_dev = ENI_DEV(dev);
error = pci_error = 0;
@@ -1672,7 +1672,7 @@ static int get_esi_asic(struct atm_dev *dev)
}
if (pci_error) {
printk(KERN_ERR DEV_LABEL "(itf %d): error reading ESI "
- "(0x%02x)\n",dev->number,pci_error);
+ "(0x%02x)\n", dev->number, pci_error);
error = -EIO;
}
return error;
@@ -1701,7 +1701,7 @@ static int eni_do_init(struct atm_dev *dev)
struct pci_dev *pci_dev;
unsigned long real_base;
void __iomem *base;
- int error,i,last;
+ int error, i, last;
DPRINTK(">eni_init\n");
dev->ci_range.vpi_bits = 0;
@@ -1711,19 +1711,19 @@ static int eni_do_init(struct atm_dev *dev)
pci_dev = eni_dev->pci_dev;
real_base = pci_resource_start(pci_dev, 0);
eni_dev->irq = pci_dev->irq;
- if ((error = pci_write_config_word(pci_dev,PCI_COMMAND,
+ if ((error = pci_write_config_word(pci_dev, PCI_COMMAND,
PCI_COMMAND_MEMORY |
(eni_dev->asic ? PCI_COMMAND_PARITY | PCI_COMMAND_SERR : 0)))) {
printk(KERN_ERR DEV_LABEL "(itf %d): can't enable memory "
- "(0x%02x)\n",dev->number,error);
+ "(0x%02x)\n", dev->number, error);
return -EIO;
}
printk(KERN_NOTICE DEV_LABEL "(itf %d): rev.%d,base=0x%lx,irq=%d,",
- dev->number,pci_dev->revision,real_base,eni_dev->irq);
- if (!(base = ioremap_nocache(real_base,MAP_MAX_SIZE))) {
+ dev->number, pci_dev->revision, real_base, eni_dev->irq);
+ if (!(base = ioremap_nocache(real_base, MAP_MAX_SIZE))) {
printk("\n");
printk(KERN_ERR DEV_LABEL "(itf %d): can't set up page "
- "mapping\n",dev->number);
+ "mapping\n", dev->number);
return error;
}
eni_dev->ioaddr = base;
@@ -1746,35 +1746,35 @@ static int eni_do_init(struct atm_dev *dev)
eni_dev->ram = base+RAM_BASE;
last = MAP_MAX_SIZE-RAM_BASE;
for (i = last-RAM_INCREMENT; i >= 0; i -= RAM_INCREMENT) {
- writel(0x55555555,eni_dev->ram+i);
+ writel(0x55555555, eni_dev->ram+i);
if (readl(eni_dev->ram+i) != 0x55555555) last = i;
else {
- writel(0xAAAAAAAA,eni_dev->ram+i);
+ writel(0xAAAAAAAA, eni_dev->ram+i);
if (readl(eni_dev->ram+i) != 0xAAAAAAAA) last = i;
- else writel(i,eni_dev->ram+i);
+ else writel(i, eni_dev->ram+i);
}
}
for (i = 0; i < last; i += RAM_INCREMENT)
if (readl(eni_dev->ram+i) != i) break;
eni_dev->mem = i;
- memset_io(eni_dev->ram,0,eni_dev->mem);
+ memset_io(eni_dev->ram, 0, eni_dev->mem);
/* TODO: should shrink allocation now */
- printk("mem=%dkB (",eni_dev->mem >> 10);
+ printk("mem=%dkB (", eni_dev->mem >> 10);
/* TODO: check for non-SUNI, check for TAXI ? */
if (!(eni_in(MID_RES_ID_MCON) & 0x200) != !eni_dev->asic) {
printk(")\n");
printk(KERN_ERR DEV_LABEL "(itf %d): ERROR - wrong id 0x%x\n",
- dev->number,(unsigned) eni_in(MID_RES_ID_MCON));
+ dev->number, (unsigned) eni_in(MID_RES_ID_MCON));
error = -EINVAL;
goto unmap;
}
- error = eni_dev->asic ? get_esi_asic(dev) : get_esi_fpga(dev,base);
+ error = eni_dev->asic ? get_esi_asic(dev) : get_esi_fpga(dev, base);
if (error)
goto unmap;
for (i = 0; i < ESI_LEN; i++)
- printk("%s%02X",i ? "-" : "",dev->esi[i]);
+ printk("%s%02X", i ? "-" : "", dev->esi[i]);
printk(")\n");
- printk(KERN_NOTICE DEV_LABEL "(itf %d): %s,%s\n",dev->number,
+ printk(KERN_NOTICE DEV_LABEL "(itf %d): %s,%s\n", dev->number,
eni_in(MID_RES_ID_MCON) & 0x200 ? "ASIC" : "FPGA",
media_name[eni_in(MID_RES_ID_MCON) & DAUGTHER_ID]);
@@ -1807,24 +1807,24 @@ static int eni_start(struct atm_dev *dev)
DPRINTK(">eni_start\n");
eni_dev = ENI_DEV(dev);
- if (request_irq(eni_dev->irq,&eni_int,IRQF_SHARED,DEV_LABEL,dev)) {
+ if (request_irq(eni_dev->irq, &eni_int, IRQF_SHARED, DEV_LABEL, dev)) {
printk(KERN_ERR DEV_LABEL "(itf %d): IRQ%d is already in use\n",
- dev->number,eni_dev->irq);
+ dev->number, eni_dev->irq);
error = -EAGAIN;
goto out;
}
pci_set_master(eni_dev->pci_dev);
- if ((error = pci_write_config_word(eni_dev->pci_dev,PCI_COMMAND,
+ if ((error = pci_write_config_word(eni_dev->pci_dev, PCI_COMMAND,
PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER |
(eni_dev->asic ? PCI_COMMAND_PARITY | PCI_COMMAND_SERR : 0)))) {
printk(KERN_ERR DEV_LABEL "(itf %d): can't enable memory+"
- "master (0x%02x)\n",dev->number,error);
+ "master (0x%02x)\n", dev->number, error);
goto free_irq;
}
- if ((error = pci_write_config_byte(eni_dev->pci_dev,PCI_TONGA_CTRL,
- END_SWAP_DMA))) {
+ if ((error = pci_write_config_byte(eni_dev->pci_dev, PCI_TONGA_CTRL,
+ END_SWAP_DMA))) {
printk(KERN_ERR DEV_LABEL "(itf %d): can't set endian swap "
- "(0x%02x)\n",dev->number,error);
+ "(0x%02x)\n", dev->number, error);
goto free_irq;
}
/* determine addresses of internal tables */
@@ -1834,16 +1834,16 @@ static int eni_start(struct atm_dev *dev)
eni_dev->service = eni_dev->tx_dma+NR_DMA_TX*8;
buf = eni_dev->service+NR_SERVICE*4;
DPRINTK("vci 0x%lx,rx 0x%lx, tx 0x%lx,srv 0x%lx,buf 0x%lx\n",
- eni_dev->vci,eni_dev->rx_dma,eni_dev->tx_dma,
- eni_dev->service,buf);
+ eni_dev->vci, eni_dev->rx_dma, eni_dev->tx_dma,
+ eni_dev->service, buf);
spin_lock_init(&eni_dev->lock);
- tasklet_init(&eni_dev->task,eni_tasklet,(unsigned long) dev);
+ tasklet_init(&eni_dev->task, eni_tasklet, (unsigned long) dev);
eni_dev->events = 0;
/* initialize memory management */
buffer_mem = eni_dev->mem - (buf - eni_dev->ram);
eni_dev->free_list_size = buffer_mem/MID_MIN_BUF_SIZE/2;
eni_dev->free_list = kmalloc(
- sizeof(struct eni_free)*(eni_dev->free_list_size+1),GFP_KERNEL);
+ sizeof(struct eni_free)*(eni_dev->free_list_size+1), GFP_KERNEL);
if (!eni_dev->free_list) {
printk(KERN_ERR DEV_LABEL "(itf %d): couldn't get free page\n",
dev->number);
@@ -1851,8 +1851,8 @@ static int eni_start(struct atm_dev *dev)
goto free_irq;
}
eni_dev->free_len = 0;
- eni_put_free(eni_dev,buf,buffer_mem);
- memset_io(eni_dev->vci,0,16*NR_VCI); /* clear VCI table */
+ eni_put_free(eni_dev, buf, buffer_mem);
+ memset_io(eni_dev->vci, 0, 16*NR_VCI); /* clear VCI table */
/*
* byte_addr free (k)
* 0x00000000 512 VCI table
@@ -1862,7 +1862,7 @@ static int eni_start(struct atm_dev *dev)
* 0x00007000 484 buffers
* 0x00080000 0 end (512kB)
*/
- eni_out(0xffffffff,MID_IE);
+ eni_out(0xffffffff, MID_IE);
error = start_tx(dev);
if (error) goto free_list;
error = start_rx(dev);
@@ -1891,14 +1891,14 @@ static void eni_close(struct atm_vcc *vcc)
{
DPRINTK(">eni_close\n");
if (!ENI_VCC(vcc)) return;
- clear_bit(ATM_VF_READY,&vcc->flags);
+ clear_bit(ATM_VF_READY, &vcc->flags);
close_rx(vcc);
close_tx(vcc);
DPRINTK("eni_close: done waiting\n");
/* deallocate memory */
kfree(ENI_VCC(vcc));
vcc->dev_data = NULL;
- clear_bit(ATM_VF_ADDR,&vcc->flags);
+ clear_bit(ATM_VF_ADDR, &vcc->flags);
/*foo();*/
}
@@ -1911,17 +1911,17 @@ static int eni_open(struct atm_vcc *vcc)
int vci = vcc->vci;
DPRINTK(">eni_open\n");
- EVENT("eni_open\n",0,0);
- if (!test_bit(ATM_VF_PARTIAL,&vcc->flags))
+ EVENT("eni_open\n", 0, 0);
+ if (!test_bit(ATM_VF_PARTIAL, &vcc->flags))
vcc->dev_data = NULL;
if (vci != ATM_VPI_UNSPEC && vpi != ATM_VCI_UNSPEC)
- set_bit(ATM_VF_ADDR,&vcc->flags);
+ set_bit(ATM_VF_ADDR, &vcc->flags);
if (vcc->qos.aal != ATM_AAL0 && vcc->qos.aal != ATM_AAL5)
return -EINVAL;
- DPRINTK(DEV_LABEL "(itf %d): open %d.%d\n",vcc->dev->number,vcc->vpi,
- vcc->vci);
- if (!test_bit(ATM_VF_PARTIAL,&vcc->flags)) {
- eni_vcc = kmalloc(sizeof(struct eni_vcc),GFP_KERNEL);
+ DPRINTK(DEV_LABEL "(itf %d): open %d.%d\n", vcc->dev->number, vcc->vpi,
+ vcc->vci);
+ if (!test_bit(ATM_VF_PARTIAL, &vcc->flags)) {
+ eni_vcc = kmalloc(sizeof(struct eni_vcc), GFP_KERNEL);
if (!eni_vcc) return -ENOMEM;
vcc->dev_data = eni_vcc;
eni_vcc->tx = NULL; /* for eni_close after open_rx */
@@ -1943,18 +1943,18 @@ static int eni_open(struct atm_vcc *vcc)
eni_close(vcc);
return error;
}
- set_bit(ATM_VF_READY,&vcc->flags);
+ set_bit(ATM_VF_READY, &vcc->flags);
/* should power down SUNI while !ref_count @@@ */
return 0;
}
-static int eni_change_qos(struct atm_vcc *vcc,struct atm_qos *qos,int flgs)
+static int eni_change_qos(struct atm_vcc *vcc, struct atm_qos *qos, int flgs)
{
struct eni_dev *eni_dev = ENI_DEV(vcc->dev);
struct eni_tx *tx = ENI_VCC(vcc)->tx;
struct sk_buff *skb;
- int error,rate,rsv,shp;
+ int error, rate, rsv, shp;
if (qos->txtp.traffic_class == ATM_NONE) return 0;
if (tx == eni_dev->ubr) return -EBADFD;
@@ -1966,7 +1966,7 @@ static int eni_change_qos(struct atm_vcc *vcc,struct atm_qos *qos,int flgs)
if ((flgs & ATM_MF_DEC_SHP) && rate && rate < tx->shaping) shp = 1;
if ((flgs & ATM_MF_INC_SHP) && (!rate || rate > tx->shaping)) shp = 1;
if (!rsv && !shp) return 0;
- error = reserve_or_set_tx(vcc,&qos->txtp,rsv,shp);
+ error = reserve_or_set_tx(vcc, &qos->txtp, rsv, shp);
if (error) return error;
if (shp && !(flgs & ATM_MF_IMMED)) return 0;
/*
@@ -1988,14 +1988,14 @@ static int eni_change_qos(struct atm_vcc *vcc,struct atm_qos *qos,int flgs)
}
-static int eni_ioctl(struct atm_dev *dev,unsigned int cmd,void __user *arg)
+static int eni_ioctl(struct atm_dev *dev, unsigned int cmd, void __user *arg)
{
struct eni_dev *eni_dev = ENI_DEV(dev);
if (cmd == ENI_MEMDUMP) {
if (!capable(CAP_NET_ADMIN)) return -EPERM;
printk(KERN_WARNING "Please use /proc/atm/" DEV_LABEL ":%d "
- "instead of obsolete ioctl ENI_MEMDUMP\n",dev->number);
+ "instead of obsolete ioctl ENI_MEMDUMP\n", dev->number);
dump(dev);
return 0;
}
@@ -2016,7 +2016,7 @@ static int eni_ioctl(struct atm_dev *dev,unsigned int cmd,void __user *arg)
if (cmd == ATM_SETCIRANGE) {
struct atm_cirange ci;
- if (copy_from_user(&ci, arg,sizeof(struct atm_cirange)))
+ if (copy_from_user(&ci, arg, sizeof(struct atm_cirange)))
return -EFAULT;
if ((ci.vpi_bits == 0 || ci.vpi_bits == ATM_CI_MAX) &&
(ci.vci_bits == NR_VCI_LD || ci.vpi_bits == ATM_CI_MAX))
@@ -2024,42 +2024,42 @@ static int eni_ioctl(struct atm_dev *dev,unsigned int cmd,void __user *arg)
return -EINVAL;
}
if (!dev->phy->ioctl) return -ENOIOCTLCMD;
- return dev->phy->ioctl(dev,cmd,arg);
+ return dev->phy->ioctl(dev, cmd, arg);
}
-static int eni_getsockopt(struct atm_vcc *vcc,int level,int optname,
- void __user *optval,int optlen)
+static int eni_getsockopt(struct atm_vcc *vcc, int level, int optname,
+ void __user *optval, int optlen)
{
return -EINVAL;
}
-static int eni_setsockopt(struct atm_vcc *vcc,int level,int optname,
- void __user *optval,unsigned int optlen)
+static int eni_setsockopt(struct atm_vcc *vcc, int level, int optname,
+ void __user *optval, unsigned int optlen)
{
return -EINVAL;
}
-static int eni_send(struct atm_vcc *vcc,struct sk_buff *skb)
+static int eni_send(struct atm_vcc *vcc, struct sk_buff *skb)
{
enum enq_res res;
DPRINTK(">eni_send\n");
if (!ENI_VCC(vcc)->tx) {
- if (vcc->pop) vcc->pop(vcc,skb);
+ if (vcc->pop) vcc->pop(vcc, skb);
else dev_kfree_skb(skb);
return -EINVAL;
}
if (!skb) {
printk(KERN_CRIT "!skb in eni_send ?\n");
- if (vcc->pop) vcc->pop(vcc,skb);
+ if (vcc->pop) vcc->pop(vcc, skb);
return -EINVAL;
}
if (vcc->qos.aal == ATM_AAL0) {
if (skb->len != ATM_CELL_SIZE-1) {
- if (vcc->pop) vcc->pop(vcc,skb);
+ if (vcc->pop) vcc->pop(vcc, skb);
else dev_kfree_skb(skb);
return -EINVAL;
}
@@ -2071,42 +2071,42 @@ submitted++;
res = do_tx(skb);
tasklet_enable(&ENI_DEV(vcc->dev)->task);
if (res == enq_ok) return 0;
- skb_queue_tail(&ENI_VCC(vcc)->tx->backlog,skb);
+ skb_queue_tail(&ENI_VCC(vcc)->tx->backlog, skb);
backlogged++;
tasklet_schedule(&ENI_DEV(vcc->dev)->task);
return 0;
}
-static void eni_phy_put(struct atm_dev *dev,unsigned char value,
- unsigned long addr)
+static void eni_phy_put(struct atm_dev *dev, unsigned char value,
+ unsigned long addr)
{
- writel(value,ENI_DEV(dev)->phy+addr*4);
+ writel(value, ENI_DEV(dev)->phy+addr*4);
}
-static unsigned char eni_phy_get(struct atm_dev *dev,unsigned long addr)
+static unsigned char eni_phy_get(struct atm_dev *dev, unsigned long addr)
{
return readl(ENI_DEV(dev)->phy+addr*4);
}
-static int eni_proc_read(struct atm_dev *dev,loff_t *pos,char *page)
+static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
{
struct hlist_node *node;
struct sock *s;
- static const char *signal[] = { "LOST","unknown","okay" };
+ static const char *signal[] = { "LOST", "unknown", "okay" };
struct eni_dev *eni_dev = ENI_DEV(dev);
struct atm_vcc *vcc;
- int left,i;
+ int left, i;
left = *pos;
if (!left)
- return sprintf(page,DEV_LABEL "(itf %d) signal %s, %dkB, "
- "%d cps remaining\n",dev->number,signal[(int) dev->signal],
- eni_dev->mem >> 10,eni_dev->tx_bw);
+ return sprintf(page, DEV_LABEL "(itf %d) signal %s, %dkB, "
+ "%d cps remaining\n", dev->number, signal[(int) dev->signal],
+ eni_dev->mem >> 10, eni_dev->tx_bw);
if (!--left)
- return sprintf(page,"%4sBursts: TX"
+ return sprintf(page, "%4sBursts: TX"
#if !defined(CONFIG_ATM_ENI_BURST_TX_16W) && \
!defined(CONFIG_ATM_ENI_BURST_TX_8W) && \
!defined(CONFIG_ATM_ENI_BURST_TX_4W) && \
@@ -2147,24 +2147,24 @@ static int eni_proc_read(struct atm_dev *dev,loff_t *pos,char *page)
#ifndef CONFIG_ATM_ENI_TUNE_BURST
" (default)"
#endif
- "\n","");
+ "\n", "");
if (!--left)
- return sprintf(page,"%4sBuffer multipliers: tx %d%%, rx %d%%\n",
- "",eni_dev->tx_mult,eni_dev->rx_mult);
+ return sprintf(page, "%4sBuffer multipliers: tx %d%%, rx %d%%\n",
+ "", eni_dev->tx_mult, eni_dev->rx_mult);
for (i = 0; i < NR_CHAN; i++) {
struct eni_tx *tx = eni_dev->tx+i;
if (!tx->send) continue;
if (!--left) {
- return sprintf(page,"tx[%d]: 0x%ld-0x%ld "
- "(%6ld bytes), rsv %d cps, shp %d cps%s\n",i,
+ return sprintf(page, "tx[%d]: 0x%ld-0x%ld "
+ "(%6ld bytes), rsv %d cps, shp %d cps%s\n", i,
(unsigned long) (tx->send - eni_dev->ram),
- tx->send-eni_dev->ram+tx->words*4-1,tx->words*4,
- tx->reserved,tx->shaping,
+ tx->send-eni_dev->ram+tx->words*4-1, tx->words*4,
+ tx->reserved, tx->shaping,
tx == eni_dev->ubr ? " (UBR)" : "");
}
if (--left) continue;
- return sprintf(page,"%10sbacklog %u packets\n","",
+ return sprintf(page, "%10sbacklog %u packets\n", "",
skb_queue_len(&tx->backlog));
}
read_lock(&vcc_sklist_lock);
@@ -2180,18 +2180,18 @@ static int eni_proc_read(struct atm_dev *dev,loff_t *pos,char *page)
continue;
eni_vcc = ENI_VCC(vcc);
if (--left) continue;
- length = sprintf(page,"vcc %4d: ",vcc->vci);
+ length = sprintf(page, "vcc %4d: ", vcc->vci);
if (eni_vcc->rx) {
- length += sprintf(page+length,"0x%ld-0x%ld "
+ length += sprintf(page+length, "0x%ld-0x%ld "
"(%6ld bytes)",
(unsigned long) (eni_vcc->recv - eni_dev->ram),
eni_vcc->recv-eni_dev->ram+eni_vcc->words*4-1,
eni_vcc->words*4);
- if (eni_vcc->tx) length += sprintf(page+length,", ");
+ if (eni_vcc->tx) length += sprintf(page+length, ", ");
}
if (eni_vcc->tx)
- length += sprintf(page+length,"tx[%d], txing %d bytes",
- eni_vcc->tx->index,eni_vcc->txing);
+ length += sprintf(page+length, "tx[%d], txing %d bytes",
+ eni_vcc->tx->index, eni_vcc->txing);
page[length] = '\n';
read_unlock(&vcc_sklist_lock);
return length+1;
@@ -2204,8 +2204,8 @@ static int eni_proc_read(struct atm_dev *dev,loff_t *pos,char *page)
if (--left) continue;
offset = (unsigned long) eni_dev->ram+eni_dev->base_diff;
- return sprintf(page,"free %p-%p (%6d bytes)\n",
- fe->start-offset,fe->start-offset+(1 << fe->order)-1,
+ return sprintf(page, "free %p-%p (%6d bytes)\n",
+ fe->start-offset, fe->start-offset+(1 << fe->order)-1,
1 << fe->order);
}
return 0;
@@ -2289,7 +2289,7 @@ static struct pci_device_id eni_pci_tbl[] = {
{ PCI_VDEVICE(EF, PCI_DEVICE_ID_EF_ATM_ASIC), 1 /* ASIC */ },
{ 0, }
};
-MODULE_DEVICE_TABLE(pci,eni_pci_tbl);
+MODULE_DEVICE_TABLE(pci, eni_pci_tbl);
static void eni_remove_one(struct pci_dev *pdev)
@@ -2320,7 +2320,7 @@ static int __init eni_init(void)
if (sizeof(skb->cb) < sizeof(struct eni_skb_prv)) {
printk(KERN_ERR "eni_detect: skb->cb is too small (%Zd < %Zd)\n",
- sizeof(skb->cb),sizeof(struct eni_skb_prv));
+ sizeof(skb->cb), sizeof(struct eni_skb_prv));
return -EIO;
}
return pci_register_driver(&eni_driver);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/5] drivers: atm: checkpatch.pl fixes removed all trailing whitespaces in eni.c
2013-01-20 23:12 [PATCH 1/5] drivers: atm: checkpatch.pl fixed coding style errors in eni.c Patrik Karlin
@ 2013-01-20 23:12 ` Patrik Karlin
2013-01-20 23:12 ` [PATCH 3/5] drivers: atm: checkpatch.pl fixed tabbed indentation " Patrik Karlin
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Patrik Karlin @ 2013-01-20 23:12 UTC (permalink / raw)
To: chas; +Cc: linux-atm-general, netdev, linux-kernel, Patrik Karlin
This patch removes all trailing whitespace's in eni.c
as sugested by checkpatch.pl
Signed-off-by: Patrik Kårlin <patrik.karlin@gmail.com>
---
drivers/atm/eni.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/atm/eni.c b/drivers/atm/eni.c
index 6ae0d89..1d5adc2 100644
--- a/drivers/atm/eni.c
+++ b/drivers/atm/eni.c
@@ -1,7 +1,7 @@
/* drivers/atm/eni.c - Efficient Networks ENI155P device driver */
-
+
/* Written 1995-2000 by Werner Almesberger, EPFL LRC/ICA */
-
+
#include <linux/module.h>
#include <linux/kernel.h>
@@ -33,7 +33,7 @@
#if !defined(__i386__) && !defined(__x86_64__)
#ifndef ioremap_nocache
#define ioremap_nocache(X, Y) ioremap(X, Y)
-#endif
+#endif
#endif
/*
@@ -90,7 +90,7 @@ static void event_dump(void)
#else
-/*
+/*
* NULL pointer checking
*/
@@ -112,7 +112,7 @@ static int ec = 0;
static void EVENT(const char *s, unsigned long a, unsigned long b)
{
- ev[ec] = s;
+ ev[ec] = s;
ev_a[ec] = a;
ev_b[ec] = b;
ec = (ec+1) % EV;
@@ -1800,7 +1800,7 @@ static void eni_do_release(struct atm_dev *dev)
static int eni_start(struct atm_dev *dev)
{
struct eni_dev *eni_dev;
-
+
void __iomem *buf;
unsigned long buffer_mem;
int error;
@@ -2148,7 +2148,7 @@ static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
" (default)"
#endif
"\n", "");
- if (!--left)
+ if (!--left)
return sprintf(page, "%4sBuffer multipliers: tx %d%%, rx %d%%\n",
"", eni_dev->tx_mult, eni_dev->rx_mult);
for (i = 0; i < NR_CHAN; i++) {
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/5] drivers: atm: checkpatch.pl fixed tabbed indentation in eni.c
2013-01-20 23:12 [PATCH 1/5] drivers: atm: checkpatch.pl fixed coding style errors in eni.c Patrik Karlin
2013-01-20 23:12 ` [PATCH 2/5] drivers: atm: checkpatch.pl fixes removed all trailing whitespaces " Patrik Karlin
@ 2013-01-20 23:12 ` Patrik Karlin
2013-01-20 23:12 ` [PATCH 4/5] drivers: atm: checkpatch.pl changed headers from asm/* to linux/* " Patrik Karlin
2013-01-20 23:12 ` [PATCH 5/5] drivers: atm: checkpatch.pl fixed coding style issues " Patrik Karlin
3 siblings, 0 replies; 6+ messages in thread
From: Patrik Karlin @ 2013-01-20 23:12 UTC (permalink / raw)
To: chas; +Cc: linux-atm-general, netdev, linux-kernel, Patrik Karlin
This removes all dont use spaces att beginning of lines errors
from checkpatch.pl
Signed-off-by: Patrik Kårlin <patrik.karlin@gmail.com>
---
drivers/atm/eni.c | 512 ++++++++++++++++++++++++++---------------------------
1 file changed, 256 insertions(+), 256 deletions(-)
diff --git a/drivers/atm/eni.c b/drivers/atm/eni.c
index 1d5adc2..f4a136a 100644
--- a/drivers/atm/eni.c
+++ b/drivers/atm/eni.c
@@ -94,8 +94,8 @@ static void event_dump(void)
* NULL pointer checking
*/
-#define NULLCHECK(x) \
- if ((unsigned long) (x) < 0x30) \
+#define NULLCHECK(x) \
+ if ((unsigned long) (x) < 0x30) \
printk(KERN_CRIT #x "==0x%lx\n", (unsigned long) (x))
/*
@@ -135,23 +135,23 @@ static void event_dump(void)
/*
- * NExx must not be equal at end
- * EExx may be equal at end
+ * NExx must not be equal at end
+ * EExx may be equal at end
* xxPJOK verify validity of pointer jumps
* xxPMOK operating on a circular buffer of "c" words
*/
-#define NEPJOK(a0, a1, b) \
- ((a0) < (a1) ? (b) <= (a0) || (b) > (a1) : (b) <= (a0) && (b) > (a1))
-#define EEPJOK(a0, a1, b) \
- ((a0) < (a1) ? (b) < (a0) || (b) >= (a1) : (b) < (a0) && (b) >= (a1))
+#define NEPJOK(a0, a1, b) \
+ ((a0) < (a1) ? (b) <= (a0) || (b) > (a1) : (b) <= (a0) && (b) > (a1))
+#define EEPJOK(a0, a1, b) \
+ ((a0) < (a1) ? (b) < (a0) || (b) >= (a1) : (b) < (a0) && (b) >= (a1))
#define NEPMOK(a0, d, b, c) NEPJOK(a0, (a0+d) & (c-1), b)
#define EEPMOK(a0, d, b, c) EEPJOK(a0, (a0+d) & (c-1), b)
static int tx_complete = 0, dma_complete = 0, queued = 0, requeued = 0,
- backlogged = 0, rx_enqueued = 0, rx_dequeued = 0, pushed = 0, submitted = 0,
- putting = 0;
+ backlogged = 0, rx_enqueued = 0, rx_dequeued = 0, pushed = 0, submitted = 0,
+ putting = 0;
static struct atm_dev *eni_boards = NULL;
@@ -166,11 +166,11 @@ static struct atm_dev *eni_boards = NULL;
static void dump_mem(struct eni_dev *eni_dev)
{
int i;
-
+
for (i = 0; i < eni_dev->free_len; i++)
printk(KERN_DEBUG " %d: %p %d\n", i,
- eni_dev->free_list[i].start,
- 1 << eni_dev->free_list[i].order);
+ eni_dev->free_list[i].start,
+ 1 << eni_dev->free_list[i].order);
}
@@ -187,19 +187,19 @@ static void dump(struct atm_dev *dev)
for (i = 0; i < NR_CHAN; i++)
if (eni_dev->tx[i].send)
printk(KERN_NOTICE " TX %d @ %p: %ld\n", i,
- eni_dev->tx[i].send, eni_dev->tx[i].words*4);
+ eni_dev->tx[i].send, eni_dev->tx[i].words*4);
printk(KERN_NOTICE "RX buffers\n");
for (i = 0; i < 1024; i++)
if (eni_dev->rx_map[i] && ENI_VCC(eni_dev->rx_map[i])->rx)
printk(KERN_NOTICE " RX %d @ %p: %ld\n", i,
- ENI_VCC(eni_dev->rx_map[i])->recv,
- ENI_VCC(eni_dev->rx_map[i])->words*4);
+ ENI_VCC(eni_dev->rx_map[i])->recv,
+ ENI_VCC(eni_dev->rx_map[i])->words*4);
printk(KERN_NOTICE "----\n");
}
static void eni_put_free(struct eni_dev *eni_dev, void __iomem *start,
- unsigned long size)
+ unsigned long size)
{
struct eni_free *list;
int len, order;
@@ -211,13 +211,13 @@ static void eni_put_free(struct eni_dev *eni_dev, void __iomem *start,
while (size) {
if (len >= eni_dev->free_list_size) {
printk(KERN_CRIT "eni_put_free overflow (%p,%ld)\n",
- start, size);
+ start, size);
break;
}
for (order = 0; !(((unsigned long)start | size) & (1 << order)); order++);
if (MID_MIN_BUF_SIZE > (1 << order)) {
printk(KERN_CRIT "eni_put_free: order %d too small\n",
- order);
+ order);
break;
}
list[len].start = (void __iomem *) start;
@@ -252,9 +252,9 @@ static void __iomem *eni_alloc_mem(struct eni_dev *eni_dev, unsigned long *size)
break;
}
else if (best_order > list[i].order && list[i].order > order) {
- best_order = list[i].order;
- index = i;
- }
+ best_order = list[i].order;
+ index = i;
+ }
if (best_order == 65) return NULL;
start = list[index].start-eni_dev->base_diff;
list[index] = list[--len];
@@ -262,14 +262,14 @@ static void __iomem *eni_alloc_mem(struct eni_dev *eni_dev, unsigned long *size)
*size = 1 << order;
eni_put_free(eni_dev, start+*size, (1 << best_order)-*size);
DPRINTK("%ld bytes (order %d) at 0x%lx\n", *size, order, start);
- memset_io(start, 0, *size); /* never leak data */
+ memset_io(start, 0, *size); /* never leak data */
/*dump_mem(eni_dev);*/
return start;
}
static void eni_free_mem(struct eni_dev *eni_dev, void __iomem *start,
- unsigned long size)
+ unsigned long size)
{
struct eni_free *list;
int len, i, order;
@@ -283,7 +283,7 @@ static void eni_free_mem(struct eni_dev *eni_dev, void __iomem *start,
if (((unsigned long) list[i].start) == ((unsigned long)start^(1 << order)) &&
list[i].order == order) {
DPRINTK("match[%d]: 0x%lx/0x%lx(0x%x), %d/%d\n", i,
- list[i].start, start, 1 << order, list[i].order, order);
+ list[i].start, start, 1 << order, list[i].order, order);
list[i] = list[--len];
start = (void __iomem *) ((unsigned long) start & ~(unsigned long) (1 << order));
order++;
@@ -292,7 +292,7 @@ static void eni_free_mem(struct eni_dev *eni_dev, void __iomem *start,
}
if (len >= eni_dev->free_list_size) {
printk(KERN_ALERT "eni_free_mem overflow (%p,%d)\n", start,
- order);
+ order);
return;
}
list[len].start = start;
@@ -318,18 +318,18 @@ static void rx_ident_err(struct atm_vcc *vcc)
eni_dev = ENI_DEV(dev);
/* immediately halt adapter */
eni_out(eni_in(MID_MC_S) &
- ~(MID_DMA_ENABLE | MID_TX_ENABLE | MID_RX_ENABLE), MID_MC_S);
+ ~(MID_DMA_ENABLE | MID_TX_ENABLE | MID_RX_ENABLE), MID_MC_S);
/* dump useful information */
eni_vcc = ENI_VCC(vcc);
printk(KERN_ALERT DEV_LABEL "(itf %d): driver error - RX ident "
- "mismatch\n", dev->number);
+ "mismatch\n", dev->number);
printk(KERN_ALERT " VCI %d, rxing %d, words %ld\n", vcc->vci,
- eni_vcc->rxing, eni_vcc->words);
+ eni_vcc->rxing, eni_vcc->words);
printk(KERN_ALERT " host descr 0x%lx, rx pos 0x%lx, descr value "
- "0x%x\n", eni_vcc->descr, eni_vcc->rx_pos,
- (unsigned) readl(eni_vcc->recv+eni_vcc->descr*4));
+ "0x%x\n", eni_vcc->descr, eni_vcc->rx_pos,
+ (unsigned) readl(eni_vcc->recv+eni_vcc->descr*4));
printk(KERN_ALERT " last %p, servicing %d\n", eni_vcc->last,
- eni_vcc->servicing);
+ eni_vcc->servicing);
EVENT("---dump ends here---\n", 0, 0);
printk(KERN_NOTICE "---recent events---\n");
event_dump();
@@ -340,7 +340,7 @@ static void rx_ident_err(struct atm_vcc *vcc)
static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
- unsigned long skip, unsigned long size, unsigned long eff)
+ unsigned long skip, unsigned long size, unsigned long eff)
{
struct eni_dev *eni_dev;
struct eni_vcc *eni_vcc;
@@ -355,21 +355,21 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
paddr = 0; /* GCC, shut up */
if (skb) {
paddr = pci_map_single(eni_dev->pci_dev, skb->data, skb->len,
- PCI_DMA_FROMDEVICE);
+ PCI_DMA_FROMDEVICE);
ENI_PRV_PADDR(skb) = paddr;
if (paddr & 3)
printk(KERN_CRIT DEV_LABEL "(itf %d): VCI %d has "
- "mis-aligned RX data (0x%lx)\n", vcc->dev->number,
- vcc->vci, (unsigned long) paddr);
+ "mis-aligned RX data (0x%lx)\n", vcc->dev->number,
+ vcc->vci, (unsigned long) paddr);
ENI_PRV_SIZE(skb) = size+skip;
- /* PDU plus descriptor */
+ /* PDU plus descriptor */
ATM_SKB(skb)->vcc = vcc;
}
j = 0;
if ((eff && skip) || 1) { /* @@@ actually, skip is always == 1 ... */
here = (eni_vcc->descr+skip) & (eni_vcc->words-1);
dma[j++] = (here << MID_DMA_COUNT_SHIFT) | (vcc->vci
- << MID_DMA_VCI_SHIFT) | MID_DT_JK;
+ << MID_DMA_VCI_SHIFT) | MID_DT_JK;
j++;
}
here = (eni_vcc->descr+size+skip) & (eni_vcc->words-1);
@@ -380,7 +380,7 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
if (!size) {
DPRINTK("strange things happen ...\n");
EVENT("strange things happen ... (skip=%ld,eff=%ld)\n",
- size, eff);
+ size, eff);
}
words = eff;
if (paddr & 15) {
@@ -389,7 +389,7 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
init = 4-((paddr & 15) >> 2);
if (init > words) init = words;
dma[j++] = MID_DT_WORD | (init << MID_DMA_COUNT_SHIFT) |
- (vcc->vci << MID_DMA_VCI_SHIFT);
+ (vcc->vci << MID_DMA_VCI_SHIFT);
dma[j++] = paddr;
paddr += init << 2;
words -= init;
@@ -397,8 +397,8 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
#ifdef CONFIG_ATM_ENI_BURST_RX_16W /* may work with some PCI chipsets ... */
if (words & ~15) {
dma[j++] = MID_DT_16W | ((words >> 4) <<
- MID_DMA_COUNT_SHIFT) | (vcc->vci <<
- MID_DMA_VCI_SHIFT);
+ MID_DMA_COUNT_SHIFT) | (vcc->vci <<
+ MID_DMA_VCI_SHIFT);
dma[j++] = paddr;
paddr += (words & ~15) << 2;
words &= 15;
@@ -407,8 +407,8 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
#ifdef CONFIG_ATM_ENI_BURST_RX_8W /* works only with *some* PCI chipsets ... */
if (words & ~7) {
dma[j++] = MID_DT_8W | ((words >> 3) <<
- MID_DMA_COUNT_SHIFT) | (vcc->vci <<
- MID_DMA_VCI_SHIFT);
+ MID_DMA_COUNT_SHIFT) | (vcc->vci <<
+ MID_DMA_VCI_SHIFT);
dma[j++] = paddr;
paddr += (words & ~7) << 2;
words &= 7;
@@ -417,8 +417,8 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
#ifdef CONFIG_ATM_ENI_BURST_RX_4W /* recommended */
if (words & ~3) {
dma[j++] = MID_DT_4W | ((words >> 2) <<
- MID_DMA_COUNT_SHIFT) | (vcc->vci <<
- MID_DMA_VCI_SHIFT);
+ MID_DMA_COUNT_SHIFT) | (vcc->vci <<
+ MID_DMA_VCI_SHIFT);
dma[j++] = paddr;
paddr += (words & ~3) << 2;
words &= 3;
@@ -427,8 +427,8 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
#ifdef CONFIG_ATM_ENI_BURST_RX_2W /* probably useless if RX_4W, RX_8W, ... */
if (words & ~1) {
dma[j++] = MID_DT_2W | ((words >> 1) <<
- MID_DMA_COUNT_SHIFT) | (vcc->vci <<
- MID_DMA_VCI_SHIFT);
+ MID_DMA_COUNT_SHIFT) | (vcc->vci <<
+ MID_DMA_VCI_SHIFT);
dma[j++] = paddr;
paddr += (words & ~1) << 2;
words &= 1;
@@ -436,13 +436,13 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
#endif
if (words) {
dma[j++] = MID_DT_WORD | (words << MID_DMA_COUNT_SHIFT)
- | (vcc->vci << MID_DMA_VCI_SHIFT);
+ | (vcc->vci << MID_DMA_VCI_SHIFT);
dma[j++] = paddr;
}
}
if (size != eff) {
dma[j++] = (here << MID_DMA_COUNT_SHIFT) |
- (vcc->vci << MID_DMA_VCI_SHIFT) | MID_DT_JK;
+ (vcc->vci << MID_DMA_VCI_SHIFT) | MID_DT_JK;
j++;
}
if (!j || j > 2*RX_DMA_BUF) {
@@ -459,19 +459,19 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
*/
if (!NEPMOK(dma_wr, j+j+1, dma_rd, NR_DMA_RX)) { /* @@@ +1 is ugly */
printk(KERN_WARNING DEV_LABEL "(itf %d): RX DMA full\n",
- vcc->dev->number);
+ vcc->dev->number);
goto trouble;
}
- for (i = 0; i < j; i++) {
+ for (i = 0; i < j; i++) {
writel(dma[i*2], eni_dev->rx_dma+dma_wr*8);
writel(dma[i*2+1], eni_dev->rx_dma+dma_wr*8+4);
dma_wr = (dma_wr+1) & (NR_DMA_RX-1);
- }
+ }
if (skb) {
ENI_PRV_POS(skb) = eni_vcc->descr+size+1;
skb_queue_tail(&eni_dev->rx_queue, skb);
eni_vcc->last = skb;
-rx_enqueued++;
+ rx_enqueued++;
}
eni_vcc->descr = here;
eni_out(dma_wr, MID_DMA_WR_RX);
@@ -480,7 +480,7 @@ rx_enqueued++;
trouble:
if (paddr)
pci_unmap_single(eni_dev->pci_dev, paddr, skb->len,
- PCI_DMA_FROMDEVICE);
+ PCI_DMA_FROMDEVICE);
if (skb) dev_kfree_skb_irq(skb);
return -1;
}
@@ -493,7 +493,7 @@ static void discard(struct atm_vcc *vcc, unsigned long size)
eni_vcc = ENI_VCC(vcc);
EVENT("discard (size=%ld)\n", size, 0);
while (do_rx_dma(vcc, NULL, 1, size, 0)) EVENT("BUSY LOOP", 0, 0);
- /* could do a full fallback, but that might be more expensive */
+ /* could do a full fallback, but that might be more expensive */
if (eni_vcc->rxing) ENI_PRV_POS(eni_vcc->last) += size+1;
else eni_vcc->rx_pos = (eni_vcc->rx_pos+size+1) & (eni_vcc->words-1);
}
@@ -520,7 +520,7 @@ static int rx_aal0(struct atm_vcc *vcc)
}
if (descr & MID_RED_T) {
DPRINTK(DEV_LABEL "(itf %d): trashing empty cell\n",
- vcc->dev->number);
+ vcc->dev->number);
length = 0;
atomic_inc(&vcc->stats->rx_err);
}
@@ -560,7 +560,7 @@ static int rx_aal5(struct atm_vcc *vcc)
if (descr & MID_RED_T) {
EVENT("empty cell (descr=0x%lx)\n", descr, 0);
DPRINTK(DEV_LABEL "(itf %d): trashing empty cell\n",
- vcc->dev->number);
+ vcc->dev->number);
size = 0;
}
else {
@@ -568,13 +568,13 @@ static int rx_aal5(struct atm_vcc *vcc)
if (time_after(jiffies, silence) || silence == 0) {
printk(KERN_WARNING DEV_LABEL "(itf %d): "
- "discarding PDU(s) with CRC error\n",
- vcc->dev->number);
+ "discarding PDU(s) with CRC error\n",
+ vcc->dev->number);
silence = (jiffies+2*HZ)|1;
}
size = (descr & MID_RED_COUNT)*(ATM_CELL_PAYLOAD >> 2);
EVENT("CRC error (descr=0x%lx,size=%ld)\n", descr,
- size);
+ size);
}
eff = length = 0;
atomic_inc(&vcc->stats->rx_err);
@@ -583,16 +583,16 @@ static int rx_aal5(struct atm_vcc *vcc)
size = (descr & MID_RED_COUNT)*(ATM_CELL_PAYLOAD >> 2);
DPRINTK("size=%ld\n", size);
length = readl(eni_vcc->recv+(((eni_vcc->descr+size-1) &
- (eni_vcc->words-1)))*4) & 0xffff;
- /* -trailer(2)+header(1) */
+ (eni_vcc->words-1)))*4) & 0xffff;
+ /* -trailer(2)+header(1) */
if (length && length <= (size << 2)-8 && length <=
- ATM_MAX_AAL5_PDU) eff = (length+3) >> 2;
+ ATM_MAX_AAL5_PDU) eff = (length+3) >> 2;
else { /* ^ trailer length (8) */
EVENT("bad PDU (descr=0x08%lx,length=%ld)\n", descr,
- length);
+ length);
printk(KERN_ERR DEV_LABEL "(itf %d): bad AAL5 PDU "
- "(VCI=%d,length=%ld,size=%ld (descr 0x%lx))\n",
- vcc->dev->number, vcc->vci, length, size << 2, descr);
+ "(VCI=%d,length=%ld,size=%ld (descr 0x%lx))\n",
+ vcc->dev->number, vcc->vci, length, size << 2, descr);
length = eff = 0;
atomic_inc(&vcc->stats->rx_err);
}
@@ -620,12 +620,12 @@ static inline int rx_vcc(struct atm_vcc *vcc)
vci_dsc = ENI_DEV(vcc->dev)->vci+vcc->vci*16;
EVENT("rx_vcc(1)\n", 0, 0);
while (eni_vcc->descr != (tmp = (readl(vci_dsc+4) & MID_VCI_DESCR) >>
- MID_VCI_DESCR_SHIFT)) {
+ MID_VCI_DESCR_SHIFT)) {
EVENT("rx_vcc(2: host dsc=0x%lx, nic dsc=0x%lx)\n",
- eni_vcc->descr, tmp);
+ eni_vcc->descr, tmp);
DPRINTK("CB_DESCR %ld REG_DESCR %d\n", ENI_VCC(vcc)->descr,
- (((unsigned) readl(vci_dsc+4) & MID_VCI_DESCR) >>
- MID_VCI_DESCR_SHIFT));
+ (((unsigned) readl(vci_dsc+4) & MID_VCI_DESCR) >>
+ MID_VCI_DESCR_SHIFT));
if (ENI_VCC(vcc)->rx(vcc)) return 1;
}
/* clear IN_SERVICE flag */
@@ -637,12 +637,12 @@ static inline int rx_vcc(struct atm_vcc *vcc)
*/
EVENT("rx_vcc(3)\n", 0, 0);
while (ENI_VCC(vcc)->descr != (tmp = (readl(vci_dsc+4) & MID_VCI_DESCR)
- >> MID_VCI_DESCR_SHIFT)) {
+ >> MID_VCI_DESCR_SHIFT)) {
EVENT("rx_vcc(4: host dsc=0x%lx, nic dsc=0x%lx)\n",
- eni_vcc->descr, tmp);
+ eni_vcc->descr, tmp);
DPRINTK("CB_DESCR %ld REG_DESCR %d\n", ENI_VCC(vcc)->descr,
- (((unsigned) readl(vci_dsc+4) & MID_VCI_DESCR) >>
- MID_VCI_DESCR_SHIFT));
+ (((unsigned) readl(vci_dsc+4) & MID_VCI_DESCR) >>
+ MID_VCI_DESCR_SHIFT));
if (ENI_VCC(vcc)->rx(vcc)) return 1;
}
return 0;
@@ -688,7 +688,7 @@ static void get_service(struct atm_dev *dev)
vcc = eni_dev->rx_map[vci & 1023];
if (!vcc) {
printk(KERN_CRIT DEV_LABEL "(itf %d): VCI %ld not "
- "found\n", dev->number, vci);
+ "found\n", dev->number, vci);
continue; /* nasty but we try to go on anyway */
/* @@@ nope, doesn't work */
}
@@ -712,7 +712,7 @@ static void get_service(struct atm_dev *dev)
else eni_dev->slow = vcc;
eni_dev->last_slow = vcc;
}
-putting++;
+ putting++;
ENI_VCC(vcc)->servicing++;
}
}
@@ -734,21 +734,21 @@ static void dequeue_rx(struct atm_dev *dev)
if (!skb) {
if (first) {
DPRINTK(DEV_LABEL "(itf %d): RX but not "
- "rxing\n", dev->number);
+ "rxing\n", dev->number);
EVENT("nothing to dequeue\n", 0, 0);
}
break;
}
EVENT("dequeued (size=%ld,pos=0x%lx)\n", ENI_PRV_SIZE(skb),
- ENI_PRV_POS(skb));
-rx_dequeued++;
+ ENI_PRV_POS(skb));
+ rx_dequeued++;
vcc = ATM_SKB(skb)->vcc;
eni_vcc = ENI_VCC(vcc);
first = 0;
vci_dsc = eni_dev->vci+vcc->vci*16;
if (!EEPMOK(eni_vcc->rx_pos, ENI_PRV_SIZE(skb),
- (readl(vci_dsc+4) & MID_VCI_READ) >> MID_VCI_READ_SHIFT,
- eni_vcc->words)) {
+ (readl(vci_dsc+4) & MID_VCI_READ) >> MID_VCI_READ_SHIFT,
+ eni_vcc->words)) {
EVENT("requeuing\n", 0, 0);
skb_queue_head(&eni_dev->rx_queue, skb);
break;
@@ -756,13 +756,13 @@ rx_dequeued++;
eni_vcc->rxing--;
eni_vcc->rx_pos = ENI_PRV_POS(skb) & (eni_vcc->words-1);
pci_unmap_single(eni_dev->pci_dev, ENI_PRV_PADDR(skb), skb->len,
- PCI_DMA_TODEVICE);
+ PCI_DMA_TODEVICE);
if (!skb->len) dev_kfree_skb_irq(skb);
else {
EVENT("pushing (len=%ld)\n", skb->len, 0);
if (vcc->qos.aal == ATM_AAL0)
*(unsigned long *) skb->data =
- ntohl(*(unsigned long *) skb->data);
+ ntohl(*(unsigned long *) skb->data);
memset(skb->cb, 0, sizeof(struct eni_skb_prv));
vcc->push(vcc, skb);
pushed++;
@@ -823,12 +823,12 @@ static int open_rx_second(struct atm_vcc *vcc)
writel(0, here+8); /* write, state, count = 0 */
if (eni_dev->rx_map[vcc->vci])
printk(KERN_CRIT DEV_LABEL "(itf %d): BUG - VCI %d already "
- "in use\n", vcc->dev->number, vcc->vci);
+ "in use\n", vcc->dev->number, vcc->vci);
eni_dev->rx_map[vcc->vci] = vcc; /* now it counts */
writel(((vcc->qos.aal != ATM_AAL5 ? MID_MODE_RAW : MID_MODE_AAL5) <<
- MID_VCI_MODE_SHIFT) | MID_VCI_PTI_MODE |
- (((eni_vcc->recv-eni_dev->ram) >> (MID_LOC_SKIP+2)) <<
- MID_VCI_LOCATION_SHIFT) | (order << MID_VCI_SIZE_SHIFT), here);
+ MID_VCI_MODE_SHIFT) | MID_VCI_PTI_MODE |
+ (((eni_vcc->recv-eni_dev->ram) >> (MID_LOC_SKIP+2)) <<
+ MID_VCI_LOCATION_SHIFT) | (order << MID_VCI_SIZE_SHIFT), here);
return 0;
}
@@ -847,7 +847,7 @@ static void close_rx(struct atm_vcc *vcc)
here = eni_dev->vci+vcc->vci*16;
/* block receiver */
writel((readl(here) & ~MID_VCI_MODE) | (MID_MODE_TRASH <<
- MID_VCI_MODE_SHIFT), here);
+ MID_VCI_MODE_SHIFT), here);
/* wait for receiver to become idle */
udelay(27);
/* discard pending cell */
@@ -867,9 +867,9 @@ static void close_rx(struct atm_vcc *vcc)
if (!eni_vcc->rxing) break;
}
EVENT("drain PDUs (rx %ld, serv %ld)\n", eni_vcc->rxing,
- eni_vcc->servicing);
+ eni_vcc->servicing);
printk(KERN_INFO "%d+%d RX left\n", eni_vcc->servicing,
- eni_vcc->rxing);
+ eni_vcc->rxing);
schedule();
set_current_state(TASK_UNINTERRUPTIBLE);
}
@@ -883,9 +883,9 @@ static void close_rx(struct atm_vcc *vcc)
tasklet_enable(&eni_dev->task);
if (at_end) break;
EVENT("drain discard (host 0x%lx, nic 0x%lx)\n",
- eni_vcc->rx_pos, tmp);
+ eni_vcc->rx_pos, tmp);
printk(KERN_INFO "draining RX: host 0x%lx, nic 0x%x\n",
- eni_vcc->rx_pos, tmp);
+ eni_vcc->rx_pos, tmp);
schedule();
set_current_state(TASK_UNINTERRUPTIBLE);
}
@@ -905,7 +905,7 @@ static int start_rx(struct atm_dev *dev)
eni_dev->rx_map = (struct atm_vcc **) get_zeroed_page(GFP_KERNEL);
if (!eni_dev->rx_map) {
printk(KERN_ERR DEV_LABEL "(itf %d): couldn't get free page\n",
- dev->number);
+ dev->number);
free_page((unsigned long) eni_dev->free_list);
return -ENOMEM;
}
@@ -927,7 +927,7 @@ enum enq_res { enq_ok, enq_next, enq_jam };
static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
- u32 size)
+ u32 size)
{
u32 init, words;
@@ -943,9 +943,9 @@ static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
init = 4-(paddr & 3);
if (init > size || size < 7) init = size;
DPRINTK("put_dma: %lx DMA: %d/%d bytes\n",
- (unsigned long) paddr, init, size);
+ (unsigned long) paddr, init, size);
dma[(*j)++] = MID_DT_BYTE | (init << MID_DMA_COUNT_SHIFT) |
- (chan << MID_DMA_CHAN_SHIFT);
+ (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
paddr += init;
size -= init;
@@ -956,9 +956,9 @@ static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
init = 8-((paddr & 31) >> 2);
if (init > words) init = words;
DPRINTK("put_dma: %lx DMA: %d/%d words\n",
- (unsigned long) paddr, init, words);
+ (unsigned long) paddr, init, words);
dma[(*j)++] = MID_DT_WORD | (init << MID_DMA_COUNT_SHIFT) |
- (chan << MID_DMA_CHAN_SHIFT);
+ (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
paddr += init << 2;
words -= init;
@@ -966,9 +966,9 @@ static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
#ifdef CONFIG_ATM_ENI_BURST_TX_16W /* may work with some PCI chipsets ... */
if (words & ~15) {
DPRINTK("put_dma: %lx DMA: %d*16/%d words\n",
- (unsigned long) paddr, words >> 4, words);
+ (unsigned long) paddr, words >> 4, words);
dma[(*j)++] = MID_DT_16W | ((words >> 4) << MID_DMA_COUNT_SHIFT)
- | (chan << MID_DMA_CHAN_SHIFT);
+ | (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
paddr += (words & ~15) << 2;
words &= 15;
@@ -977,9 +977,9 @@ static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
#ifdef CONFIG_ATM_ENI_BURST_TX_8W /* recommended */
if (words & ~7) {
DPRINTK("put_dma: %lx DMA: %d*8/%d words\n",
- (unsigned long) paddr, words >> 3, words);
+ (unsigned long) paddr, words >> 3, words);
dma[(*j)++] = MID_DT_8W | ((words >> 3) << MID_DMA_COUNT_SHIFT)
- | (chan << MID_DMA_CHAN_SHIFT);
+ | (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
paddr += (words & ~7) << 2;
words &= 7;
@@ -988,9 +988,9 @@ static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
#ifdef CONFIG_ATM_ENI_BURST_TX_4W /* probably useless if TX_8W or TX_16W */
if (words & ~3) {
DPRINTK("put_dma: %lx DMA: %d*4/%d words\n",
- (unsigned long) paddr, words >> 2, words);
+ (unsigned long) paddr, words >> 2, words);
dma[(*j)++] = MID_DT_4W | ((words >> 2) << MID_DMA_COUNT_SHIFT)
- | (chan << MID_DMA_CHAN_SHIFT);
+ | (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
paddr += (words & ~3) << 2;
words &= 3;
@@ -999,9 +999,9 @@ static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
#ifdef CONFIG_ATM_ENI_BURST_TX_2W /* probably useless if TX_4W, TX_8W, ... */
if (words & ~1) {
DPRINTK("put_dma: %lx DMA: %d*2/%d words\n",
- (unsigned long) paddr, words >> 1, words);
+ (unsigned long) paddr, words >> 1, words);
dma[(*j)++] = MID_DT_2W | ((words >> 1) << MID_DMA_COUNT_SHIFT)
- | (chan << MID_DMA_CHAN_SHIFT);
+ | (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
paddr += (words & ~1) << 2;
words &= 1;
@@ -1009,17 +1009,17 @@ static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
#endif
if (words) {
DPRINTK("put_dma: %lx DMA: %d words\n", (unsigned long) paddr,
- words);
+ words);
dma[(*j)++] = MID_DT_WORD | (words << MID_DMA_COUNT_SHIFT) |
- (chan << MID_DMA_CHAN_SHIFT);
+ (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
paddr += words << 2;
}
if (size) {
DPRINTK("put_dma: %lx DMA: %d bytes\n", (unsigned long) paddr,
- size);
+ size);
dma[(*j)++] = MID_DT_BYTE | (size << MID_DMA_COUNT_SHIFT) |
- (chan << MID_DMA_CHAN_SHIFT);
+ (chan << MID_DMA_CHAN_SHIFT);
dma[(*j)++] = paddr;
}
}
@@ -1059,7 +1059,7 @@ static enum enq_res do_tx(struct sk_buff *skb)
#if 0 /* should work now */
if ((unsigned long) skb->data & 3)
printk(KERN_ERR DEV_LABEL "(itf %d): VCI %d has mis-aligned "
- "TX data\n", vcc->dev->number, vcc->vci);
+ "TX data\n", vcc->dev->number, vcc->vci);
#endif
/*
* Potential future IP speedup: make hard_header big enough to put
@@ -1071,13 +1071,13 @@ static enum enq_res do_tx(struct sk_buff *skb)
/* check space in buffer */
if (!aal5)
size = (ATM_CELL_PAYLOAD >> 2)+TX_DESCR_SIZE;
- /* cell without HEC plus segmentation header (includes
- four-byte cell header) */
+ /* cell without HEC plus segmentation header (includes
+ four-byte cell header) */
else {
size = skb->len+4*AAL5_TRAILER+ATM_CELL_PAYLOAD-1;
- /* add AAL5 trailer */
+ /* add AAL5 trailer */
size = ((size-(size % ATM_CELL_PAYLOAD)) >> 2)+TX_DESCR_SIZE;
- /* add segmentation header */
+ /* add segmentation header */
}
/*
* Can I move tx_pos by size bytes without getting closer than TX_GAP
@@ -1085,9 +1085,9 @@ static enum enq_res do_tx(struct sk_buff *skb)
* the manual calls "too close".
*/
if (!NEPMOK(tx->tx_pos, size+TX_GAP,
- eni_in(MID_TX_RDPTR(tx->index)), tx->words)) {
+ eni_in(MID_TX_RDPTR(tx->index)), tx->words)) {
DPRINTK(DEV_LABEL "(itf %d): TX full (size %d)\n",
- vcc->dev->number, size);
+ vcc->dev->number, size);
return enq_next;
}
/* check DMA */
@@ -1095,44 +1095,44 @@ static enum enq_res do_tx(struct sk_buff *skb)
dma_rd = eni_in(MID_DMA_RD_TX);
dma_size = 3; /* JK for descriptor and final fill, plus final size
mis-alignment fix */
-DPRINTK("iovcnt = %d\n", skb_shinfo(skb)->nr_frags);
+ DPRINTK("iovcnt = %d\n", skb_shinfo(skb)->nr_frags);
if (!skb_shinfo(skb)->nr_frags) dma_size += 5;
else dma_size += 5*(skb_shinfo(skb)->nr_frags+1);
if (dma_size > TX_DMA_BUF) {
printk(KERN_CRIT DEV_LABEL "(itf %d): needs %d DMA entries "
- "(got only %d)\n", vcc->dev->number, dma_size, TX_DMA_BUF);
+ "(got only %d)\n", vcc->dev->number, dma_size, TX_DMA_BUF);
}
DPRINTK("dma_wr is %d, tx_pos is %ld\n", dma_wr, tx->tx_pos);
if (dma_wr != dma_rd && ((dma_rd+NR_DMA_TX-dma_wr) & (NR_DMA_TX-1)) <
- dma_size) {
+ dma_size) {
printk(KERN_WARNING DEV_LABEL "(itf %d): TX DMA full\n",
- vcc->dev->number);
+ vcc->dev->number);
return enq_jam;
}
paddr = pci_map_single(eni_dev->pci_dev, skb->data, skb->len,
- PCI_DMA_TODEVICE);
+ PCI_DMA_TODEVICE);
ENI_PRV_PADDR(skb) = paddr;
/* prepare DMA queue entries */
j = 0;
eni_dev->dma[j++] = (((tx->tx_pos+TX_DESCR_SIZE) & (tx->words-1)) <<
- MID_DMA_COUNT_SHIFT) | (tx->index << MID_DMA_CHAN_SHIFT) |
- MID_DT_JK;
+ MID_DMA_COUNT_SHIFT) | (tx->index << MID_DMA_CHAN_SHIFT) |
+ MID_DT_JK;
j++;
if (!skb_shinfo(skb)->nr_frags)
if (aal5) put_dma(tx->index, eni_dev->dma, &j, paddr, skb->len);
else put_dma(tx->index, eni_dev->dma, &j, paddr+4, skb->len-4);
else {
- DPRINTK("doing direct send\n"); /* @@@ well, this doesn't work anyway */
+ DPRINTK("doing direct send\n"); /* @@@ well, this doesn't work anyway */
for (i = -1; i < skb_shinfo(skb)->nr_frags; i++)
if (i == -1)
put_dma(tx->index, eni_dev->dma, &j, (unsigned long)
- skb->data,
- skb_headlen(skb));
+ skb->data,
+ skb_headlen(skb));
else
put_dma(tx->index, eni_dev->dma, &j, (unsigned long)
- skb_frag_page(&skb_shinfo(skb)->frags[i]) +
+ skb_frag_page(&skb_shinfo(skb)->frags[i]) +
skb_shinfo(skb)->frags[i].page_offset,
- skb_frag_size(&skb_shinfo(skb)->frags[i]));
+ skb_frag_size(&skb_shinfo(skb)->frags[i]));
}
if (skb->len & 3) {
put_dma(tx->index, eni_dev->dma, &j, eni_dev->zero.dma,
@@ -1140,24 +1140,24 @@ DPRINTK("iovcnt = %d\n", skb_shinfo(skb)->nr_frags);
}
/* JK for AAL5 trailer - AAL0 doesn't need it, but who cares ... */
eni_dev->dma[j++] = (((tx->tx_pos+size) & (tx->words-1)) <<
- MID_DMA_COUNT_SHIFT) | (tx->index << MID_DMA_CHAN_SHIFT) |
- MID_DMA_END | MID_DT_JK;
+ MID_DMA_COUNT_SHIFT) | (tx->index << MID_DMA_CHAN_SHIFT) |
+ MID_DMA_END | MID_DT_JK;
j++;
DPRINTK("DMA at end: %d\n", j);
/* store frame */
writel((MID_SEG_TX_ID << MID_SEG_ID_SHIFT) |
- (aal5 ? MID_SEG_AAL5 : 0) | (tx->prescaler << MID_SEG_PR_SHIFT) |
- (tx->resolution << MID_SEG_RATE_SHIFT) |
- (size/(ATM_CELL_PAYLOAD/4)), tx->send+tx->tx_pos*4);
+ (aal5 ? MID_SEG_AAL5 : 0) | (tx->prescaler << MID_SEG_PR_SHIFT) |
+ (tx->resolution << MID_SEG_RATE_SHIFT) |
+ (size/(ATM_CELL_PAYLOAD/4)), tx->send+tx->tx_pos*4);
/*printk("dsc = 0x%08lx\n",(unsigned long) readl(tx->send+tx->tx_pos*4));*/
writel((vcc->vci << MID_SEG_VCI_SHIFT) |
- (aal5 ? 0 : (skb->data[3] & 0xf)) |
- (ATM_SKB(skb)->atm_options & ATM_ATMOPT_CLP ? MID_SEG_CLP : 0),
- tx->send+((tx->tx_pos+1) & (tx->words-1))*4);
+ (aal5 ? 0 : (skb->data[3] & 0xf)) |
+ (ATM_SKB(skb)->atm_options & ATM_ATMOPT_CLP ? MID_SEG_CLP : 0),
+ tx->send+((tx->tx_pos+1) & (tx->words-1))*4);
DPRINTK("size: %d, len:%d\n", size, skb->len);
if (aal5)
writel(skb->len, tx->send+
- ((tx->tx_pos+size-AAL5_TRAILER) & (tx->words-1))*4);
+ ((tx->tx_pos+size-AAL5_TRAILER) & (tx->words-1))*4);
j = j >> 1;
for (i = 0; i < j; i++) {
writel(eni_dev->dma[i*2], eni_dev->tx_dma+dma_wr*8);
@@ -1171,7 +1171,7 @@ DPRINTK("iovcnt = %d\n", skb_shinfo(skb)->nr_frags);
DPRINTK("dma_wr set to %d, tx_pos is now %ld\n", dma_wr, tx->tx_pos);
eni_out(dma_wr, MID_DMA_WR_TX);
skb_queue_tail(&eni_dev->tx_queue, skb);
-queued++;
+ queued++;
return enq_ok;
}
@@ -1192,7 +1192,7 @@ static void poll_tx(struct atm_dev *dev)
if (res == enq_ok) continue;
DPRINTK("re-queuing TX PDU\n");
skb_queue_head(&tx->backlog, skb);
-requeued++;
+ requeued++;
if (res == enq_jam) return;
break;
}
@@ -1216,7 +1216,7 @@ static void dequeue_tx(struct atm_dev *dev)
tx = ENI_VCC(vcc)->tx;
NULLCHECK(ENI_VCC(vcc)->tx);
DPRINTK("dequeue_tx: next 0x%lx curr 0x%x\n", ENI_PRV_POS(skb),
- (unsigned) eni_in(MID_TX_DESCRSTART(tx->index)));
+ (unsigned) eni_in(MID_TX_DESCRSTART(tx->index)));
if (ENI_VCC(vcc)->txing < tx->words && ENI_PRV_POS(skb) ==
eni_in(MID_TX_DESCRSTART(tx->index))) {
skb_queue_head(&eni_dev->tx_queue, skb);
@@ -1224,12 +1224,12 @@ static void dequeue_tx(struct atm_dev *dev)
}
ENI_VCC(vcc)->txing -= ENI_PRV_SIZE(skb);
pci_unmap_single(eni_dev->pci_dev, ENI_PRV_PADDR(skb), skb->len,
- PCI_DMA_TODEVICE);
+ PCI_DMA_TODEVICE);
if (vcc->pop) vcc->pop(vcc, skb);
else dev_kfree_skb_irq(skb);
atomic_inc(&vcc->stats->tx);
wake_up(&eni_dev->tx_wait);
-dma_complete++;
+ dma_complete++;
}
}
@@ -1245,10 +1245,10 @@ static struct eni_tx *alloc_tx(struct eni_dev *eni_dev, int ubr)
static int comp_tx(struct eni_dev *eni_dev, int *pcr, int reserved, int *pre,
- int *res, int unlimited)
+ int *res, int unlimited)
{
static const int pre_div[] = { 4, 16, 128, 2048 };
- /* 2^(((x+2)^2-(x+2))/2+1) */
+ /* 2^(((x+2)^2-(x+2))/2+1) */
if (unlimited) *pre = *res = 0;
else {
@@ -1282,7 +1282,7 @@ static int comp_tx(struct eni_dev *eni_dev, int *pcr, int reserved, int *pre,
static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
- int set_rsv, int set_shp)
+ int set_rsv, int set_shp)
{
struct eni_dev *eni_dev = ENI_DEV(vcc->dev);
struct eni_vcc *eni_vcc = ENI_VCC(vcc);
@@ -1296,7 +1296,7 @@ static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
rate = atm_pcr_goal(txtp);
ubr = txtp->traffic_class == ATM_UBR;
unlimited = ubr && (!rate || rate <= -ATM_OC3_PCR ||
- rate >= ATM_OC3_PCR);
+ rate >= ATM_OC3_PCR);
if (!unlimited) {
size = txtp->max_sdu*eni_dev->tx_mult/100;
if (size > MID_MAX_BUF_SIZE && txtp->max_sdu <=
@@ -1329,10 +1329,10 @@ static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
skb_queue_head_init(&tx->backlog);
for (order = 0; size > (1 << (order+10)); order++);
eni_out((order << MID_SIZE_SHIFT) |
- ((tx->send-eni_dev->ram) >> (MID_LOC_SKIP+2)),
- MID_TX_PLACE(tx->index));
+ ((tx->send-eni_dev->ram) >> (MID_LOC_SKIP+2)),
+ MID_TX_PLACE(tx->index));
tx->tx_pos = eni_in(MID_TX_DESCRSTART(tx->index)) &
- MID_DESCR_START;
+ MID_DESCR_START;
}
error = comp_tx(eni_dev, &rate, tx->reserved, &pre, &res, unlimited);
if (!error && txtp->min_pcr > rate) error = -EINVAL;
@@ -1417,7 +1417,7 @@ static void close_tx(struct atm_vcc *vcc)
* a bit ...
*/
while (eni_in(MID_TX_RDPTR(eni_vcc->tx->index)) !=
- eni_in(MID_TX_DESCRSTART(eni_vcc->tx->index)))
+ eni_in(MID_TX_DESCRSTART(eni_vcc->tx->index)))
schedule();
eni_free_mem(eni_dev, eni_vcc->tx->send, eni_vcc->tx->words << 2);
eni_vcc->tx->send = NULL;
@@ -1455,12 +1455,12 @@ static int start_tx(struct atm_dev *dev)
static void foo(void)
{
-printk(KERN_INFO
- "tx_complete=%d,dma_complete=%d,queued=%d,requeued=%d,sub=%d,\n"
- "backlogged=%d,rx_enqueued=%d,rx_dequeued=%d,putting=%d,pushed=%d\n",
- tx_complete, dma_complete, queued, requeued, submitted, backlogged,
- rx_enqueued, rx_dequeued, putting, pushed);
-if (eni_boards) printk(KERN_INFO "loss: %ld\n", ENI_DEV(eni_boards)->lost);
+ printk(KERN_INFO
+ "tx_complete=%d,dma_complete=%d,queued=%d,requeued=%d,sub=%d,\n"
+ "backlogged=%d,rx_enqueued=%d,rx_dequeued=%d,putting=%d,pushed=%d\n",
+ tx_complete, dma_complete, queued, requeued, submitted, backlogged,
+ rx_enqueued, rx_dequeued, putting, pushed);
+ if (eni_boards) printk(KERN_INFO "loss: %ld\n", ENI_DEV(eni_boards)->lost);
}
#endif
@@ -1471,13 +1471,13 @@ static void bug_int(struct atm_dev *dev, unsigned long reason)
DPRINTK(">bug_int\n");
if (reason & MID_DMA_ERR_ACK)
printk(KERN_CRIT DEV_LABEL "(itf %d): driver error - DMA "
- "error\n", dev->number);
+ "error\n", dev->number);
if (reason & MID_TX_IDENT_MISM)
printk(KERN_CRIT DEV_LABEL "(itf %d): driver error - ident "
- "mismatch\n", dev->number);
+ "mismatch\n", dev->number);
if (reason & MID_TX_DMA_OVFL)
printk(KERN_CRIT DEV_LABEL "(itf %d): driver error - DMA "
- "overflow\n", dev->number);
+ "overflow\n", dev->number);
EVENT("---dump ends here---\n", 0, 0);
printk(KERN_NOTICE "---recent events---\n");
event_dump();
@@ -1546,13 +1546,13 @@ static void eni_tasklet(unsigned long data)
poll_rx(dev);
EVENT("poll_rx done\n", 0, 0);
}
- if (events & MID_TX_DMA_COMPLETE) {
+ if (events & MID_TX_DMA_COMPLETE) {
EVENT("INT: TX DMA COMPLETE\n", 0, 0);
dequeue_tx(dev);
}
if (events & MID_TX_COMPLETE) {
EVENT("INT: TX COMPLETE\n", 0, 0);
-tx_complete++;
+ tx_complete++;
wake_up(&eni_dev->tx_wait);
/* poll_rx ? */
}
@@ -1568,27 +1568,27 @@ tx_complete++;
static char * const media_name[] = {
- "MMF", "SMF", "MMF", "03?", /* 0- 3 */
- "UTP", "05?", "06?", "07?", /* 4- 7 */
- "TAXI", "09?", "10?", "11?", /* 8-11 */
- "12?", "13?", "14?", "15?", /* 12-15 */
- "MMF", "SMF", "18?", "19?", /* 16-19 */
- "UTP", "21?", "22?", "23?", /* 20-23 */
- "24?", "25?", "26?", "27?", /* 24-27 */
- "28?", "29?", "30?", "31?" /* 28-31 */
+ "MMF", "SMF", "MMF", "03?", /* 0- 3 */
+ "UTP", "05?", "06?", "07?", /* 4- 7 */
+ "TAXI", "09?", "10?", "11?", /* 8-11 */
+ "12?", "13?", "14?", "15?", /* 12-15 */
+ "MMF", "SMF", "18?", "19?", /* 16-19 */
+ "UTP", "21?", "22?", "23?", /* 20-23 */
+ "24?", "25?", "26?", "27?", /* 24-27 */
+ "28?", "29?", "30?", "31?" /* 28-31 */
};
-#define SET_SEPROM \
- ({ if (!error && !pci_error) { \
- pci_error = pci_write_config_byte(eni_dev->pci_dev, PCI_TONGA_CTRL, tonga); \
- udelay(10); /* 10 usecs */ \
- } })
-#define GET_SEPROM \
- ({ if (!error && !pci_error) { \
- pci_error = pci_read_config_byte(eni_dev->pci_dev, PCI_TONGA_CTRL, &tonga); \
- udelay(10); /* 10 usecs */ \
- } })
+#define SET_SEPROM \
+ ({ if (!error && !pci_error) { \
+ pci_error = pci_write_config_byte(eni_dev->pci_dev, PCI_TONGA_CTRL, tonga); \
+ udelay(10); /* 10 usecs */ \
+ } })
+#define GET_SEPROM \
+ ({ if (!error && !pci_error) { \
+ pci_error = pci_read_config_byte(eni_dev->pci_dev, PCI_TONGA_CTRL, &tonga); \
+ udelay(10); /* 10 usecs */ \
+ } })
static int get_esi_asic(struct atm_dev *dev)
@@ -1616,7 +1616,7 @@ static int get_esi_asic(struct atm_dev *dev)
address = ((i+SEPROM_ESI_BASE) << 1)+1;
for (j = 7; j >= 0; j--) {
tonga = (address >> j) & 1 ? tonga | SEPROM_DATA :
- tonga & ~SEPROM_DATA;
+ tonga & ~SEPROM_DATA;
SET_SEPROM;
tonga |= SEPROM_CLK;
SET_SEPROM;
@@ -1672,7 +1672,7 @@ static int get_esi_asic(struct atm_dev *dev)
}
if (pci_error) {
printk(KERN_ERR DEV_LABEL "(itf %d): error reading ESI "
- "(0x%02x)\n", dev->number, pci_error);
+ "(0x%02x)\n", dev->number, pci_error);
error = -EIO;
}
return error;
@@ -1712,18 +1712,18 @@ static int eni_do_init(struct atm_dev *dev)
real_base = pci_resource_start(pci_dev, 0);
eni_dev->irq = pci_dev->irq;
if ((error = pci_write_config_word(pci_dev, PCI_COMMAND,
- PCI_COMMAND_MEMORY |
- (eni_dev->asic ? PCI_COMMAND_PARITY | PCI_COMMAND_SERR : 0)))) {
+ PCI_COMMAND_MEMORY |
+ (eni_dev->asic ? PCI_COMMAND_PARITY | PCI_COMMAND_SERR : 0)))) {
printk(KERN_ERR DEV_LABEL "(itf %d): can't enable memory "
- "(0x%02x)\n", dev->number, error);
+ "(0x%02x)\n", dev->number, error);
return -EIO;
}
printk(KERN_NOTICE DEV_LABEL "(itf %d): rev.%d,base=0x%lx,irq=%d,",
- dev->number, pci_dev->revision, real_base, eni_dev->irq);
+ dev->number, pci_dev->revision, real_base, eni_dev->irq);
if (!(base = ioremap_nocache(real_base, MAP_MAX_SIZE))) {
printk("\n");
printk(KERN_ERR DEV_LABEL "(itf %d): can't set up page "
- "mapping\n", dev->number);
+ "mapping\n", dev->number);
return error;
}
eni_dev->ioaddr = base;
@@ -1764,7 +1764,7 @@ static int eni_do_init(struct atm_dev *dev)
if (!(eni_in(MID_RES_ID_MCON) & 0x200) != !eni_dev->asic) {
printk(")\n");
printk(KERN_ERR DEV_LABEL "(itf %d): ERROR - wrong id 0x%x\n",
- dev->number, (unsigned) eni_in(MID_RES_ID_MCON));
+ dev->number, (unsigned) eni_in(MID_RES_ID_MCON));
error = -EINVAL;
goto unmap;
}
@@ -1775,8 +1775,8 @@ static int eni_do_init(struct atm_dev *dev)
printk("%s%02X", i ? "-" : "", dev->esi[i]);
printk(")\n");
printk(KERN_NOTICE DEV_LABEL "(itf %d): %s,%s\n", dev->number,
- eni_in(MID_RES_ID_MCON) & 0x200 ? "ASIC" : "FPGA",
- media_name[eni_in(MID_RES_ID_MCON) & DAUGTHER_ID]);
+ eni_in(MID_RES_ID_MCON) & 0x200 ? "ASIC" : "FPGA",
+ media_name[eni_in(MID_RES_ID_MCON) & DAUGTHER_ID]);
error = suni_init(dev);
if (error)
@@ -1809,22 +1809,22 @@ static int eni_start(struct atm_dev *dev)
eni_dev = ENI_DEV(dev);
if (request_irq(eni_dev->irq, &eni_int, IRQF_SHARED, DEV_LABEL, dev)) {
printk(KERN_ERR DEV_LABEL "(itf %d): IRQ%d is already in use\n",
- dev->number, eni_dev->irq);
+ dev->number, eni_dev->irq);
error = -EAGAIN;
goto out;
}
pci_set_master(eni_dev->pci_dev);
if ((error = pci_write_config_word(eni_dev->pci_dev, PCI_COMMAND,
- PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER |
- (eni_dev->asic ? PCI_COMMAND_PARITY | PCI_COMMAND_SERR : 0)))) {
+ PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER |
+ (eni_dev->asic ? PCI_COMMAND_PARITY | PCI_COMMAND_SERR : 0)))) {
printk(KERN_ERR DEV_LABEL "(itf %d): can't enable memory+"
- "master (0x%02x)\n", dev->number, error);
+ "master (0x%02x)\n", dev->number, error);
goto free_irq;
}
if ((error = pci_write_config_byte(eni_dev->pci_dev, PCI_TONGA_CTRL,
- END_SWAP_DMA))) {
+ END_SWAP_DMA))) {
printk(KERN_ERR DEV_LABEL "(itf %d): can't set endian swap "
- "(0x%02x)\n", dev->number, error);
+ "(0x%02x)\n", dev->number, error);
goto free_irq;
}
/* determine addresses of internal tables */
@@ -1834,8 +1834,8 @@ static int eni_start(struct atm_dev *dev)
eni_dev->service = eni_dev->tx_dma+NR_DMA_TX*8;
buf = eni_dev->service+NR_SERVICE*4;
DPRINTK("vci 0x%lx,rx 0x%lx, tx 0x%lx,srv 0x%lx,buf 0x%lx\n",
- eni_dev->vci, eni_dev->rx_dma, eni_dev->tx_dma,
- eni_dev->service, buf);
+ eni_dev->vci, eni_dev->rx_dma, eni_dev->tx_dma,
+ eni_dev->service, buf);
spin_lock_init(&eni_dev->lock);
tasklet_init(&eni_dev->task, eni_tasklet, (unsigned long) dev);
eni_dev->events = 0;
@@ -1843,10 +1843,10 @@ static int eni_start(struct atm_dev *dev)
buffer_mem = eni_dev->mem - (buf - eni_dev->ram);
eni_dev->free_list_size = buffer_mem/MID_MIN_BUF_SIZE/2;
eni_dev->free_list = kmalloc(
- sizeof(struct eni_free)*(eni_dev->free_list_size+1), GFP_KERNEL);
+ sizeof(struct eni_free)*(eni_dev->free_list_size+1), GFP_KERNEL);
if (!eni_dev->free_list) {
printk(KERN_ERR DEV_LABEL "(itf %d): couldn't get free page\n",
- dev->number);
+ dev->number);
error = -ENOMEM;
goto free_irq;
}
@@ -1855,7 +1855,7 @@ static int eni_start(struct atm_dev *dev)
memset_io(eni_dev->vci, 0, 16*NR_VCI); /* clear VCI table */
/*
* byte_addr free (k)
- * 0x00000000 512 VCI table
+ * 0x00000000 512 VCI table
* 0x00004000 496 RX DMA
* 0x00005000 492 TX DMA
* 0x00006000 488 service list
@@ -1870,9 +1870,9 @@ static int eni_start(struct atm_dev *dev)
error = dev->phy->start(dev);
if (error) goto free_list;
eni_out(eni_in(MID_MC_S) | (1 << MID_INT_SEL_SHIFT) |
- MID_TX_LOCK_MODE | MID_DMA_ENABLE | MID_TX_ENABLE | MID_RX_ENABLE,
- MID_MC_S);
- /* Tonga uses SBus INTReq1 */
+ MID_TX_LOCK_MODE | MID_DMA_ENABLE | MID_TX_ENABLE | MID_RX_ENABLE,
+ MID_MC_S);
+ /* Tonga uses SBus INTReq1 */
(void) eni_in(MID_ISA); /* clear Midway interrupts */
return 0;
@@ -1919,7 +1919,7 @@ static int eni_open(struct atm_vcc *vcc)
if (vcc->qos.aal != ATM_AAL0 && vcc->qos.aal != ATM_AAL5)
return -EINVAL;
DPRINTK(DEV_LABEL "(itf %d): open %d.%d\n", vcc->dev->number, vcc->vpi,
- vcc->vci);
+ vcc->vci);
if (!test_bit(ATM_VF_PARTIAL, &vcc->flags)) {
eni_vcc = kmalloc(sizeof(struct eni_vcc), GFP_KERNEL);
if (!eni_vcc) return -ENOMEM;
@@ -1980,8 +1980,8 @@ static int eni_change_qos(struct atm_vcc *vcc, struct atm_qos *qos, int flgs)
if (ATM_SKB(skb)->vcc != vcc) continue;
dsc = tx->send+ENI_PRV_POS(skb)*4;
writel((readl(dsc) & ~(MID_SEG_RATE | MID_SEG_PR)) |
- (tx->prescaler << MID_SEG_PR_SHIFT) |
- (tx->resolution << MID_SEG_RATE_SHIFT), dsc);
+ (tx->prescaler << MID_SEG_PR_SHIFT) |
+ (tx->resolution << MID_SEG_RATE_SHIFT), dsc);
}
tasklet_enable(&eni_dev->task);
return 0;
@@ -1995,7 +1995,7 @@ static int eni_ioctl(struct atm_dev *dev, unsigned int cmd, void __user *arg)
if (cmd == ENI_MEMDUMP) {
if (!capable(CAP_NET_ADMIN)) return -EPERM;
printk(KERN_WARNING "Please use /proc/atm/" DEV_LABEL ":%d "
- "instead of obsolete ioctl ENI_MEMDUMP\n", dev->number);
+ "instead of obsolete ioctl ENI_MEMDUMP\n", dev->number);
dump(dev);
return 0;
}
@@ -2004,7 +2004,7 @@ static int eni_ioctl(struct atm_dev *dev, unsigned int cmd, void __user *arg)
if (!capable(CAP_NET_ADMIN)) return -EPERM;
if (copy_from_user(&mult, arg,
- sizeof(struct eni_multipliers)))
+ sizeof(struct eni_multipliers)))
return -EFAULT;
if ((mult.tx && mult.tx <= 100) || (mult.rx &&mult.rx <= 100) ||
mult.tx > 65536 || mult.rx > 65536)
@@ -2020,7 +2020,7 @@ static int eni_ioctl(struct atm_dev *dev, unsigned int cmd, void __user *arg)
return -EFAULT;
if ((ci.vpi_bits == 0 || ci.vpi_bits == ATM_CI_MAX) &&
(ci.vci_bits == NR_VCI_LD || ci.vpi_bits == ATM_CI_MAX))
- return 0;
+ return 0;
return -EINVAL;
}
if (!dev->phy->ioctl) return -ENOIOCTLCMD;
@@ -2029,14 +2029,14 @@ static int eni_ioctl(struct atm_dev *dev, unsigned int cmd, void __user *arg)
static int eni_getsockopt(struct atm_vcc *vcc, int level, int optname,
- void __user *optval, int optlen)
+ void __user *optval, int optlen)
{
return -EINVAL;
}
static int eni_setsockopt(struct atm_vcc *vcc, int level, int optname,
- void __user *optval, unsigned int optlen)
+ void __user *optval, unsigned int optlen)
{
return -EINVAL;
}
@@ -2065,20 +2065,20 @@ static int eni_send(struct atm_vcc *vcc, struct sk_buff *skb)
}
*(u32 *) skb->data = htonl(*(u32 *) skb->data);
}
-submitted++;
+ submitted++;
ATM_SKB(skb)->vcc = vcc;
tasklet_disable(&ENI_DEV(vcc->dev)->task);
res = do_tx(skb);
tasklet_enable(&ENI_DEV(vcc->dev)->task);
if (res == enq_ok) return 0;
skb_queue_tail(&ENI_VCC(vcc)->tx->backlog, skb);
-backlogged++;
+ backlogged++;
tasklet_schedule(&ENI_DEV(vcc->dev)->task);
return 0;
}
static void eni_phy_put(struct atm_dev *dev, unsigned char value,
- unsigned long addr)
+ unsigned long addr)
{
writel(value, ENI_DEV(dev)->phy+addr*4);
}
@@ -2103,69 +2103,69 @@ static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
left = *pos;
if (!left)
return sprintf(page, DEV_LABEL "(itf %d) signal %s, %dkB, "
- "%d cps remaining\n", dev->number, signal[(int) dev->signal],
- eni_dev->mem >> 10, eni_dev->tx_bw);
+ "%d cps remaining\n", dev->number, signal[(int) dev->signal],
+ eni_dev->mem >> 10, eni_dev->tx_bw);
if (!--left)
return sprintf(page, "%4sBursts: TX"
-#if !defined(CONFIG_ATM_ENI_BURST_TX_16W) && \
- !defined(CONFIG_ATM_ENI_BURST_TX_8W) && \
- !defined(CONFIG_ATM_ENI_BURST_TX_4W) && \
- !defined(CONFIG_ATM_ENI_BURST_TX_2W)
- " none"
+#if !defined(CONFIG_ATM_ENI_BURST_TX_16W) && \
+ !defined(CONFIG_ATM_ENI_BURST_TX_8W) && \
+ !defined(CONFIG_ATM_ENI_BURST_TX_4W) && \
+ !defined(CONFIG_ATM_ENI_BURST_TX_2W)
+ " none"
#endif
#ifdef CONFIG_ATM_ENI_BURST_TX_16W
- " 16W"
+ " 16W"
#endif
#ifdef CONFIG_ATM_ENI_BURST_TX_8W
- " 8W"
+ " 8W"
#endif
#ifdef CONFIG_ATM_ENI_BURST_TX_4W
- " 4W"
+ " 4W"
#endif
#ifdef CONFIG_ATM_ENI_BURST_TX_2W
- " 2W"
+ " 2W"
#endif
- ", RX"
-#if !defined(CONFIG_ATM_ENI_BURST_RX_16W) && \
- !defined(CONFIG_ATM_ENI_BURST_RX_8W) && \
- !defined(CONFIG_ATM_ENI_BURST_RX_4W) && \
- !defined(CONFIG_ATM_ENI_BURST_RX_2W)
- " none"
+ ", RX"
+#if !defined(CONFIG_ATM_ENI_BURST_RX_16W) && \
+ !defined(CONFIG_ATM_ENI_BURST_RX_8W) && \
+ !defined(CONFIG_ATM_ENI_BURST_RX_4W) && \
+ !defined(CONFIG_ATM_ENI_BURST_RX_2W)
+ " none"
#endif
#ifdef CONFIG_ATM_ENI_BURST_RX_16W
- " 16W"
+ " 16W"
#endif
#ifdef CONFIG_ATM_ENI_BURST_RX_8W
- " 8W"
+ " 8W"
#endif
#ifdef CONFIG_ATM_ENI_BURST_RX_4W
- " 4W"
+ " 4W"
#endif
#ifdef CONFIG_ATM_ENI_BURST_RX_2W
- " 2W"
+ " 2W"
#endif
#ifndef CONFIG_ATM_ENI_TUNE_BURST
- " (default)"
+ " (default)"
#endif
- "\n", "");
+ "\n", "");
if (!--left)
return sprintf(page, "%4sBuffer multipliers: tx %d%%, rx %d%%\n",
- "", eni_dev->tx_mult, eni_dev->rx_mult);
+ "", eni_dev->tx_mult, eni_dev->rx_mult);
for (i = 0; i < NR_CHAN; i++) {
struct eni_tx *tx = eni_dev->tx+i;
if (!tx->send) continue;
if (!--left) {
- return sprintf(page, "tx[%d]: 0x%ld-0x%ld "
- "(%6ld bytes), rsv %d cps, shp %d cps%s\n", i,
- (unsigned long) (tx->send - eni_dev->ram),
- tx->send-eni_dev->ram+tx->words*4-1, tx->words*4,
- tx->reserved, tx->shaping,
- tx == eni_dev->ubr ? " (UBR)" : "");
+ return sprintf(page, "tx[%d]: 0x%ld-0x%ld "
+ "(%6ld bytes), rsv %d cps, shp %d cps%s\n", i,
+ (unsigned long) (tx->send - eni_dev->ram),
+ tx->send-eni_dev->ram+tx->words*4-1, tx->words*4,
+ tx->reserved, tx->shaping,
+ tx == eni_dev->ubr ? " (UBR)" : "");
}
if (--left) continue;
return sprintf(page, "%10sbacklog %u packets\n", "",
- skb_queue_len(&tx->backlog));
+ skb_queue_len(&tx->backlog));
}
read_lock(&vcc_sklist_lock);
for(i = 0; i < VCC_HTABLE_SIZE; ++i) {
@@ -2183,15 +2183,15 @@ static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
length = sprintf(page, "vcc %4d: ", vcc->vci);
if (eni_vcc->rx) {
length += sprintf(page+length, "0x%ld-0x%ld "
- "(%6ld bytes)",
- (unsigned long) (eni_vcc->recv - eni_dev->ram),
- eni_vcc->recv-eni_dev->ram+eni_vcc->words*4-1,
- eni_vcc->words*4);
+ "(%6ld bytes)",
+ (unsigned long) (eni_vcc->recv - eni_dev->ram),
+ eni_vcc->recv-eni_dev->ram+eni_vcc->words*4-1,
+ eni_vcc->words*4);
if (eni_vcc->tx) length += sprintf(page+length, ", ");
}
if (eni_vcc->tx)
length += sprintf(page+length, "tx[%d], txing %d bytes",
- eni_vcc->tx->index, eni_vcc->txing);
+ eni_vcc->tx->index, eni_vcc->txing);
page[length] = '\n';
read_unlock(&vcc_sklist_lock);
return length+1;
@@ -2204,9 +2204,9 @@ static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
if (--left) continue;
offset = (unsigned long) eni_dev->ram+eni_dev->base_diff;
- return sprintf(page, "free %p-%p (%6d bytes)\n",
- fe->start-offset, fe->start-offset+(1 << fe->order)-1,
- 1 << fe->order);
+ return sprintf(page, "free %p-%p (%6d bytes)\n",
+ fe->start-offset, fe->start-offset+(1 << fe->order)-1,
+ 1 << fe->order);
}
return 0;
}
@@ -2320,7 +2320,7 @@ static int __init eni_init(void)
if (sizeof(skb->cb) < sizeof(struct eni_skb_prv)) {
printk(KERN_ERR "eni_detect: skb->cb is too small (%Zd < %Zd)\n",
- sizeof(skb->cb), sizeof(struct eni_skb_prv));
+ sizeof(skb->cb), sizeof(struct eni_skb_prv));
return -EIO;
}
return pci_register_driver(&eni_driver);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 4/5] drivers: atm: checkpatch.pl changed headers from asm/* to linux/* in eni.c
2013-01-20 23:12 [PATCH 1/5] drivers: atm: checkpatch.pl fixed coding style errors in eni.c Patrik Karlin
2013-01-20 23:12 ` [PATCH 2/5] drivers: atm: checkpatch.pl fixes removed all trailing whitespaces " Patrik Karlin
2013-01-20 23:12 ` [PATCH 3/5] drivers: atm: checkpatch.pl fixed tabbed indentation " Patrik Karlin
@ 2013-01-20 23:12 ` Patrik Karlin
2013-01-20 23:12 ` [PATCH 5/5] drivers: atm: checkpatch.pl fixed coding style issues " Patrik Karlin
3 siblings, 0 replies; 6+ messages in thread
From: Patrik Karlin @ 2013-01-20 23:12 UTC (permalink / raw)
To: chas; +Cc: linux-atm-general, netdev, linux-kernel, Patrik Karlin
This patch changes the included headers from asm/* to linux/* for
header changes sugested by checkpatch.pl
Signed-off-by: Patrik Kårlin <patrik.karlin@gmail.com>
---
drivers/atm/eni.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/atm/eni.c b/drivers/atm/eni.c
index f4a136a..2877832 100644
--- a/drivers/atm/eni.c
+++ b/drivers/atm/eni.c
@@ -19,10 +19,10 @@
#include <linux/atm_eni.h>
#include <linux/bitops.h>
#include <linux/slab.h>
-#include <asm/io.h>
+#include <linux/io.h>
#include <linux/atomic.h>
-#include <asm/uaccess.h>
-#include <asm/string.h>
+#include <linux/uaccess.h>
+#include <linux/string.h>
#include <asm/byteorder.h>
#include "tonga.h"
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 5/5] drivers: atm: checkpatch.pl fixed coding style issues in eni.c
2013-01-20 23:12 [PATCH 1/5] drivers: atm: checkpatch.pl fixed coding style errors in eni.c Patrik Karlin
` (2 preceding siblings ...)
2013-01-20 23:12 ` [PATCH 4/5] drivers: atm: checkpatch.pl changed headers from asm/* to linux/* " Patrik Karlin
@ 2013-01-20 23:12 ` Patrik Karlin
2013-01-21 4:04 ` David Miller
3 siblings, 1 reply; 6+ messages in thread
From: Patrik Karlin @ 2013-01-20 23:12 UTC (permalink / raw)
To: chas; +Cc: linux-atm-general, netdev, linux-kernel, Patrik Karlin
This patch fixes statement placement around if/else/for statments
as suggested by checkpatch.pl
Signed-off-by: Patrik Kårlin <patrik.karlin@gmail.com>
---
drivers/atm/eni.c | 318 ++++++++++++++++++++++++++++++++++-------------------
1 file changed, 205 insertions(+), 113 deletions(-)
diff --git a/drivers/atm/eni.c b/drivers/atm/eni.c
index 2877832..a8a18d4 100644
--- a/drivers/atm/eni.c
+++ b/drivers/atm/eni.c
@@ -214,7 +214,8 @@ static void eni_put_free(struct eni_dev *eni_dev, void __iomem *start,
start, size);
break;
}
- for (order = 0; !(((unsigned long)start | size) & (1 << order)); order++);
+ for (order = 0; !(((unsigned long)start | size) & (1 << order)); order++)
+ ;
if (MID_MIN_BUF_SIZE > (1 << order)) {
printk(KERN_CRIT "eni_put_free: order %d too small\n",
order);
@@ -239,9 +240,12 @@ static void __iomem *eni_alloc_mem(struct eni_dev *eni_dev, unsigned long *size)
list = eni_dev->free_list;
len = eni_dev->free_len;
- if (*size < MID_MIN_BUF_SIZE) *size = MID_MIN_BUF_SIZE;
- if (*size > MID_MAX_BUF_SIZE) return NULL;
- for (order = 0; (1 << order) < *size; order++);
+ if (*size < MID_MIN_BUF_SIZE)
+ *size = MID_MIN_BUF_SIZE;
+ if (*size > MID_MAX_BUF_SIZE)
+ return NULL;
+ for (order = 0; (1 << order) < *size; order++)
+ ;
DPRINTK("trying: %ld->%d\n", *size, order);
best_order = 65; /* we don't have more than 2^64 of anything ... */
index = 0; /* silence GCC */
@@ -250,12 +254,12 @@ static void __iomem *eni_alloc_mem(struct eni_dev *eni_dev, unsigned long *size)
best_order = order;
index = i;
break;
- }
- else if (best_order > list[i].order && list[i].order > order) {
+ } else if (best_order > list[i].order && list[i].order > order) {
best_order = list[i].order;
index = i;
}
- if (best_order == 65) return NULL;
+ if (best_order == 65)
+ return NULL;
start = list[index].start-eni_dev->base_diff;
list[index] = list[--len];
eni_dev->free_len = len;
@@ -277,7 +281,8 @@ static void eni_free_mem(struct eni_dev *eni_dev, void __iomem *start,
start += eni_dev->base_diff;
list = eni_dev->free_list;
len = eni_dev->free_len;
- for (order = -1; size; order++) size >>= 1;
+ for (order = -1; size; order++)
+ size >>= 1;
DPRINTK("eni_free_mem: %p+0x%lx (order %d)\n", start, size, order);
for (i = 0; i < len; i++)
if (((unsigned long) list[i].start) == ((unsigned long)start^(1 << order)) &&
@@ -373,7 +378,8 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
j++;
}
here = (eni_vcc->descr+size+skip) & (eni_vcc->words-1);
- if (!eff) size += skip;
+ if (!eff)
+ size += skip;
else {
unsigned long words;
@@ -387,7 +393,8 @@ static int do_rx_dma(struct atm_vcc *vcc, struct sk_buff *skb,
unsigned long init;
init = 4-((paddr & 15) >> 2);
- if (init > words) init = words;
+ if (init > words)
+ init = words;
dma[j++] = MID_DT_WORD | (init << MID_DMA_COUNT_SHIFT) |
(vcc->vci << MID_DMA_VCI_SHIFT);
dma[j++] = paddr;
@@ -481,7 +488,8 @@ trouble:
if (paddr)
pci_unmap_single(eni_dev->pci_dev, paddr, skb->len,
PCI_DMA_FROMDEVICE);
- if (skb) dev_kfree_skb_irq(skb);
+ if (skb)
+ dev_kfree_skb_irq(skb);
return -1;
}
@@ -492,10 +500,13 @@ static void discard(struct atm_vcc *vcc, unsigned long size)
eni_vcc = ENI_VCC(vcc);
EVENT("discard (size=%ld)\n", size, 0);
- while (do_rx_dma(vcc, NULL, 1, size, 0)) EVENT("BUSY LOOP", 0, 0);
+ while (do_rx_dma(vcc, NULL, 1, size, 0))
+ EVENT("BUSY LOOP", 0, 0);
/* could do a full fallback, but that might be more expensive */
- if (eni_vcc->rxing) ENI_PRV_POS(eni_vcc->last) += size+1;
- else eni_vcc->rx_pos = (eni_vcc->rx_pos+size+1) & (eni_vcc->words-1);
+ if (eni_vcc->rxing)
+ ENI_PRV_POS(eni_vcc->last) += size+1;
+ else
+ eni_vcc->rx_pos = (eni_vcc->rx_pos+size+1) & (eni_vcc->words-1);
}
@@ -523,8 +534,7 @@ static int rx_aal0(struct atm_vcc *vcc)
vcc->dev->number);
length = 0;
atomic_inc(&vcc->stats->rx_err);
- }
- else {
+ } else {
length = ATM_CELL_SIZE-1; /* no HEC */
}
skb = length ? atm_alloc_charge(vcc, length, GFP_ATOMIC) : NULL;
@@ -535,7 +545,8 @@ static int rx_aal0(struct atm_vcc *vcc)
skb_put(skb, length);
skb->tstamp = eni_vcc->timestamp;
DPRINTK("got len %ld\n", length);
- if (do_rx_dma(vcc, skb, 1, length >> 2, length >> 2)) return 1;
+ if (do_rx_dma(vcc, skb, 1, length >> 2, length >> 2))
+ return 1;
eni_vcc->rxing++;
return 0;
}
@@ -562,8 +573,7 @@ static int rx_aal5(struct atm_vcc *vcc)
DPRINTK(DEV_LABEL "(itf %d): trashing empty cell\n",
vcc->dev->number);
size = 0;
- }
- else {
+ } else {
static unsigned long silence = 0;
if (time_after(jiffies, silence) || silence == 0) {
@@ -578,15 +588,15 @@ static int rx_aal5(struct atm_vcc *vcc)
}
eff = length = 0;
atomic_inc(&vcc->stats->rx_err);
- }
- else {
+ } else {
size = (descr & MID_RED_COUNT)*(ATM_CELL_PAYLOAD >> 2);
DPRINTK("size=%ld\n", size);
length = readl(eni_vcc->recv+(((eni_vcc->descr+size-1) &
(eni_vcc->words-1)))*4) & 0xffff;
/* -trailer(2)+header(1) */
if (length && length <= (size << 2)-8 && length <=
- ATM_MAX_AAL5_PDU) eff = (length+3) >> 2;
+ ATM_MAX_AAL5_PDU)
+ eff = (length+3) >> 2;
else { /* ^ trailer length (8) */
EVENT("bad PDU (descr=0x08%lx,length=%ld)\n", descr,
length);
@@ -604,7 +614,8 @@ static int rx_aal5(struct atm_vcc *vcc)
}
skb_put(skb, length);
DPRINTK("got len %ld\n", length);
- if (do_rx_dma(vcc, skb, 1, size, eff)) return 1;
+ if (do_rx_dma(vcc, skb, 1, size, eff))
+ return 1;
eni_vcc->rxing++;
return 0;
}
@@ -626,7 +637,8 @@ static inline int rx_vcc(struct atm_vcc *vcc)
DPRINTK("CB_DESCR %ld REG_DESCR %d\n", ENI_VCC(vcc)->descr,
(((unsigned) readl(vci_dsc+4) & MID_VCI_DESCR) >>
MID_VCI_DESCR_SHIFT));
- if (ENI_VCC(vcc)->rx(vcc)) return 1;
+ if (ENI_VCC(vcc)->rx(vcc))
+ return 1;
}
/* clear IN_SERVICE flag */
writel(readl(vci_dsc) & ~MID_VCI_IN_SERVICE, vci_dsc);
@@ -643,7 +655,8 @@ static inline int rx_vcc(struct atm_vcc *vcc)
DPRINTK("CB_DESCR %ld REG_DESCR %d\n", ENI_VCC(vcc)->descr,
(((unsigned) readl(vci_dsc+4) & MID_VCI_DESCR) >>
MID_VCI_DESCR_SHIFT));
- if (ENI_VCC(vcc)->rx(vcc)) return 1;
+ if (ENI_VCC(vcc)->rx(vcc))
+ return 1;
}
return 0;
}
@@ -657,7 +670,8 @@ static void poll_rx(struct atm_dev *dev)
eni_dev = ENI_DEV(dev);
while ((curr = eni_dev->fast)) {
EVENT("poll_rx.fast\n", 0, 0);
- if (rx_vcc(curr)) return;
+ if (rx_vcc(curr))
+ return;
eni_dev->fast = ENI_VCC(curr)->next;
ENI_VCC(curr)->next = ENI_VCC_NOS;
barrier();
@@ -665,7 +679,8 @@ static void poll_rx(struct atm_dev *dev)
}
while ((curr = eni_dev->slow)) {
EVENT("poll_rx.slow\n", 0, 0);
- if (rx_vcc(curr)) return;
+ if (rx_vcc(curr))
+ return;
eni_dev->slow = ENI_VCC(curr)->next;
ENI_VCC(curr)->next = ENI_VCC_NOS;
barrier();
@@ -703,13 +718,14 @@ static void get_service(struct atm_dev *dev)
if (vcc->qos.rxtp.traffic_class == ATM_CBR) {
if (eni_dev->fast)
ENI_VCC(eni_dev->last_fast)->next = vcc;
- else eni_dev->fast = vcc;
+ else
+ eni_dev->fast = vcc;
eni_dev->last_fast = vcc;
- }
- else {
+ } else {
if (eni_dev->slow)
ENI_VCC(eni_dev->last_slow)->next = vcc;
- else eni_dev->slow = vcc;
+ else
+ eni_dev->slow = vcc;
eni_dev->last_slow = vcc;
}
putting++;
@@ -757,7 +773,8 @@ static void dequeue_rx(struct atm_dev *dev)
eni_vcc->rx_pos = ENI_PRV_POS(skb) & (eni_vcc->words-1);
pci_unmap_single(eni_dev->pci_dev, ENI_PRV_PADDR(skb), skb->len,
PCI_DMA_TODEVICE);
- if (!skb->len) dev_kfree_skb_irq(skb);
+ if (!skb->len)
+ dev_kfree_skb_irq(skb);
else {
EVENT("pushing (len=%ld)\n", skb->len, 0);
if (vcc->qos.aal == ATM_AAL0)
@@ -783,7 +800,8 @@ static int open_rx_first(struct atm_vcc *vcc)
eni_dev = ENI_DEV(vcc->dev);
eni_vcc = ENI_VCC(vcc);
eni_vcc->rx = NULL;
- if (vcc->qos.rxtp.traffic_class == ATM_NONE) return 0;
+ if (vcc->qos.rxtp.traffic_class == ATM_NONE)
+ return 0;
size = vcc->qos.rxtp.max_sdu*eni_dev->rx_mult/100;
if (size > MID_MAX_BUF_SIZE && vcc->qos.rxtp.max_sdu <=
MID_MAX_BUF_SIZE)
@@ -791,7 +809,8 @@ static int open_rx_first(struct atm_vcc *vcc)
eni_vcc->recv = eni_alloc_mem(eni_dev, &size);
DPRINTK("rx at 0x%lx\n", eni_vcc->recv);
eni_vcc->words = size >> 2;
- if (!eni_vcc->recv) return -ENOBUFS;
+ if (!eni_vcc->recv)
+ return -ENOBUFS;
eni_vcc->rx = vcc->qos.aal == ATM_AAL5 ? rx_aal5 : rx_aal0;
eni_vcc->descr = 0;
eni_vcc->rx_pos = 0;
@@ -813,12 +832,14 @@ static int open_rx_second(struct atm_vcc *vcc)
DPRINTK("open_rx_second\n");
eni_dev = ENI_DEV(vcc->dev);
eni_vcc = ENI_VCC(vcc);
- if (!eni_vcc->rx) return 0;
+ if (!eni_vcc->rx)
+ return 0;
/* set up VCI descriptor */
here = eni_dev->vci+vcc->vci*16;
DPRINTK("loc 0x%x\n", (unsigned) (eni_vcc->recv-eni_dev->ram)/4);
size = eni_vcc->words >> 8;
- for (order = -1; size; order++) size >>= 1;
+ for (order = -1; size; order++)
+ size >>= 1;
writel(0, here+4); /* descr, read = 0 */
writel(0, here+8); /* write, state, count = 0 */
if (eni_dev->rx_map[vcc->vci])
@@ -841,7 +862,8 @@ static void close_rx(struct atm_vcc *vcc)
struct eni_vcc *eni_vcc;
eni_vcc = ENI_VCC(vcc);
- if (!eni_vcc->rx) return;
+ if (!eni_vcc->rx)
+ return;
eni_dev = ENI_DEV(vcc->dev);
if (vcc->vpi != ATM_VPI_UNSPEC && vcc->vci != ATM_VCI_UNSPEC) {
here = eni_dev->vci+vcc->vci*16;
@@ -864,7 +886,8 @@ static void close_rx(struct atm_vcc *vcc)
/* transition service->rx: rxing++, servicing-- */
if (!eni_vcc->servicing) {
barrier();
- if (!eni_vcc->rxing) break;
+ if (!eni_vcc->rxing)
+ break;
}
EVENT("drain PDUs (rx %ld, serv %ld)\n", eni_vcc->rxing,
eni_vcc->servicing);
@@ -881,7 +904,8 @@ static void close_rx(struct atm_vcc *vcc)
tmp = readl(eni_dev->vci+vcc->vci*16+4) & MID_VCI_READ;
at_end = eni_vcc->rx_pos == tmp >> MID_VCI_READ_SHIFT;
tasklet_enable(&eni_dev->task);
- if (at_end) break;
+ if (at_end)
+ break;
EVENT("drain discard (host 0x%lx, nic 0x%lx)\n",
eni_vcc->rx_pos, tmp);
printk(KERN_INFO "draining RX: host 0x%lx, nic 0x%x\n",
@@ -941,7 +965,8 @@ static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
#endif
if (paddr & 3) {
init = 4-(paddr & 3);
- if (init > size || size < 7) init = size;
+ if (init > size || size < 7)
+ init = size;
DPRINTK("put_dma: %lx DMA: %d/%d bytes\n",
(unsigned long) paddr, init, size);
dma[(*j)++] = MID_DT_BYTE | (init << MID_DMA_COUNT_SHIFT) |
@@ -954,7 +979,8 @@ static inline void put_dma(int chan, u32 *dma, int *j, dma_addr_t paddr,
size &= 3;
if (words && (paddr & 31)) {
init = 8-((paddr & 31) >> 2);
- if (init > words) init = words;
+ if (init > words)
+ init = words;
DPRINTK("put_dma: %lx DMA: %d/%d words\n",
(unsigned long) paddr, init, words);
dma[(*j)++] = MID_DT_WORD | (init << MID_DMA_COUNT_SHIFT) |
@@ -1096,8 +1122,10 @@ static enum enq_res do_tx(struct sk_buff *skb)
dma_size = 3; /* JK for descriptor and final fill, plus final size
mis-alignment fix */
DPRINTK("iovcnt = %d\n", skb_shinfo(skb)->nr_frags);
- if (!skb_shinfo(skb)->nr_frags) dma_size += 5;
- else dma_size += 5*(skb_shinfo(skb)->nr_frags+1);
+ if (!skb_shinfo(skb)->nr_frags)
+ dma_size += 5;
+ else
+ dma_size += 5*(skb_shinfo(skb)->nr_frags+1);
if (dma_size > TX_DMA_BUF) {
printk(KERN_CRIT DEV_LABEL "(itf %d): needs %d DMA entries "
"(got only %d)\n", vcc->dev->number, dma_size, TX_DMA_BUF);
@@ -1119,8 +1147,10 @@ static enum enq_res do_tx(struct sk_buff *skb)
MID_DT_JK;
j++;
if (!skb_shinfo(skb)->nr_frags)
- if (aal5) put_dma(tx->index, eni_dev->dma, &j, paddr, skb->len);
- else put_dma(tx->index, eni_dev->dma, &j, paddr+4, skb->len-4);
+ if (aal5)
+ put_dma(tx->index, eni_dev->dma, &j, paddr, skb->len);
+ else
+ put_dma(tx->index, eni_dev->dma, &j, paddr+4, skb->len-4);
else {
DPRINTK("doing direct send\n"); /* @@@ well, this doesn't work anyway */
for (i = -1; i < skb_shinfo(skb)->nr_frags; i++)
@@ -1189,11 +1219,13 @@ static void poll_tx(struct atm_dev *dev)
if (tx->send)
while ((skb = skb_dequeue(&tx->backlog))) {
res = do_tx(skb);
- if (res == enq_ok) continue;
+ if (res == enq_ok)
+ continue;
DPRINTK("re-queuing TX PDU\n");
skb_queue_head(&tx->backlog, skb);
requeued++;
- if (res == enq_jam) return;
+ if (res == enq_jam)
+ return;
break;
}
}
@@ -1225,8 +1257,10 @@ static void dequeue_tx(struct atm_dev *dev)
ENI_VCC(vcc)->txing -= ENI_PRV_SIZE(skb);
pci_unmap_single(eni_dev->pci_dev, ENI_PRV_PADDR(skb), skb->len,
PCI_DMA_TODEVICE);
- if (vcc->pop) vcc->pop(vcc, skb);
- else dev_kfree_skb_irq(skb);
+ if (vcc->pop)
+ vcc->pop(vcc, skb);
+ else
+ dev_kfree_skb_irq(skb);
atomic_inc(&vcc->stats->tx);
wake_up(&eni_dev->tx_wait);
dma_complete++;
@@ -1239,7 +1273,8 @@ static struct eni_tx *alloc_tx(struct eni_dev *eni_dev, int ubr)
int i;
for (i = !ubr; i < NR_CHAN; i++)
- if (!eni_dev->tx[i].send) return eni_dev->tx+i;
+ if (!eni_dev->tx[i].send)
+ return eni_dev->tx+i;
return NULL;
}
@@ -1250,30 +1285,36 @@ static int comp_tx(struct eni_dev *eni_dev, int *pcr, int reserved, int *pre,
static const int pre_div[] = { 4, 16, 128, 2048 };
/* 2^(((x+2)^2-(x+2))/2+1) */
- if (unlimited) *pre = *res = 0;
+ if (unlimited)
+ *pre = *res = 0;
else {
if (*pcr > 0) {
int div;
for (*pre = 0; *pre < 3; (*pre)++)
- if (TS_CLOCK/pre_div[*pre]/64 <= *pcr) break;
+ if (TS_CLOCK/pre_div[*pre]/64 <= *pcr)
+ break;
div = pre_div[*pre]**pcr;
DPRINTK("min div %d\n", div);
*res = TS_CLOCK/div-1;
- }
- else {
+ } else {
int div;
- if (!*pcr) *pcr = eni_dev->tx_bw+reserved;
+ if (!*pcr)
+ *pcr = eni_dev->tx_bw+reserved;
for (*pre = 3; *pre >= 0; (*pre)--)
- if (TS_CLOCK/pre_div[*pre]/64 > -*pcr) break;
- if (*pre < 3) (*pre)++; /* else fail later */
+ if (TS_CLOCK/pre_div[*pre]/64 > -*pcr)
+ break;
+ if (*pre < 3)
+ (*pre)++; /* else fail later */
div = pre_div[*pre]*-*pcr;
DPRINTK("max div %d\n", div);
*res = DIV_ROUND_UP(TS_CLOCK, div)-1;
}
- if (*res < 0) *res = 0;
- if (*res > MID_SEG_MAX_RATE) *res = MID_SEG_MAX_RATE;
+ if (*res < 0)
+ *res = 0;
+ if (*res > MID_SEG_MAX_RATE)
+ *res = MID_SEG_MAX_RATE;
}
*pcr = TS_CLOCK/pre_div[*pre]/(*res+1);
DPRINTK("out pcr: %d (%d:%d)\n", *pcr, *pre, *res);
@@ -1302,8 +1343,7 @@ static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
if (size > MID_MAX_BUF_SIZE && txtp->max_sdu <=
MID_MAX_BUF_SIZE)
size = MID_MAX_BUF_SIZE;
- }
- else {
+ } else {
if (eni_dev->ubr) {
eni_vcc->tx = eni_dev->ubr;
txtp->pcr = ATM_OC3_PCR;
@@ -1313,10 +1353,12 @@ static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
}
new_tx = !eni_vcc->tx;
mem = NULL; /* for gcc */
- if (!new_tx) tx = eni_vcc->tx;
+ if (!new_tx)
+ tx = eni_vcc->tx;
else {
mem = eni_alloc_mem(eni_dev, &size);
- if (!mem) return -ENOBUFS;
+ if (!mem)
+ return -ENOBUFS;
tx = alloc_tx(eni_dev, unlimited);
if (!tx) {
eni_free_mem(eni_dev, mem, size);
@@ -1327,7 +1369,8 @@ static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
tx->send = mem;
tx->words = size >> 2;
skb_queue_head_init(&tx->backlog);
- for (order = 0; size > (1 << (order+10)); order++);
+ for (order = 0; size > (1 << (order+10)); order++)
+ ;
eni_out((order << MID_SIZE_SHIFT) |
((tx->send-eni_dev->ram) >> (MID_LOC_SKIP+2)),
MID_TX_PLACE(tx->index));
@@ -1335,9 +1378,11 @@ static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
MID_DESCR_START;
}
error = comp_tx(eni_dev, &rate, tx->reserved, &pre, &res, unlimited);
- if (!error && txtp->min_pcr > rate) error = -EINVAL;
+ if (!error && txtp->min_pcr > rate)
+ error = -EINVAL;
if (!error && txtp->max_pcr && txtp->max_pcr != ATM_MAX_PCR &&
- txtp->max_pcr < rate) error = -EINVAL;
+ txtp->max_pcr < rate)
+ error = -EINVAL;
if (!error && !ubr && rate > eni_dev->tx_bw+tx->reserved)
error = -EINVAL;
if (!error && set_rsv && !set_shp && rate < tx->shaping)
@@ -1358,12 +1403,14 @@ static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
eni_dev->tx_bw -= rate;
}
if (set_shp || (unlimited && new_tx)) {
- if (unlimited && new_tx) eni_dev->ubr = tx;
+ if (unlimited && new_tx)
+ eni_dev->ubr = tx;
tx->prescaler = pre;
tx->resolution = res;
tx->shaping = rate;
}
- if (set_shp) eni_vcc->tx = tx;
+ if (set_shp)
+ eni_vcc->tx = tx;
DPRINTK("rsv %d shp %d\n", tx->reserved, tx->shaping);
return 0;
}
@@ -1372,7 +1419,8 @@ static int reserve_or_set_tx(struct atm_vcc *vcc, struct atm_trafprm *txtp,
static int open_tx_first(struct atm_vcc *vcc)
{
ENI_VCC(vcc)->tx = NULL;
- if (vcc->qos.txtp.traffic_class == ATM_NONE) return 0;
+ if (vcc->qos.txtp.traffic_class == ATM_NONE)
+ return 0;
ENI_VCC(vcc)->txing = 0;
return reserve_or_set_tx(vcc, &vcc->qos.txtp, 1, 1);
}
@@ -1391,7 +1439,8 @@ static void close_tx(struct atm_vcc *vcc)
struct eni_vcc *eni_vcc;
eni_vcc = ENI_VCC(vcc);
- if (!eni_vcc->tx) return;
+ if (!eni_vcc->tx)
+ return;
eni_dev = ENI_DEV(vcc->dev);
/* wait for TX queue to drain */
DPRINTK("eni_close: waiting for TX ...\n");
@@ -1403,7 +1452,8 @@ static void close_tx(struct atm_vcc *vcc)
tasklet_disable(&eni_dev->task);
txing = skb_peek(&eni_vcc->tx->backlog) || eni_vcc->txing;
tasklet_enable(&eni_dev->task);
- if (!txing) break;
+ if (!txing)
+ break;
DPRINTK("%d TX left\n", eni_vcc->txing);
schedule();
set_current_state(TASK_UNINTERRUPTIBLE);
@@ -1460,7 +1510,8 @@ static void foo(void)
"backlogged=%d,rx_enqueued=%d,rx_dequeued=%d,putting=%d,pushed=%d\n",
tx_complete, dma_complete, queued, requeued, submitted, backlogged,
rx_enqueued, rx_dequeued, putting, pushed);
- if (eni_boards) printk(KERN_INFO "loss: %ld\n", ENI_DEV(eni_boards)->lost);
+ if (eni_boards)
+ printk(KERN_INFO "loss: %ld\n", ENI_DEV(eni_boards)->lost);
}
#endif
@@ -1634,7 +1685,8 @@ static int get_esi_asic(struct atm_dev *dev)
SET_SEPROM;
tonga |= SEPROM_DATA;
SET_SEPROM;
- if (failed) error = -EIO;
+ if (failed)
+ error = -EIO;
else {
dev->esi[i] = 0;
for (j = 7; j >= 0; j--) {
@@ -1644,7 +1696,8 @@ static int get_esi_asic(struct atm_dev *dev)
tonga |= SEPROM_CLK;
SET_SEPROM;
GET_SEPROM;
- if (tonga & SEPROM_DATA) dev->esi[i] |= 1;
+ if (tonga & SEPROM_DATA)
+ dev->esi[i] |= 1;
tonga &= ~SEPROM_CLK;
SET_SEPROM;
tonga |= SEPROM_DATA;
@@ -1656,7 +1709,8 @@ static int get_esi_asic(struct atm_dev *dev)
tonga |= SEPROM_CLK;
SET_SEPROM;
GET_SEPROM;
- if (!(tonga & SEPROM_DATA)) error = -EIO;
+ if (!(tonga & SEPROM_DATA))
+ error = -EIO;
tonga &= ~SEPROM_CLK;
SET_SEPROM;
tonga |= SEPROM_DATA;
@@ -1689,7 +1743,8 @@ static int get_esi_fpga(struct atm_dev *dev, void __iomem *base)
int i;
mac_base = base+EPROM_SIZE-sizeof(struct midway_eprom);
- for (i = 0; i < ESI_LEN; i++) dev->esi[i] = readb(mac_base+(i^3));
+ for (i = 0; i < ESI_LEN; i++)
+ dev->esi[i] = readb(mac_base+(i^3));
return 0;
}
@@ -1747,15 +1802,19 @@ static int eni_do_init(struct atm_dev *dev)
last = MAP_MAX_SIZE-RAM_BASE;
for (i = last-RAM_INCREMENT; i >= 0; i -= RAM_INCREMENT) {
writel(0x55555555, eni_dev->ram+i);
- if (readl(eni_dev->ram+i) != 0x55555555) last = i;
+ if (readl(eni_dev->ram+i) != 0x55555555)
+ last = i;
else {
writel(0xAAAAAAAA, eni_dev->ram+i);
- if (readl(eni_dev->ram+i) != 0xAAAAAAAA) last = i;
- else writel(i, eni_dev->ram+i);
+ if (readl(eni_dev->ram+i) != 0xAAAAAAAA)
+ last = i;
+ else
+ writel(i, eni_dev->ram+i);
}
}
for (i = 0; i < last; i += RAM_INCREMENT)
- if (readl(eni_dev->ram+i) != i) break;
+ if (readl(eni_dev->ram+i) != i)
+ break;
eni_dev->mem = i;
memset_io(eni_dev->ram, 0, eni_dev->mem);
/* TODO: should shrink allocation now */
@@ -1864,11 +1923,14 @@ static int eni_start(struct atm_dev *dev)
*/
eni_out(0xffffffff, MID_IE);
error = start_tx(dev);
- if (error) goto free_list;
+ if (error)
+ goto free_list;
error = start_rx(dev);
- if (error) goto free_list;
+ if (error)
+ goto free_list;
error = dev->phy->start(dev);
- if (error) goto free_list;
+ if (error)
+ goto free_list;
eni_out(eni_in(MID_MC_S) | (1 << MID_INT_SEL_SHIFT) |
MID_TX_LOCK_MODE | MID_DMA_ENABLE | MID_TX_ENABLE | MID_RX_ENABLE,
MID_MC_S);
@@ -1890,7 +1952,8 @@ out:
static void eni_close(struct atm_vcc *vcc)
{
DPRINTK(">eni_close\n");
- if (!ENI_VCC(vcc)) return;
+ if (!ENI_VCC(vcc))
+ return;
clear_bit(ATM_VF_READY, &vcc->flags);
close_rx(vcc);
close_tx(vcc);
@@ -1922,7 +1985,8 @@ static int eni_open(struct atm_vcc *vcc)
vcc->vci);
if (!test_bit(ATM_VF_PARTIAL, &vcc->flags)) {
eni_vcc = kmalloc(sizeof(struct eni_vcc), GFP_KERNEL);
- if (!eni_vcc) return -ENOMEM;
+ if (!eni_vcc)
+ return -ENOMEM;
vcc->dev_data = eni_vcc;
eni_vcc->tx = NULL; /* for eni_close after open_rx */
if ((error = open_rx_first(vcc))) {
@@ -1934,7 +1998,8 @@ static int eni_open(struct atm_vcc *vcc)
return error;
}
}
- if (vci == ATM_VPI_UNSPEC || vpi == ATM_VCI_UNSPEC) return 0;
+ if (vci == ATM_VPI_UNSPEC || vpi == ATM_VCI_UNSPEC)
+ return 0;
if ((error = open_rx_second(vcc))) {
eni_close(vcc);
return error;
@@ -1956,19 +2021,29 @@ static int eni_change_qos(struct atm_vcc *vcc, struct atm_qos *qos, int flgs)
struct sk_buff *skb;
int error, rate, rsv, shp;
- if (qos->txtp.traffic_class == ATM_NONE) return 0;
- if (tx == eni_dev->ubr) return -EBADFD;
+ if (qos->txtp.traffic_class == ATM_NONE)
+ return 0;
+ if (tx == eni_dev->ubr)
+ return -EBADFD;
rate = atm_pcr_goal(&qos->txtp);
- if (rate < 0) rate = -rate;
+ if (rate < 0)
+ rate = -rate;
rsv = shp = 0;
- if ((flgs & ATM_MF_DEC_RSV) && rate && rate < tx->reserved) rsv = 1;
- if ((flgs & ATM_MF_INC_RSV) && (!rate || rate > tx->reserved)) rsv = 1;
- if ((flgs & ATM_MF_DEC_SHP) && rate && rate < tx->shaping) shp = 1;
- if ((flgs & ATM_MF_INC_SHP) && (!rate || rate > tx->shaping)) shp = 1;
- if (!rsv && !shp) return 0;
+ if ((flgs & ATM_MF_DEC_RSV) && rate && rate < tx->reserved)
+ rsv = 1;
+ if ((flgs & ATM_MF_INC_RSV) && (!rate || rate > tx->reserved))
+ rsv = 1;
+ if ((flgs & ATM_MF_DEC_SHP) && rate && rate < tx->shaping)
+ shp = 1;
+ if ((flgs & ATM_MF_INC_SHP) && (!rate || rate > tx->shaping))
+ shp = 1;
+ if (!rsv && !shp)
+ return 0;
error = reserve_or_set_tx(vcc, &qos->txtp, rsv, shp);
- if (error) return error;
- if (shp && !(flgs & ATM_MF_IMMED)) return 0;
+ if (error)
+ return error;
+ if (shp && !(flgs & ATM_MF_IMMED))
+ return 0;
/*
* Walk through the send buffer and patch the rate information in all
* segmentation buffer descriptors of this VCC.
@@ -1977,7 +2052,8 @@ static int eni_change_qos(struct atm_vcc *vcc, struct atm_qos *qos, int flgs)
skb_queue_walk(&eni_dev->tx_queue, skb) {
void __iomem *dsc;
- if (ATM_SKB(skb)->vcc != vcc) continue;
+ if (ATM_SKB(skb)->vcc != vcc)
+ continue;
dsc = tx->send+ENI_PRV_POS(skb)*4;
writel((readl(dsc) & ~(MID_SEG_RATE | MID_SEG_PR)) |
(tx->prescaler << MID_SEG_PR_SHIFT) |
@@ -1993,7 +2069,8 @@ static int eni_ioctl(struct atm_dev *dev, unsigned int cmd, void __user *arg)
struct eni_dev *eni_dev = ENI_DEV(dev);
if (cmd == ENI_MEMDUMP) {
- if (!capable(CAP_NET_ADMIN)) return -EPERM;
+ if (!capable(CAP_NET_ADMIN))
+ return -EPERM;
printk(KERN_WARNING "Please use /proc/atm/" DEV_LABEL ":%d "
"instead of obsolete ioctl ENI_MEMDUMP\n", dev->number);
dump(dev);
@@ -2002,15 +2079,18 @@ static int eni_ioctl(struct atm_dev *dev, unsigned int cmd, void __user *arg)
if (cmd == ENI_SETMULT) {
struct eni_multipliers mult;
- if (!capable(CAP_NET_ADMIN)) return -EPERM;
+ if (!capable(CAP_NET_ADMIN))
+ return -EPERM;
if (copy_from_user(&mult, arg,
sizeof(struct eni_multipliers)))
return -EFAULT;
if ((mult.tx && mult.tx <= 100) || (mult.rx &&mult.rx <= 100) ||
mult.tx > 65536 || mult.rx > 65536)
return -EINVAL;
- if (mult.tx) eni_dev->tx_mult = mult.tx;
- if (mult.rx) eni_dev->rx_mult = mult.rx;
+ if (mult.tx)
+ eni_dev->tx_mult = mult.tx;
+ if (mult.rx)
+ eni_dev->rx_mult = mult.rx;
return 0;
}
if (cmd == ATM_SETCIRANGE) {
@@ -2023,7 +2103,8 @@ static int eni_ioctl(struct atm_dev *dev, unsigned int cmd, void __user *arg)
return 0;
return -EINVAL;
}
- if (!dev->phy->ioctl) return -ENOIOCTLCMD;
+ if (!dev->phy->ioctl)
+ return -ENOIOCTLCMD;
return dev->phy->ioctl(dev, cmd, arg);
}
@@ -2048,19 +2129,24 @@ static int eni_send(struct atm_vcc *vcc, struct sk_buff *skb)
DPRINTK(">eni_send\n");
if (!ENI_VCC(vcc)->tx) {
- if (vcc->pop) vcc->pop(vcc, skb);
- else dev_kfree_skb(skb);
+ if (vcc->pop)
+ vcc->pop(vcc, skb);
+ else
+ dev_kfree_skb(skb);
return -EINVAL;
}
if (!skb) {
printk(KERN_CRIT "!skb in eni_send ?\n");
- if (vcc->pop) vcc->pop(vcc, skb);
+ if (vcc->pop)
+ vcc->pop(vcc, skb);
return -EINVAL;
}
if (vcc->qos.aal == ATM_AAL0) {
if (skb->len != ATM_CELL_SIZE-1) {
- if (vcc->pop) vcc->pop(vcc, skb);
- else dev_kfree_skb(skb);
+ if (vcc->pop)
+ vcc->pop(vcc, skb);
+ else
+ dev_kfree_skb(skb);
return -EINVAL;
}
*(u32 *) skb->data = htonl(*(u32 *) skb->data);
@@ -2070,7 +2156,8 @@ static int eni_send(struct atm_vcc *vcc, struct sk_buff *skb)
tasklet_disable(&ENI_DEV(vcc->dev)->task);
res = do_tx(skb);
tasklet_enable(&ENI_DEV(vcc->dev)->task);
- if (res == enq_ok) return 0;
+ if (res == enq_ok)
+ return 0;
skb_queue_tail(&ENI_VCC(vcc)->tx->backlog, skb);
backlogged++;
tasklet_schedule(&ENI_DEV(vcc->dev)->task);
@@ -2154,7 +2241,8 @@ static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
for (i = 0; i < NR_CHAN; i++) {
struct eni_tx *tx = eni_dev->tx+i;
- if (!tx->send) continue;
+ if (!tx->send)
+ continue;
if (!--left) {
return sprintf(page, "tx[%d]: 0x%ld-0x%ld "
"(%6ld bytes), rsv %d cps, shp %d cps%s\n", i,
@@ -2163,7 +2251,8 @@ static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
tx->reserved, tx->shaping,
tx == eni_dev->ubr ? " (UBR)" : "");
}
- if (--left) continue;
+ if (--left)
+ continue;
return sprintf(page, "%10sbacklog %u packets\n", "",
skb_queue_len(&tx->backlog));
}
@@ -2179,7 +2268,8 @@ static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
if (vcc->dev != dev)
continue;
eni_vcc = ENI_VCC(vcc);
- if (--left) continue;
+ if (--left)
+ continue;
length = sprintf(page, "vcc %4d: ", vcc->vci);
if (eni_vcc->rx) {
length += sprintf(page+length, "0x%ld-0x%ld "
@@ -2187,7 +2277,8 @@ static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
(unsigned long) (eni_vcc->recv - eni_dev->ram),
eni_vcc->recv-eni_dev->ram+eni_vcc->words*4-1,
eni_vcc->words*4);
- if (eni_vcc->tx) length += sprintf(page+length, ", ");
+ if (eni_vcc->tx)
+ length += sprintf(page+length, ", ");
}
if (eni_vcc->tx)
length += sprintf(page+length, "tx[%d], txing %d bytes",
@@ -2202,7 +2293,8 @@ static int eni_proc_read(struct atm_dev *dev, loff_t *pos, char *page)
struct eni_free *fe = eni_dev->free_list+i;
unsigned long offset;
- if (--left) continue;
+ if (--left)
+ continue;
offset = (unsigned long) eni_dev->ram+eni_dev->base_diff;
return sprintf(page, "free %p-%p (%6d bytes)\n",
fe->start-offset, fe->start-offset+(1 << fe->order)-1,
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 5/5] drivers: atm: checkpatch.pl fixed coding style issues in eni.c
2013-01-20 23:12 ` [PATCH 5/5] drivers: atm: checkpatch.pl fixed coding style issues " Patrik Karlin
@ 2013-01-21 4:04 ` David Miller
0 siblings, 0 replies; 6+ messages in thread
From: David Miller @ 2013-01-21 4:04 UTC (permalink / raw)
To: patrik.karlin; +Cc: chas, linux-atm-general, netdev, linux-kernel
From: Patrik Karlin <patrik.karlin@gmail.com>
Date: Mon, 21 Jan 2013 00:12:55 +0100
> This patch fixes statement placement around if/else/for statments
> as suggested by checkpatch.pl
>
> Signed-off-by: Patrik Kårlin <patrik.karlin@gmail.com>
This patch set is a good example of why nobody should
fix up coding style in such a robotic way in response
to codingstyle.pl complaints.
> - ATM_MAX_AAL5_PDU) eff = (length+3) >> 2;
> + ATM_MAX_AAL5_PDU)
I bet you didn't even notice that in this change you are adding
trailing whitespace, the exact problem you fixed up for this file in a
previous patch of the series.
I really would encourage you to work on something else entirely.
Thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-01-21 4:04 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-20 23:12 [PATCH 1/5] drivers: atm: checkpatch.pl fixed coding style errors in eni.c Patrik Karlin
2013-01-20 23:12 ` [PATCH 2/5] drivers: atm: checkpatch.pl fixes removed all trailing whitespaces " Patrik Karlin
2013-01-20 23:12 ` [PATCH 3/5] drivers: atm: checkpatch.pl fixed tabbed indentation " Patrik Karlin
2013-01-20 23:12 ` [PATCH 4/5] drivers: atm: checkpatch.pl changed headers from asm/* to linux/* " Patrik Karlin
2013-01-20 23:12 ` [PATCH 5/5] drivers: atm: checkpatch.pl fixed coding style issues " Patrik Karlin
2013-01-21 4:04 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).