diff options
| author | Linus Torvalds <torvalds@g5.osdl.org> | 2006-06-20 23:01:16 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-06-20 23:01:16 -0400 |
| commit | 34641a58a227e498adf471ab016bd054cc399d7e (patch) | |
| tree | da1e0af965d06d0c6e13c546ac01f6a54a904d86 | |
| parent | 2eec0e0842ef747027eb9181d5f50d7157184d57 (diff) | |
| parent | 7136b8073f0123918e3e50269ae021bbb09e1a81 (diff) | |
Merge git://git.kernel.org/pub/scm/linux/kernel/git/bcollins/linux1394-2.6
* git://git.kernel.org/pub/scm/linux/kernel/git/bcollins/linux1394-2.6: (28 commits)
eth1394: replace __constant_htons by htons
ieee1394: adjust code formatting in highlevel.c
ieee1394: hl_irqs_lock is taken in hardware interrupt context
ieee1394_core: switch to kthread API
ieee1394: sbp2: Kconfig fix
ieee1394: add preprocessor constant for invalid csr address
sbp2: fix deregistration of status fifo address space
[PATCH] eth1394: endian fixes
Fix broken suspend/resume in ohci1394
sbp2: use __attribute__((packed)) for on-the-wire structures
sbp2: provide helptext for CONFIG_IEEE1394_SBP2_PHYS_DMA and mark it experimental
Update feature removal of obsolete raw1394 ISO requests.
sbp2: fix S800 transfers if phys_dma is off
sbp2: remove ohci1394 specific constant
ohci1394: make phys_dma parameter read-only
ohci1394: set address range properties
ieee1394: extend lowlevel API for address range properties
sbp2: log number of supported concurrent logins
sbp2: remove manipulation of inquiry response
ieee1394: save RAM by using a single tlabel for broadcast transactions
...
| -rw-r--r-- | Documentation/feature-removal-schedule.txt | 4 | ||||
| -rw-r--r-- | drivers/ieee1394/Kconfig | 13 | ||||
| -rw-r--r-- | drivers/ieee1394/csr1212.c | 2 | ||||
| -rw-r--r-- | drivers/ieee1394/csr1212.h | 1 | ||||
| -rw-r--r-- | drivers/ieee1394/dma.c | 18 | ||||
| -rw-r--r-- | drivers/ieee1394/eth1394.c | 51 | ||||
| -rw-r--r-- | drivers/ieee1394/eth1394.h | 2 | ||||
| -rw-r--r-- | drivers/ieee1394/highlevel.c | 445 | ||||
| -rw-r--r-- | drivers/ieee1394/hosts.c | 7 | ||||
| -rw-r--r-- | drivers/ieee1394/hosts.h | 17 | ||||
| -rw-r--r-- | drivers/ieee1394/ieee1394_core.c | 62 | ||||
| -rw-r--r-- | drivers/ieee1394/ieee1394_transactions.c | 10 | ||||
| -rw-r--r-- | drivers/ieee1394/nodemgr.c | 61 | ||||
| -rw-r--r-- | drivers/ieee1394/ohci1394.c | 37 | ||||
| -rw-r--r-- | drivers/ieee1394/ohci1394.h | 10 | ||||
| -rw-r--r-- | drivers/ieee1394/raw1394.c | 54 | ||||
| -rw-r--r-- | drivers/ieee1394/sbp2.c | 85 | ||||
| -rw-r--r-- | drivers/ieee1394/sbp2.h | 23 | ||||
| -rw-r--r-- | drivers/ieee1394/video1394.c | 16 |
19 files changed, 462 insertions, 456 deletions
diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt index f50cf8fac3f0..f7293297f326 100644 --- a/Documentation/feature-removal-schedule.txt +++ b/Documentation/feature-removal-schedule.txt | |||
| @@ -49,11 +49,11 @@ Who: Paul E. McKenney <paulmck@us.ibm.com> | |||
| 49 | --------------------------- | 49 | --------------------------- |
| 50 | 50 | ||
| 51 | What: raw1394: requests of type RAW1394_REQ_ISO_SEND, RAW1394_REQ_ISO_LISTEN | 51 | What: raw1394: requests of type RAW1394_REQ_ISO_SEND, RAW1394_REQ_ISO_LISTEN |
| 52 | When: November 2005 | 52 | When: November 2006 |
| 53 | Why: Deprecated in favour of the new ioctl-based rawiso interface, which is | 53 | Why: Deprecated in favour of the new ioctl-based rawiso interface, which is |
| 54 | more efficient. You should really be using libraw1394 for raw1394 | 54 | more efficient. You should really be using libraw1394 for raw1394 |
| 55 | access anyway. | 55 | access anyway. |
| 56 | Who: Jody McIntyre <scjody@steamballoon.com> | 56 | Who: Jody McIntyre <scjody@modernduck.com> |
| 57 | 57 | ||
| 58 | --------------------------- | 58 | --------------------------- |
| 59 | 59 | ||
diff --git a/drivers/ieee1394/Kconfig b/drivers/ieee1394/Kconfig index 39142e2f804b..186737539cf5 100644 --- a/drivers/ieee1394/Kconfig +++ b/drivers/ieee1394/Kconfig | |||
| @@ -128,8 +128,17 @@ config IEEE1394_SBP2 | |||
| 128 | 1394 bus. SBP-2 devices include harddrives and DVD devices. | 128 | 1394 bus. SBP-2 devices include harddrives and DVD devices. |
| 129 | 129 | ||
| 130 | config IEEE1394_SBP2_PHYS_DMA | 130 | config IEEE1394_SBP2_PHYS_DMA |
| 131 | bool "Enable Phys DMA support for SBP2 (Debug)" | 131 | bool "Enable replacement for physical DMA in SBP2" |
| 132 | depends on IEEE1394 && IEEE1394_SBP2 | 132 | depends on IEEE1394 && IEEE1394_SBP2 && EXPERIMENTAL && (X86_32 || PPC_32) |
| 133 | help | ||
| 134 | This builds sbp2 for use with non-OHCI host adapters which do not | ||
| 135 | support physical DMA or for when ohci1394 is run with phys_dma=0. | ||
| 136 | Physical DMA is data movement without assistence of the drivers' | ||
| 137 | interrupt handlers. This option includes the interrupt handlers | ||
| 138 | that are required in absence of this hardware feature. | ||
| 139 | |||
| 140 | This option is buggy and currently broken on some architectures. | ||
| 141 | If unsure, say N. | ||
| 133 | 142 | ||
| 134 | config IEEE1394_ETH1394 | 143 | config IEEE1394_ETH1394 |
| 135 | tristate "Ethernet over 1394" | 144 | tristate "Ethernet over 1394" |
diff --git a/drivers/ieee1394/csr1212.c b/drivers/ieee1394/csr1212.c index 15773544234b..586f71e7346a 100644 --- a/drivers/ieee1394/csr1212.c +++ b/drivers/ieee1394/csr1212.c | |||
| @@ -779,7 +779,7 @@ static int csr1212_append_new_cache(struct csr1212_csr *csr, size_t romsize) | |||
| 779 | romsize = (romsize + (csr->max_rom - 1)) & ~(csr->max_rom - 1); | 779 | romsize = (romsize + (csr->max_rom - 1)) & ~(csr->max_rom - 1); |
| 780 | 780 | ||
| 781 | csr_addr = csr->ops->allocate_addr_range(romsize, csr->max_rom, csr->private); | 781 | csr_addr = csr->ops->allocate_addr_range(romsize, csr->max_rom, csr->private); |
| 782 | if (csr_addr == ~0ULL) { | 782 | if (csr_addr == CSR1212_INVALID_ADDR_SPACE) { |
| 783 | return CSR1212_ENOMEM; | 783 | return CSR1212_ENOMEM; |
| 784 | } | 784 | } |
| 785 | if (csr_addr < CSR1212_REGISTER_SPACE_BASE) { | 785 | if (csr_addr < CSR1212_REGISTER_SPACE_BASE) { |
diff --git a/drivers/ieee1394/csr1212.h b/drivers/ieee1394/csr1212.h index cecd5871f2de..17ddd72dee4e 100644 --- a/drivers/ieee1394/csr1212.h +++ b/drivers/ieee1394/csr1212.h | |||
| @@ -192,6 +192,7 @@ | |||
| 192 | 192 | ||
| 193 | #define CSR1212_EXTENDED_ROM_SIZE (0x10000 * sizeof(u_int32_t)) | 193 | #define CSR1212_EXTENDED_ROM_SIZE (0x10000 * sizeof(u_int32_t)) |
| 194 | 194 | ||
| 195 | #define CSR1212_INVALID_ADDR_SPACE -1 | ||
| 195 | 196 | ||
| 196 | /* Config ROM image structures */ | 197 | /* Config ROM image structures */ |
| 197 | struct csr1212_bus_info_block_img { | 198 | struct csr1212_bus_info_block_img { |
diff --git a/drivers/ieee1394/dma.c b/drivers/ieee1394/dma.c index 9fb2769d9abc..ca5167de707d 100644 --- a/drivers/ieee1394/dma.c +++ b/drivers/ieee1394/dma.c | |||
| @@ -145,12 +145,12 @@ void dma_region_free(struct dma_region *dma) | |||
| 145 | /* find the scatterlist index and remaining offset corresponding to a | 145 | /* find the scatterlist index and remaining offset corresponding to a |
| 146 | given offset from the beginning of the buffer */ | 146 | given offset from the beginning of the buffer */ |
| 147 | static inline int dma_region_find(struct dma_region *dma, unsigned long offset, | 147 | static inline int dma_region_find(struct dma_region *dma, unsigned long offset, |
| 148 | unsigned long *rem) | 148 | unsigned int start, unsigned long *rem) |
| 149 | { | 149 | { |
| 150 | int i; | 150 | int i; |
| 151 | unsigned long off = offset; | 151 | unsigned long off = offset; |
| 152 | 152 | ||
| 153 | for (i = 0; i < dma->n_dma_pages; i++) { | 153 | for (i = start; i < dma->n_dma_pages; i++) { |
| 154 | if (off < sg_dma_len(&dma->sglist[i])) { | 154 | if (off < sg_dma_len(&dma->sglist[i])) { |
| 155 | *rem = off; | 155 | *rem = off; |
| 156 | break; | 156 | break; |
| @@ -170,7 +170,7 @@ dma_addr_t dma_region_offset_to_bus(struct dma_region * dma, | |||
| 170 | unsigned long rem = 0; | 170 | unsigned long rem = 0; |
| 171 | 171 | ||
| 172 | struct scatterlist *sg = | 172 | struct scatterlist *sg = |
| 173 | &dma->sglist[dma_region_find(dma, offset, &rem)]; | 173 | &dma->sglist[dma_region_find(dma, offset, 0, &rem)]; |
| 174 | return sg_dma_address(sg) + rem; | 174 | return sg_dma_address(sg) + rem; |
| 175 | } | 175 | } |
| 176 | 176 | ||
| @@ -178,13 +178,13 @@ void dma_region_sync_for_cpu(struct dma_region *dma, unsigned long offset, | |||
| 178 | unsigned long len) | 178 | unsigned long len) |
| 179 | { | 179 | { |
| 180 | int first, last; | 180 | int first, last; |
| 181 | unsigned long rem; | 181 | unsigned long rem = 0; |
| 182 | 182 | ||
| 183 | if (!len) | 183 | if (!len) |
| 184 | len = 1; | 184 | len = 1; |
| 185 | 185 | ||
| 186 | first = dma_region_find(dma, offset, &rem); | 186 | first = dma_region_find(dma, offset, 0, &rem); |
| 187 | last = dma_region_find(dma, offset + len - 1, &rem); | 187 | last = dma_region_find(dma, rem + len - 1, first, &rem); |
| 188 | 188 | ||
| 189 | pci_dma_sync_sg_for_cpu(dma->dev, &dma->sglist[first], last - first + 1, | 189 | pci_dma_sync_sg_for_cpu(dma->dev, &dma->sglist[first], last - first + 1, |
| 190 | dma->direction); | 190 | dma->direction); |
| @@ -194,13 +194,13 @@ void dma_region_sync_for_device(struct dma_region *dma, unsigned long offset, | |||
| 194 | unsigned long len) | 194 | unsigned long len) |
| 195 | { | 195 | { |
| 196 | int first, last; | 196 | int first, last; |
| 197 | unsigned long rem; | 197 | unsigned long rem = 0; |
| 198 | 198 | ||
| 199 | if (!len) | 199 | if (!len) |
| 200 | len = 1; | 200 | len = 1; |
| 201 | 201 | ||
| 202 | first = dma_region_find(dma, offset, &rem); | 202 | first = dma_region_find(dma, offset, 0, &rem); |
| 203 | last = dma_region_find(dma, offset + len - 1, &rem); | 203 | last = dma_region_find(dma, rem + len - 1, first, &rem); |
| 204 | 204 | ||
| 205 | pci_dma_sync_sg_for_device(dma->dev, &dma->sglist[first], | 205 | pci_dma_sync_sg_for_device(dma->dev, &dma->sglist[first], |
| 206 | last - first + 1, dma->direction); | 206 | last - first + 1, dma->direction); |
diff --git a/drivers/ieee1394/eth1394.c b/drivers/ieee1394/eth1394.c index 30fa0d43a43a..5bda15904a08 100644 --- a/drivers/ieee1394/eth1394.c +++ b/drivers/ieee1394/eth1394.c | |||
| @@ -367,7 +367,7 @@ static int eth1394_probe(struct device *dev) | |||
| 367 | spin_lock_init(&node_info->pdg.lock); | 367 | spin_lock_init(&node_info->pdg.lock); |
| 368 | INIT_LIST_HEAD(&node_info->pdg.list); | 368 | INIT_LIST_HEAD(&node_info->pdg.list); |
| 369 | node_info->pdg.sz = 0; | 369 | node_info->pdg.sz = 0; |
| 370 | node_info->fifo = ETHER1394_INVALID_ADDR; | 370 | node_info->fifo = CSR1212_INVALID_ADDR_SPACE; |
| 371 | 371 | ||
| 372 | ud->device.driver_data = node_info; | 372 | ud->device.driver_data = node_info; |
| 373 | new_node->ud = ud; | 373 | new_node->ud = ud; |
| @@ -502,10 +502,8 @@ static void ether1394_reset_priv (struct net_device *dev, int set_mtu) | |||
| 502 | 502 | ||
| 503 | /* Determine speed limit */ | 503 | /* Determine speed limit */ |
| 504 | for (i = 0; i < host->node_count; i++) | 504 | for (i = 0; i < host->node_count; i++) |
| 505 | if (max_speed > host->speed_map[NODEID_TO_NODE(host->node_id) * | 505 | if (max_speed > host->speed[i]) |
| 506 | 64 + i]) | 506 | max_speed = host->speed[i]; |
| 507 | max_speed = host->speed_map[NODEID_TO_NODE(host->node_id) * | ||
| 508 | 64 + i]; | ||
| 509 | priv->bc_sspd = max_speed; | 507 | priv->bc_sspd = max_speed; |
| 510 | 508 | ||
| 511 | /* We'll use our maxpayload as the default mtu */ | 509 | /* We'll use our maxpayload as the default mtu */ |
| @@ -568,13 +566,11 @@ static void ether1394_add_host (struct hpsb_host *host) | |||
| 568 | if (!(host->config_roms & HPSB_CONFIG_ROM_ENTRY_IP1394)) | 566 | if (!(host->config_roms & HPSB_CONFIG_ROM_ENTRY_IP1394)) |
| 569 | return; | 567 | return; |
| 570 | 568 | ||
| 571 | fifo_addr = hpsb_allocate_and_register_addrspace(ð1394_highlevel, | 569 | fifo_addr = hpsb_allocate_and_register_addrspace( |
| 572 | host, | 570 | ð1394_highlevel, host, &addr_ops, |
| 573 | &addr_ops, | 571 | ETHER1394_REGION_ADDR_LEN, ETHER1394_REGION_ADDR_LEN, |
| 574 | ETHER1394_REGION_ADDR_LEN, | 572 | CSR1212_INVALID_ADDR_SPACE, CSR1212_INVALID_ADDR_SPACE); |
| 575 | ETHER1394_REGION_ADDR_LEN, | 573 | if (fifo_addr == CSR1212_INVALID_ADDR_SPACE) |
| 576 | -1, -1); | ||
| 577 | if (fifo_addr == ~0ULL) | ||
| 578 | goto out; | 574 | goto out; |
| 579 | 575 | ||
| 580 | /* We should really have our own alloc_hpsbdev() function in | 576 | /* We should really have our own alloc_hpsbdev() function in |
| @@ -774,7 +770,7 @@ static int ether1394_rebuild_header(struct sk_buff *skb) | |||
| 774 | default: | 770 | default: |
| 775 | ETH1394_PRINT(KERN_DEBUG, dev->name, | 771 | ETH1394_PRINT(KERN_DEBUG, dev->name, |
| 776 | "unable to resolve type %04x addresses.\n", | 772 | "unable to resolve type %04x addresses.\n", |
| 777 | eth->h_proto); | 773 | ntohs(eth->h_proto)); |
| 778 | break; | 774 | break; |
| 779 | } | 775 | } |
| 780 | 776 | ||
| @@ -796,9 +792,8 @@ static int ether1394_header_cache(struct neighbour *neigh, struct hh_cache *hh) | |||
| 796 | (16 - ETH1394_HLEN)); | 792 | (16 - ETH1394_HLEN)); |
| 797 | struct net_device *dev = neigh->dev; | 793 | struct net_device *dev = neigh->dev; |
| 798 | 794 | ||
| 799 | if (type == __constant_htons(ETH_P_802_3)) { | 795 | if (type == htons(ETH_P_802_3)) |
| 800 | return -1; | 796 | return -1; |
| 801 | } | ||
| 802 | 797 | ||
| 803 | eth->h_proto = type; | 798 | eth->h_proto = type; |
| 804 | memcpy(eth->h_dest, neigh->ha, dev->addr_len); | 799 | memcpy(eth->h_dest, neigh->ha, dev->addr_len); |
| @@ -887,7 +882,7 @@ static inline u16 ether1394_parse_encap(struct sk_buff *skb, | |||
| 887 | /* If this is an ARP packet, convert it. First, we want to make | 882 | /* If this is an ARP packet, convert it. First, we want to make |
| 888 | * use of some of the fields, since they tell us a little bit | 883 | * use of some of the fields, since they tell us a little bit |
| 889 | * about the sending machine. */ | 884 | * about the sending machine. */ |
| 890 | if (ether_type == __constant_htons (ETH_P_ARP)) { | 885 | if (ether_type == htons(ETH_P_ARP)) { |
| 891 | struct eth1394_arp *arp1394 = (struct eth1394_arp*)skb->data; | 886 | struct eth1394_arp *arp1394 = (struct eth1394_arp*)skb->data; |
| 892 | struct arphdr *arp = (struct arphdr *)skb->data; | 887 | struct arphdr *arp = (struct arphdr *)skb->data; |
| 893 | unsigned char *arp_ptr = (unsigned char *)(arp + 1); | 888 | unsigned char *arp_ptr = (unsigned char *)(arp + 1); |
| @@ -935,7 +930,7 @@ static inline u16 ether1394_parse_encap(struct sk_buff *skb, | |||
| 935 | *(u32*)arp_ptr = arp1394->sip; /* move sender IP addr */ | 930 | *(u32*)arp_ptr = arp1394->sip; /* move sender IP addr */ |
| 936 | arp_ptr += arp->ar_pln; /* skip over sender IP addr */ | 931 | arp_ptr += arp->ar_pln; /* skip over sender IP addr */ |
| 937 | 932 | ||
| 938 | if (arp->ar_op == 1) | 933 | if (arp->ar_op == htons(ARPOP_REQUEST)) |
| 939 | /* just set ARP req target unique ID to 0 */ | 934 | /* just set ARP req target unique ID to 0 */ |
| 940 | *((u64*)arp_ptr) = 0; | 935 | *((u64*)arp_ptr) = 0; |
| 941 | else | 936 | else |
| @@ -943,8 +938,8 @@ static inline u16 ether1394_parse_encap(struct sk_buff *skb, | |||
| 943 | } | 938 | } |
| 944 | 939 | ||
| 945 | /* Now add the ethernet header. */ | 940 | /* Now add the ethernet header. */ |
| 946 | if (dev->hard_header (skb, dev, __constant_ntohs (ether_type), | 941 | if (dev->hard_header(skb, dev, ntohs(ether_type), &dest_hw, NULL, |
| 947 | &dest_hw, NULL, skb->len) >= 0) | 942 | skb->len) >= 0) |
| 948 | ret = ether1394_type_trans(skb, dev); | 943 | ret = ether1394_type_trans(skb, dev); |
| 949 | 944 | ||
| 950 | return ret; | 945 | return ret; |
| @@ -1395,7 +1390,7 @@ static inline void ether1394_arp_to_1394arp(struct sk_buff *skb, | |||
| 1395 | /* We need to encapsulate the standard header with our own. We use the | 1390 | /* We need to encapsulate the standard header with our own. We use the |
| 1396 | * ethernet header's proto for our own. */ | 1391 | * ethernet header's proto for our own. */ |
| 1397 | static inline unsigned int ether1394_encapsulate_prep(unsigned int max_payload, | 1392 | static inline unsigned int ether1394_encapsulate_prep(unsigned int max_payload, |
| 1398 | int proto, | 1393 | __be16 proto, |
| 1399 | union eth1394_hdr *hdr, | 1394 | union eth1394_hdr *hdr, |
| 1400 | u16 dg_size, u16 dgl) | 1395 | u16 dg_size, u16 dgl) |
| 1401 | { | 1396 | { |
| @@ -1514,8 +1509,8 @@ static inline void ether1394_prep_gasp_packet(struct hpsb_packet *p, | |||
| 1514 | p->data = ((quadlet_t*)skb->data) - 2; | 1509 | p->data = ((quadlet_t*)skb->data) - 2; |
| 1515 | p->data[0] = cpu_to_be32((priv->host->node_id << 16) | | 1510 | p->data[0] = cpu_to_be32((priv->host->node_id << 16) | |
| 1516 | ETHER1394_GASP_SPECIFIER_ID_HI); | 1511 | ETHER1394_GASP_SPECIFIER_ID_HI); |
| 1517 | p->data[1] = __constant_cpu_to_be32((ETHER1394_GASP_SPECIFIER_ID_LO << 24) | | 1512 | p->data[1] = cpu_to_be32((ETHER1394_GASP_SPECIFIER_ID_LO << 24) | |
| 1518 | ETHER1394_GASP_VERSION); | 1513 | ETHER1394_GASP_VERSION); |
| 1519 | 1514 | ||
| 1520 | /* Setting the node id to ALL_NODES (not LOCAL_BUS | ALL_NODES) | 1515 | /* Setting the node id to ALL_NODES (not LOCAL_BUS | ALL_NODES) |
| 1521 | * prevents hpsb_send_packet() from setting the speed to an arbitrary | 1516 | * prevents hpsb_send_packet() from setting the speed to an arbitrary |
| @@ -1626,7 +1621,7 @@ static int ether1394_tx (struct sk_buff *skb, struct net_device *dev) | |||
| 1626 | gfp_t kmflags = in_interrupt() ? GFP_ATOMIC : GFP_KERNEL; | 1621 | gfp_t kmflags = in_interrupt() ? GFP_ATOMIC : GFP_KERNEL; |
| 1627 | struct eth1394hdr *eth; | 1622 | struct eth1394hdr *eth; |
| 1628 | struct eth1394_priv *priv = netdev_priv(dev); | 1623 | struct eth1394_priv *priv = netdev_priv(dev); |
| 1629 | int proto; | 1624 | __be16 proto; |
| 1630 | unsigned long flags; | 1625 | unsigned long flags; |
| 1631 | nodeid_t dest_node; | 1626 | nodeid_t dest_node; |
| 1632 | eth1394_tx_type tx_type; | 1627 | eth1394_tx_type tx_type; |
| @@ -1670,9 +1665,9 @@ static int ether1394_tx (struct sk_buff *skb, struct net_device *dev) | |||
| 1670 | /* Set the transmission type for the packet. ARP packets and IP | 1665 | /* Set the transmission type for the packet. ARP packets and IP |
| 1671 | * broadcast packets are sent via GASP. */ | 1666 | * broadcast packets are sent via GASP. */ |
| 1672 | if (memcmp(eth->h_dest, dev->broadcast, ETH1394_ALEN) == 0 || | 1667 | if (memcmp(eth->h_dest, dev->broadcast, ETH1394_ALEN) == 0 || |
| 1673 | proto == __constant_htons(ETH_P_ARP) || | 1668 | proto == htons(ETH_P_ARP) || |
| 1674 | (proto == __constant_htons(ETH_P_IP) && | 1669 | (proto == htons(ETH_P_IP) && |
| 1675 | IN_MULTICAST(__constant_ntohl(skb->nh.iph->daddr)))) { | 1670 | IN_MULTICAST(ntohl(skb->nh.iph->daddr)))) { |
| 1676 | tx_type = ETH1394_GASP; | 1671 | tx_type = ETH1394_GASP; |
| 1677 | dest_node = LOCAL_BUS | ALL_NODES; | 1672 | dest_node = LOCAL_BUS | ALL_NODES; |
| 1678 | max_payload = priv->bc_maxpayload - ETHER1394_GASP_OVERHEAD; | 1673 | max_payload = priv->bc_maxpayload - ETHER1394_GASP_OVERHEAD; |
| @@ -1688,7 +1683,7 @@ static int ether1394_tx (struct sk_buff *skb, struct net_device *dev) | |||
| 1688 | goto fail; | 1683 | goto fail; |
| 1689 | } | 1684 | } |
| 1690 | node_info = (struct eth1394_node_info*)node->ud->device.driver_data; | 1685 | node_info = (struct eth1394_node_info*)node->ud->device.driver_data; |
| 1691 | if (node_info->fifo == ETHER1394_INVALID_ADDR) { | 1686 | if (node_info->fifo == CSR1212_INVALID_ADDR_SPACE) { |
| 1692 | ret = -EAGAIN; | 1687 | ret = -EAGAIN; |
| 1693 | goto fail; | 1688 | goto fail; |
| 1694 | } | 1689 | } |
| @@ -1704,7 +1699,7 @@ static int ether1394_tx (struct sk_buff *skb, struct net_device *dev) | |||
| 1704 | } | 1699 | } |
| 1705 | 1700 | ||
| 1706 | /* If this is an ARP packet, convert it */ | 1701 | /* If this is an ARP packet, convert it */ |
| 1707 | if (proto == __constant_htons (ETH_P_ARP)) | 1702 | if (proto == htons(ETH_P_ARP)) |
| 1708 | ether1394_arp_to_1394arp (skb, dev); | 1703 | ether1394_arp_to_1394arp (skb, dev); |
| 1709 | 1704 | ||
| 1710 | ptask->hdr.words.word1 = 0; | 1705 | ptask->hdr.words.word1 = 0; |
diff --git a/drivers/ieee1394/eth1394.h b/drivers/ieee1394/eth1394.h index a77213cfc483..c45cbff9138d 100644 --- a/drivers/ieee1394/eth1394.h +++ b/drivers/ieee1394/eth1394.h | |||
| @@ -32,8 +32,6 @@ | |||
| 32 | * S3200 (per Table 16-3 of IEEE 1394b-2002). */ | 32 | * S3200 (per Table 16-3 of IEEE 1394b-2002). */ |
| 33 | #define ETHER1394_REGION_ADDR_LEN 4096 | 33 | #define ETHER1394_REGION_ADDR_LEN 4096 |
| 34 | 34 | ||
| 35 | #define ETHER1394_INVALID_ADDR ~0ULL | ||
| 36 | |||
| 37 | /* GASP identifier numbers for IPv4 over IEEE 1394 */ | 35 | /* GASP identifier numbers for IPv4 over IEEE 1394 */ |
| 38 | #define ETHER1394_GASP_SPECIFIER_ID 0x00005E | 36 | #define ETHER1394_GASP_SPECIFIER_ID 0x00005E |
| 39 | #define ETHER1394_GASP_SPECIFIER_ID_HI ((ETHER1394_GASP_SPECIFIER_ID >> 8) & 0xffff) | 37 | #define ETHER1394_GASP_SPECIFIER_ID_HI ((ETHER1394_GASP_SPECIFIER_ID >> 8) & 0xffff) |
diff --git a/drivers/ieee1394/highlevel.c b/drivers/ieee1394/highlevel.c index 491e6032bdec..25b22609e793 100644 --- a/drivers/ieee1394/highlevel.c +++ b/drivers/ieee1394/highlevel.c | |||
| @@ -53,7 +53,7 @@ static struct hpsb_address_serve dummy_zero_addr, dummy_max_addr; | |||
| 53 | 53 | ||
| 54 | 54 | ||
| 55 | static struct hl_host_info *hl_get_hostinfo(struct hpsb_highlevel *hl, | 55 | static struct hl_host_info *hl_get_hostinfo(struct hpsb_highlevel *hl, |
| 56 | struct hpsb_host *host) | 56 | struct hpsb_host *host) |
| 57 | { | 57 | { |
| 58 | struct hl_host_info *hi = NULL; | 58 | struct hl_host_info *hi = NULL; |
| 59 | 59 | ||
| @@ -68,24 +68,18 @@ static struct hl_host_info *hl_get_hostinfo(struct hpsb_highlevel *hl, | |||
| 68 | } | 68 | } |
| 69 | } | 69 | } |
| 70 | read_unlock(&hl->host_info_lock); | 70 | read_unlock(&hl->host_info_lock); |
| 71 | |||
| 72 | return NULL; | 71 | return NULL; |
| 73 | } | 72 | } |
| 74 | 73 | ||
| 75 | |||
| 76 | /* Returns a per host/driver data structure that was previously stored by | 74 | /* Returns a per host/driver data structure that was previously stored by |
| 77 | * hpsb_create_hostinfo. */ | 75 | * hpsb_create_hostinfo. */ |
| 78 | void *hpsb_get_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host) | 76 | void *hpsb_get_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host) |
| 79 | { | 77 | { |
| 80 | struct hl_host_info *hi = hl_get_hostinfo(hl, host); | 78 | struct hl_host_info *hi = hl_get_hostinfo(hl, host); |
| 81 | 79 | ||
| 82 | if (hi) | 80 | return hi ? hi->data : NULL; |
| 83 | return hi->data; | ||
| 84 | |||
| 85 | return NULL; | ||
| 86 | } | 81 | } |
| 87 | 82 | ||
| 88 | |||
| 89 | /* If size is zero, then the return here is only valid for error checking */ | 83 | /* If size is zero, then the return here is only valid for error checking */ |
| 90 | void *hpsb_create_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host, | 84 | void *hpsb_create_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host, |
| 91 | size_t data_size) | 85 | size_t data_size) |
| @@ -96,8 +90,8 @@ void *hpsb_create_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host, | |||
| 96 | 90 | ||
| 97 | hi = hl_get_hostinfo(hl, host); | 91 | hi = hl_get_hostinfo(hl, host); |
| 98 | if (hi) { | 92 | if (hi) { |
| 99 | HPSB_ERR("%s called hpsb_create_hostinfo when hostinfo already exists", | 93 | HPSB_ERR("%s called hpsb_create_hostinfo when hostinfo already" |
| 100 | hl->name); | 94 | " exists", hl->name); |
| 101 | return NULL; | 95 | return NULL; |
| 102 | } | 96 | } |
| 103 | 97 | ||
| @@ -120,7 +114,6 @@ void *hpsb_create_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host, | |||
| 120 | return data; | 114 | return data; |
| 121 | } | 115 | } |
| 122 | 116 | ||
| 123 | |||
| 124 | int hpsb_set_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host, | 117 | int hpsb_set_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host, |
| 125 | void *data) | 118 | void *data) |
| 126 | { | 119 | { |
| @@ -132,16 +125,14 @@ int hpsb_set_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host, | |||
| 132 | hi->data = data; | 125 | hi->data = data; |
| 133 | return 0; | 126 | return 0; |
| 134 | } else | 127 | } else |
| 135 | HPSB_ERR("%s called hpsb_set_hostinfo when hostinfo already has data", | 128 | HPSB_ERR("%s called hpsb_set_hostinfo when hostinfo " |
| 136 | hl->name); | 129 | "already has data", hl->name); |
| 137 | } else | 130 | } else |
| 138 | HPSB_ERR("%s called hpsb_set_hostinfo when no hostinfo exists", | 131 | HPSB_ERR("%s called hpsb_set_hostinfo when no hostinfo exists", |
| 139 | hl->name); | 132 | hl->name); |
| 140 | |||
| 141 | return -EINVAL; | 133 | return -EINVAL; |
| 142 | } | 134 | } |
| 143 | 135 | ||
| 144 | |||
| 145 | void hpsb_destroy_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host) | 136 | void hpsb_destroy_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host) |
| 146 | { | 137 | { |
| 147 | struct hl_host_info *hi; | 138 | struct hl_host_info *hi; |
| @@ -154,23 +145,20 @@ void hpsb_destroy_hostinfo(struct hpsb_highlevel *hl, struct hpsb_host *host) | |||
| 154 | write_unlock_irqrestore(&hl->host_info_lock, flags); | 145 | write_unlock_irqrestore(&hl->host_info_lock, flags); |
| 155 | kfree(hi); | 146 | kfree(hi); |
| 156 | } | 147 | } |
| 157 | |||
| 158 | return; | 148 | return; |
| 159 | } | 149 | } |
| 160 | 150 | ||
| 161 | 151 | void hpsb_set_hostinfo_key(struct hpsb_highlevel *hl, struct hpsb_host *host, | |
| 162 | void hpsb_set_hostinfo_key(struct hpsb_highlevel *hl, struct hpsb_host *host, unsigned long key) | 152 | unsigned long key) |
| 163 | { | 153 | { |
| 164 | struct hl_host_info *hi; | 154 | struct hl_host_info *hi; |
| 165 | 155 | ||
| 166 | hi = hl_get_hostinfo(hl, host); | 156 | hi = hl_get_hostinfo(hl, host); |
| 167 | if (hi) | 157 | if (hi) |
| 168 | hi->key = key; | 158 | hi->key = key; |
| 169 | |||
| 170 | return; | 159 | return; |
| 171 | } | 160 | } |
| 172 | 161 | ||
| 173 | |||
| 174 | void *hpsb_get_hostinfo_bykey(struct hpsb_highlevel *hl, unsigned long key) | 162 | void *hpsb_get_hostinfo_bykey(struct hpsb_highlevel *hl, unsigned long key) |
| 175 | { | 163 | { |
| 176 | struct hl_host_info *hi; | 164 | struct hl_host_info *hi; |
| @@ -187,46 +175,41 @@ void *hpsb_get_hostinfo_bykey(struct hpsb_highlevel *hl, unsigned long key) | |||
| 187 | } | 175 | } |
| 188 | } | 176 | } |
| 189 | read_unlock(&hl->host_info_lock); | 177 | read_unlock(&hl->host_info_lock); |
| 190 | |||
| 191 | return data; | 178 | return data; |
| 192 | } | 179 | } |
| 193 | 180 | ||
| 194 | |||
| 195 | static int highlevel_for_each_host_reg(struct hpsb_host *host, void *__data) | 181 | static int highlevel_for_each_host_reg(struct hpsb_host *host, void *__data) |
| 196 | { | 182 | { |
| 197 | struct hpsb_highlevel *hl = __data; | 183 | struct hpsb_highlevel *hl = __data; |
| 198 | 184 | ||
| 199 | hl->add_host(host); | 185 | hl->add_host(host); |
| 200 | 186 | ||
| 201 | if (host->update_config_rom) { | 187 | if (host->update_config_rom && hpsb_update_config_rom_image(host) < 0) |
| 202 | if (hpsb_update_config_rom_image(host) < 0) { | 188 | HPSB_ERR("Failed to generate Configuration ROM image for host " |
| 203 | HPSB_ERR("Failed to generate Configuration ROM image for host " | 189 | "%s-%d", hl->name, host->id); |
| 204 | "%s-%d", hl->name, host->id); | ||
| 205 | } | ||
| 206 | } | ||
| 207 | |||
| 208 | return 0; | 190 | return 0; |
| 209 | } | 191 | } |
| 210 | 192 | ||
| 211 | void hpsb_register_highlevel(struct hpsb_highlevel *hl) | 193 | void hpsb_register_highlevel(struct hpsb_highlevel *hl) |
| 212 | { | 194 | { |
| 213 | INIT_LIST_HEAD(&hl->addr_list); | 195 | unsigned long flags; |
| 196 | |||
| 197 | INIT_LIST_HEAD(&hl->addr_list); | ||
| 214 | INIT_LIST_HEAD(&hl->host_info_list); | 198 | INIT_LIST_HEAD(&hl->host_info_list); |
| 215 | 199 | ||
| 216 | rwlock_init(&hl->host_info_lock); | 200 | rwlock_init(&hl->host_info_lock); |
| 217 | 201 | ||
| 218 | down_write(&hl_drivers_sem); | 202 | down_write(&hl_drivers_sem); |
| 219 | list_add_tail(&hl->hl_list, &hl_drivers); | 203 | list_add_tail(&hl->hl_list, &hl_drivers); |
| 220 | up_write(&hl_drivers_sem); | 204 | up_write(&hl_drivers_sem); |
| 221 | 205 | ||
| 222 | write_lock(&hl_irqs_lock); | 206 | write_lock_irqsave(&hl_irqs_lock, flags); |
| 223 | list_add_tail(&hl->irq_list, &hl_irqs); | 207 | list_add_tail(&hl->irq_list, &hl_irqs); |
| 224 | write_unlock(&hl_irqs_lock); | 208 | write_unlock_irqrestore(&hl_irqs_lock, flags); |
| 225 | 209 | ||
| 226 | if (hl->add_host) | 210 | if (hl->add_host) |
| 227 | nodemgr_for_each_host(hl, highlevel_for_each_host_reg); | 211 | nodemgr_for_each_host(hl, highlevel_for_each_host_reg); |
| 228 | 212 | return; | |
| 229 | return; | ||
| 230 | } | 213 | } |
| 231 | 214 | ||
| 232 | static void __delete_addr(struct hpsb_address_serve *as) | 215 | static void __delete_addr(struct hpsb_address_serve *as) |
| @@ -236,7 +219,8 @@ static void __delete_addr(struct hpsb_address_serve *as) | |||
| 236 | kfree(as); | 219 | kfree(as); |
| 237 | } | 220 | } |
| 238 | 221 | ||
| 239 | static void __unregister_host(struct hpsb_highlevel *hl, struct hpsb_host *host, int update_cr) | 222 | static void __unregister_host(struct hpsb_highlevel *hl, struct hpsb_host *host, |
| 223 | int update_cr) | ||
| 240 | { | 224 | { |
| 241 | unsigned long flags; | 225 | unsigned long flags; |
| 242 | struct list_head *lh, *next; | 226 | struct list_head *lh, *next; |
| @@ -251,7 +235,6 @@ static void __unregister_host(struct hpsb_highlevel *hl, struct hpsb_host *host, | |||
| 251 | write_lock_irqsave(&addr_space_lock, flags); | 235 | write_lock_irqsave(&addr_space_lock, flags); |
| 252 | list_for_each_safe (lh, next, &hl->addr_list) { | 236 | list_for_each_safe (lh, next, &hl->addr_list) { |
| 253 | as = list_entry(lh, struct hpsb_address_serve, hl_list); | 237 | as = list_entry(lh, struct hpsb_address_serve, hl_list); |
| 254 | |||
| 255 | if (as->host == host) | 238 | if (as->host == host) |
| 256 | __delete_addr(as); | 239 | __delete_addr(as); |
| 257 | } | 240 | } |
| @@ -259,15 +242,12 @@ static void __unregister_host(struct hpsb_highlevel *hl, struct hpsb_host *host, | |||
| 259 | 242 | ||
| 260 | /* Now update the config-rom to reflect anything removed by the | 243 | /* Now update the config-rom to reflect anything removed by the |
| 261 | * highlevel driver. */ | 244 | * highlevel driver. */ |
| 262 | if (update_cr && host->update_config_rom) { | 245 | if (update_cr && host->update_config_rom && |
| 263 | if (hpsb_update_config_rom_image(host) < 0) { | 246 | hpsb_update_config_rom_image(host) < 0) |
| 264 | HPSB_ERR("Failed to generate Configuration ROM image for host " | 247 | HPSB_ERR("Failed to generate Configuration ROM image for host " |
| 265 | "%s-%d", hl->name, host->id); | 248 | "%s-%d", hl->name, host->id); |
| 266 | } | ||
| 267 | } | ||
| 268 | 249 | ||
| 269 | /* And finally, remove all the host info associated between these | 250 | /* Finally remove all the host info associated between these two. */ |
| 270 | * two. */ | ||
| 271 | hpsb_destroy_hostinfo(hl, host); | 251 | hpsb_destroy_hostinfo(hl, host); |
| 272 | } | 252 | } |
| 273 | 253 | ||
| @@ -276,18 +256,19 @@ static int highlevel_for_each_host_unreg(struct hpsb_host *host, void *__data) | |||
| 276 | struct hpsb_highlevel *hl = __data; | 256 | struct hpsb_highlevel *hl = __data; |
| 277 | 257 | ||
| 278 | __unregister_host(hl, host, 1); | 258 | __unregister_host(hl, host, 1); |
| 279 | |||
| 280 | return 0; | 259 | return 0; |
| 281 | } | 260 | } |
| 282 | 261 | ||
| 283 | void hpsb_unregister_highlevel(struct hpsb_highlevel *hl) | 262 | void hpsb_unregister_highlevel(struct hpsb_highlevel *hl) |
| 284 | { | 263 | { |
| 285 | write_lock(&hl_irqs_lock); | 264 | unsigned long flags; |
| 265 | |||
| 266 | write_lock_irqsave(&hl_irqs_lock, flags); | ||
| 286 | list_del(&hl->irq_list); | 267 | list_del(&hl->irq_list); |
| 287 | write_unlock(&hl_irqs_lock); | 268 | write_unlock_irqrestore(&hl_irqs_lock, flags); |
| 288 | 269 | ||
| 289 | down_write(&hl_drivers_sem); | 270 | down_write(&hl_drivers_sem); |
| 290 | list_del(&hl->hl_list); | 271 | list_del(&hl->hl_list); |
| 291 | up_write(&hl_drivers_sem); | 272 | up_write(&hl_drivers_sem); |
| 292 | 273 | ||
| 293 | nodemgr_for_each_host(hl, highlevel_for_each_host_unreg); | 274 | nodemgr_for_each_host(hl, highlevel_for_each_host_unreg); |
| @@ -301,7 +282,7 @@ u64 hpsb_allocate_and_register_addrspace(struct hpsb_highlevel *hl, | |||
| 301 | { | 282 | { |
| 302 | struct hpsb_address_serve *as, *a1, *a2; | 283 | struct hpsb_address_serve *as, *a1, *a2; |
| 303 | struct list_head *entry; | 284 | struct list_head *entry; |
| 304 | u64 retval = ~0ULL; | 285 | u64 retval = CSR1212_INVALID_ADDR_SPACE; |
| 305 | unsigned long flags; | 286 | unsigned long flags; |
| 306 | u64 align_mask = ~(alignment - 1); | 287 | u64 align_mask = ~(alignment - 1); |
| 307 | 288 | ||
| @@ -312,14 +293,19 @@ u64 hpsb_allocate_and_register_addrspace(struct hpsb_highlevel *hl, | |||
| 312 | return retval; | 293 | return retval; |
| 313 | } | 294 | } |
| 314 | 295 | ||
| 315 | if (start == ~0ULL && end == ~0ULL) { | 296 | /* default range, |
| 316 | start = CSR1212_ALL_SPACE_BASE + 0xffff00000000ULL; /* ohci1394.c limit */ | 297 | * avoids controller's posted write area (see OHCI 1.1 clause 1.5) */ |
| 317 | end = CSR1212_ALL_SPACE_END; | 298 | if (start == CSR1212_INVALID_ADDR_SPACE && |
| 299 | end == CSR1212_INVALID_ADDR_SPACE) { | ||
| 300 | start = host->middle_addr_space; | ||
| 301 | end = CSR1212_ALL_SPACE_END; | ||
| 318 | } | 302 | } |
| 319 | 303 | ||
| 320 | if (((start|end) & ~align_mask) || (start >= end) || (end > 0x1000000000000ULL)) { | 304 | if (((start|end) & ~align_mask) || (start >= end) || |
| 321 | HPSB_ERR("%s called with invalid addresses (start = %012Lx end = %012Lx)", | 305 | (end > CSR1212_ALL_SPACE_END)) { |
| 322 | __FUNCTION__, (unsigned long long)start, (unsigned long long)end); | 306 | HPSB_ERR("%s called with invalid addresses " |
| 307 | "(start = %012Lx end = %012Lx)", __FUNCTION__, | ||
| 308 | (unsigned long long)start,(unsigned long long)end); | ||
| 323 | return retval; | 309 | return retval; |
| 324 | } | 310 | } |
| 325 | 311 | ||
| @@ -333,20 +319,21 @@ u64 hpsb_allocate_and_register_addrspace(struct hpsb_highlevel *hl, | |||
| 333 | as->host = host; | 319 | as->host = host; |
| 334 | 320 | ||
| 335 | write_lock_irqsave(&addr_space_lock, flags); | 321 | write_lock_irqsave(&addr_space_lock, flags); |
| 336 | |||
| 337 | list_for_each(entry, &host->addr_space) { | 322 | list_for_each(entry, &host->addr_space) { |
| 338 | u64 a1sa, a1ea; | 323 | u64 a1sa, a1ea; |
| 339 | u64 a2sa, a2ea; | 324 | u64 a2sa, a2ea; |
| 340 | 325 | ||
| 341 | a1 = list_entry(entry, struct hpsb_address_serve, host_list); | 326 | a1 = list_entry(entry, struct hpsb_address_serve, host_list); |
| 342 | a2 = list_entry(entry->next, struct hpsb_address_serve, host_list); | 327 | a2 = list_entry(entry->next, struct hpsb_address_serve, |
| 328 | host_list); | ||
| 343 | 329 | ||
| 344 | a1sa = a1->start & align_mask; | 330 | a1sa = a1->start & align_mask; |
| 345 | a1ea = (a1->end + alignment -1) & align_mask; | 331 | a1ea = (a1->end + alignment -1) & align_mask; |
| 346 | a2sa = a2->start & align_mask; | 332 | a2sa = a2->start & align_mask; |
| 347 | a2ea = (a2->end + alignment -1) & align_mask; | 333 | a2ea = (a2->end + alignment -1) & align_mask; |
| 348 | 334 | ||
| 349 | if ((a2sa - a1ea >= size) && (a2sa - start >= size) && (a2sa > start)) { | 335 | if ((a2sa - a1ea >= size) && (a2sa - start >= size) && |
| 336 | (a2sa > start)) { | ||
| 350 | as->start = max(start, a1ea); | 337 | as->start = max(start, a1ea); |
| 351 | as->end = as->start + size; | 338 | as->end = as->start + size; |
| 352 | list_add(&as->host_list, entry); | 339 | list_add(&as->host_list, entry); |
| @@ -355,47 +342,45 @@ u64 hpsb_allocate_and_register_addrspace(struct hpsb_highlevel *hl, | |||
| 355 | break; | 342 | break; |
| 356 | } | 343 | } |
| 357 | } | 344 | } |
| 358 | |||
| 359 | write_unlock_irqrestore(&addr_space_lock, flags); | 345 | write_unlock_irqrestore(&addr_space_lock, flags); |
| 360 | 346 | ||
| 361 | if (retval == ~0ULL) { | 347 | if (retval == CSR1212_INVALID_ADDR_SPACE) |
| 362 | kfree(as); | 348 | kfree(as); |
| 363 | } | ||
| 364 | |||
| 365 | return retval; | 349 | return retval; |
| 366 | } | 350 | } |
| 367 | 351 | ||
| 368 | int hpsb_register_addrspace(struct hpsb_highlevel *hl, struct hpsb_host *host, | 352 | int hpsb_register_addrspace(struct hpsb_highlevel *hl, struct hpsb_host *host, |
| 369 | struct hpsb_address_ops *ops, u64 start, u64 end) | 353 | struct hpsb_address_ops *ops, u64 start, u64 end) |
| 370 | { | 354 | { |
| 371 | struct hpsb_address_serve *as; | 355 | struct hpsb_address_serve *as; |
| 372 | struct list_head *lh; | 356 | struct list_head *lh; |
| 373 | int retval = 0; | 357 | int retval = 0; |
| 374 | unsigned long flags; | 358 | unsigned long flags; |
| 375 | 359 | ||
| 376 | if (((start|end) & 3) || (start >= end) || (end > 0x1000000000000ULL)) { | 360 | if (((start|end) & 3) || (start >= end) || |
| 377 | HPSB_ERR("%s called with invalid addresses", __FUNCTION__); | 361 | (end > CSR1212_ALL_SPACE_END)) { |
| 378 | return 0; | 362 | HPSB_ERR("%s called with invalid addresses", __FUNCTION__); |
| 379 | } | 363 | return 0; |
| 364 | } | ||
| 380 | 365 | ||
| 381 | as = kmalloc(sizeof(*as), GFP_ATOMIC); | 366 | as = kmalloc(sizeof(*as), GFP_ATOMIC); |
| 382 | if (!as) | 367 | if (!as) |
| 383 | return 0; | 368 | return 0; |
| 384 | 369 | ||
| 385 | INIT_LIST_HEAD(&as->host_list); | 370 | INIT_LIST_HEAD(&as->host_list); |
| 386 | INIT_LIST_HEAD(&as->hl_list); | 371 | INIT_LIST_HEAD(&as->hl_list); |
| 387 | as->op = ops; | 372 | as->op = ops; |
| 388 | as->start = start; | 373 | as->start = start; |
| 389 | as->end = end; | 374 | as->end = end; |
| 390 | as->host = host; | 375 | as->host = host; |
| 391 | 376 | ||
| 392 | write_lock_irqsave(&addr_space_lock, flags); | 377 | write_lock_irqsave(&addr_space_lock, flags); |
| 393 | |||
| 394 | list_for_each(lh, &host->addr_space) { | 378 | list_for_each(lh, &host->addr_space) { |
| 395 | struct hpsb_address_serve *as_this = | 379 | struct hpsb_address_serve *as_this = |
| 396 | list_entry(lh, struct hpsb_address_serve, host_list); | 380 | list_entry(lh, struct hpsb_address_serve, host_list); |
| 397 | struct hpsb_address_serve *as_next = | 381 | struct hpsb_address_serve *as_next = |
| 398 | list_entry(lh->next, struct hpsb_address_serve, host_list); | 382 | list_entry(lh->next, struct hpsb_address_serve, |
| 383 | host_list); | ||
| 399 | 384 | ||
| 400 | if (as_this->end > as->start) | 385 | if (as_this->end > as->start) |
| 401 | break; | 386 | break; |
| @@ -411,60 +396,51 @@ int hpsb_register_addrspace(struct hpsb_highlevel *hl, struct hpsb_host *host, | |||
| 411 | 396 | ||
| 412 | if (retval == 0) | 397 | if (retval == 0) |
| 413 | kfree(as); | 398 | kfree(as); |
| 414 | 399 | return retval; | |
| 415 | return retval; | ||
| 416 | } | 400 | } |
| 417 | 401 | ||
| 418 | int hpsb_unregister_addrspace(struct hpsb_highlevel *hl, struct hpsb_host *host, | 402 | int hpsb_unregister_addrspace(struct hpsb_highlevel *hl, struct hpsb_host *host, |
| 419 | u64 start) | 403 | u64 start) |
| 420 | { | 404 | { |
| 421 | int retval = 0; | 405 | int retval = 0; |
| 422 | struct hpsb_address_serve *as; | 406 | struct hpsb_address_serve *as; |
| 423 | struct list_head *lh, *next; | 407 | struct list_head *lh, *next; |
| 424 | unsigned long flags; | 408 | unsigned long flags; |
| 425 | |||
| 426 | write_lock_irqsave(&addr_space_lock, flags); | ||
| 427 | 409 | ||
| 410 | write_lock_irqsave(&addr_space_lock, flags); | ||
| 428 | list_for_each_safe (lh, next, &hl->addr_list) { | 411 | list_for_each_safe (lh, next, &hl->addr_list) { |
| 429 | as = list_entry(lh, struct hpsb_address_serve, hl_list); | 412 | as = list_entry(lh, struct hpsb_address_serve, hl_list); |
| 430 | if (as->start == start && as->host == host) { | 413 | if (as->start == start && as->host == host) { |
| 431 | __delete_addr(as); | 414 | __delete_addr(as); |
| 432 | retval = 1; | 415 | retval = 1; |
| 433 | break; | 416 | break; |
| 434 | } | 417 | } |
| 435 | } | 418 | } |
| 436 | 419 | write_unlock_irqrestore(&addr_space_lock, flags); | |
| 437 | write_unlock_irqrestore(&addr_space_lock, flags); | 420 | return retval; |
| 438 | |||
| 439 | return retval; | ||
| 440 | } | 421 | } |
| 441 | 422 | ||
| 442 | int hpsb_listen_channel(struct hpsb_highlevel *hl, struct hpsb_host *host, | 423 | int hpsb_listen_channel(struct hpsb_highlevel *hl, struct hpsb_host *host, |
| 443 | unsigned int channel) | 424 | unsigned int channel) |
| 444 | { | 425 | { |
| 445 | if (channel > 63) { | 426 | if (channel > 63) { |
| 446 | HPSB_ERR("%s called with invalid channel", __FUNCTION__); | 427 | HPSB_ERR("%s called with invalid channel", __FUNCTION__); |
| 447 | return -EINVAL; | 428 | return -EINVAL; |
| 448 | } | 429 | } |
| 449 | 430 | if (host->iso_listen_count[channel]++ == 0) | |
| 450 | if (host->iso_listen_count[channel]++ == 0) { | 431 | return host->driver->devctl(host, ISO_LISTEN_CHANNEL, channel); |
| 451 | return host->driver->devctl(host, ISO_LISTEN_CHANNEL, channel); | ||
| 452 | } | ||
| 453 | |||
| 454 | return 0; | 432 | return 0; |
| 455 | } | 433 | } |
| 456 | 434 | ||
| 457 | void hpsb_unlisten_channel(struct hpsb_highlevel *hl, struct hpsb_host *host, | 435 | void hpsb_unlisten_channel(struct hpsb_highlevel *hl, struct hpsb_host *host, |
| 458 | unsigned int channel) | 436 | unsigned int channel) |
| 459 | { | 437 | { |
| 460 | if (channel > 63) { | 438 | if (channel > 63) { |
| 461 | HPSB_ERR("%s called with invalid channel", __FUNCTION__); | 439 | HPSB_ERR("%s called with invalid channel", __FUNCTION__); |
| 462 | return; | 440 | return; |
| 463 | } | 441 | } |
| 464 | 442 | if (--host->iso_listen_count[channel] == 0) | |
| 465 | if (--host->iso_listen_count[channel] == 0) { | 443 | host->driver->devctl(host, ISO_UNLISTEN_CHANNEL, channel); |
| 466 | host->driver->devctl(host, ISO_UNLISTEN_CHANNEL, channel); | ||
| 467 | } | ||
| 468 | } | 444 | } |
| 469 | 445 | ||
| 470 | static void init_hpsb_highlevel(struct hpsb_host *host) | 446 | static void init_hpsb_highlevel(struct hpsb_host *host) |
| @@ -485,26 +461,24 @@ static void init_hpsb_highlevel(struct hpsb_host *host) | |||
| 485 | 461 | ||
| 486 | void highlevel_add_host(struct hpsb_host *host) | 462 | void highlevel_add_host(struct hpsb_host *host) |
| 487 | { | 463 | { |
| 488 | struct hpsb_highlevel *hl; | 464 | struct hpsb_highlevel *hl; |
| 489 | 465 | ||
| 490 | init_hpsb_highlevel(host); | 466 | init_hpsb_highlevel(host); |
| 491 | 467 | ||
| 492 | down_read(&hl_drivers_sem); | 468 | down_read(&hl_drivers_sem); |
| 493 | list_for_each_entry(hl, &hl_drivers, hl_list) { | 469 | list_for_each_entry(hl, &hl_drivers, hl_list) { |
| 494 | if (hl->add_host) | 470 | if (hl->add_host) |
| 495 | hl->add_host(host); | 471 | hl->add_host(host); |
| 496 | } | ||
| 497 | up_read(&hl_drivers_sem); | ||
| 498 | if (host->update_config_rom) { | ||
| 499 | if (hpsb_update_config_rom_image(host) < 0) | ||
| 500 | HPSB_ERR("Failed to generate Configuration ROM image for " | ||
| 501 | "host %s-%d", hl->name, host->id); | ||
| 502 | } | 472 | } |
| 473 | up_read(&hl_drivers_sem); | ||
| 474 | if (host->update_config_rom && hpsb_update_config_rom_image(host) < 0) | ||
| 475 | HPSB_ERR("Failed to generate Configuration ROM image for host " | ||
| 476 | "%s-%d", hl->name, host->id); | ||
| 503 | } | 477 | } |
| 504 | 478 | ||
| 505 | void highlevel_remove_host(struct hpsb_host *host) | 479 | void highlevel_remove_host(struct hpsb_host *host) |
| 506 | { | 480 | { |
| 507 | struct hpsb_highlevel *hl; | 481 | struct hpsb_highlevel *hl; |
| 508 | 482 | ||
| 509 | down_read(&hl_drivers_sem); | 483 | down_read(&hl_drivers_sem); |
| 510 | list_for_each_entry(hl, &hl_drivers, hl_list) | 484 | list_for_each_entry(hl, &hl_drivers, hl_list) |
| @@ -514,184 +488,169 @@ void highlevel_remove_host(struct hpsb_host *host) | |||
| 514 | 488 | ||
| 515 | void highlevel_host_reset(struct hpsb_host *host) | 489 | void highlevel_host_reset(struct hpsb_host *host) |
| 516 | { | 490 | { |
| 517 | struct hpsb_highlevel *hl; | 491 | unsigned long flags; |
| 492 | struct hpsb_highlevel *hl; | ||
| 518 | 493 | ||
| 519 | read_lock(&hl_irqs_lock); | 494 | read_lock_irqsave(&hl_irqs_lock, flags); |
| 520 | list_for_each_entry(hl, &hl_irqs, irq_list) { | 495 | list_for_each_entry(hl, &hl_irqs, irq_list) { |
| 521 | if (hl->host_reset) | 496 | if (hl->host_reset) |
| 522 | hl->host_reset(host); | 497 | hl->host_reset(host); |
| 523 | } | 498 | } |
| 524 | read_unlock(&hl_irqs_lock); | 499 | read_unlock_irqrestore(&hl_irqs_lock, flags); |
| 525 | } | 500 | } |
| 526 | 501 | ||
| 527 | void highlevel_iso_receive(struct hpsb_host *host, void *data, size_t length) | 502 | void highlevel_iso_receive(struct hpsb_host *host, void *data, size_t length) |
| 528 | { | 503 | { |
| 529 | struct hpsb_highlevel *hl; | 504 | unsigned long flags; |
| 530 | int channel = (((quadlet_t *)data)[0] >> 8) & 0x3f; | 505 | struct hpsb_highlevel *hl; |
| 506 | int channel = (((quadlet_t *)data)[0] >> 8) & 0x3f; | ||
| 531 | 507 | ||
| 532 | read_lock(&hl_irqs_lock); | 508 | read_lock_irqsave(&hl_irqs_lock, flags); |
| 533 | list_for_each_entry(hl, &hl_irqs, irq_list) { | 509 | list_for_each_entry(hl, &hl_irqs, irq_list) { |
| 534 | if (hl->iso_receive) | 510 | if (hl->iso_receive) |
| 535 | hl->iso_receive(host, channel, data, length); | 511 | hl->iso_receive(host, channel, data, length); |
| 536 | } | 512 | } |
| 537 | read_unlock(&hl_irqs_lock); | 513 | read_unlock_irqrestore(&hl_irqs_lock, flags); |
| 538 | } | 514 | } |
| 539 | 515 | ||
| 540 | void highlevel_fcp_request(struct hpsb_host *host, int nodeid, int direction, | 516 | void highlevel_fcp_request(struct hpsb_host *host, int nodeid, int direction, |
| 541 | void *data, size_t length) | 517 | void *data, size_t length) |
| 542 | { | 518 | { |
| 543 | struct hpsb_highlevel *hl; | 519 | unsigned long flags; |
| 544 | int cts = ((quadlet_t *)data)[0] >> 4; | 520 | struct hpsb_highlevel *hl; |
| 521 | int cts = ((quadlet_t *)data)[0] >> 4; | ||
| 545 | 522 | ||
| 546 | read_lock(&hl_irqs_lock); | 523 | read_lock_irqsave(&hl_irqs_lock, flags); |
| 547 | list_for_each_entry(hl, &hl_irqs, irq_list) { | 524 | list_for_each_entry(hl, &hl_irqs, irq_list) { |
| 548 | if (hl->fcp_request) | 525 | if (hl->fcp_request) |
| 549 | hl->fcp_request(host, nodeid, direction, cts, data, | 526 | hl->fcp_request(host, nodeid, direction, cts, data, |
| 550 | length); | 527 | length); |
| 551 | } | 528 | } |
| 552 | read_unlock(&hl_irqs_lock); | 529 | read_unlock_irqrestore(&hl_irqs_lock, flags); |
| 553 | } | 530 | } |
| 554 | 531 | ||
| 555 | int highlevel_read(struct hpsb_host *host, int nodeid, void *data, | 532 | int highlevel_read(struct hpsb_host *host, int nodeid, void *data, u64 addr, |
| 556 | u64 addr, unsigned int length, u16 flags) | 533 | unsigned int length, u16 flags) |
| 557 | { | 534 | { |
| 558 | struct hpsb_address_serve *as; | 535 | struct hpsb_address_serve *as; |
| 559 | unsigned int partlength; | 536 | unsigned int partlength; |
| 560 | int rcode = RCODE_ADDRESS_ERROR; | 537 | int rcode = RCODE_ADDRESS_ERROR; |
| 561 | |||
| 562 | read_lock(&addr_space_lock); | ||
| 563 | 538 | ||
| 539 | read_lock(&addr_space_lock); | ||
| 564 | list_for_each_entry(as, &host->addr_space, host_list) { | 540 | list_for_each_entry(as, &host->addr_space, host_list) { |
| 565 | if (as->start > addr) | 541 | if (as->start > addr) |
| 566 | break; | 542 | break; |
| 567 | 543 | ||
| 568 | if (as->end > addr) { | 544 | if (as->end > addr) { |
| 569 | partlength = min(as->end - addr, (u64) length); | 545 | partlength = min(as->end - addr, (u64) length); |
| 570 | 546 | ||
| 571 | if (as->op->read) { | 547 | if (as->op->read) |
| 572 | rcode = as->op->read(host, nodeid, data, | 548 | rcode = as->op->read(host, nodeid, data, |
| 573 | addr, partlength, flags); | 549 | addr, partlength, flags); |
| 574 | } else { | 550 | else |
| 575 | rcode = RCODE_TYPE_ERROR; | 551 | rcode = RCODE_TYPE_ERROR; |
| 576 | } | ||
| 577 | 552 | ||
| 578 | data += partlength; | 553 | data += partlength; |
| 579 | length -= partlength; | 554 | length -= partlength; |
| 580 | addr += partlength; | 555 | addr += partlength; |
| 581 | |||
| 582 | if ((rcode != RCODE_COMPLETE) || !length) { | ||
| 583 | break; | ||
| 584 | } | ||
| 585 | } | ||
| 586 | } | ||
| 587 | |||
| 588 | read_unlock(&addr_space_lock); | ||
| 589 | 556 | ||
| 590 | if (length && (rcode == RCODE_COMPLETE)) { | 557 | if ((rcode != RCODE_COMPLETE) || !length) |
| 591 | rcode = RCODE_ADDRESS_ERROR; | 558 | break; |
| 592 | } | 559 | } |
| 560 | } | ||
| 561 | read_unlock(&addr_space_lock); | ||
| 593 | 562 | ||
| 594 | return rcode; | 563 | if (length && (rcode == RCODE_COMPLETE)) |
| 564 | rcode = RCODE_ADDRESS_ERROR; | ||
| 565 | return rcode; | ||
| 595 | } | 566 | } |
| 596 | 567 | ||
| 597 | int highlevel_write(struct hpsb_host *host, int nodeid, int destid, | 568 | int highlevel_write(struct hpsb_host *host, int nodeid, int destid, void *data, |
| 598 | void *data, u64 addr, unsigned int length, u16 flags) | 569 | u64 addr, unsigned int length, u16 flags) |
| 599 | { | 570 | { |
| 600 | struct hpsb_address_serve *as; | 571 | struct hpsb_address_serve *as; |
| 601 | unsigned int partlength; | 572 | unsigned int partlength; |
| 602 | int rcode = RCODE_ADDRESS_ERROR; | 573 | int rcode = RCODE_ADDRESS_ERROR; |
| 603 | |||
| 604 | read_lock(&addr_space_lock); | ||
| 605 | 574 | ||
| 575 | read_lock(&addr_space_lock); | ||
| 606 | list_for_each_entry(as, &host->addr_space, host_list) { | 576 | list_for_each_entry(as, &host->addr_space, host_list) { |
| 607 | if (as->start > addr) | 577 | if (as->start > addr) |
| 608 | break; | 578 | break; |
| 609 | 579 | ||
| 610 | if (as->end > addr) { | 580 | if (as->end > addr) { |
| 611 | partlength = min(as->end - addr, (u64) length); | 581 | partlength = min(as->end - addr, (u64) length); |
| 612 | 582 | ||
| 613 | if (as->op->write) { | 583 | if (as->op->write) |
| 614 | rcode = as->op->write(host, nodeid, destid, | 584 | rcode = as->op->write(host, nodeid, destid, |
| 615 | data, addr, partlength, flags); | 585 | data, addr, partlength, |
| 616 | } else { | 586 | flags); |
| 617 | rcode = RCODE_TYPE_ERROR; | 587 | else |
| 618 | } | 588 | rcode = RCODE_TYPE_ERROR; |
| 619 | 589 | ||
| 620 | data += partlength; | 590 | data += partlength; |
| 621 | length -= partlength; | 591 | length -= partlength; |
| 622 | addr += partlength; | 592 | addr += partlength; |
| 623 | |||
| 624 | if ((rcode != RCODE_COMPLETE) || !length) { | ||
| 625 | break; | ||
| 626 | } | ||
| 627 | } | ||
| 628 | } | ||
| 629 | |||
| 630 | read_unlock(&addr_space_lock); | ||
| 631 | 593 | ||
| 632 | if (length && (rcode == RCODE_COMPLETE)) { | 594 | if ((rcode != RCODE_COMPLETE) || !length) |
| 633 | rcode = RCODE_ADDRESS_ERROR; | 595 | break; |
| 634 | } | 596 | } |
| 597 | } | ||
| 598 | read_unlock(&addr_space_lock); | ||
| 635 | 599 | ||
| 636 | return rcode; | 600 | if (length && (rcode == RCODE_COMPLETE)) |
| 601 | rcode = RCODE_ADDRESS_ERROR; | ||
| 602 | return rcode; | ||
| 637 | } | 603 | } |
| 638 | 604 | ||
| 639 | |||
| 640 | int highlevel_lock(struct hpsb_host *host, int nodeid, quadlet_t *store, | 605 | int highlevel_lock(struct hpsb_host *host, int nodeid, quadlet_t *store, |
| 641 | u64 addr, quadlet_t data, quadlet_t arg, int ext_tcode, u16 flags) | 606 | u64 addr, quadlet_t data, quadlet_t arg, int ext_tcode, |
| 607 | u16 flags) | ||
| 642 | { | 608 | { |
| 643 | struct hpsb_address_serve *as; | 609 | struct hpsb_address_serve *as; |
| 644 | int rcode = RCODE_ADDRESS_ERROR; | 610 | int rcode = RCODE_ADDRESS_ERROR; |
| 645 | |||
| 646 | read_lock(&addr_space_lock); | ||
| 647 | 611 | ||
| 612 | read_lock(&addr_space_lock); | ||
| 648 | list_for_each_entry(as, &host->addr_space, host_list) { | 613 | list_for_each_entry(as, &host->addr_space, host_list) { |
| 649 | if (as->start > addr) | 614 | if (as->start > addr) |
| 650 | break; | 615 | break; |
| 651 | 616 | ||
| 652 | if (as->end > addr) { | 617 | if (as->end > addr) { |
| 653 | if (as->op->lock) { | 618 | if (as->op->lock) |
| 654 | rcode = as->op->lock(host, nodeid, store, addr, | 619 | rcode = as->op->lock(host, nodeid, store, addr, |
| 655 | data, arg, ext_tcode, flags); | 620 | data, arg, ext_tcode, |
| 656 | } else { | 621 | flags); |
| 657 | rcode = RCODE_TYPE_ERROR; | 622 | else |
| 658 | } | 623 | rcode = RCODE_TYPE_ERROR; |
| 659 | 624 | break; | |
| 660 | break; | 625 | } |
| 661 | } | 626 | } |
| 662 | } | 627 | read_unlock(&addr_space_lock); |
| 663 | 628 | return rcode; | |
| 664 | read_unlock(&addr_space_lock); | ||
| 665 | |||
| 666 | return rcode; | ||
| 667 | } | 629 | } |
| 668 | 630 | ||
| 669 | int highlevel_lock64(struct hpsb_host *host, int nodeid, octlet_t *store, | 631 | int highlevel_lock64(struct hpsb_host *host, int nodeid, octlet_t *store, |
| 670 | u64 addr, octlet_t data, octlet_t arg, int ext_tcode, u16 flags) | 632 | u64 addr, octlet_t data, octlet_t arg, int ext_tcode, |
| 633 | u16 flags) | ||
| 671 | { | 634 | { |
| 672 | struct hpsb_address_serve *as; | 635 | struct hpsb_address_serve *as; |
| 673 | int rcode = RCODE_ADDRESS_ERROR; | 636 | int rcode = RCODE_ADDRESS_ERROR; |
| 674 | 637 | ||
| 675 | read_lock(&addr_space_lock); | 638 | read_lock(&addr_space_lock); |
| 676 | 639 | ||
| 677 | list_for_each_entry(as, &host->addr_space, host_list) { | 640 | list_for_each_entry(as, &host->addr_space, host_list) { |
| 678 | if (as->start > addr) | 641 | if (as->start > addr) |
| 679 | break; | 642 | break; |
| 680 | 643 | ||
| 681 | if (as->end > addr) { | 644 | if (as->end > addr) { |
| 682 | if (as->op->lock64) { | 645 | if (as->op->lock64) |
| 683 | rcode = as->op->lock64(host, nodeid, store, | 646 | rcode = as->op->lock64(host, nodeid, store, |
| 684 | addr, data, arg, | 647 | addr, data, arg, |
| 685 | ext_tcode, flags); | 648 | ext_tcode, flags); |
| 686 | } else { | 649 | else |
| 687 | rcode = RCODE_TYPE_ERROR; | 650 | rcode = RCODE_TYPE_ERROR; |
| 688 | } | 651 | break; |
| 689 | 652 | } | |
| 690 | break; | 653 | } |
| 691 | } | 654 | read_unlock(&addr_space_lock); |
| 692 | } | 655 | return rcode; |
| 693 | |||
| 694 | read_unlock(&addr_space_lock); | ||
| 695 | |||
| 696 | return rcode; | ||
| 697 | } | 656 | } |
diff --git a/drivers/ieee1394/hosts.c b/drivers/ieee1394/hosts.c index ba09741fc826..2d47b11777a5 100644 --- a/drivers/ieee1394/hosts.c +++ b/drivers/ieee1394/hosts.c | |||
| @@ -19,6 +19,7 @@ | |||
| 19 | #include <linux/pci.h> | 19 | #include <linux/pci.h> |
| 20 | #include <linux/timer.h> | 20 | #include <linux/timer.h> |
| 21 | #include <linux/jiffies.h> | 21 | #include <linux/jiffies.h> |
| 22 | #include <linux/mutex.h> | ||
| 22 | 23 | ||
| 23 | #include "csr1212.h" | 24 | #include "csr1212.h" |
| 24 | #include "ieee1394.h" | 25 | #include "ieee1394.h" |
| @@ -105,7 +106,7 @@ static int alloc_hostnum_cb(struct hpsb_host *host, void *__data) | |||
| 105 | * Return Value: a pointer to the &hpsb_host if succesful, %NULL if | 106 | * Return Value: a pointer to the &hpsb_host if succesful, %NULL if |
| 106 | * no memory was available. | 107 | * no memory was available. |
| 107 | */ | 108 | */ |
| 108 | static DECLARE_MUTEX(host_num_alloc); | 109 | static DEFINE_MUTEX(host_num_alloc); |
| 109 | 110 | ||
| 110 | struct hpsb_host *hpsb_alloc_host(struct hpsb_host_driver *drv, size_t extra, | 111 | struct hpsb_host *hpsb_alloc_host(struct hpsb_host_driver *drv, size_t extra, |
| 111 | struct device *dev) | 112 | struct device *dev) |
| @@ -148,7 +149,7 @@ struct hpsb_host *hpsb_alloc_host(struct hpsb_host_driver *drv, size_t extra, | |||
| 148 | h->topology_map = h->csr.topology_map + 3; | 149 | h->topology_map = h->csr.topology_map + 3; |
| 149 | h->speed_map = (u8 *)(h->csr.speed_map + 2); | 150 | h->speed_map = (u8 *)(h->csr.speed_map + 2); |
| 150 | 151 | ||
| 151 | down(&host_num_alloc); | 152 | mutex_lock(&host_num_alloc); |
| 152 | 153 | ||
| 153 | while (nodemgr_for_each_host(&hostnum, alloc_hostnum_cb)) | 154 | while (nodemgr_for_each_host(&hostnum, alloc_hostnum_cb)) |
| 154 | hostnum++; | 155 | hostnum++; |
| @@ -167,7 +168,7 @@ struct hpsb_host *hpsb_alloc_host(struct hpsb_host_driver *drv, size_t extra, | |||
| 167 | class_device_register(&h->class_dev); | 168 | class_device_register(&h->class_dev); |
| 168 | get_device(&h->device); | 169 | get_device(&h->device); |
| 169 | 170 | ||
| 170 | up(&host_num_alloc); | 171 | mutex_unlock(&host_num_alloc); |
| 171 | 172 | ||
| 172 | return h; | 173 | return h; |
| 173 | } | 174 | } |
diff --git a/drivers/ieee1394/hosts.h b/drivers/ieee1394/hosts.h index 07d188ca8495..9ad4b2463077 100644 --- a/drivers/ieee1394/hosts.h +++ b/drivers/ieee1394/hosts.h | |||
| @@ -30,13 +30,14 @@ struct hpsb_host { | |||
| 30 | 30 | ||
| 31 | unsigned char iso_listen_count[64]; | 31 | unsigned char iso_listen_count[64]; |
| 32 | 32 | ||
| 33 | int node_count; /* number of identified nodes on this bus */ | 33 | int node_count; /* number of identified nodes on this bus */ |
| 34 | int selfid_count; /* total number of SelfIDs received */ | 34 | int selfid_count; /* total number of SelfIDs received */ |
| 35 | int nodes_active; /* number of nodes that are actually active */ | 35 | int nodes_active; /* number of nodes with active link layer */ |
| 36 | u8 speed[ALL_NODES]; /* speed between each node and local node */ | ||
| 36 | 37 | ||
| 37 | nodeid_t node_id; /* node ID of this host */ | 38 | nodeid_t node_id; /* node ID of this host */ |
| 38 | nodeid_t irm_id; /* ID of this bus' isochronous resource manager */ | 39 | nodeid_t irm_id; /* ID of this bus' isochronous resource manager */ |
| 39 | nodeid_t busmgr_id; /* ID of this bus' bus manager */ | 40 | nodeid_t busmgr_id; /* ID of this bus' bus manager */ |
| 40 | 41 | ||
| 41 | /* this nodes state */ | 42 | /* this nodes state */ |
| 42 | unsigned in_bus_reset:1; | 43 | unsigned in_bus_reset:1; |
| @@ -55,7 +56,7 @@ struct hpsb_host { | |||
| 55 | struct csr_control csr; | 56 | struct csr_control csr; |
| 56 | 57 | ||
| 57 | /* Per node tlabel pool allocation */ | 58 | /* Per node tlabel pool allocation */ |
| 58 | struct hpsb_tlabel_pool tpool[64]; | 59 | struct hpsb_tlabel_pool tpool[ALL_NODES]; |
| 59 | 60 | ||
| 60 | struct hpsb_host_driver *driver; | 61 | struct hpsb_host_driver *driver; |
| 61 | 62 | ||
| @@ -72,6 +73,8 @@ struct hpsb_host { | |||
| 72 | unsigned int config_roms; | 73 | unsigned int config_roms; |
| 73 | 74 | ||
| 74 | struct list_head addr_space; | 75 | struct list_head addr_space; |
| 76 | u64 low_addr_space; /* upper bound of physical DMA area */ | ||
| 77 | u64 middle_addr_space; /* upper bound of posted write area */ | ||
| 75 | }; | 78 | }; |
| 76 | 79 | ||
| 77 | 80 | ||
diff --git a/drivers/ieee1394/ieee1394_core.c b/drivers/ieee1394/ieee1394_core.c index be6854e25ad4..49354de9fb8a 100644 --- a/drivers/ieee1394/ieee1394_core.c +++ b/drivers/ieee1394/ieee1394_core.c | |||
| @@ -33,6 +33,7 @@ | |||
| 33 | #include <linux/kdev_t.h> | 33 | #include <linux/kdev_t.h> |
| 34 | #include <linux/skbuff.h> | 34 | #include <linux/skbuff.h> |
| 35 | #include <linux/suspend.h> | 35 | #include <linux/suspend.h> |
| 36 | #include <linux/kthread.h> | ||
| 36 | 37 | ||
| 37 | #include <asm/byteorder.h> | 38 | #include <asm/byteorder.h> |
| 38 | #include <asm/semaphore.h> | 39 | #include <asm/semaphore.h> |
| @@ -285,9 +286,9 @@ static int check_selfids(struct hpsb_host *host) | |||
| 285 | 286 | ||
| 286 | static void build_speed_map(struct hpsb_host *host, int nodecount) | 287 | static void build_speed_map(struct hpsb_host *host, int nodecount) |
| 287 | { | 288 | { |
| 288 | u8 speedcap[nodecount]; | ||
| 289 | u8 cldcnt[nodecount]; | 289 | u8 cldcnt[nodecount]; |
| 290 | u8 *map = host->speed_map; | 290 | u8 *map = host->speed_map; |
| 291 | u8 *speedcap = host->speed; | ||
| 291 | struct selfid *sid; | 292 | struct selfid *sid; |
| 292 | struct ext_selfid *esid; | 293 | struct ext_selfid *esid; |
| 293 | int i, j, n; | 294 | int i, j, n; |
| @@ -354,6 +355,11 @@ static void build_speed_map(struct hpsb_host *host, int nodecount) | |||
| 354 | } | 355 | } |
| 355 | } | 356 | } |
| 356 | } | 357 | } |
| 358 | |||
| 359 | /* assume maximum speed for 1394b PHYs, nodemgr will correct it */ | ||
| 360 | for (n = 0; n < nodecount; n++) | ||
| 361 | if (speedcap[n] == 3) | ||
| 362 | speedcap[n] = IEEE1394_SPEED_MAX; | ||
| 357 | } | 363 | } |
| 358 | 364 | ||
| 359 | 365 | ||
| @@ -554,11 +560,10 @@ int hpsb_send_packet(struct hpsb_packet *packet) | |||
| 554 | return 0; | 560 | return 0; |
| 555 | } | 561 | } |
| 556 | 562 | ||
| 557 | if (packet->type == hpsb_async && packet->node_id != ALL_NODES) { | 563 | if (packet->type == hpsb_async && |
| 564 | NODEID_TO_NODE(packet->node_id) != ALL_NODES) | ||
| 558 | packet->speed_code = | 565 | packet->speed_code = |
| 559 | host->speed_map[NODEID_TO_NODE(host->node_id) * 64 | 566 | host->speed[NODEID_TO_NODE(packet->node_id)]; |
| 560 | + NODEID_TO_NODE(packet->node_id)]; | ||
| 561 | } | ||
| 562 | 567 | ||
| 563 | dump_packet("send packet", packet->header, packet->header_size, packet->speed_code); | 568 | dump_packet("send packet", packet->header, packet->header_size, packet->speed_code); |
| 564 | 569 | ||
| @@ -997,11 +1002,8 @@ void abort_timedouts(unsigned long __opaque) | |||
| 997 | * packets that have a "complete" function are sent here. This way, the | 1002 | * packets that have a "complete" function are sent here. This way, the |
| 998 | * completion is run out of kernel context, and doesn't block the rest of | 1003 | * completion is run out of kernel context, and doesn't block the rest of |
| 999 | * the stack. */ | 1004 | * the stack. */ |
| 1000 | static int khpsbpkt_pid = -1, khpsbpkt_kill; | 1005 | static struct task_struct *khpsbpkt_thread; |
| 1001 | static DECLARE_COMPLETION(khpsbpkt_complete); | ||
| 1002 | static struct sk_buff_head hpsbpkt_queue; | 1006 | static struct sk_buff_head hpsbpkt_queue; |
| 1003 | static DECLARE_MUTEX_LOCKED(khpsbpkt_sig); | ||
| 1004 | |||
| 1005 | 1007 | ||
| 1006 | static void queue_packet_complete(struct hpsb_packet *packet) | 1008 | static void queue_packet_complete(struct hpsb_packet *packet) |
| 1007 | { | 1009 | { |
| @@ -1011,9 +1013,7 @@ static void queue_packet_complete(struct hpsb_packet *packet) | |||
| 1011 | } | 1013 | } |
| 1012 | if (packet->complete_routine != NULL) { | 1014 | if (packet->complete_routine != NULL) { |
| 1013 | skb_queue_tail(&hpsbpkt_queue, packet->skb); | 1015 | skb_queue_tail(&hpsbpkt_queue, packet->skb); |
| 1014 | 1016 | wake_up_process(khpsbpkt_thread); | |
| 1015 | /* Signal the kernel thread to handle this */ | ||
| 1016 | up(&khpsbpkt_sig); | ||
| 1017 | } | 1017 | } |
| 1018 | return; | 1018 | return; |
| 1019 | } | 1019 | } |
| @@ -1025,19 +1025,9 @@ static int hpsbpkt_thread(void *__hi) | |||
| 1025 | void (*complete_routine)(void*); | 1025 | void (*complete_routine)(void*); |
| 1026 | void *complete_data; | 1026 | void *complete_data; |
| 1027 | 1027 | ||
| 1028 | daemonize("khpsbpkt"); | ||
| 1029 | |||
| 1030 | current->flags |= PF_NOFREEZE; | 1028 | current->flags |= PF_NOFREEZE; |
| 1031 | 1029 | ||
| 1032 | while (1) { | 1030 | while (!kthread_should_stop()) { |
| 1033 | if (down_interruptible(&khpsbpkt_sig)) { | ||
| 1034 | printk("khpsbpkt: received unexpected signal?!\n" ); | ||
| 1035 | break; | ||
| 1036 | } | ||
| 1037 | |||
| 1038 | if (khpsbpkt_kill) | ||
| 1039 | break; | ||
| 1040 | |||
| 1041 | while ((skb = skb_dequeue(&hpsbpkt_queue)) != NULL) { | 1031 | while ((skb = skb_dequeue(&hpsbpkt_queue)) != NULL) { |
| 1042 | packet = (struct hpsb_packet *)skb->data; | 1032 | packet = (struct hpsb_packet *)skb->data; |
| 1043 | 1033 | ||
| @@ -1048,9 +1038,13 @@ static int hpsbpkt_thread(void *__hi) | |||
| 1048 | 1038 | ||
| 1049 | complete_routine(complete_data); | 1039 | complete_routine(complete_data); |
| 1050 | } | 1040 | } |
| 1051 | } | ||
| 1052 | 1041 | ||
| 1053 | complete_and_exit(&khpsbpkt_complete, 0); | 1042 | set_current_state(TASK_INTERRUPTIBLE); |
| 1043 | if (!skb_peek(&hpsbpkt_queue)) | ||
| 1044 | schedule(); | ||
| 1045 | __set_current_state(TASK_RUNNING); | ||
| 1046 | } | ||
| 1047 | return 0; | ||
| 1054 | } | 1048 | } |
| 1055 | 1049 | ||
| 1056 | static int __init ieee1394_init(void) | 1050 | static int __init ieee1394_init(void) |
| @@ -1065,10 +1059,10 @@ static int __init ieee1394_init(void) | |||
| 1065 | HPSB_ERR("Some features may not be available\n"); | 1059 | HPSB_ERR("Some features may not be available\n"); |
| 1066 | } | 1060 | } |
| 1067 | 1061 | ||
| 1068 | khpsbpkt_pid = kernel_thread(hpsbpkt_thread, NULL, CLONE_KERNEL); | 1062 | khpsbpkt_thread = kthread_run(hpsbpkt_thread, NULL, "khpsbpkt"); |
| 1069 | if (khpsbpkt_pid < 0) { | 1063 | if (IS_ERR(khpsbpkt_thread)) { |
| 1070 | HPSB_ERR("Failed to start hpsbpkt thread!\n"); | 1064 | HPSB_ERR("Failed to start hpsbpkt thread!\n"); |
| 1071 | ret = -ENOMEM; | 1065 | ret = PTR_ERR(khpsbpkt_thread); |
| 1072 | goto exit_cleanup_config_roms; | 1066 | goto exit_cleanup_config_roms; |
| 1073 | } | 1067 | } |
| 1074 | 1068 | ||
| @@ -1148,10 +1142,7 @@ release_all_bus: | |||
| 1148 | release_chrdev: | 1142 | release_chrdev: |
| 1149 | unregister_chrdev_region(IEEE1394_CORE_DEV, 256); | 1143 | unregister_chrdev_region(IEEE1394_CORE_DEV, 256); |
| 1150 | exit_release_kernel_thread: | 1144 | exit_release_kernel_thread: |
| 1151 | if (khpsbpkt_pid >= 0) { | 1145 | kthread_stop(khpsbpkt_thread); |
| 1152 | kill_proc(khpsbpkt_pid, SIGTERM, 1); | ||
| 1153 | wait_for_completion(&khpsbpkt_complete); | ||
| 1154 | } | ||
| 1155 | exit_cleanup_config_roms: | 1146 | exit_cleanup_config_roms: |
| 1156 | hpsb_cleanup_config_roms(); | 1147 | hpsb_cleanup_config_roms(); |
| 1157 | return ret; | 1148 | return ret; |
| @@ -1172,12 +1163,7 @@ static void __exit ieee1394_cleanup(void) | |||
| 1172 | bus_remove_file(&ieee1394_bus_type, fw_bus_attrs[i]); | 1163 | bus_remove_file(&ieee1394_bus_type, fw_bus_attrs[i]); |
| 1173 | bus_unregister(&ieee1394_bus_type); | 1164 | bus_unregister(&ieee1394_bus_type); |
| 1174 | 1165 | ||
| 1175 | if (khpsbpkt_pid >= 0) { | 1166 | kthread_stop(khpsbpkt_thread); |
| 1176 | khpsbpkt_kill = 1; | ||
| 1177 | mb(); | ||
| 1178 | up(&khpsbpkt_sig); | ||
| 1179 | wait_for_completion(&khpsbpkt_complete); | ||
| 1180 | } | ||
| 1181 | 1167 | ||
| 1182 | hpsb_cleanup_config_roms(); | 1168 | hpsb_cleanup_config_roms(); |
| 1183 | 1169 | ||
diff --git a/drivers/ieee1394/ieee1394_transactions.c b/drivers/ieee1394/ieee1394_transactions.c index 3fe2f6c4a253..a114b91d606d 100644 --- a/drivers/ieee1394/ieee1394_transactions.c +++ b/drivers/ieee1394/ieee1394_transactions.c | |||
| @@ -136,8 +136,11 @@ int hpsb_get_tlabel(struct hpsb_packet *packet) | |||
| 136 | { | 136 | { |
| 137 | unsigned long flags; | 137 | unsigned long flags; |
| 138 | struct hpsb_tlabel_pool *tp; | 138 | struct hpsb_tlabel_pool *tp; |
| 139 | int n = NODEID_TO_NODE(packet->node_id); | ||
| 139 | 140 | ||
| 140 | tp = &packet->host->tpool[packet->node_id & NODE_MASK]; | 141 | if (unlikely(n == ALL_NODES)) |
| 142 | return 0; | ||
| 143 | tp = &packet->host->tpool[n]; | ||
| 141 | 144 | ||
| 142 | if (irqs_disabled() || in_atomic()) { | 145 | if (irqs_disabled() || in_atomic()) { |
| 143 | if (down_trylock(&tp->count)) | 146 | if (down_trylock(&tp->count)) |
| @@ -175,8 +178,11 @@ void hpsb_free_tlabel(struct hpsb_packet *packet) | |||
| 175 | { | 178 | { |
| 176 | unsigned long flags; | 179 | unsigned long flags; |
| 177 | struct hpsb_tlabel_pool *tp; | 180 | struct hpsb_tlabel_pool *tp; |
| 181 | int n = NODEID_TO_NODE(packet->node_id); | ||
| 178 | 182 | ||
| 179 | tp = &packet->host->tpool[packet->node_id & NODE_MASK]; | 183 | if (unlikely(n == ALL_NODES)) |
| 184 | return; | ||
| 185 | tp = &packet->host->tpool[n]; | ||
| 180 | 186 | ||
| 181 | BUG_ON(packet->tlabel > 63 || packet->tlabel < 0); | 187 | BUG_ON(packet->tlabel > 63 || packet->tlabel < 0); |
| 182 | 188 | ||
diff --git a/drivers/ieee1394/nodemgr.c b/drivers/ieee1394/nodemgr.c index 082c7fd239f5..948f1b8c4238 100644 --- a/drivers/ieee1394/nodemgr.c +++ b/drivers/ieee1394/nodemgr.c | |||
| @@ -38,6 +38,7 @@ struct nodemgr_csr_info { | |||
| 38 | struct hpsb_host *host; | 38 | struct hpsb_host *host; |
| 39 | nodeid_t nodeid; | 39 | nodeid_t nodeid; |
| 40 | unsigned int generation; | 40 | unsigned int generation; |
| 41 | unsigned int speed_unverified:1; | ||
| 41 | }; | 42 | }; |
| 42 | 43 | ||
| 43 | 44 | ||
| @@ -57,23 +58,75 @@ static char *nodemgr_find_oui_name(int oui) | |||
| 57 | return NULL; | 58 | return NULL; |
| 58 | } | 59 | } |
| 59 | 60 | ||
| 61 | /* | ||
| 62 | * Correct the speed map entry. This is necessary | ||
| 63 | * - for nodes with link speed < phy speed, | ||
| 64 | * - for 1394b nodes with negotiated phy port speed < IEEE1394_SPEED_MAX. | ||
| 65 | * A possible speed is determined by trial and error, using quadlet reads. | ||
| 66 | */ | ||
| 67 | static int nodemgr_check_speed(struct nodemgr_csr_info *ci, u64 addr, | ||
| 68 | quadlet_t *buffer) | ||
| 69 | { | ||
| 70 | quadlet_t q; | ||
| 71 | u8 i, *speed, old_speed, good_speed; | ||
| 72 | int ret; | ||
| 73 | |||
| 74 | speed = ci->host->speed + NODEID_TO_NODE(ci->nodeid); | ||
| 75 | old_speed = *speed; | ||
| 76 | good_speed = IEEE1394_SPEED_MAX + 1; | ||
| 77 | |||
| 78 | /* Try every speed from S100 to old_speed. | ||
| 79 | * If we did it the other way around, a too low speed could be caught | ||
| 80 | * if the retry succeeded for some other reason, e.g. because the link | ||
| 81 | * just finished its initialization. */ | ||
| 82 | for (i = IEEE1394_SPEED_100; i <= old_speed; i++) { | ||
| 83 | *speed = i; | ||
| 84 | ret = hpsb_read(ci->host, ci->nodeid, ci->generation, addr, | ||
| 85 | &q, sizeof(quadlet_t)); | ||
| 86 | if (ret) | ||
| 87 | break; | ||
| 88 | *buffer = q; | ||
| 89 | good_speed = i; | ||
| 90 | } | ||
| 91 | if (good_speed <= IEEE1394_SPEED_MAX) { | ||
| 92 | HPSB_DEBUG("Speed probe of node " NODE_BUS_FMT " yields %s", | ||
| 93 | NODE_BUS_ARGS(ci->host, ci->nodeid), | ||
| 94 | hpsb_speedto_str[good_speed]); | ||
| 95 | *speed = good_speed; | ||
| 96 | ci->speed_unverified = 0; | ||
| 97 | return 0; | ||
| 98 | } | ||
| 99 | *speed = old_speed; | ||
| 100 | return ret; | ||
| 101 | } | ||
| 60 | 102 | ||
| 61 | static int nodemgr_bus_read(struct csr1212_csr *csr, u64 addr, u16 length, | 103 | static int nodemgr_bus_read(struct csr1212_csr *csr, u64 addr, u16 length, |
| 62 | void *buffer, void *__ci) | 104 | void *buffer, void *__ci) |
| 63 | { | 105 | { |
| 64 | struct nodemgr_csr_info *ci = (struct nodemgr_csr_info*)__ci; | 106 | struct nodemgr_csr_info *ci = (struct nodemgr_csr_info*)__ci; |
| 65 | int i, ret = 0; | 107 | int i, ret; |
| 66 | 108 | ||
| 67 | for (i = 1; ; i++) { | 109 | for (i = 1; ; i++) { |
| 68 | ret = hpsb_read(ci->host, ci->nodeid, ci->generation, addr, | 110 | ret = hpsb_read(ci->host, ci->nodeid, ci->generation, addr, |
| 69 | buffer, length); | 111 | buffer, length); |
| 70 | if (!ret || i == 3) | 112 | if (!ret) { |
| 113 | ci->speed_unverified = 0; | ||
| 114 | break; | ||
| 115 | } | ||
| 116 | /* Give up after 3rd failure. */ | ||
| 117 | if (i == 3) | ||
| 71 | break; | 118 | break; |
| 72 | 119 | ||
| 120 | /* The ieee1394_core guessed the node's speed capability from | ||
| 121 | * the self ID. Check whether a lower speed works. */ | ||
| 122 | if (ci->speed_unverified && length == sizeof(quadlet_t)) { | ||
| 123 | ret = nodemgr_check_speed(ci, addr, buffer); | ||
| 124 | if (!ret) | ||
| 125 | break; | ||
| 126 | } | ||
| 73 | if (msleep_interruptible(334)) | 127 | if (msleep_interruptible(334)) |
| 74 | return -EINTR; | 128 | return -EINTR; |
| 75 | } | 129 | } |
| 76 | |||
| 77 | return ret; | 130 | return ret; |
| 78 | } | 131 | } |
| 79 | 132 | ||
| @@ -1204,6 +1257,8 @@ static void nodemgr_node_scan_one(struct host_info *hi, | |||
| 1204 | ci->host = host; | 1257 | ci->host = host; |
| 1205 | ci->nodeid = nodeid; | 1258 | ci->nodeid = nodeid; |
| 1206 | ci->generation = generation; | 1259 | ci->generation = generation; |
| 1260 | ci->speed_unverified = | ||
| 1261 | host->speed[NODEID_TO_NODE(nodeid)] > IEEE1394_SPEED_100; | ||
| 1207 | 1262 | ||
| 1208 | /* We need to detect when the ConfigROM's generation has changed, | 1263 | /* We need to detect when the ConfigROM's generation has changed, |
| 1209 | * so we only update the node's info when it needs to be. */ | 1264 | * so we only update the node's info when it needs to be. */ |
diff --git a/drivers/ieee1394/ohci1394.c b/drivers/ieee1394/ohci1394.c index c031650a1c74..3d278412e1ca 100644 --- a/drivers/ieee1394/ohci1394.c +++ b/drivers/ieee1394/ohci1394.c | |||
| @@ -163,7 +163,7 @@ printk(level "%s: fw-host%d: " fmt "\n" , OHCI1394_DRIVER_NAME, ohci->host->id , | |||
| 163 | 163 | ||
| 164 | /* Module Parameters */ | 164 | /* Module Parameters */ |
| 165 | static int phys_dma = 1; | 165 | static int phys_dma = 1; |
| 166 | module_param(phys_dma, int, 0644); | 166 | module_param(phys_dma, int, 0444); |
| 167 | MODULE_PARM_DESC(phys_dma, "Enable physical dma (default = 1)."); | 167 | MODULE_PARM_DESC(phys_dma, "Enable physical dma (default = 1)."); |
| 168 | 168 | ||
| 169 | static void dma_trm_tasklet(unsigned long data); | 169 | static void dma_trm_tasklet(unsigned long data); |
| @@ -553,7 +553,8 @@ static void ohci_initialize(struct ti_ohci *ohci) | |||
| 553 | * register content. | 553 | * register content. |
| 554 | * To actually enable physical responses is the job of our interrupt | 554 | * To actually enable physical responses is the job of our interrupt |
| 555 | * handler which programs the physical request filter. */ | 555 | * handler which programs the physical request filter. */ |
| 556 | reg_write(ohci, OHCI1394_PhyUpperBound, 0x01000000); | 556 | reg_write(ohci, OHCI1394_PhyUpperBound, |
| 557 | OHCI1394_PHYS_UPPER_BOUND_PROGRAMMED >> 16); | ||
| 557 | 558 | ||
| 558 | DBGMSG("physUpperBoundOffset=%08x", | 559 | DBGMSG("physUpperBoundOffset=%08x", |
| 559 | reg_read(ohci, OHCI1394_PhyUpperBound)); | 560 | reg_read(ohci, OHCI1394_PhyUpperBound)); |
| @@ -580,6 +581,7 @@ static void ohci_initialize(struct ti_ohci *ohci) | |||
| 580 | OHCI1394_isochRx | | 581 | OHCI1394_isochRx | |
| 581 | OHCI1394_isochTx | | 582 | OHCI1394_isochTx | |
| 582 | OHCI1394_postedWriteErr | | 583 | OHCI1394_postedWriteErr | |
| 584 | OHCI1394_cycleTooLong | | ||
| 583 | OHCI1394_cycleInconsistent); | 585 | OHCI1394_cycleInconsistent); |
| 584 | 586 | ||
| 585 | /* Enable link */ | 587 | /* Enable link */ |
| @@ -2382,6 +2384,15 @@ static irqreturn_t ohci_irq_handler(int irq, void *dev_id, | |||
| 2382 | PRINT(KERN_ERR, "physical posted write error"); | 2384 | PRINT(KERN_ERR, "physical posted write error"); |
| 2383 | /* no recovery strategy yet, had to involve protocol drivers */ | 2385 | /* no recovery strategy yet, had to involve protocol drivers */ |
| 2384 | } | 2386 | } |
| 2387 | if (event & OHCI1394_cycleTooLong) { | ||
| 2388 | if(printk_ratelimit()) | ||
| 2389 | PRINT(KERN_WARNING, "isochronous cycle too long"); | ||
| 2390 | else | ||
| 2391 | DBGMSG("OHCI1394_cycleTooLong"); | ||
| 2392 | reg_write(ohci, OHCI1394_LinkControlSet, | ||
| 2393 | OHCI1394_LinkControl_CycleMaster); | ||
| 2394 | event &= ~OHCI1394_cycleTooLong; | ||
| 2395 | } | ||
| 2385 | if (event & OHCI1394_cycleInconsistent) { | 2396 | if (event & OHCI1394_cycleInconsistent) { |
| 2386 | /* We subscribe to the cycleInconsistent event only to | 2397 | /* We subscribe to the cycleInconsistent event only to |
| 2387 | * clear the corresponding event bit... otherwise, | 2398 | * clear the corresponding event bit... otherwise, |
| @@ -3400,6 +3411,14 @@ static int __devinit ohci1394_pci_probe(struct pci_dev *dev, | |||
| 3400 | host->csr.max_rec = (reg_read(ohci, OHCI1394_BusOptions) >> 12) & 0xf; | 3411 | host->csr.max_rec = (reg_read(ohci, OHCI1394_BusOptions) >> 12) & 0xf; |
| 3401 | host->csr.lnk_spd = reg_read(ohci, OHCI1394_BusOptions) & 0x7; | 3412 | host->csr.lnk_spd = reg_read(ohci, OHCI1394_BusOptions) & 0x7; |
| 3402 | 3413 | ||
| 3414 | if (phys_dma) { | ||
| 3415 | host->low_addr_space = | ||
| 3416 | (u64) reg_read(ohci, OHCI1394_PhyUpperBound) << 16; | ||
| 3417 | if (!host->low_addr_space) | ||
| 3418 | host->low_addr_space = OHCI1394_PHYS_UPPER_BOUND_FIXED; | ||
| 3419 | } | ||
| 3420 | host->middle_addr_space = OHCI1394_MIDDLE_ADDRESS_SPACE; | ||
| 3421 | |||
| 3403 | /* Tell the highlevel this host is ready */ | 3422 | /* Tell the highlevel this host is ready */ |
| 3404 | if (hpsb_add_host(host)) | 3423 | if (hpsb_add_host(host)) |
| 3405 | FAIL(-ENOMEM, "Failed to register host with highlevel"); | 3424 | FAIL(-ENOMEM, "Failed to register host with highlevel"); |
| @@ -3458,24 +3477,13 @@ static void ohci1394_pci_remove(struct pci_dev *pdev) | |||
| 3458 | case OHCI_INIT_HAVE_TXRX_BUFFERS__MAYBE: | 3477 | case OHCI_INIT_HAVE_TXRX_BUFFERS__MAYBE: |
| 3459 | /* The ohci_soft_reset() stops all DMA contexts, so we | 3478 | /* The ohci_soft_reset() stops all DMA contexts, so we |
| 3460 | * dont need to do this. */ | 3479 | * dont need to do this. */ |
| 3461 | /* Free AR dma */ | ||
| 3462 | free_dma_rcv_ctx(&ohci->ar_req_context); | 3480 | free_dma_rcv_ctx(&ohci->ar_req_context); |
| 3463 | free_dma_rcv_ctx(&ohci->ar_resp_context); | 3481 | free_dma_rcv_ctx(&ohci->ar_resp_context); |
| 3464 | |||
| 3465 | /* Free AT dma */ | ||
| 3466 | free_dma_trm_ctx(&ohci->at_req_context); | 3482 | free_dma_trm_ctx(&ohci->at_req_context); |
| 3467 | free_dma_trm_ctx(&ohci->at_resp_context); | 3483 | free_dma_trm_ctx(&ohci->at_resp_context); |
| 3468 | |||
| 3469 | /* Free IR dma */ | ||
| 3470 | free_dma_rcv_ctx(&ohci->ir_legacy_context); | 3484 | free_dma_rcv_ctx(&ohci->ir_legacy_context); |
| 3471 | |||
| 3472 | /* Free IT dma */ | ||
| 3473 | free_dma_trm_ctx(&ohci->it_legacy_context); | 3485 | free_dma_trm_ctx(&ohci->it_legacy_context); |
| 3474 | 3486 | ||
| 3475 | /* Free IR legacy dma */ | ||
| 3476 | free_dma_rcv_ctx(&ohci->ir_legacy_context); | ||
| 3477 | |||
| 3478 | |||
| 3479 | case OHCI_INIT_HAVE_SELFID_BUFFER: | 3487 | case OHCI_INIT_HAVE_SELFID_BUFFER: |
| 3480 | pci_free_consistent(ohci->dev, OHCI1394_SI_DMA_BUF_SIZE, | 3488 | pci_free_consistent(ohci->dev, OHCI1394_SI_DMA_BUF_SIZE, |
| 3481 | ohci->selfid_buf_cpu, | 3489 | ohci->selfid_buf_cpu, |
| @@ -3535,6 +3543,7 @@ static int ohci1394_pci_resume (struct pci_dev *pdev) | |||
| 3535 | } | 3543 | } |
| 3536 | #endif /* CONFIG_PPC_PMAC */ | 3544 | #endif /* CONFIG_PPC_PMAC */ |
| 3537 | 3545 | ||
| 3546 | pci_restore_state(pdev); | ||
| 3538 | pci_enable_device(pdev); | 3547 | pci_enable_device(pdev); |
| 3539 | 3548 | ||
| 3540 | return 0; | 3549 | return 0; |
| @@ -3554,6 +3563,8 @@ static int ohci1394_pci_suspend (struct pci_dev *pdev, pm_message_t state) | |||
| 3554 | } | 3563 | } |
| 3555 | #endif | 3564 | #endif |
| 3556 | 3565 | ||
| 3566 | pci_save_state(pdev); | ||
| 3567 | |||
| 3557 | return 0; | 3568 | return 0; |
| 3558 | } | 3569 | } |
| 3559 | 3570 | ||
diff --git a/drivers/ieee1394/ohci1394.h b/drivers/ieee1394/ohci1394.h index 7df0962144e3..fa05f113f7f0 100644 --- a/drivers/ieee1394/ohci1394.h +++ b/drivers/ieee1394/ohci1394.h | |||
| @@ -443,6 +443,16 @@ static inline u32 reg_read(const struct ti_ohci *ohci, int offset) | |||
| 443 | 443 | ||
| 444 | #define OHCI1394_TCODE_PHY 0xE | 444 | #define OHCI1394_TCODE_PHY 0xE |
| 445 | 445 | ||
| 446 | /* Node offset map (phys DMA area, posted write area). | ||
| 447 | * The value of OHCI1394_PHYS_UPPER_BOUND_PROGRAMMED may be modified but must | ||
| 448 | * be lower than OHCI1394_MIDDLE_ADDRESS_SPACE. | ||
| 449 | * OHCI1394_PHYS_UPPER_BOUND_FIXED and OHCI1394_MIDDLE_ADDRESS_SPACE are | ||
| 450 | * constants given by the OHCI spec. | ||
| 451 | */ | ||
| 452 | #define OHCI1394_PHYS_UPPER_BOUND_FIXED 0x000100000000ULL /* 4 GB */ | ||
| 453 | #define OHCI1394_PHYS_UPPER_BOUND_PROGRAMMED 0x010000000000ULL /* 1 TB */ | ||
| 454 | #define OHCI1394_MIDDLE_ADDRESS_SPACE 0xffff00000000ULL | ||
| 455 | |||
| 446 | void ohci1394_init_iso_tasklet(struct ohci1394_iso_tasklet *tasklet, | 456 | void ohci1394_init_iso_tasklet(struct ohci1394_iso_tasklet *tasklet, |
| 447 | int type, | 457 | int type, |
| 448 | void (*func)(unsigned long), | 458 | void (*func)(unsigned long), |
diff --git a/drivers/ieee1394/raw1394.c b/drivers/ieee1394/raw1394.c index f7de546f2ed6..20ce539580f1 100644 --- a/drivers/ieee1394/raw1394.c +++ b/drivers/ieee1394/raw1394.c | |||
| @@ -408,34 +408,34 @@ static void fcp_request(struct hpsb_host *host, int nodeid, int direction, | |||
| 408 | 408 | ||
| 409 | #ifdef CONFIG_COMPAT | 409 | #ifdef CONFIG_COMPAT |
| 410 | struct compat_raw1394_req { | 410 | struct compat_raw1394_req { |
| 411 | __u32 type; | 411 | __u32 type; |
| 412 | __s32 error; | 412 | __s32 error; |
| 413 | __u32 misc; | 413 | __u32 misc; |
| 414 | 414 | ||
| 415 | __u32 generation; | 415 | __u32 generation; |
| 416 | __u32 length; | 416 | __u32 length; |
| 417 | 417 | ||
| 418 | __u64 address; | 418 | __u64 address; |
| 419 | 419 | ||
| 420 | __u64 tag; | 420 | __u64 tag; |
| 421 | 421 | ||
| 422 | __u64 sendb; | 422 | __u64 sendb; |
| 423 | __u64 recvb; | 423 | __u64 recvb; |
| 424 | } __attribute__((packed)); | 424 | } __attribute__((packed)); |
| 425 | 425 | ||
| 426 | static const char __user *raw1394_compat_write(const char __user *buf) | 426 | static const char __user *raw1394_compat_write(const char __user *buf) |
| 427 | { | 427 | { |
| 428 | struct compat_raw1394_req __user *cr = (typeof(cr)) buf; | 428 | struct compat_raw1394_req __user *cr = (typeof(cr)) buf; |
| 429 | struct raw1394_request __user *r; | 429 | struct raw1394_request __user *r; |
| 430 | r = compat_alloc_user_space(sizeof(struct raw1394_request)); | 430 | r = compat_alloc_user_space(sizeof(struct raw1394_request)); |
| 431 | 431 | ||
| 432 | #define C(x) __copy_in_user(&r->x, &cr->x, sizeof(r->x)) | 432 | #define C(x) __copy_in_user(&r->x, &cr->x, sizeof(r->x)) |
| 433 | 433 | ||
| 434 | if (copy_in_user(r, cr, sizeof(struct compat_raw1394_req)) || | 434 | if (copy_in_user(r, cr, sizeof(struct compat_raw1394_req)) || |
| 435 | C(address) || | 435 | C(address) || |
| 436 | C(tag) || | 436 | C(tag) || |
| 437 | C(sendb) || | 437 | C(sendb) || |
| 438 | C(recvb)) | 438 | C(recvb)) |
| 439 | return ERR_PTR(-EFAULT); | 439 | return ERR_PTR(-EFAULT); |
| 440 | return (const char __user *)r; | 440 | return (const char __user *)r; |
| 441 | } | 441 | } |
| @@ -443,11 +443,11 @@ static const char __user *raw1394_compat_write(const char __user *buf) | |||
| 443 | 443 | ||
| 444 | #define P(x) __put_user(r->x, &cr->x) | 444 | #define P(x) __put_user(r->x, &cr->x) |
| 445 | 445 | ||
| 446 | static int | 446 | static int |
| 447 | raw1394_compat_read(const char __user *buf, struct raw1394_request *r) | 447 | raw1394_compat_read(const char __user *buf, struct raw1394_request *r) |
| 448 | { | 448 | { |
| 449 | struct compat_raw1394_req __user *cr = (typeof(cr)) r; | 449 | struct compat_raw1394_req __user *cr = (typeof(cr)) r; |
| 450 | if (!access_ok(VERIFY_WRITE,cr,sizeof(struct compat_raw1394_req)) || | 450 | if (!access_ok(VERIFY_WRITE, cr, sizeof(struct compat_raw1394_req)) || |
| 451 | P(type) || | 451 | P(type) || |
| 452 | P(error) || | 452 | P(error) || |
| 453 | P(misc) || | 453 | P(misc) || |
| @@ -512,18 +512,17 @@ static ssize_t raw1394_read(struct file *file, char __user * buffer, | |||
| 512 | } | 512 | } |
| 513 | 513 | ||
| 514 | #ifdef CONFIG_COMPAT | 514 | #ifdef CONFIG_COMPAT |
| 515 | if (count == sizeof(struct compat_raw1394_req) && | 515 | if (count == sizeof(struct compat_raw1394_req) && |
| 516 | sizeof(struct compat_raw1394_req) != | 516 | sizeof(struct compat_raw1394_req) != |
| 517 | sizeof(struct raw1394_request)) { | 517 | sizeof(struct raw1394_request)) { |
| 518 | ret = raw1394_compat_read(buffer, &req->req); | 518 | ret = raw1394_compat_read(buffer, &req->req); |
| 519 | 519 | } else | |
| 520 | } else | ||
| 521 | #endif | 520 | #endif |
| 522 | { | 521 | { |
| 523 | if (copy_to_user(buffer, &req->req, sizeof(req->req))) { | 522 | if (copy_to_user(buffer, &req->req, sizeof(req->req))) { |
| 524 | ret = -EFAULT; | 523 | ret = -EFAULT; |
| 525 | goto out; | 524 | goto out; |
| 526 | } | 525 | } |
| 527 | ret = (ssize_t) sizeof(struct raw1394_request); | 526 | ret = (ssize_t) sizeof(struct raw1394_request); |
| 528 | } | 527 | } |
| 529 | out: | 528 | out: |
| @@ -2348,7 +2347,6 @@ static int state_connected(struct file_info *fi, struct pending_request *req) | |||
| 2348 | return handle_async_request(fi, req, node); | 2347 | return handle_async_request(fi, req, node); |
| 2349 | } | 2348 | } |
| 2350 | 2349 | ||
| 2351 | |||
| 2352 | static ssize_t raw1394_write(struct file *file, const char __user * buffer, | 2350 | static ssize_t raw1394_write(struct file *file, const char __user * buffer, |
| 2353 | size_t count, loff_t * offset_is_ignored) | 2351 | size_t count, loff_t * offset_is_ignored) |
| 2354 | { | 2352 | { |
| @@ -2357,9 +2355,9 @@ static ssize_t raw1394_write(struct file *file, const char __user * buffer, | |||
| 2357 | ssize_t retval = 0; | 2355 | ssize_t retval = 0; |
| 2358 | 2356 | ||
| 2359 | #ifdef CONFIG_COMPAT | 2357 | #ifdef CONFIG_COMPAT |
| 2360 | if (count == sizeof(struct compat_raw1394_req) && | 2358 | if (count == sizeof(struct compat_raw1394_req) && |
| 2361 | sizeof(struct compat_raw1394_req) != | 2359 | sizeof(struct compat_raw1394_req) != |
| 2362 | sizeof(struct raw1394_request)) { | 2360 | sizeof(struct raw1394_request)) { |
| 2363 | buffer = raw1394_compat_write(buffer); | 2361 | buffer = raw1394_compat_write(buffer); |
| 2364 | if (IS_ERR(buffer)) | 2362 | if (IS_ERR(buffer)) |
| 2365 | return PTR_ERR(buffer); | 2363 | return PTR_ERR(buffer); |
diff --git a/drivers/ieee1394/sbp2.c b/drivers/ieee1394/sbp2.c index 5413dc43b9f1..1d5ceb7ecc83 100644 --- a/drivers/ieee1394/sbp2.c +++ b/drivers/ieee1394/sbp2.c | |||
| @@ -127,10 +127,12 @@ MODULE_PARM_DESC(max_sectors, "Change max sectors per I/O supported (default = " | |||
| 127 | * talking to a single sbp2 device at the same time (filesystem coherency, | 127 | * talking to a single sbp2 device at the same time (filesystem coherency, |
| 128 | * etc.). If you're running an sbp2 device that supports multiple logins, | 128 | * etc.). If you're running an sbp2 device that supports multiple logins, |
| 129 | * and you're either running read-only filesystems or some sort of special | 129 | * and you're either running read-only filesystems or some sort of special |
| 130 | * filesystem supporting multiple hosts (one such filesystem is OpenGFS, | 130 | * filesystem supporting multiple hosts, e.g. OpenGFS, Oracle Cluster |
| 131 | * see opengfs.sourceforge.net for more info), then set exclusive_login | 131 | * File System, or Lustre, then set exclusive_login to zero. |
| 132 | * to zero. Note: The Oxsemi OXFW911 sbp2 chipset supports up to four | 132 | * |
| 133 | * concurrent logins. | 133 | * So far only bridges from Oxford Semiconductor are known to support |
| 134 | * concurrent logins. Depending on firmware, four or two concurrent logins | ||
| 135 | * are possible on OXFW911 and newer Oxsemi bridges. | ||
| 134 | */ | 136 | */ |
| 135 | static int exclusive_login = 1; | 137 | static int exclusive_login = 1; |
| 136 | module_param(exclusive_login, int, 0644); | 138 | module_param(exclusive_login, int, 0644); |
| @@ -306,8 +308,9 @@ static const struct { | |||
| 306 | u32 model_id; | 308 | u32 model_id; |
| 307 | unsigned workarounds; | 309 | unsigned workarounds; |
| 308 | } sbp2_workarounds_table[] = { | 310 | } sbp2_workarounds_table[] = { |
| 309 | /* TSB42AA9 */ { | 311 | /* DViCO Momobay CX-1 with TSB42AA9 bridge */ { |
| 310 | .firmware_revision = 0x002800, | 312 | .firmware_revision = 0x002800, |
| 313 | .model_id = 0x001010, | ||
| 311 | .workarounds = SBP2_WORKAROUND_INQUIRY_36 | | 314 | .workarounds = SBP2_WORKAROUND_INQUIRY_36 | |
| 312 | SBP2_WORKAROUND_MODE_SENSE_8, | 315 | SBP2_WORKAROUND_MODE_SENSE_8, |
| 313 | }, | 316 | }, |
| @@ -791,12 +794,12 @@ static struct scsi_id_instance_data *sbp2_alloc_device(struct unit_directory *ud | |||
| 791 | scsi_id->ud = ud; | 794 | scsi_id->ud = ud; |
| 792 | scsi_id->speed_code = IEEE1394_SPEED_100; | 795 | scsi_id->speed_code = IEEE1394_SPEED_100; |
| 793 | scsi_id->max_payload_size = sbp2_speedto_max_payload[IEEE1394_SPEED_100]; | 796 | scsi_id->max_payload_size = sbp2_speedto_max_payload[IEEE1394_SPEED_100]; |
| 797 | scsi_id->status_fifo_addr = CSR1212_INVALID_ADDR_SPACE; | ||
| 794 | atomic_set(&scsi_id->sbp2_login_complete, 0); | 798 | atomic_set(&scsi_id->sbp2_login_complete, 0); |
| 795 | INIT_LIST_HEAD(&scsi_id->sbp2_command_orb_inuse); | 799 | INIT_LIST_HEAD(&scsi_id->sbp2_command_orb_inuse); |
| 796 | INIT_LIST_HEAD(&scsi_id->sbp2_command_orb_completed); | 800 | INIT_LIST_HEAD(&scsi_id->sbp2_command_orb_completed); |
| 797 | INIT_LIST_HEAD(&scsi_id->scsi_list); | 801 | INIT_LIST_HEAD(&scsi_id->scsi_list); |
| 798 | spin_lock_init(&scsi_id->sbp2_command_orb_lock); | 802 | spin_lock_init(&scsi_id->sbp2_command_orb_lock); |
| 799 | scsi_id->sbp2_lun = 0; | ||
| 800 | 803 | ||
| 801 | ud->device.driver_data = scsi_id; | 804 | ud->device.driver_data = scsi_id; |
| 802 | 805 | ||
| @@ -844,8 +847,8 @@ static struct scsi_id_instance_data *sbp2_alloc_device(struct unit_directory *ud | |||
| 844 | scsi_id->status_fifo_addr = hpsb_allocate_and_register_addrspace( | 847 | scsi_id->status_fifo_addr = hpsb_allocate_and_register_addrspace( |
| 845 | &sbp2_highlevel, ud->ne->host, &sbp2_ops, | 848 | &sbp2_highlevel, ud->ne->host, &sbp2_ops, |
| 846 | sizeof(struct sbp2_status_block), sizeof(quadlet_t), | 849 | sizeof(struct sbp2_status_block), sizeof(quadlet_t), |
| 847 | 0x010000000000ULL, CSR1212_ALL_SPACE_END); | 850 | ud->ne->host->low_addr_space, CSR1212_ALL_SPACE_END); |
| 848 | if (scsi_id->status_fifo_addr == ~0ULL) { | 851 | if (scsi_id->status_fifo_addr == CSR1212_INVALID_ADDR_SPACE) { |
| 849 | SBP2_ERR("failed to allocate status FIFO address range"); | 852 | SBP2_ERR("failed to allocate status FIFO address range"); |
| 850 | goto failed_alloc; | 853 | goto failed_alloc; |
| 851 | } | 854 | } |
| @@ -1087,9 +1090,9 @@ static void sbp2_remove_device(struct scsi_id_instance_data *scsi_id) | |||
| 1087 | SBP2_DMA_FREE("single query logins data"); | 1090 | SBP2_DMA_FREE("single query logins data"); |
| 1088 | } | 1091 | } |
| 1089 | 1092 | ||
| 1090 | if (scsi_id->status_fifo_addr) | 1093 | if (scsi_id->status_fifo_addr != CSR1212_INVALID_ADDR_SPACE) |
| 1091 | hpsb_unregister_addrspace(&sbp2_highlevel, hi->host, | 1094 | hpsb_unregister_addrspace(&sbp2_highlevel, hi->host, |
| 1092 | scsi_id->status_fifo_addr); | 1095 | scsi_id->status_fifo_addr); |
| 1093 | 1096 | ||
| 1094 | scsi_id->ud->device.driver_data = NULL; | 1097 | scsi_id->ud->device.driver_data = NULL; |
| 1095 | 1098 | ||
| @@ -1213,13 +1216,11 @@ static int sbp2_query_logins(struct scsi_id_instance_data *scsi_id) | |||
| 1213 | SBP2_DEBUG("length_max_logins = %x", | 1216 | SBP2_DEBUG("length_max_logins = %x", |
| 1214 | (unsigned int)scsi_id->query_logins_response->length_max_logins); | 1217 | (unsigned int)scsi_id->query_logins_response->length_max_logins); |
| 1215 | 1218 | ||
| 1216 | SBP2_DEBUG("Query logins to SBP-2 device successful"); | ||
| 1217 | |||
| 1218 | max_logins = RESPONSE_GET_MAX_LOGINS(scsi_id->query_logins_response->length_max_logins); | 1219 | max_logins = RESPONSE_GET_MAX_LOGINS(scsi_id->query_logins_response->length_max_logins); |
| 1219 | SBP2_DEBUG("Maximum concurrent logins supported: %d", max_logins); | 1220 | SBP2_INFO("Maximum concurrent logins supported: %d", max_logins); |
| 1220 | 1221 | ||
| 1221 | active_logins = RESPONSE_GET_ACTIVE_LOGINS(scsi_id->query_logins_response->length_max_logins); | 1222 | active_logins = RESPONSE_GET_ACTIVE_LOGINS(scsi_id->query_logins_response->length_max_logins); |
| 1222 | SBP2_DEBUG("Number of active logins: %d", active_logins); | 1223 | SBP2_INFO("Number of active logins: %d", active_logins); |
| 1223 | 1224 | ||
| 1224 | if (active_logins >= max_logins) { | 1225 | if (active_logins >= max_logins) { |
| 1225 | return -EIO; | 1226 | return -EIO; |
| @@ -1648,6 +1649,8 @@ static void sbp2_parse_unit_directory(struct scsi_id_instance_data *scsi_id, | |||
| 1648 | } | 1649 | } |
| 1649 | } | 1650 | } |
| 1650 | 1651 | ||
| 1652 | #define SBP2_PAYLOAD_TO_BYTES(p) (1 << ((p) + 2)) | ||
| 1653 | |||
| 1651 | /* | 1654 | /* |
| 1652 | * This function is called in order to determine the max speed and packet | 1655 | * This function is called in order to determine the max speed and packet |
| 1653 | * size we can use in our ORBs. Note, that we (the driver and host) only | 1656 | * size we can use in our ORBs. Note, that we (the driver and host) only |
| @@ -1660,13 +1663,12 @@ static void sbp2_parse_unit_directory(struct scsi_id_instance_data *scsi_id, | |||
| 1660 | static int sbp2_max_speed_and_size(struct scsi_id_instance_data *scsi_id) | 1663 | static int sbp2_max_speed_and_size(struct scsi_id_instance_data *scsi_id) |
| 1661 | { | 1664 | { |
| 1662 | struct sbp2scsi_host_info *hi = scsi_id->hi; | 1665 | struct sbp2scsi_host_info *hi = scsi_id->hi; |
| 1666 | u8 payload; | ||
| 1663 | 1667 | ||
| 1664 | SBP2_DEBUG_ENTER(); | 1668 | SBP2_DEBUG_ENTER(); |
| 1665 | 1669 | ||
| 1666 | /* Initial setting comes from the hosts speed map */ | ||
| 1667 | scsi_id->speed_code = | 1670 | scsi_id->speed_code = |
| 1668 | hi->host->speed_map[NODEID_TO_NODE(hi->host->node_id) * 64 + | 1671 | hi->host->speed[NODEID_TO_NODE(scsi_id->ne->nodeid)]; |
| 1669 | NODEID_TO_NODE(scsi_id->ne->nodeid)]; | ||
| 1670 | 1672 | ||
| 1671 | /* Bump down our speed if the user requested it */ | 1673 | /* Bump down our speed if the user requested it */ |
| 1672 | if (scsi_id->speed_code > max_speed) { | 1674 | if (scsi_id->speed_code > max_speed) { |
| @@ -1677,15 +1679,22 @@ static int sbp2_max_speed_and_size(struct scsi_id_instance_data *scsi_id) | |||
| 1677 | 1679 | ||
| 1678 | /* Payload size is the lesser of what our speed supports and what | 1680 | /* Payload size is the lesser of what our speed supports and what |
| 1679 | * our host supports. */ | 1681 | * our host supports. */ |
| 1680 | scsi_id->max_payload_size = | 1682 | payload = min(sbp2_speedto_max_payload[scsi_id->speed_code], |
| 1681 | min(sbp2_speedto_max_payload[scsi_id->speed_code], | 1683 | (u8) (hi->host->csr.max_rec - 1)); |
| 1682 | (u8) (hi->host->csr.max_rec - 1)); | 1684 | |
| 1685 | /* If physical DMA is off, work around limitation in ohci1394: | ||
| 1686 | * packet size must not exceed PAGE_SIZE */ | ||
| 1687 | if (scsi_id->ne->host->low_addr_space < (1ULL << 32)) | ||
| 1688 | while (SBP2_PAYLOAD_TO_BYTES(payload) + 24 > PAGE_SIZE && | ||
| 1689 | payload) | ||
| 1690 | payload--; | ||
| 1683 | 1691 | ||
| 1684 | HPSB_DEBUG("Node " NODE_BUS_FMT ": Max speed [%s] - Max payload [%u]", | 1692 | HPSB_DEBUG("Node " NODE_BUS_FMT ": Max speed [%s] - Max payload [%u]", |
| 1685 | NODE_BUS_ARGS(hi->host, scsi_id->ne->nodeid), | 1693 | NODE_BUS_ARGS(hi->host, scsi_id->ne->nodeid), |
| 1686 | hpsb_speedto_str[scsi_id->speed_code], | 1694 | hpsb_speedto_str[scsi_id->speed_code], |
| 1687 | 1 << ((u32) scsi_id->max_payload_size + 2)); | 1695 | SBP2_PAYLOAD_TO_BYTES(payload)); |
| 1688 | 1696 | ||
| 1697 | scsi_id->max_payload_size = payload; | ||
| 1689 | return 0; | 1698 | return 0; |
| 1690 | } | 1699 | } |
| 1691 | 1700 | ||
| @@ -2113,33 +2122,6 @@ static unsigned int sbp2_status_to_sense_data(unchar *sbp2_status, unchar *sense | |||
| 2113 | } | 2122 | } |
| 2114 | 2123 | ||
| 2115 | /* | 2124 | /* |
| 2116 | * This function is called after a command is completed, in order to do any necessary SBP-2 | ||
| 2117 | * response data translations for the SCSI stack | ||
| 2118 | */ | ||
| 2119 | static void sbp2_check_sbp2_response(struct scsi_id_instance_data *scsi_id, | ||
| 2120 | struct scsi_cmnd *SCpnt) | ||
| 2121 | { | ||
| 2122 | u8 *scsi_buf = SCpnt->request_buffer; | ||
| 2123 | |||
| 2124 | SBP2_DEBUG_ENTER(); | ||
| 2125 | |||
| 2126 | if (SCpnt->cmnd[0] == INQUIRY && (SCpnt->cmnd[1] & 3) == 0) { | ||
| 2127 | /* | ||
| 2128 | * Make sure data length is ok. Minimum length is 36 bytes | ||
| 2129 | */ | ||
| 2130 | if (scsi_buf[4] == 0) { | ||
| 2131 | scsi_buf[4] = 36 - 5; | ||
| 2132 | } | ||
| 2133 | |||
| 2134 | /* | ||
| 2135 | * Fix ansi revision and response data format | ||
| 2136 | */ | ||
| 2137 | scsi_buf[2] |= 2; | ||
| 2138 | scsi_buf[3] = (scsi_buf[3] & 0xf0) | 2; | ||
| 2139 | } | ||
| 2140 | } | ||
| 2141 | |||
| 2142 | /* | ||
| 2143 | * This function deals with status writes from the SBP-2 device | 2125 | * This function deals with status writes from the SBP-2 device |
| 2144 | */ | 2126 | */ |
| 2145 | static int sbp2_handle_status_write(struct hpsb_host *host, int nodeid, int destid, | 2127 | static int sbp2_handle_status_write(struct hpsb_host *host, int nodeid, int destid, |
| @@ -2478,13 +2460,6 @@ static void sbp2scsi_complete_command(struct scsi_id_instance_data *scsi_id, | |||
| 2478 | } | 2460 | } |
| 2479 | 2461 | ||
| 2480 | /* | 2462 | /* |
| 2481 | * Take care of any sbp2 response data mucking here (RBC stuff, etc.) | ||
| 2482 | */ | ||
| 2483 | if (SCpnt->result == DID_OK << 16) { | ||
| 2484 | sbp2_check_sbp2_response(scsi_id, SCpnt); | ||
| 2485 | } | ||
| 2486 | |||
| 2487 | /* | ||
| 2488 | * If a bus reset is in progress and there was an error, complete | 2463 | * If a bus reset is in progress and there was an error, complete |
| 2489 | * the command as busy so that it will get retried. | 2464 | * the command as busy so that it will get retried. |
| 2490 | */ | 2465 | */ |
diff --git a/drivers/ieee1394/sbp2.h b/drivers/ieee1394/sbp2.h index f4ccc9d0fba4..b22ce1aa8fe4 100644 --- a/drivers/ieee1394/sbp2.h +++ b/drivers/ieee1394/sbp2.h | |||
| @@ -52,7 +52,7 @@ struct sbp2_command_orb { | |||
| 52 | u32 data_descriptor_lo; | 52 | u32 data_descriptor_lo; |
| 53 | u32 misc; | 53 | u32 misc; |
| 54 | u8 cdb[12]; | 54 | u8 cdb[12]; |
| 55 | }; | 55 | } __attribute__((packed)); |
| 56 | 56 | ||
| 57 | #define SBP2_LOGIN_REQUEST 0x0 | 57 | #define SBP2_LOGIN_REQUEST 0x0 |
| 58 | #define SBP2_QUERY_LOGINS_REQUEST 0x1 | 58 | #define SBP2_QUERY_LOGINS_REQUEST 0x1 |
| @@ -80,7 +80,7 @@ struct sbp2_login_orb { | |||
| 80 | u32 passwd_resp_lengths; | 80 | u32 passwd_resp_lengths; |
| 81 | u32 status_fifo_hi; | 81 | u32 status_fifo_hi; |
| 82 | u32 status_fifo_lo; | 82 | u32 status_fifo_lo; |
| 83 | }; | 83 | } __attribute__((packed)); |
| 84 | 84 | ||
| 85 | #define RESPONSE_GET_LOGIN_ID(value) (value & 0xffff) | 85 | #define RESPONSE_GET_LOGIN_ID(value) (value & 0xffff) |
| 86 | #define RESPONSE_GET_LENGTH(value) ((value >> 16) & 0xffff) | 86 | #define RESPONSE_GET_LENGTH(value) ((value >> 16) & 0xffff) |
| @@ -91,7 +91,7 @@ struct sbp2_login_response { | |||
| 91 | u32 command_block_agent_hi; | 91 | u32 command_block_agent_hi; |
| 92 | u32 command_block_agent_lo; | 92 | u32 command_block_agent_lo; |
| 93 | u32 reconnect_hold; | 93 | u32 reconnect_hold; |
| 94 | }; | 94 | } __attribute__((packed)); |
| 95 | 95 | ||
| 96 | #define ORB_SET_LOGIN_ID(value) (value & 0xffff) | 96 | #define ORB_SET_LOGIN_ID(value) (value & 0xffff) |
| 97 | 97 | ||
| @@ -106,7 +106,7 @@ struct sbp2_query_logins_orb { | |||
| 106 | u32 reserved_resp_length; | 106 | u32 reserved_resp_length; |
| 107 | u32 status_fifo_hi; | 107 | u32 status_fifo_hi; |
| 108 | u32 status_fifo_lo; | 108 | u32 status_fifo_lo; |
| 109 | }; | 109 | } __attribute__((packed)); |
| 110 | 110 | ||
| 111 | #define RESPONSE_GET_MAX_LOGINS(value) (value & 0xffff) | 111 | #define RESPONSE_GET_MAX_LOGINS(value) (value & 0xffff) |
| 112 | #define RESPONSE_GET_ACTIVE_LOGINS(value) ((RESPONSE_GET_LENGTH(value) - 4) / 12) | 112 | #define RESPONSE_GET_ACTIVE_LOGINS(value) ((RESPONSE_GET_LENGTH(value) - 4) / 12) |
| @@ -116,7 +116,7 @@ struct sbp2_query_logins_response { | |||
| 116 | u32 misc_IDs; | 116 | u32 misc_IDs; |
| 117 | u32 initiator_misc_hi; | 117 | u32 initiator_misc_hi; |
| 118 | u32 initiator_misc_lo; | 118 | u32 initiator_misc_lo; |
| 119 | }; | 119 | } __attribute__((packed)); |
| 120 | 120 | ||
| 121 | struct sbp2_reconnect_orb { | 121 | struct sbp2_reconnect_orb { |
| 122 | u32 reserved1; | 122 | u32 reserved1; |
| @@ -127,7 +127,7 @@ struct sbp2_reconnect_orb { | |||
| 127 | u32 reserved5; | 127 | u32 reserved5; |
| 128 | u32 status_fifo_hi; | 128 | u32 status_fifo_hi; |
| 129 | u32 status_fifo_lo; | 129 | u32 status_fifo_lo; |
| 130 | }; | 130 | } __attribute__((packed)); |
| 131 | 131 | ||
| 132 | struct sbp2_logout_orb { | 132 | struct sbp2_logout_orb { |
| 133 | u32 reserved1; | 133 | u32 reserved1; |
| @@ -138,7 +138,7 @@ struct sbp2_logout_orb { | |||
| 138 | u32 reserved5; | 138 | u32 reserved5; |
| 139 | u32 status_fifo_hi; | 139 | u32 status_fifo_hi; |
| 140 | u32 status_fifo_lo; | 140 | u32 status_fifo_lo; |
| 141 | }; | 141 | } __attribute__((packed)); |
| 142 | 142 | ||
| 143 | #define PAGE_TABLE_SET_SEGMENT_BASE_HI(value) (value & 0xffff) | 143 | #define PAGE_TABLE_SET_SEGMENT_BASE_HI(value) (value & 0xffff) |
| 144 | #define PAGE_TABLE_SET_SEGMENT_LENGTH(value) ((value & 0xffff) << 16) | 144 | #define PAGE_TABLE_SET_SEGMENT_LENGTH(value) ((value & 0xffff) << 16) |
| @@ -146,7 +146,7 @@ struct sbp2_logout_orb { | |||
| 146 | struct sbp2_unrestricted_page_table { | 146 | struct sbp2_unrestricted_page_table { |
| 147 | u32 length_segment_base_hi; | 147 | u32 length_segment_base_hi; |
| 148 | u32 segment_base_lo; | 148 | u32 segment_base_lo; |
| 149 | }; | 149 | } __attribute__((packed)); |
| 150 | 150 | ||
| 151 | #define RESP_STATUS_REQUEST_COMPLETE 0x0 | 151 | #define RESP_STATUS_REQUEST_COMPLETE 0x0 |
| 152 | #define RESP_STATUS_TRANSPORT_FAILURE 0x1 | 152 | #define RESP_STATUS_TRANSPORT_FAILURE 0x1 |
| @@ -191,7 +191,7 @@ struct sbp2_status_block { | |||
| 191 | u32 ORB_offset_hi_misc; | 191 | u32 ORB_offset_hi_misc; |
| 192 | u32 ORB_offset_lo; | 192 | u32 ORB_offset_lo; |
| 193 | u8 command_set_dependent[24]; | 193 | u8 command_set_dependent[24]; |
| 194 | }; | 194 | } __attribute__((packed)); |
| 195 | 195 | ||
| 196 | /* | 196 | /* |
| 197 | * Miscellaneous SBP2 related config rom defines | 197 | * Miscellaneous SBP2 related config rom defines |
| @@ -395,9 +395,8 @@ static int sbp2_link_orb_command(struct scsi_id_instance_data *scsi_id, | |||
| 395 | static int sbp2_send_command(struct scsi_id_instance_data *scsi_id, | 395 | static int sbp2_send_command(struct scsi_id_instance_data *scsi_id, |
| 396 | struct scsi_cmnd *SCpnt, | 396 | struct scsi_cmnd *SCpnt, |
| 397 | void (*done)(struct scsi_cmnd *)); | 397 | void (*done)(struct scsi_cmnd *)); |
| 398 | static unsigned int sbp2_status_to_sense_data(unchar *sbp2_status, unchar *sense_data); | 398 | static unsigned int sbp2_status_to_sense_data(unchar *sbp2_status, |
| 399 | static void sbp2_check_sbp2_response(struct scsi_id_instance_data *scsi_id, | 399 | unchar *sense_data); |
| 400 | struct scsi_cmnd *SCpnt); | ||
| 401 | static void sbp2_parse_unit_directory(struct scsi_id_instance_data *scsi_id, | 400 | static void sbp2_parse_unit_directory(struct scsi_id_instance_data *scsi_id, |
| 402 | struct unit_directory *ud); | 401 | struct unit_directory *ud); |
| 403 | static int sbp2_set_busy_timeout(struct scsi_id_instance_data *scsi_id); | 402 | static int sbp2_set_busy_timeout(struct scsi_id_instance_data *scsi_id); |
diff --git a/drivers/ieee1394/video1394.c b/drivers/ieee1394/video1394.c index 4e3bd62c458d..295d0f8c3d06 100644 --- a/drivers/ieee1394/video1394.c +++ b/drivers/ieee1394/video1394.c | |||
| @@ -331,7 +331,7 @@ alloc_dma_iso_ctx(struct ti_ohci *ohci, int type, int num_desc, | |||
| 331 | 331 | ||
| 332 | spin_lock_init(&d->lock); | 332 | spin_lock_init(&d->lock); |
| 333 | 333 | ||
| 334 | PRINT(KERN_INFO, ohci->host->id, "Iso %s DMA: %d buffers " | 334 | DBGMSG(ohci->host->id, "Iso %s DMA: %d buffers " |
| 335 | "of size %d allocated for a frame size %d, each with %d prgs", | 335 | "of size %d allocated for a frame size %d, each with %d prgs", |
| 336 | (type == OHCI_ISO_RECEIVE) ? "receive" : "transmit", | 336 | (type == OHCI_ISO_RECEIVE) ? "receive" : "transmit", |
| 337 | d->num_desc - 1, d->buf_size, d->frame_size, d->nb_cmd); | 337 | d->num_desc - 1, d->buf_size, d->frame_size, d->nb_cmd); |
| @@ -759,7 +759,7 @@ static int __video1394_ioctl(struct file *file, | |||
| 759 | } else { | 759 | } else { |
| 760 | mask = (u64)0x1<<v.channel; | 760 | mask = (u64)0x1<<v.channel; |
| 761 | } | 761 | } |
| 762 | PRINT(KERN_INFO, ohci->host->id, "mask: %08X%08X usage: %08X%08X\n", | 762 | DBGMSG(ohci->host->id, "mask: %08X%08X usage: %08X%08X\n", |
| 763 | (u32)(mask>>32),(u32)(mask&0xffffffff), | 763 | (u32)(mask>>32),(u32)(mask&0xffffffff), |
| 764 | (u32)(ohci->ISO_channel_usage>>32), | 764 | (u32)(ohci->ISO_channel_usage>>32), |
| 765 | (u32)(ohci->ISO_channel_usage&0xffffffff)); | 765 | (u32)(ohci->ISO_channel_usage&0xffffffff)); |
| @@ -805,7 +805,7 @@ static int __video1394_ioctl(struct file *file, | |||
| 805 | v.buf_size = d->buf_size; | 805 | v.buf_size = d->buf_size; |
| 806 | list_add_tail(&d->link, &ctx->context_list); | 806 | list_add_tail(&d->link, &ctx->context_list); |
| 807 | 807 | ||
| 808 | PRINT(KERN_INFO, ohci->host->id, | 808 | DBGMSG(ohci->host->id, |
| 809 | "iso context %d listen on channel %d", | 809 | "iso context %d listen on channel %d", |
| 810 | d->ctx, v.channel); | 810 | d->ctx, v.channel); |
| 811 | } | 811 | } |
| @@ -828,7 +828,7 @@ static int __video1394_ioctl(struct file *file, | |||
| 828 | 828 | ||
| 829 | list_add_tail(&d->link, &ctx->context_list); | 829 | list_add_tail(&d->link, &ctx->context_list); |
| 830 | 830 | ||
| 831 | PRINT(KERN_INFO, ohci->host->id, | 831 | DBGMSG(ohci->host->id, |
| 832 | "Iso context %d talk on channel %d", d->ctx, | 832 | "Iso context %d talk on channel %d", d->ctx, |
| 833 | v.channel); | 833 | v.channel); |
| 834 | } | 834 | } |
| @@ -873,7 +873,7 @@ static int __video1394_ioctl(struct file *file, | |||
| 873 | d = find_ctx(&ctx->context_list, OHCI_ISO_TRANSMIT, channel); | 873 | d = find_ctx(&ctx->context_list, OHCI_ISO_TRANSMIT, channel); |
| 874 | 874 | ||
| 875 | if (d == NULL) return -ESRCH; | 875 | if (d == NULL) return -ESRCH; |
| 876 | PRINT(KERN_INFO, ohci->host->id, "Iso context %d " | 876 | DBGMSG(ohci->host->id, "Iso context %d " |
| 877 | "stop talking on channel %d", d->ctx, channel); | 877 | "stop talking on channel %d", d->ctx, channel); |
| 878 | free_dma_iso_ctx(d); | 878 | free_dma_iso_ctx(d); |
| 879 | 879 | ||
| @@ -935,7 +935,7 @@ static int __video1394_ioctl(struct file *file, | |||
| 935 | else { | 935 | else { |
| 936 | /* Wake up dma context if necessary */ | 936 | /* Wake up dma context if necessary */ |
| 937 | if (!(reg_read(ohci, d->ctrlSet) & 0x400)) { | 937 | if (!(reg_read(ohci, d->ctrlSet) & 0x400)) { |
| 938 | PRINT(KERN_INFO, ohci->host->id, | 938 | DBGMSG(ohci->host->id, |
| 939 | "Waking up iso dma ctx=%d", d->ctx); | 939 | "Waking up iso dma ctx=%d", d->ctx); |
| 940 | reg_write(ohci, d->ctrlSet, 0x1000); | 940 | reg_write(ohci, d->ctrlSet, 0x1000); |
| 941 | } | 941 | } |
| @@ -1106,7 +1106,7 @@ static int __video1394_ioctl(struct file *file, | |||
| 1106 | else { | 1106 | else { |
| 1107 | /* Wake up dma context if necessary */ | 1107 | /* Wake up dma context if necessary */ |
| 1108 | if (!(reg_read(ohci, d->ctrlSet) & 0x400)) { | 1108 | if (!(reg_read(ohci, d->ctrlSet) & 0x400)) { |
| 1109 | PRINT(KERN_INFO, ohci->host->id, | 1109 | DBGMSG(ohci->host->id, |
| 1110 | "Waking up iso transmit dma ctx=%d", | 1110 | "Waking up iso transmit dma ctx=%d", |
| 1111 | d->ctx); | 1111 | d->ctx); |
| 1112 | put_timestamp(ohci, d, d->last_buffer); | 1112 | put_timestamp(ohci, d, d->last_buffer); |
| @@ -1232,7 +1232,7 @@ static int video1394_release(struct inode *inode, struct file *file) | |||
| 1232 | "is not being used", d->channel); | 1232 | "is not being used", d->channel); |
| 1233 | else | 1233 | else |
| 1234 | ohci->ISO_channel_usage &= ~mask; | 1234 | ohci->ISO_channel_usage &= ~mask; |
| 1235 | PRINT(KERN_INFO, ohci->host->id, "On release: Iso %s context " | 1235 | DBGMSG(ohci->host->id, "On release: Iso %s context " |
| 1236 | "%d stop listening on channel %d", | 1236 | "%d stop listening on channel %d", |
| 1237 | d->type == OHCI_ISO_RECEIVE ? "receive" : "transmit", | 1237 | d->type == OHCI_ISO_RECEIVE ? "receive" : "transmit", |
| 1238 | d->ctx, d->channel); | 1238 | d->ctx, d->channel); |
