pcnet: code style cleanup

Fix up the code to match Documentation/CodingStyle. This is mostly
removing extraneous spaces.

No functional change is intended.

Signed-off-by: Paul Burton <paul.burton@imgtec.com>
This commit is contained in:
Paul Burton 2013-11-08 11:18:43 +00:00 committed by Daniel Schwierzeck
parent fa476f75bf
commit 6011dabd0a

View file

@ -89,39 +89,39 @@ static pcnet_priv_t *lp;
#define PCNET_RESET 0x14 #define PCNET_RESET 0x14
#define PCNET_BDP 0x16 #define PCNET_BDP 0x16
static u16 pcnet_read_csr (struct eth_device *dev, int index) static u16 pcnet_read_csr(struct eth_device *dev, int index)
{ {
outw (index, dev->iobase + PCNET_RAP); outw(index, dev->iobase + PCNET_RAP);
return inw (dev->iobase + PCNET_RDP); return inw(dev->iobase + PCNET_RDP);
} }
static void pcnet_write_csr (struct eth_device *dev, int index, u16 val) static void pcnet_write_csr(struct eth_device *dev, int index, u16 val)
{ {
outw (index, dev->iobase + PCNET_RAP); outw(index, dev->iobase + PCNET_RAP);
outw (val, dev->iobase + PCNET_RDP); outw(val, dev->iobase + PCNET_RDP);
} }
static u16 pcnet_read_bcr (struct eth_device *dev, int index) static u16 pcnet_read_bcr(struct eth_device *dev, int index)
{ {
outw (index, dev->iobase + PCNET_RAP); outw(index, dev->iobase + PCNET_RAP);
return inw (dev->iobase + PCNET_BDP); return inw(dev->iobase + PCNET_BDP);
} }
static void pcnet_write_bcr (struct eth_device *dev, int index, u16 val) static void pcnet_write_bcr(struct eth_device *dev, int index, u16 val)
{ {
outw (index, dev->iobase + PCNET_RAP); outw(index, dev->iobase + PCNET_RAP);
outw (val, dev->iobase + PCNET_BDP); outw(val, dev->iobase + PCNET_BDP);
} }
static void pcnet_reset (struct eth_device *dev) static void pcnet_reset(struct eth_device *dev)
{ {
inw (dev->iobase + PCNET_RESET); inw(dev->iobase + PCNET_RESET);
} }
static int pcnet_check (struct eth_device *dev) static int pcnet_check(struct eth_device *dev)
{ {
outw (88, dev->iobase + PCNET_RAP); outw(88, dev->iobase + PCNET_RAP);
return (inw (dev->iobase + PCNET_RAP) == 88); return inw(dev->iobase + PCNET_RAP) == 88;
} }
static int pcnet_init (struct eth_device *dev, bd_t * bis); static int pcnet_init (struct eth_device *dev, bd_t * bis);
@ -139,63 +139,64 @@ static struct pci_device_id supported[] = {
}; };
int pcnet_initialize (bd_t * bis) int pcnet_initialize(bd_t *bis)
{ {
pci_dev_t devbusfn; pci_dev_t devbusfn;
struct eth_device *dev; struct eth_device *dev;
u16 command, status; u16 command, status;
int dev_nr = 0; int dev_nr = 0;
PCNET_DEBUG1 ("\npcnet_initialize...\n"); PCNET_DEBUG1("\npcnet_initialize...\n");
for (dev_nr = 0;; dev_nr++) { for (dev_nr = 0;; dev_nr++) {
/* /*
* Find the PCnet PCI device(s). * Find the PCnet PCI device(s).
*/ */
if ((devbusfn = pci_find_devices (supported, dev_nr)) < 0) { devbusfn = pci_find_devices(supported, dev_nr);
if (devbusfn < 0)
break; break;
}
/* /*
* Allocate and pre-fill the device structure. * Allocate and pre-fill the device structure.
*/ */
dev = (struct eth_device *) malloc (sizeof *dev); dev = (struct eth_device *)malloc(sizeof(*dev));
if (!dev) { if (!dev) {
printf("pcnet: Can not allocate memory\n"); printf("pcnet: Can not allocate memory\n");
break; break;
} }
memset(dev, 0, sizeof(*dev)); memset(dev, 0, sizeof(*dev));
dev->priv = (void *) devbusfn; dev->priv = (void *)devbusfn;
sprintf (dev->name, "pcnet#%d", dev_nr); sprintf(dev->name, "pcnet#%d", dev_nr);
/* /*
* Setup the PCI device. * Setup the PCI device.
*/ */
pci_read_config_dword (devbusfn, PCI_BASE_ADDRESS_0, pci_read_config_dword(devbusfn, PCI_BASE_ADDRESS_0,
(unsigned int *) &dev->iobase); (unsigned int *)&dev->iobase);
dev->iobase=pci_io_to_phys (devbusfn, dev->iobase); dev->iobase = pci_io_to_phys(devbusfn, dev->iobase);
dev->iobase &= ~0xf; dev->iobase &= ~0xf;
PCNET_DEBUG1 ("%s: devbusfn=0x%x iobase=0x%x: ", PCNET_DEBUG1("%s: devbusfn=0x%x iobase=0x%x: ",
dev->name, devbusfn, dev->iobase); dev->name, devbusfn, dev->iobase);
command = PCI_COMMAND_IO | PCI_COMMAND_MASTER; command = PCI_COMMAND_IO | PCI_COMMAND_MASTER;
pci_write_config_word (devbusfn, PCI_COMMAND, command); pci_write_config_word(devbusfn, PCI_COMMAND, command);
pci_read_config_word (devbusfn, PCI_COMMAND, &status); pci_read_config_word(devbusfn, PCI_COMMAND, &status);
if ((status & command) != command) { if ((status & command) != command) {
printf ("%s: Couldn't enable IO access or Bus Mastering\n", dev->name); printf("%s: Couldn't enable IO access or Bus Mastering\n",
free (dev); dev->name);
free(dev);
continue; continue;
} }
pci_write_config_byte (devbusfn, PCI_LATENCY_TIMER, 0x40); pci_write_config_byte(devbusfn, PCI_LATENCY_TIMER, 0x40);
/* /*
* Probe the PCnet chip. * Probe the PCnet chip.
*/ */
if (pcnet_probe (dev, bis, dev_nr) < 0) { if (pcnet_probe(dev, bis, dev_nr) < 0) {
free (dev); free(dev);
continue; continue;
} }
@ -207,15 +208,15 @@ int pcnet_initialize (bd_t * bis)
dev->send = pcnet_send; dev->send = pcnet_send;
dev->recv = pcnet_recv; dev->recv = pcnet_recv;
eth_register (dev); eth_register(dev);
} }
udelay (10 * 1000); udelay(10 * 1000);
return dev_nr; return dev_nr;
} }
static int pcnet_probe (struct eth_device *dev, bd_t * bis, int dev_nr) static int pcnet_probe(struct eth_device *dev, bd_t *bis, int dev_nr)
{ {
int chip_version; int chip_version;
char *chipname; char *chipname;
@ -225,17 +226,17 @@ static int pcnet_probe (struct eth_device *dev, bd_t * bis, int dev_nr)
#endif #endif
/* Reset the PCnet controller */ /* Reset the PCnet controller */
pcnet_reset (dev); pcnet_reset(dev);
/* Check if register access is working */ /* Check if register access is working */
if (pcnet_read_csr (dev, 0) != 4 || !pcnet_check (dev)) { if (pcnet_read_csr(dev, 0) != 4 || !pcnet_check(dev)) {
printf ("%s: CSR register access check failed\n", dev->name); printf("%s: CSR register access check failed\n", dev->name);
return -1; return -1;
} }
/* Identify the chip */ /* Identify the chip */
chip_version = chip_version =
pcnet_read_csr (dev, 88) | (pcnet_read_csr (dev, 89) << 16); pcnet_read_csr(dev, 88) | (pcnet_read_csr(dev, 89) << 16);
if ((chip_version & 0xfff) != 0x003) if ((chip_version & 0xfff) != 0x003)
return -1; return -1;
chip_version = (chip_version >> 12) & 0xffff; chip_version = (chip_version >> 12) & 0xffff;
@ -254,12 +255,12 @@ static int pcnet_probe (struct eth_device *dev, bd_t * bis, int dev_nr)
break; break;
#endif #endif
default: default:
printf ("%s: PCnet version %#x not supported\n", printf("%s: PCnet version %#x not supported\n",
dev->name, chip_version); dev->name, chip_version);
return -1; return -1;
} }
PCNET_DEBUG1 ("AMD %s\n", chipname); PCNET_DEBUG1("AMD %s\n", chipname);
#ifdef PCNET_HAS_PROM #ifdef PCNET_HAS_PROM
/* /*
@ -270,7 +271,7 @@ static int pcnet_probe (struct eth_device *dev, bd_t * bis, int dev_nr)
for (i = 0; i < 3; i++) { for (i = 0; i < 3; i++) {
unsigned int val; unsigned int val;
val = pcnet_read_csr (dev, i + 12) & 0x0ffff; val = pcnet_read_csr(dev, i + 12) & 0x0ffff;
/* There may be endianness issues here. */ /* There may be endianness issues here. */
dev->enetaddr[2 * i] = val & 0x0ff; dev->enetaddr[2 * i] = val & 0x0ff;
dev->enetaddr[2 * i + 1] = (val >> 8) & 0x0ff; dev->enetaddr[2 * i + 1] = (val >> 8) & 0x0ff;
@ -280,35 +281,35 @@ static int pcnet_probe (struct eth_device *dev, bd_t * bis, int dev_nr)
return 0; return 0;
} }
static int pcnet_init (struct eth_device *dev, bd_t * bis) static int pcnet_init(struct eth_device *dev, bd_t *bis)
{ {
int i, val; int i, val;
u32 addr; u32 addr;
PCNET_DEBUG1 ("%s: pcnet_init...\n", dev->name); PCNET_DEBUG1("%s: pcnet_init...\n", dev->name);
/* Switch pcnet to 32bit mode */ /* Switch pcnet to 32bit mode */
pcnet_write_bcr (dev, 20, 2); pcnet_write_bcr(dev, 20, 2);
#ifdef CONFIG_PN62 #ifdef CONFIG_PN62
/* Setup LED registers */ /* Setup LED registers */
val = pcnet_read_bcr (dev, 2) | 0x1000; val = pcnet_read_bcr(dev, 2) | 0x1000;
pcnet_write_bcr (dev, 2, val); /* enable LEDPE */ pcnet_write_bcr(dev, 2, val); /* enable LEDPE */
pcnet_write_bcr (dev, 4, 0x5080); /* 100MBit */ pcnet_write_bcr(dev, 4, 0x5080); /* 100MBit */
pcnet_write_bcr (dev, 5, 0x40c0); /* LNKSE */ pcnet_write_bcr(dev, 5, 0x40c0); /* LNKSE */
pcnet_write_bcr (dev, 6, 0x4090); /* TX Activity */ pcnet_write_bcr(dev, 6, 0x4090); /* TX Activity */
pcnet_write_bcr (dev, 7, 0x4084); /* RX Activity */ pcnet_write_bcr(dev, 7, 0x4084); /* RX Activity */
#endif #endif
/* Set/reset autoselect bit */ /* Set/reset autoselect bit */
val = pcnet_read_bcr (dev, 2) & ~2; val = pcnet_read_bcr(dev, 2) & ~2;
val |= 2; val |= 2;
pcnet_write_bcr (dev, 2, val); pcnet_write_bcr(dev, 2, val);
/* Enable auto negotiate, setup, disable fd */ /* Enable auto negotiate, setup, disable fd */
val = pcnet_read_bcr (dev, 32) & ~0x98; val = pcnet_read_bcr(dev, 32) & ~0x98;
val |= 0x20; val |= 0x20;
pcnet_write_bcr (dev, 32, val); pcnet_write_bcr(dev, 32, val);
/* /*
* We only maintain one structure because the drivers will never * We only maintain one structure because the drivers will never
@ -316,12 +317,12 @@ static int pcnet_init (struct eth_device *dev, bd_t * bis)
* must be aligned on 16-byte boundaries. * must be aligned on 16-byte boundaries.
*/ */
if (lp == NULL) { if (lp == NULL) {
addr = (u32) malloc (sizeof (pcnet_priv_t) + 0x10); addr = (u32)malloc(sizeof(pcnet_priv_t) + 0x10);
addr = (addr + 0xf) & ~0xf; addr = (addr + 0xf) & ~0xf;
lp = (pcnet_priv_t *) addr; lp = (pcnet_priv_t *)addr;
} }
lp->init_block.mode = cpu_to_le16 (0x0000); lp->init_block.mode = cpu_to_le16(0x0000);
lp->init_block.filter[0] = 0x00000000; lp->init_block.filter[0] = 0x00000000;
lp->init_block.filter[1] = 0x00000000; lp->init_block.filter[1] = 0x00000000;
@ -330,9 +331,9 @@ static int pcnet_init (struct eth_device *dev, bd_t * bis)
*/ */
lp->cur_rx = 0; lp->cur_rx = 0;
for (i = 0; i < RX_RING_SIZE; i++) { for (i = 0; i < RX_RING_SIZE; i++) {
lp->rx_ring[i].base = PCI_TO_MEM_LE (dev, lp->rx_buf[i]); lp->rx_ring[i].base = PCI_TO_MEM_LE(dev, lp->rx_buf[i]);
lp->rx_ring[i].buf_length = cpu_to_le16 (-PKT_BUF_SZ); lp->rx_ring[i].buf_length = cpu_to_le16(-PKT_BUF_SZ);
lp->rx_ring[i].status = cpu_to_le16 (0x8000); lp->rx_ring[i].status = cpu_to_le16(0x8000);
PCNET_DEBUG1 PCNET_DEBUG1
("Rx%d: base=0x%x buf_length=0x%hx status=0x%hx\n", i, ("Rx%d: base=0x%x buf_length=0x%hx status=0x%hx\n", i,
lp->rx_ring[i].base, lp->rx_ring[i].buf_length, lp->rx_ring[i].base, lp->rx_ring[i].buf_length,
@ -352,48 +353,48 @@ static int pcnet_init (struct eth_device *dev, bd_t * bis)
/* /*
* Setup Init Block. * Setup Init Block.
*/ */
PCNET_DEBUG1 ("Init block at 0x%p: MAC", &lp->init_block); PCNET_DEBUG1("Init block at 0x%p: MAC", &lp->init_block);
for (i = 0; i < 6; i++) { for (i = 0; i < 6; i++) {
lp->init_block.phys_addr[i] = dev->enetaddr[i]; lp->init_block.phys_addr[i] = dev->enetaddr[i];
PCNET_DEBUG1 (" %02x", lp->init_block.phys_addr[i]); PCNET_DEBUG1(" %02x", lp->init_block.phys_addr[i]);
} }
lp->init_block.tlen_rlen = cpu_to_le16 (TX_RING_LEN_BITS | lp->init_block.tlen_rlen = cpu_to_le16(TX_RING_LEN_BITS |
RX_RING_LEN_BITS); RX_RING_LEN_BITS);
lp->init_block.rx_ring = PCI_TO_MEM_LE (dev, lp->rx_ring); lp->init_block.rx_ring = PCI_TO_MEM_LE(dev, lp->rx_ring);
lp->init_block.tx_ring = PCI_TO_MEM_LE (dev, lp->tx_ring); lp->init_block.tx_ring = PCI_TO_MEM_LE(dev, lp->tx_ring);
PCNET_DEBUG1 ("\ntlen_rlen=0x%x rx_ring=0x%x tx_ring=0x%x\n", PCNET_DEBUG1("\ntlen_rlen=0x%x rx_ring=0x%x tx_ring=0x%x\n",
lp->init_block.tlen_rlen, lp->init_block.tlen_rlen,
lp->init_block.rx_ring, lp->init_block.tx_ring); lp->init_block.rx_ring, lp->init_block.tx_ring);
/* /*
* Tell the controller where the Init Block is located. * Tell the controller where the Init Block is located.
*/ */
addr = PCI_TO_MEM (dev, &lp->init_block); addr = PCI_TO_MEM(dev, &lp->init_block);
pcnet_write_csr (dev, 1, addr & 0xffff); pcnet_write_csr(dev, 1, addr & 0xffff);
pcnet_write_csr (dev, 2, (addr >> 16) & 0xffff); pcnet_write_csr(dev, 2, (addr >> 16) & 0xffff);
pcnet_write_csr (dev, 4, 0x0915); pcnet_write_csr(dev, 4, 0x0915);
pcnet_write_csr (dev, 0, 0x0001); /* start */ pcnet_write_csr(dev, 0, 0x0001); /* start */
/* Wait for Init Done bit */ /* Wait for Init Done bit */
for (i = 10000; i > 0; i--) { for (i = 10000; i > 0; i--) {
if (pcnet_read_csr (dev, 0) & 0x0100) if (pcnet_read_csr(dev, 0) & 0x0100)
break; break;
udelay (10); udelay(10);
} }
if (i <= 0) { if (i <= 0) {
printf ("%s: TIMEOUT: controller init failed\n", dev->name); printf("%s: TIMEOUT: controller init failed\n", dev->name);
pcnet_reset (dev); pcnet_reset(dev);
return -1; return -1;
} }
/* /*
* Finally start network controller operation. * Finally start network controller operation.
*/ */
pcnet_write_csr (dev, 0, 0x0002); pcnet_write_csr(dev, 0, 0x0002);
return 0; return 0;
} }
@ -403,20 +404,20 @@ static int pcnet_send(struct eth_device *dev, void *packet, int pkt_len)
int i, status; int i, status;
struct pcnet_tx_head *entry = &lp->tx_ring[lp->cur_tx]; struct pcnet_tx_head *entry = &lp->tx_ring[lp->cur_tx];
PCNET_DEBUG2 ("Tx%d: %d bytes from 0x%p ", lp->cur_tx, pkt_len, PCNET_DEBUG2("Tx%d: %d bytes from 0x%p ", lp->cur_tx, pkt_len,
packet); packet);
/* Wait for completion by testing the OWN bit */ /* Wait for completion by testing the OWN bit */
for (i = 1000; i > 0; i--) { for (i = 1000; i > 0; i--) {
status = le16_to_cpu (entry->status); status = le16_to_cpu(entry->status);
if ((status & 0x8000) == 0) if ((status & 0x8000) == 0)
break; break;
udelay (100); udelay(100);
PCNET_DEBUG2 ("."); PCNET_DEBUG2(".");
} }
if (i <= 0) { if (i <= 0) {
printf ("%s: TIMEOUT: Tx%d failed (status = 0x%x)\n", printf("%s: TIMEOUT: Tx%d failed (status = 0x%x)\n",
dev->name, lp->cur_tx, status); dev->name, lp->cur_tx, status);
pkt_len = 0; pkt_len = 0;
goto failure; goto failure;
} }
@ -426,19 +427,19 @@ static int pcnet_send(struct eth_device *dev, void *packet, int pkt_len)
* set the status with the "ownership" bits last. * set the status with the "ownership" bits last.
*/ */
status = 0x8300; status = 0x8300;
entry->length = le16_to_cpu (-pkt_len); entry->length = le16_to_cpu(-pkt_len);
entry->misc = 0x00000000; entry->misc = 0x00000000;
entry->base = PCI_TO_MEM_LE (dev, packet); entry->base = PCI_TO_MEM_LE(dev, packet);
entry->status = le16_to_cpu (status); entry->status = le16_to_cpu(status);
/* Trigger an immediate send poll. */ /* Trigger an immediate send poll. */
pcnet_write_csr (dev, 0, 0x0008); pcnet_write_csr(dev, 0, 0x0008);
failure: failure:
if (++lp->cur_tx >= TX_RING_SIZE) if (++lp->cur_tx >= TX_RING_SIZE)
lp->cur_tx = 0; lp->cur_tx = 0;
PCNET_DEBUG2 ("done\n"); PCNET_DEBUG2("done\n");
return pkt_len; return pkt_len;
} }
@ -453,40 +454,38 @@ static int pcnet_recv (struct eth_device *dev)
/* /*
* If we own the next entry, it's a new packet. Send it up. * If we own the next entry, it's a new packet. Send it up.
*/ */
if (((status = le16_to_cpu (entry->status)) & 0x8000) != 0) { status = le16_to_cpu(entry->status);
if ((status & 0x8000) != 0)
break; break;
}
status >>= 8; status >>= 8;
if (status != 0x03) { /* There was an error. */ if (status != 0x03) { /* There was an error. */
printf("%s: Rx%d", dev->name, lp->cur_rx);
printf ("%s: Rx%d", dev->name, lp->cur_rx); PCNET_DEBUG1(" (status=0x%x)", status);
PCNET_DEBUG1 (" (status=0x%x)", status);
if (status & 0x20) if (status & 0x20)
printf (" Frame"); printf(" Frame");
if (status & 0x10) if (status & 0x10)
printf (" Overflow"); printf(" Overflow");
if (status & 0x08) if (status & 0x08)
printf (" CRC"); printf(" CRC");
if (status & 0x04) if (status & 0x04)
printf (" Fifo"); printf(" Fifo");
printf (" Error\n"); printf(" Error\n");
entry->status &= le16_to_cpu (0x03ff); entry->status &= le16_to_cpu(0x03ff);
} else { } else {
pkt_len = (le32_to_cpu(entry->msg_length) & 0xfff) - 4;
pkt_len =
(le32_to_cpu (entry->msg_length) & 0xfff) - 4;
if (pkt_len < 60) { if (pkt_len < 60) {
printf ("%s: Rx%d: invalid packet length %d\n", dev->name, lp->cur_rx, pkt_len); printf("%s: Rx%d: invalid packet length %d\n",
dev->name, lp->cur_rx, pkt_len);
} else { } else {
NetReceive (lp->rx_buf[lp->cur_rx], pkt_len); NetReceive(lp->rx_buf[lp->cur_rx], pkt_len);
PCNET_DEBUG2 ("Rx%d: %d bytes from 0x%p\n", PCNET_DEBUG2("Rx%d: %d bytes from 0x%p\n",
lp->cur_rx, pkt_len, lp->cur_rx, pkt_len,
lp->rx_buf[lp->cur_rx]); lp->rx_buf[lp->cur_rx]);
} }
} }
entry->status |= cpu_to_le16 (0x8000); entry->status |= cpu_to_le16(0x8000);
if (++lp->cur_rx >= RX_RING_SIZE) if (++lp->cur_rx >= RX_RING_SIZE)
lp->cur_rx = 0; lp->cur_rx = 0;
@ -494,22 +493,21 @@ static int pcnet_recv (struct eth_device *dev)
return pkt_len; return pkt_len;
} }
static void pcnet_halt (struct eth_device *dev) static void pcnet_halt(struct eth_device *dev)
{ {
int i; int i;
PCNET_DEBUG1 ("%s: pcnet_halt...\n", dev->name); PCNET_DEBUG1("%s: pcnet_halt...\n", dev->name);
/* Reset the PCnet controller */ /* Reset the PCnet controller */
pcnet_reset (dev); pcnet_reset(dev);
/* Wait for Stop bit */ /* Wait for Stop bit */
for (i = 1000; i > 0; i--) { for (i = 1000; i > 0; i--) {
if (pcnet_read_csr (dev, 0) & 0x4) if (pcnet_read_csr(dev, 0) & 0x4)
break; break;
udelay (10); udelay(10);
}
if (i <= 0) {
printf ("%s: TIMEOUT: controller reset failed\n", dev->name);
} }
if (i <= 0)
printf("%s: TIMEOUT: controller reset failed\n", dev->name);
} }