提交 b5e47729 编写于 作者: S Stefan Richter

firewire: nosy: misc cleanups

Extend copyright note to 2007, c.f. Kristian's git log.

Includes:
  - replace some <asm/*.h> by <linux/*.h>
  - add required indirectly included <linux/spinlock.h>
  - order alphabetically

Coding style related changes:
  - change to utf8
  - normalize whitespace
  - normalize comment style
  - remove usages of __FUNCTION__
  - remove an unnecessary cast from void *

Const and static declarations:
  - driver_name is not const in pci_driver.name, drop const qualifier
  - driver_name can be taken from KBUILD_MODNAME
  - the global variable minors[] can and should be static
  - constify struct file_operations instance

Data types:
  - Remove unused struct member struct packet.code.  struct packet is
    only used for driver-internal bookkeeping; it does not appear on the
    wire or in DMA programs or the userspace ABI.  Hence the unused
    member .code can be removed without worries.

Preprocessor macros:
  - unroll a preprocessor macro that containd a return
  - use list_for_each_entry

Printk:
  - add missing terminating \n in some format strings
Signed-off-by: NStefan Richter <stefanr@s5r6.in-berlin.de>
上级 28646821
#ifndef __nosy_user_h #ifndef __nosy_user_h
#define __nosy_user_h #define __nosy_user_h
#include <asm/ioctl.h> #include <linux/ioctl.h>
#include <asm/types.h> #include <linux/types.h>
#define NOSY_IOC_GET_STATS _IOR('&', 0, struct nosy_stats) #define NOSY_IOC_GET_STATS _IOR('&', 0, struct nosy_stats)
#define NOSY_IOC_START _IO('&', 1) #define NOSY_IOC_START _IO('&', 1)
...@@ -10,11 +10,11 @@ ...@@ -10,11 +10,11 @@
#define NOSY_IOC_FILTER _IOW('&', 2, __u32) #define NOSY_IOC_FILTER _IOW('&', 2, __u32)
struct nosy_stats { struct nosy_stats {
__u32 total_packet_count; __u32 total_packet_count;
__u32 lost_packet_count; __u32 lost_packet_count;
}; };
/* /*
* Format of packets returned from the kernel driver: * Format of packets returned from the kernel driver:
* *
* quadlet with timestamp (microseconds) * quadlet with timestamp (microseconds)
......
/* -*- c-file-style: "linux" -*- /*
* * nosy - Snoop mode driver for TI PCILynx 1394 controllers
* nosy.c - Snoop mode driver for TI pcilynx 1394 controllers * Copyright (C) 2002-2007 Kristian Høgsberg
* Copyright (C) 2002 Kristian Hgsberg
* *
* This program is free software; you can redistribute it and/or modify * This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by * it under the terms of the GNU General Public License as published by
...@@ -18,23 +17,25 @@ ...@@ -18,23 +17,25 @@
* Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. * Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
*/ */
#include <linux/kernel.h>
#include <linux/slab.h>
#include <linux/interrupt.h>
#include <linux/sched.h> /* required for linux/wait.h */
#include <linux/wait.h>
#include <linux/errno.h> #include <linux/errno.h>
#include <linux/module.h> #include <linux/fs.h>
#include <linux/init.h> #include <linux/init.h>
#include <linux/interrupt.h>
#include <linux/io.h>
#include <linux/kernel.h>
#include <linux/miscdevice.h>
#include <linux/module.h>
#include <linux/pci.h> #include <linux/pci.h>
#include <linux/fs.h>
#include <linux/poll.h> #include <linux/poll.h>
#include <linux/miscdevice.h> #include <linux/sched.h> /* required for linux/wait.h */
#include <asm/byteorder.h> #include <linux/slab.h>
#include <linux/spinlock.h>
#include <linux/timex.h>
#include <linux/uaccess.h>
#include <linux/wait.h>
#include <asm/atomic.h> #include <asm/atomic.h>
#include <asm/io.h> #include <asm/byteorder.h>
#include <asm/uaccess.h>
#include <asm/timex.h>
#include "nosy.h" #include "nosy.h"
#include "nosy-user.h" #include "nosy-user.h"
...@@ -46,7 +47,7 @@ ...@@ -46,7 +47,7 @@
#define error(s, args...) printk(KERN_ERR s, ## args) #define error(s, args...) printk(KERN_ERR s, ## args)
#define debug(s, args...) printk(KERN_DEBUG s, ## args) #define debug(s, args...) printk(KERN_DEBUG s, ## args)
static const char driver_name[] = "nosy"; static char driver_name[] = KBUILD_MODNAME;
struct pcl_status { struct pcl_status {
unsigned int transfer_count : 13; unsigned int transfer_count : 13;
...@@ -77,8 +78,7 @@ struct pcl { ...@@ -77,8 +78,7 @@ struct pcl {
} __attribute__ ((packed)); } __attribute__ ((packed));
struct packet { struct packet {
unsigned int length : 16; unsigned int length;
unsigned int code : 16;
char data[0]; char data[0];
}; };
...@@ -87,8 +87,8 @@ struct packet_buffer { ...@@ -87,8 +87,8 @@ struct packet_buffer {
size_t capacity; size_t capacity;
long total_packet_count, lost_packet_count; long total_packet_count, lost_packet_count;
atomic_t size; atomic_t size;
struct packet *head, *tail; struct packet *head, *tail;
wait_queue_head_t wait; wait_queue_head_t wait;
}; };
struct pcilynx { struct pcilynx {
...@@ -106,7 +106,6 @@ struct pcilynx { ...@@ -106,7 +106,6 @@ struct pcilynx {
struct miscdevice misc; struct miscdevice misc;
}; };
struct client { struct client {
struct pcilynx *lynx; struct pcilynx *lynx;
unsigned long tcode_mask; unsigned long tcode_mask;
...@@ -115,7 +114,7 @@ struct client { ...@@ -115,7 +114,7 @@ struct client {
}; };
#define MAX_MINORS 64 #define MAX_MINORS 64
struct pcilynx *minors[MAX_MINORS]; static struct pcilynx *minors[MAX_MINORS];
static int static int
packet_buffer_init(struct packet_buffer *buffer, size_t capacity) packet_buffer_init(struct packet_buffer *buffer, size_t capacity)
...@@ -128,7 +127,7 @@ packet_buffer_init(struct packet_buffer *buffer, size_t capacity) ...@@ -128,7 +127,7 @@ packet_buffer_init(struct packet_buffer *buffer, size_t capacity)
buffer->capacity = capacity; buffer->capacity = capacity;
buffer->lost_packet_count = 0; buffer->lost_packet_count = 0;
atomic_set(&buffer->size, 0); atomic_set(&buffer->size, 0);
init_waitqueue_head(&buffer->wait); init_waitqueue_head(&buffer->wait);
return 0; return 0;
} }
...@@ -158,8 +157,7 @@ packet_buffer_get(struct packet_buffer *buffer, void *data, size_t user_length) ...@@ -158,8 +157,7 @@ packet_buffer_get(struct packet_buffer *buffer, void *data, size_t user_length)
if (copy_to_user(data, buffer->head->data, length)) if (copy_to_user(data, buffer->head->data, length))
return -EFAULT; return -EFAULT;
buffer->head = (struct packet *) &buffer->head->data[length]; buffer->head = (struct packet *) &buffer->head->data[length];
} } else {
else {
size_t split = end - buffer->head->data; size_t split = end - buffer->head->data;
if (copy_to_user(data, buffer->head->data, split)) if (copy_to_user(data, buffer->head->data, split))
...@@ -169,11 +167,12 @@ packet_buffer_get(struct packet_buffer *buffer, void *data, size_t user_length) ...@@ -169,11 +167,12 @@ packet_buffer_get(struct packet_buffer *buffer, void *data, size_t user_length)
buffer->head = (struct packet *) &buffer->data[length - split]; buffer->head = (struct packet *) &buffer->data[length - split];
} }
/* Decrease buffer->size as the last thing, since this is what /*
* Decrease buffer->size as the last thing, since this is what
* keeps the interrupt from overwriting the packet we are * keeps the interrupt from overwriting the packet we are
* retrieving from the buffer. */ * retrieving from the buffer.
*/
atomic_sub(sizeof (struct packet) + length, &buffer->size); atomic_sub(sizeof(struct packet) + length, &buffer->size);
return length; return length;
} }
...@@ -185,8 +184,8 @@ packet_buffer_put(struct packet_buffer *buffer, void *data, size_t length) ...@@ -185,8 +184,8 @@ packet_buffer_put(struct packet_buffer *buffer, void *data, size_t length)
buffer->total_packet_count++; buffer->total_packet_count++;
if (buffer->capacity < if (buffer->capacity <
atomic_read(&buffer->size) + sizeof (struct packet) + length) { atomic_read(&buffer->size) + sizeof(struct packet) + length) {
buffer->lost_packet_count++; buffer->lost_packet_count++;
return; return;
} }
...@@ -197,69 +196,68 @@ packet_buffer_put(struct packet_buffer *buffer, void *data, size_t length) ...@@ -197,69 +196,68 @@ packet_buffer_put(struct packet_buffer *buffer, void *data, size_t length)
if (&buffer->tail->data[length] < end) { if (&buffer->tail->data[length] < end) {
memcpy(buffer->tail->data, data, length); memcpy(buffer->tail->data, data, length);
buffer->tail = (struct packet *) &buffer->tail->data[length]; buffer->tail = (struct packet *) &buffer->tail->data[length];
} } else {
else {
size_t split = end - buffer->tail->data; size_t split = end - buffer->tail->data;
memcpy(buffer->tail->data, data, split); memcpy(buffer->tail->data, data, split);
memcpy(buffer->data, data + split, length - split); memcpy(buffer->data, data + split, length - split);
buffer->tail = (struct packet *) &buffer->data[length - split]; buffer->tail = (struct packet *) &buffer->data[length - split];
} }
/* Finally, adjust buffer size and wake up userspace reader. */ /* Finally, adjust buffer size and wake up userspace reader. */
atomic_add(sizeof (struct packet) + length, &buffer->size); atomic_add(sizeof(struct packet) + length, &buffer->size);
wake_up_interruptible(&buffer->wait); wake_up_interruptible(&buffer->wait);
} }
static inline void static inline void
reg_write(struct pcilynx *lynx, int offset, u32 data) reg_write(struct pcilynx *lynx, int offset, u32 data)
{ {
writel(data, lynx->registers + offset); writel(data, lynx->registers + offset);
} }
static inline u32 static inline u32
reg_read(struct pcilynx *lynx, int offset) reg_read(struct pcilynx *lynx, int offset)
{ {
return readl(lynx->registers + offset); return readl(lynx->registers + offset);
} }
static inline void static inline void
reg_set_bits(struct pcilynx *lynx, int offset, u32 mask) reg_set_bits(struct pcilynx *lynx, int offset, u32 mask)
{ {
reg_write(lynx, offset, (reg_read(lynx, offset) | mask)); reg_write(lynx, offset, (reg_read(lynx, offset) | mask));
} }
/* Maybe the pcl programs could be setup to just append data instead /*
* of using a whole packet. */ * Maybe the pcl programs could be set up to just append data instead
* of using a whole packet.
static inline void */
run_pcl(struct pcilynx *lynx, dma_addr_t pcl_bus, int dmachan) static inline void
run_pcl(struct pcilynx *lynx, dma_addr_t pcl_bus,
int dmachan)
{ {
reg_write(lynx, DMA0_CURRENT_PCL + dmachan * 0x20, pcl_bus); reg_write(lynx, DMA0_CURRENT_PCL + dmachan * 0x20, pcl_bus);
reg_write(lynx, DMA0_CHAN_CTRL + dmachan * 0x20, reg_write(lynx, DMA0_CHAN_CTRL + dmachan * 0x20,
DMA_CHAN_CTRL_ENABLE | DMA_CHAN_CTRL_LINK); DMA_CHAN_CTRL_ENABLE | DMA_CHAN_CTRL_LINK);
} }
static int static int
set_phy_reg(struct pcilynx *lynx, int addr, int val) set_phy_reg(struct pcilynx *lynx, int addr, int val)
{ {
if (addr > 15) { if (addr > 15) {
debug("%s: PHY register address %d out of range", debug("PHY register address %d out of range\n", addr);
__FUNCTION__, addr); return -1;
return -1; }
}
if (val > 0xff) {
if (val > 0xff) { debug("PHY register value %d out of range\n", val);
debug("%s: PHY register value %d out of range", return -1;
__FUNCTION__, val); }
return -1;
} reg_write(lynx, LINK_PHY, LINK_PHY_WRITE |
reg_write(lynx, LINK_PHY, LINK_PHY_WRITE |
LINK_PHY_ADDR(addr) | LINK_PHY_WDATA(val)); LINK_PHY_ADDR(addr) | LINK_PHY_WDATA(val));
return 0; return 0;
} }
static void static void
...@@ -317,7 +315,7 @@ nosy_open(struct inode *inode, struct file *file) ...@@ -317,7 +315,7 @@ nosy_open(struct inode *inode, struct file *file)
if (minor > MAX_MINORS || minors[minor] == NULL) if (minor > MAX_MINORS || minors[minor] == NULL)
return -ENODEV; return -ENODEV;
file->private_data = nosy_add_client(minors[minor]); file->private_data = nosy_add_client(minors[minor]);
if (file->private_data == NULL) if (file->private_data == NULL)
return -ENOMEM; return -ENOMEM;
else else
...@@ -329,7 +327,7 @@ nosy_release(struct inode *inode, struct file *file) ...@@ -329,7 +327,7 @@ nosy_release(struct inode *inode, struct file *file)
{ {
nosy_remove_client(file->private_data); nosy_remove_client(file->private_data);
return 0; return 0;
} }
static unsigned int static unsigned int
...@@ -358,19 +356,17 @@ nosy_ioctl(struct inode *inode, struct file *file, ...@@ -358,19 +356,17 @@ nosy_ioctl(struct inode *inode, struct file *file,
unsigned int cmd, unsigned long arg) unsigned int cmd, unsigned long arg)
{ {
struct client *client = file->private_data; struct client *client = file->private_data;
struct nosy_stats stats;
switch (cmd) { switch (cmd) {
case NOSY_IOC_GET_STATS: { case NOSY_IOC_GET_STATS:
struct nosy_stats stats;
stats.total_packet_count = client->buffer.total_packet_count; stats.total_packet_count = client->buffer.total_packet_count;
stats.lost_packet_count = client->buffer.lost_packet_count; stats.lost_packet_count = client->buffer.lost_packet_count;
if (copy_to_user((void *) arg, &stats, sizeof stats)) if (copy_to_user((void *) arg, &stats, sizeof stats))
return -EFAULT; return -EFAULT;
else else
return 0; return 0;
}
case NOSY_IOC_START: case NOSY_IOC_START:
nosy_start_snoop(client); nosy_start_snoop(client);
return 0; return 0;
...@@ -389,13 +385,13 @@ nosy_ioctl(struct inode *inode, struct file *file, ...@@ -389,13 +385,13 @@ nosy_ioctl(struct inode *inode, struct file *file,
} }
} }
static struct file_operations nosy_ops = { static const struct file_operations nosy_ops = {
.owner = THIS_MODULE, .owner = THIS_MODULE,
.read = nosy_read, .read = nosy_read,
.ioctl = nosy_ioctl, .ioctl = nosy_ioctl,
.poll = nosy_poll, .poll = nosy_poll,
.open = nosy_open, .open = nosy_open,
.release = nosy_release, .release = nosy_release,
}; };
#define PHY_PACKET_SIZE 12 /* 1 payload, 1 inverse, 1 ack = 3 quadlets */ #define PHY_PACKET_SIZE 12 /* 1 payload, 1 inverse, 1 ack = 3 quadlets */
...@@ -412,7 +408,6 @@ static void ...@@ -412,7 +408,6 @@ static void
packet_handler(struct pcilynx *lynx) packet_handler(struct pcilynx *lynx)
{ {
unsigned long flags; unsigned long flags;
struct list_head *pos;
struct client *client; struct client *client;
unsigned long tcode_mask; unsigned long tcode_mask;
size_t length; size_t length;
...@@ -434,12 +429,10 @@ packet_handler(struct pcilynx *lynx) ...@@ -434,12 +429,10 @@ packet_handler(struct pcilynx *lynx)
spin_lock_irqsave(&lynx->client_list_lock, flags); spin_lock_irqsave(&lynx->client_list_lock, flags);
list_for_each(pos, &lynx->client_list) { list_for_each_entry(client, &lynx->client_list, link)
client = list_entry(pos, struct client, link);
if (client->tcode_mask & tcode_mask) if (client->tcode_mask & tcode_mask)
packet_buffer_put(&client->buffer, packet_buffer_put(&client->buffer,
lynx->rcv_buffer, length + 4); lynx->rcv_buffer, length + 4);
}
spin_unlock_irqrestore(&lynx->client_list_lock, flags); spin_unlock_irqrestore(&lynx->client_list_lock, flags);
} }
...@@ -448,7 +441,6 @@ static void ...@@ -448,7 +441,6 @@ static void
bus_reset_handler(struct pcilynx *lynx) bus_reset_handler(struct pcilynx *lynx)
{ {
unsigned long flags; unsigned long flags;
struct list_head *pos;
struct client *client; struct client *client;
struct timeval tv; struct timeval tv;
...@@ -456,23 +448,19 @@ bus_reset_handler(struct pcilynx *lynx) ...@@ -456,23 +448,19 @@ bus_reset_handler(struct pcilynx *lynx)
spin_lock_irqsave(&lynx->client_list_lock, flags); spin_lock_irqsave(&lynx->client_list_lock, flags);
list_for_each(pos, &lynx->client_list) { list_for_each_entry(client, &lynx->client_list, link)
client = list_entry(pos, struct client, link);
packet_buffer_put(&client->buffer, &tv.tv_usec, 4); packet_buffer_put(&client->buffer, &tv.tv_usec, 4);
}
spin_unlock_irqrestore(&lynx->client_list_lock, flags); spin_unlock_irqrestore(&lynx->client_list_lock, flags);
} }
static irqreturn_t static irqreturn_t
irq_handler(int irq, void *device) irq_handler(int irq, void *device)
{ {
struct pcilynx *lynx = (struct pcilynx *) device; struct pcilynx *lynx = device;
u32 pci_int_status; u32 pci_int_status;
pci_int_status = reg_read(lynx, PCI_INT_STATUS); pci_int_status = reg_read(lynx, PCI_INT_STATUS);
if ((pci_int_status & PCI_INT_INT_PEND) == 0) if ((pci_int_status & PCI_INT_INT_PEND) == 0)
/* Not our interrupt, bail out quickly. */ /* Not our interrupt, bail out quickly. */
...@@ -505,19 +493,19 @@ irq_handler(int irq, void *device) ...@@ -505,19 +493,19 @@ irq_handler(int irq, void *device)
static void static void
remove_card(struct pci_dev *dev) remove_card(struct pci_dev *dev)
{ {
struct pcilynx *lynx; struct pcilynx *lynx;
lynx = pci_get_drvdata(dev); lynx = pci_get_drvdata(dev);
if (!lynx) if (!lynx)
return; return;
pci_set_drvdata(dev, NULL); pci_set_drvdata(dev, NULL);
reg_write(lynx, PCI_INT_ENABLE, 0); reg_write(lynx, PCI_INT_ENABLE, 0);
free_irq(lynx->pci_device->irq, lynx); free_irq(lynx->pci_device->irq, lynx);
pci_free_consistent(lynx->pci_device, sizeof (struct pcl), pci_free_consistent(lynx->pci_device, sizeof(struct pcl),
lynx->rcv_start_pcl, lynx->rcv_start_pcl_bus); lynx->rcv_start_pcl, lynx->rcv_start_pcl_bus);
pci_free_consistent(lynx->pci_device, sizeof (struct pcl), pci_free_consistent(lynx->pci_device, sizeof(struct pcl),
lynx->rcv_pcl, lynx->rcv_pcl_bus); lynx->rcv_pcl, lynx->rcv_pcl_bus);
pci_free_consistent(lynx->pci_device, PAGE_SIZE, pci_free_consistent(lynx->pci_device, PAGE_SIZE,
lynx->rcv_buffer, lynx->rcv_buffer_bus); lynx->rcv_buffer, lynx->rcv_buffer_bus);
...@@ -532,64 +520,58 @@ remove_card(struct pci_dev *dev) ...@@ -532,64 +520,58 @@ remove_card(struct pci_dev *dev)
#define RCV_BUFFER_SIZE (16 * 1024) #define RCV_BUFFER_SIZE (16 * 1024)
#define FAIL(s, args...) \
do { \
error(s, ## args); \
return err; \
} while (0)
static int __devinit static int __devinit
add_card(struct pci_dev *dev, const struct pci_device_id *unused) add_card(struct pci_dev *dev, const struct pci_device_id *unused)
{ {
struct pcilynx *lynx; struct pcilynx *lynx;
u32 p, end; u32 p, end;
int err, i; int i;
err = -ENXIO;
if (pci_set_dma_mask(dev, 0xffffffff)) if (pci_set_dma_mask(dev, 0xffffffff)) {
FAIL("DMA address limits not supported " error("DMA address limits not supported "
"for PCILynx hardware.\n"); "for PCILynx hardware\n");
if (pci_enable_device(dev)) return -ENXIO;
FAIL("Failed to enable PCILynx hardware.\n"); }
pci_set_master(dev); if (pci_enable_device(dev)) {
error("Failed to enable PCILynx hardware\n");
err = -ENOMEM; return -ENXIO;
}
pci_set_master(dev);
lynx = kzalloc(sizeof *lynx, GFP_KERNEL); lynx = kzalloc(sizeof *lynx, GFP_KERNEL);
if (lynx == NULL) if (lynx == NULL) {
FAIL("Failed to allocate control structure memory.\n"); error("Failed to allocate control structure memory\n");
return -ENOMEM;
lynx->pci_device = dev; }
pci_set_drvdata(dev, lynx); lynx->pci_device = dev;
pci_set_drvdata(dev, lynx);
spin_lock_init(&lynx->client_list_lock); spin_lock_init(&lynx->client_list_lock);
INIT_LIST_HEAD(&lynx->client_list); INIT_LIST_HEAD(&lynx->client_list);
lynx->registers = ioremap_nocache(pci_resource_start(dev, 0), lynx->registers = ioremap_nocache(pci_resource_start(dev, 0),
PCILYNX_MAX_REGISTER); PCILYNX_MAX_REGISTER);
lynx->rcv_start_pcl = pci_alloc_consistent(lynx->pci_device, lynx->rcv_start_pcl = pci_alloc_consistent(lynx->pci_device,
sizeof(struct pcl), sizeof(struct pcl), &lynx->rcv_start_pcl_bus);
&lynx->rcv_start_pcl_bus); lynx->rcv_pcl = pci_alloc_consistent(lynx->pci_device,
lynx->rcv_pcl = pci_alloc_consistent(lynx->pci_device, sizeof(struct pcl), &lynx->rcv_pcl_bus);
sizeof(struct pcl), lynx->rcv_buffer = pci_alloc_consistent(lynx->pci_device,
&lynx->rcv_pcl_bus); RCV_BUFFER_SIZE, &lynx->rcv_buffer_bus);
lynx->rcv_buffer = pci_alloc_consistent(lynx->pci_device, RCV_BUFFER_SIZE, if (lynx->rcv_start_pcl == NULL ||
&lynx->rcv_buffer_bus);
if (lynx->rcv_start_pcl == NULL ||
lynx->rcv_pcl == NULL || lynx->rcv_pcl == NULL ||
lynx->rcv_buffer == NULL) lynx->rcv_buffer == NULL) {
/* FIXME: do proper error handling. */ /* FIXME: do proper error handling. */
FAIL("Failed to allocate receive buffer.\n"); error("Failed to allocate receive buffer\n");
return -ENOMEM;
}
lynx->rcv_start_pcl->next = lynx->rcv_pcl_bus; lynx->rcv_start_pcl->next = lynx->rcv_pcl_bus;
lynx->rcv_pcl->next = PCL_NEXT_INVALID; lynx->rcv_pcl->next = PCL_NEXT_INVALID;
lynx->rcv_pcl->async_error_next = PCL_NEXT_INVALID; lynx->rcv_pcl->async_error_next = PCL_NEXT_INVALID;
lynx->rcv_pcl->buffer[0].control = lynx->rcv_pcl->buffer[0].control =
PCL_CMD_RCV | PCL_BIGENDIAN | 2044; PCL_CMD_RCV | PCL_BIGENDIAN | 2044;
lynx->rcv_pcl->buffer[0].pointer = lynx->rcv_buffer_bus + 4; lynx->rcv_pcl->buffer[0].pointer = lynx->rcv_buffer_bus + 4;
p = lynx->rcv_buffer_bus + 2048; p = lynx->rcv_buffer_bus + 2048;
end = lynx->rcv_buffer_bus + RCV_BUFFER_SIZE; end = lynx->rcv_buffer_bus + RCV_BUFFER_SIZE;
for (i = 1; p < end; i++, p += 2048) { for (i = 1; p < end; i++, p += 2048) {
...@@ -599,31 +581,31 @@ add_card(struct pci_dev *dev, const struct pci_device_id *unused) ...@@ -599,31 +581,31 @@ add_card(struct pci_dev *dev, const struct pci_device_id *unused)
} }
lynx->rcv_pcl->buffer[i - 1].control |= PCL_LAST_BUFF; lynx->rcv_pcl->buffer[i - 1].control |= PCL_LAST_BUFF;
reg_set_bits(lynx, MISC_CONTROL, MISC_CONTROL_SWRESET); reg_set_bits(lynx, MISC_CONTROL, MISC_CONTROL_SWRESET);
/* Fix buggy cards with autoboot pin not tied low: */ /* Fix buggy cards with autoboot pin not tied low: */
reg_write(lynx, DMA0_CHAN_CTRL, 0); reg_write(lynx, DMA0_CHAN_CTRL, 0);
reg_write(lynx, DMA_GLOBAL_REGISTER, 0x00 << 24); reg_write(lynx, DMA_GLOBAL_REGISTER, 0x00 << 24);
#if 0 #if 0
/* now, looking for PHY register set */ /* now, looking for PHY register set */
if ((get_phy_reg(lynx, 2) & 0xe0) == 0xe0) { if ((get_phy_reg(lynx, 2) & 0xe0) == 0xe0) {
lynx->phyic.reg_1394a = 1; lynx->phyic.reg_1394a = 1;
PRINT(KERN_INFO, lynx->id, PRINT(KERN_INFO, lynx->id,
"found 1394a conform PHY (using extended register set)"); "found 1394a conform PHY (using extended register set)");
lynx->phyic.vendor = get_phy_vendorid(lynx); lynx->phyic.vendor = get_phy_vendorid(lynx);
lynx->phyic.product = get_phy_productid(lynx); lynx->phyic.product = get_phy_productid(lynx);
} else { } else {
lynx->phyic.reg_1394a = 0; lynx->phyic.reg_1394a = 0;
PRINT(KERN_INFO, lynx->id, "found old 1394 PHY"); PRINT(KERN_INFO, lynx->id, "found old 1394 PHY");
} }
#endif #endif
/* Setup the general receive FIFO max size. */ /* Setup the general receive FIFO max size. */
reg_write(lynx, FIFO_SIZES, 255); reg_write(lynx, FIFO_SIZES, 255);
reg_set_bits(lynx, PCI_INT_ENABLE, PCI_INT_DMA_ALL); reg_set_bits(lynx, PCI_INT_ENABLE, PCI_INT_DMA_ALL);
reg_write(lynx, LINK_INT_ENABLE, reg_write(lynx, LINK_INT_ENABLE,
LINK_INT_PHY_TIME_OUT | LINK_INT_PHY_REG_RCVD | LINK_INT_PHY_TIME_OUT | LINK_INT_PHY_REG_RCVD |
LINK_INT_PHY_BUSRESET | LINK_INT_IT_STUCK | LINK_INT_PHY_BUSRESET | LINK_INT_IT_STUCK |
LINK_INT_AT_STUCK | LINK_INT_SNTRJ | LINK_INT_AT_STUCK | LINK_INT_SNTRJ |
...@@ -638,58 +620,60 @@ add_card(struct pci_dev *dev, const struct pci_device_id *unused) ...@@ -638,58 +620,60 @@ add_card(struct pci_dev *dev, const struct pci_device_id *unused)
run_pcl(lynx, lynx->rcv_start_pcl_bus, 0); run_pcl(lynx, lynx->rcv_start_pcl_bus, 0);
if (request_irq(dev->irq, irq_handler, IRQF_SHARED, driver_name, lynx)) if (request_irq(dev->irq, irq_handler, IRQF_SHARED,
FAIL("Failed to allocate shared interrupt %d.", dev->irq); driver_name, lynx)) {
error("Failed to allocate shared interrupt %d\n", dev->irq);
return -EIO;
}
lynx->misc.parent = &dev->dev; lynx->misc.parent = &dev->dev;
lynx->misc.minor = MISC_DYNAMIC_MINOR; lynx->misc.minor = MISC_DYNAMIC_MINOR;
lynx->misc.name = "nosy"; lynx->misc.name = "nosy";
lynx->misc.fops = &nosy_ops; lynx->misc.fops = &nosy_ops;
if (misc_register(&lynx->misc)) if (misc_register(&lynx->misc)) {
FAIL("Failed to register misc char device."); error("Failed to register misc char device\n");
return -ENOMEM;
}
minors[lynx->misc.minor] = lynx; minors[lynx->misc.minor] = lynx;
notify("Initialized PCILynx IEEE1394 card, irq=%d\n", dev->irq); notify("Initialized PCILynx IEEE1394 card, irq=%d\n", dev->irq);
return 0; return 0;
} }
static struct pci_device_id pci_table[] __devinitdata = { static struct pci_device_id pci_table[] __devinitdata = {
{ {
.vendor = PCI_VENDOR_ID_TI, .vendor = PCI_VENDOR_ID_TI,
.device = PCI_DEVICE_ID_TI_PCILYNX, .device = PCI_DEVICE_ID_TI_PCILYNX,
.subvendor = PCI_ANY_ID, .subvendor = PCI_ANY_ID,
.subdevice = PCI_ANY_ID, .subdevice = PCI_ANY_ID,
}, },
{ } /* Terminating entry */ { } /* Terminating entry */
}; };
static struct pci_driver lynx_pci_driver = { static struct pci_driver lynx_pci_driver = {
.name = (char *) driver_name, .name = driver_name,
.id_table = pci_table, .id_table = pci_table,
.probe = add_card, .probe = add_card,
.remove = __devexit_p(remove_card), .remove = remove_card,
}; };
MODULE_AUTHOR("Kristian Hgsberg"); MODULE_AUTHOR("Kristian Hoegsberg");
MODULE_DESCRIPTION("Snoop mode driver for TI pcilynx 1394 controllers"); MODULE_DESCRIPTION("Snoop mode driver for TI pcilynx 1394 controllers");
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
MODULE_DEVICE_TABLE(pci, pci_table); MODULE_DEVICE_TABLE(pci, pci_table);
static int __init nosy_init(void) static int __init nosy_init(void)
{ {
/* notify("Loaded %s version %s.\n", driver_name, VERSION); */ return pci_register_driver(&lynx_pci_driver);
return pci_register_driver(&lynx_pci_driver);
} }
static void __exit nosy_cleanup(void) static void __exit nosy_cleanup(void)
{ {
pci_unregister_driver(&lynx_pci_driver); pci_unregister_driver(&lynx_pci_driver);
notify("Unloaded %s.\n", driver_name); notify("Unloaded %s.\n", driver_name);
} }
module_init(nosy_init); module_init(nosy_init);
module_exit(nosy_cleanup); module_exit(nosy_cleanup);
/* Chip register definitions for PCILynx chipset. Based on pcilynx.h /*
* Chip register definitions for PCILynx chipset. Based on pcilynx.h
* from the Linux 1394 drivers, but modified a bit so the names here * from the Linux 1394 drivers, but modified a bit so the names here
* match the specification exactly (even though they have weird names, * match the specification exactly (even though they have weird names,
* like xxx_OVER_FLOW, or arbitrary abbreviations like SNTRJ for "sent * like xxx_OVER_FLOW, or arbitrary abbreviations like SNTRJ for "sent
...@@ -16,7 +17,7 @@ ...@@ -16,7 +17,7 @@
#define SERIAL_EEPROM_CONTROL 0x44 #define SERIAL_EEPROM_CONTROL 0x44
#define PCI_INT_STATUS 0x48 #define PCI_INT_STATUS 0x48
#define PCI_INT_ENABLE 0x4c #define PCI_INT_ENABLE 0x4c
/* status and enable have identical bit numbers */ /* status and enable have identical bit numbers */
#define PCI_INT_INT_PEND (1<<31) #define PCI_INT_INT_PEND (1<<31)
#define PCI_INT_FRC_INT (1<<30) #define PCI_INT_FRC_INT (1<<30)
...@@ -48,7 +49,7 @@ ...@@ -48,7 +49,7 @@
#define LBUS_ADDR_SEL_RAM (0x0<<16) #define LBUS_ADDR_SEL_RAM (0x0<<16)
#define LBUS_ADDR_SEL_ROM (0x1<<16) #define LBUS_ADDR_SEL_ROM (0x1<<16)
#define LBUS_ADDR_SEL_AUX (0x2<<16) #define LBUS_ADDR_SEL_AUX (0x2<<16)
#define LBUS_ADDR_SEL_ZV (0x3<<16) #define LBUS_ADDR_SEL_ZV (0x3<<16)
#define GPIO_CTRL_A 0xb8 #define GPIO_CTRL_A 0xb8
#define GPIO_CTRL_B 0xbc #define GPIO_CTRL_B 0xbc
...@@ -90,14 +91,14 @@ ...@@ -90,14 +91,14 @@
#define PCL_BIGENDIAN (1<<16) #define PCL_BIGENDIAN (1<<16)
#define PCL_ISOMODE (1<<12) #define PCL_ISOMODE (1<<12)
#define DMA0_PREV_PCL 0x100 #define DMA0_PREV_PCL 0x100
#define DMA1_PREV_PCL 0x120 #define DMA1_PREV_PCL 0x120
#define DMA2_PREV_PCL 0x140 #define DMA2_PREV_PCL 0x140
#define DMA3_PREV_PCL 0x160 #define DMA3_PREV_PCL 0x160
#define DMA4_PREV_PCL 0x180 #define DMA4_PREV_PCL 0x180
#define DMA_PREV_PCL(chan) (DMA_BREG(DMA0_PREV_PCL, chan)) #define DMA_PREV_PCL(chan) (DMA_BREG(DMA0_PREV_PCL, chan))
#define DMA0_CURRENT_PCL 0x104 #define DMA0_CURRENT_PCL 0x104
#define DMA1_CURRENT_PCL 0x124 #define DMA1_CURRENT_PCL 0x124
#define DMA2_CURRENT_PCL 0x144 #define DMA2_CURRENT_PCL 0x144
#define DMA3_CURRENT_PCL 0x164 #define DMA3_CURRENT_PCL 0x164
...@@ -118,15 +119,14 @@ ...@@ -118,15 +119,14 @@
#define DMA_CHAN_STAT_PKTCMPL (1<<27) #define DMA_CHAN_STAT_PKTCMPL (1<<27)
#define DMA_CHAN_STAT_SPECIALACK (1<<14) #define DMA_CHAN_STAT_SPECIALACK (1<<14)
#define DMA0_CHAN_CTRL 0x110
#define DMA0_CHAN_CTRL 0x110
#define DMA1_CHAN_CTRL 0x130 #define DMA1_CHAN_CTRL 0x130
#define DMA2_CHAN_CTRL 0x150 #define DMA2_CHAN_CTRL 0x150
#define DMA3_CHAN_CTRL 0x170 #define DMA3_CHAN_CTRL 0x170
#define DMA4_CHAN_CTRL 0x190 #define DMA4_CHAN_CTRL 0x190
#define DMA_CHAN_CTRL(chan) (DMA_BREG(DMA0_CHAN_CTRL, chan)) #define DMA_CHAN_CTRL(chan) (DMA_BREG(DMA0_CHAN_CTRL, chan))
/* CHAN_CTRL registers share bits */ /* CHAN_CTRL registers share bits */
#define DMA_CHAN_CTRL_ENABLE (1<<31) #define DMA_CHAN_CTRL_ENABLE (1<<31)
#define DMA_CHAN_CTRL_BUSY (1<<30) #define DMA_CHAN_CTRL_BUSY (1<<30)
#define DMA_CHAN_CTRL_LINK (1<<29) #define DMA_CHAN_CTRL_LINK (1<<29)
...@@ -153,28 +153,28 @@ ...@@ -153,28 +153,28 @@
#define DMA2_WORD0_CMP_VALUE 0xb20 #define DMA2_WORD0_CMP_VALUE 0xb20
#define DMA3_WORD0_CMP_VALUE 0xb30 #define DMA3_WORD0_CMP_VALUE 0xb30
#define DMA4_WORD0_CMP_VALUE 0xb40 #define DMA4_WORD0_CMP_VALUE 0xb40
#define DMA_WORD0_CMP_VALUE(chan) (DMA_SREG(DMA0_WORD0_CMP_VALUE, chan)) #define DMA_WORD0_CMP_VALUE(chan) (DMA_SREG(DMA0_WORD0_CMP_VALUE, chan))
#define DMA0_WORD0_CMP_ENABLE 0xb04 #define DMA0_WORD0_CMP_ENABLE 0xb04
#define DMA1_WORD0_CMP_ENABLE 0xb14 #define DMA1_WORD0_CMP_ENABLE 0xb14
#define DMA2_WORD0_CMP_ENABLE 0xb24 #define DMA2_WORD0_CMP_ENABLE 0xb24
#define DMA3_WORD0_CMP_ENABLE 0xb34 #define DMA3_WORD0_CMP_ENABLE 0xb34
#define DMA4_WORD0_CMP_ENABLE 0xb44 #define DMA4_WORD0_CMP_ENABLE 0xb44
#define DMA_WORD0_CMP_ENABLE(chan) (DMA_SREG(DMA0_WORD0_CMP_ENABLE,chan)) #define DMA_WORD0_CMP_ENABLE(chan) (DMA_SREG(DMA0_WORD0_CMP_ENABLE, chan))
#define DMA0_WORD1_CMP_VALUE 0xb08 #define DMA0_WORD1_CMP_VALUE 0xb08
#define DMA1_WORD1_CMP_VALUE 0xb18 #define DMA1_WORD1_CMP_VALUE 0xb18
#define DMA2_WORD1_CMP_VALUE 0xb28 #define DMA2_WORD1_CMP_VALUE 0xb28
#define DMA3_WORD1_CMP_VALUE 0xb38 #define DMA3_WORD1_CMP_VALUE 0xb38
#define DMA4_WORD1_CMP_VALUE 0xb48 #define DMA4_WORD1_CMP_VALUE 0xb48
#define DMA_WORD1_CMP_VALUE(chan) (DMA_SREG(DMA0_WORD1_CMP_VALUE, chan)) #define DMA_WORD1_CMP_VALUE(chan) (DMA_SREG(DMA0_WORD1_CMP_VALUE, chan))
#define DMA0_WORD1_CMP_ENABLE 0xb0c #define DMA0_WORD1_CMP_ENABLE 0xb0c
#define DMA1_WORD1_CMP_ENABLE 0xb1c #define DMA1_WORD1_CMP_ENABLE 0xb1c
#define DMA2_WORD1_CMP_ENABLE 0xb2c #define DMA2_WORD1_CMP_ENABLE 0xb2c
#define DMA3_WORD1_CMP_ENABLE 0xb3c #define DMA3_WORD1_CMP_ENABLE 0xb3c
#define DMA4_WORD1_CMP_ENABLE 0xb4c #define DMA4_WORD1_CMP_ENABLE 0xb4c
#define DMA_WORD1_CMP_ENABLE(chan) (DMA_SREG(DMA0_WORD1_CMP_ENABLE,chan)) #define DMA_WORD1_CMP_ENABLE(chan) (DMA_SREG(DMA0_WORD1_CMP_ENABLE, chan))
/* word 1 compare enable flags */ /* word 1 compare enable flags */
#define DMA_WORD1_CMP_MATCH_OTHERBUS (1<<15) #define DMA_WORD1_CMP_MATCH_OTHERBUS (1<<15)
#define DMA_WORD1_CMP_MATCH_BROADCAST (1<<14) #define DMA_WORD1_CMP_MATCH_BROADCAST (1<<14)
...@@ -211,7 +211,6 @@ ...@@ -211,7 +211,6 @@
#define LINK_PHY_WDATA(data) (data<<16) #define LINK_PHY_WDATA(data) (data<<16)
#define LINK_PHY_RADDR(addr) (addr<<8) #define LINK_PHY_RADDR(addr) (addr<<8)
#define LINK_INT_STATUS 0xf14 #define LINK_INT_STATUS 0xf14
#define LINK_INT_ENABLE 0xf18 #define LINK_INT_ENABLE 0xf18
/* status and enable have identical bit numbers */ /* status and enable have identical bit numbers */
...@@ -235,4 +234,4 @@ ...@@ -235,4 +234,4 @@
#define LINK_INT_GRF_OVER_FLOW (1<<5) #define LINK_INT_GRF_OVER_FLOW (1<<5)
#define LINK_INT_ITF_UNDER_FLOW (1<<4) #define LINK_INT_ITF_UNDER_FLOW (1<<4)
#define LINK_INT_ATF_UNDER_FLOW (1<<3) #define LINK_INT_ATF_UNDER_FLOW (1<<3)
#define LINK_INT_IARB_FAILED (1<<0) #define LINK_INT_IARB_FAILED (1<<0)
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册