Commit 9e2ef36b authored by Dave Olson's avatar Dave Olson Committed by Roland Dreier

IB/ipath: Clean up some comments

Signed-off-by: default avatarDave Olson <dave.olson@qlogic.com>
Signed-off-by: default avatarRoland Dreier <rolandd@cisco.com>
parent 3029fcc3
...@@ -1196,7 +1196,7 @@ void ipath_kreceive(struct ipath_portdata *pd) ...@@ -1196,7 +1196,7 @@ void ipath_kreceive(struct ipath_portdata *pd)
be32_to_cpu(hdr->bth[0]) & 0xff); be32_to_cpu(hdr->bth[0]) & 0xff);
else { else {
/* /*
* error packet, type of error unknown. * error packet, type of error unknown.
* Probably type 3, but we don't know, so don't * Probably type 3, but we don't know, so don't
* even try to print the opcode, etc. * even try to print the opcode, etc.
*/ */
......
...@@ -1050,11 +1050,6 @@ static int mmap_piobufs(struct vm_area_struct *vma, ...@@ -1050,11 +1050,6 @@ static int mmap_piobufs(struct vm_area_struct *vma,
phys = dd->ipath_physaddr + piobufs; phys = dd->ipath_physaddr + piobufs;
/*
* Don't mark this as non-cached, or we don't get the
* write combining behavior we want on the PIO buffers!
*/
#if defined(__powerpc__) #if defined(__powerpc__)
/* There isn't a generic way to specify writethrough mappings */ /* There isn't a generic way to specify writethrough mappings */
pgprot_val(vma->vm_page_prot) |= _PAGE_NO_CACHE; pgprot_val(vma->vm_page_prot) |= _PAGE_NO_CACHE;
......
...@@ -1022,7 +1022,8 @@ static int ipath_setup_ht_config(struct ipath_devdata *dd, ...@@ -1022,7 +1022,8 @@ static int ipath_setup_ht_config(struct ipath_devdata *dd,
do { do {
u8 cap_type; u8 cap_type;
/* the HT capability type byte is 3 bytes after the /*
* The HT capability type byte is 3 bytes after the
* capability byte. * capability byte.
*/ */
if (pci_read_config_byte(pdev, pos + 3, &cap_type)) { if (pci_read_config_byte(pdev, pos + 3, &cap_type)) {
......
...@@ -535,7 +535,7 @@ static void ipath_pe_handle_hwerrors(struct ipath_devdata *dd, char *msg, ...@@ -535,7 +535,7 @@ static void ipath_pe_handle_hwerrors(struct ipath_devdata *dd, char *msg,
if (hwerrs & INFINIPATH_HWE_SERDESPLLFAILED) { if (hwerrs & INFINIPATH_HWE_SERDESPLLFAILED) {
/* /*
* If it occurs, it is left masked since the eternal * If it occurs, it is left masked since the external
* interface is unused * interface is unused
*/ */
dd->ipath_hwerrmask &= ~INFINIPATH_HWE_SERDESPLLFAILED; dd->ipath_hwerrmask &= ~INFINIPATH_HWE_SERDESPLLFAILED;
......
...@@ -128,9 +128,8 @@ int ipath_lkey_ok(struct ipath_qp *qp, struct ipath_sge *isge, ...@@ -128,9 +128,8 @@ int ipath_lkey_ok(struct ipath_qp *qp, struct ipath_sge *isge,
int ret; int ret;
/* /*
* We use LKEY == zero to mean a physical kmalloc() address. * We use LKEY == zero for kernel virtual addresses
* This is a bit of a hack since we rely on dma_map_single() * (see ipath_get_dma_mr and ipath_dma.c).
* being reversible by calling bus_to_virt().
*/ */
if (sge->lkey == 0) { if (sge->lkey == 0) {
struct ipath_pd *pd = to_ipd(qp->ibqp.pd); struct ipath_pd *pd = to_ipd(qp->ibqp.pd);
......
...@@ -943,7 +943,7 @@ static int ipath_verbs_send_pio(struct ipath_qp *qp, u32 *hdr, u32 hdrwords, ...@@ -943,7 +943,7 @@ static int ipath_verbs_send_pio(struct ipath_qp *qp, u32 *hdr, u32 hdrwords,
* ipath_verbs_send - send a packet * ipath_verbs_send - send a packet
* @qp: the QP to send on * @qp: the QP to send on
* @hdr: the packet header * @hdr: the packet header
* @hdrwords: the number of words in the header * @hdrwords: the number of 32-bit words in the header
* @ss: the SGE to send * @ss: the SGE to send
* @len: the length of the packet in bytes * @len: the length of the packet in bytes
*/ */
...@@ -955,7 +955,10 @@ int ipath_verbs_send(struct ipath_qp *qp, struct ipath_ib_header *hdr, ...@@ -955,7 +955,10 @@ int ipath_verbs_send(struct ipath_qp *qp, struct ipath_ib_header *hdr,
int ret; int ret;
u32 dwords = (len + 3) >> 2; u32 dwords = (len + 3) >> 2;
/* +1 is for the qword padding of pbc */ /*
* Calculate the send buffer trigger address.
* The +1 counts for the pbc control dword following the pbc length.
*/
plen = hdrwords + dwords + 1; plen = hdrwords + dwords + 1;
/* Drop non-VL15 packets if we are not in the active state */ /* Drop non-VL15 packets if we are not in the active state */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment