Use a more typical logging style.
Miscellanea:
o Obsolete the c4iw_debug module parameter
o Coalesce formats
o Realign arguments
Signed-off-by: Joe Perches <joe@perches.com>
Reviewed-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
 
 int c4iw_debug;
 module_param(c4iw_debug, int, 0644);
-MODULE_PARM_DESC(c4iw_debug, "Enable debug logging (default=0)");
+MODULE_PARM_DESC(c4iw_debug, "obsolete");
 
 static int peer2peer = 1;
 module_param(peer2peer, int, 0644);
 
 static void start_ep_timer(struct c4iw_ep *ep)
 {
-       PDBG("%s ep %p\n", __func__, ep);
+       pr_debug("%s ep %p\n", __func__, ep);
        if (timer_pending(&ep->timer)) {
                pr_err("%s timer already started! ep %p\n",
                       __func__, ep);
 
 static int stop_ep_timer(struct c4iw_ep *ep)
 {
-       PDBG("%s ep %p stopping\n", __func__, ep);
+       pr_debug("%s ep %p stopping\n", __func__, ep);
        del_timer_sync(&ep->timer);
        if (!test_and_set_bit(TIMEOUT, &ep->com.flags)) {
                c4iw_put_ep(&ep->com);
 
        if (c4iw_fatal_error(rdev)) {
                kfree_skb(skb);
-               PDBG("%s - device in error state - dropping\n", __func__);
+               pr_debug("%s - device in error state - dropping\n", __func__);
                return -EIO;
        }
        error = cxgb4_l2t_send(rdev->lldi.ports[0], skb, l2e);
 
        if (c4iw_fatal_error(rdev)) {
                kfree_skb(skb);
-               PDBG("%s - device in error state - dropping\n", __func__);
+               pr_debug("%s - device in error state - dropping\n", __func__);
                return -EIO;
        }
        error = cxgb4_ofld_send(rdev->lldi.ports[0], skb);
        if (ep->emss < 128)
                ep->emss = 128;
        if (ep->emss & 7)
-               PDBG("Warning: misaligned mtu idx %u mss %u emss=%u\n",
-                    TCPOPT_MSS_G(opt), ep->mss, ep->emss);
-       PDBG("%s mss_idx %u mss %u emss=%u\n", __func__, TCPOPT_MSS_G(opt),
-            ep->mss, ep->emss);
+               pr_debug("Warning: misaligned mtu idx %u mss %u emss=%u\n",
+                        TCPOPT_MSS_G(opt), ep->mss, ep->emss);
+       pr_debug("%s mss_idx %u mss %u emss=%u\n", __func__, TCPOPT_MSS_G(opt),
+                ep->mss, ep->emss);
 }
 
 static enum c4iw_ep_state state_read(struct c4iw_ep_common *epc)
 static void state_set(struct c4iw_ep_common *epc, enum c4iw_ep_state new)
 {
        mutex_lock(&epc->mutex);
-       PDBG("%s - %s -> %s\n", __func__, states[epc->state], states[new]);
+       pr_debug("%s - %s -> %s\n", __func__, states[epc->state], states[new]);
        __state_set(epc, new);
        mutex_unlock(&epc->mutex);
        return;
                mutex_init(&epc->mutex);
                c4iw_init_wr_wait(&epc->wr_wait);
        }
-       PDBG("%s alloc ep %p\n", __func__, epc);
+       pr_debug("%s alloc ep %p\n", __func__, epc);
        return epc;
 }
 
        struct c4iw_ep *ep;
 
        ep = container_of(kref, struct c4iw_ep, com.kref);
-       PDBG("%s ep %p state %s\n", __func__, ep, states[ep->com.state]);
+       pr_debug("%s ep %p state %s\n", __func__, ep, states[ep->com.state]);
        if (test_bit(QP_REFERENCED, &ep->com.flags))
                deref_qp(ep);
        if (test_bit(RELEASE_RESOURCES, &ep->com.flags)) {
        struct c4iw_rdev *rdev = &ep->com.dev->rdev;
        struct cpl_abort_req *req = cplhdr(skb);
 
-       PDBG("%s rdev %p\n", __func__, rdev);
+       pr_debug("%s rdev %p\n", __func__, rdev);
        req->cmd = CPL_ABORT_NO_RST;
        ret = c4iw_ofld_send(rdev, skb);
        if (ret) {
        struct sk_buff *skb = skb_dequeue(&ep->com.ep_skb_list);
        u32 wrlen = roundup(sizeof(struct cpl_close_con_req), 16);
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        if (WARN_ON(!skb))
                return -ENOMEM;
 
        u32 wrlen = roundup(sizeof(struct cpl_abort_req), 16);
        struct sk_buff *req_skb = skb_dequeue(&ep->com.ep_skb_list);
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        if (WARN_ON(!req_skb))
                return -ENOMEM;
 
                        roundup(sizev4, 16) :
                        roundup(sizev6, 16);
 
-       PDBG("%s ep %p atid %u\n", __func__, ep, ep->atid);
+       pr_debug("%s ep %p atid %u\n", __func__, ep, ep->atid);
 
        skb = get_skb(NULL, wrlen, GFP_KERNEL);
        if (!skb) {
                                t5req->params =
                                          cpu_to_be64(FILTER_TUPLE_V(params));
                                t5req->rsvd = cpu_to_be32(isn);
-                               PDBG("%s snd_isn %u\n", __func__, t5req->rsvd);
+                       pr_debug("%s snd_isn %u\n", __func__, t5req->rsvd);
                                t5req->opt2 = cpu_to_be32(opt2);
                        } else {
                                t6req->params =
                                t5req6->params =
                                            cpu_to_be64(FILTER_TUPLE_V(params));
                                t5req6->rsvd = cpu_to_be32(isn);
-                               PDBG("%s snd_isn %u\n", __func__, t5req6->rsvd);
+                       pr_debug("%s snd_isn %u\n", __func__, t5req6->rsvd);
                                t5req6->opt2 = cpu_to_be32(opt2);
                        } else {
                                t6req6->params =
        struct mpa_message *mpa;
        struct mpa_v2_conn_params mpa_v2_params;
 
-       PDBG("%s ep %p tid %u pd_len %d\n", __func__, ep, ep->hwtid, ep->plen);
+       pr_debug("%s ep %p tid %u pd_len %d\n",
+                __func__, ep, ep->hwtid, ep->plen);
 
        BUG_ON(skb_cloned(skb));
 
        if (mpa_rev_to_use == 2) {
                mpa->private_data_size = htons(ntohs(mpa->private_data_size) +
                                               sizeof (struct mpa_v2_conn_params));
-               PDBG("%s initiator ird %u ord %u\n", __func__, ep->ird,
-                    ep->ord);
+               pr_debug("%s initiator ird %u ord %u\n", __func__, ep->ird,
+                        ep->ord);
                mpa_v2_params.ird = htons((u16)ep->ird);
                mpa_v2_params.ord = htons((u16)ep->ord);
 
        struct sk_buff *skb;
        struct mpa_v2_conn_params mpa_v2_params;
 
-       PDBG("%s ep %p tid %u pd_len %d\n", __func__, ep, ep->hwtid, ep->plen);
+       pr_debug("%s ep %p tid %u pd_len %d\n",
+                __func__, ep, ep->hwtid, ep->plen);
 
        mpalen = sizeof(*mpa) + plen;
        if (ep->mpa_attr.version == 2 && ep->mpa_attr.enhanced_rdma_conn)
        struct sk_buff *skb;
        struct mpa_v2_conn_params mpa_v2_params;
 
-       PDBG("%s ep %p tid %u pd_len %d\n", __func__, ep, ep->hwtid, ep->plen);
+       pr_debug("%s ep %p tid %u pd_len %d\n",
+                __func__, ep, ep->hwtid, ep->plen);
 
        mpalen = sizeof(*mpa) + plen;
        if (ep->mpa_attr.version == 2 && ep->mpa_attr.enhanced_rdma_conn)
 
        ep = lookup_atid(t, atid);
 
-       PDBG("%s ep %p tid %u snd_isn %u rcv_isn %u\n", __func__, ep, tid,
-            be32_to_cpu(req->snd_isn), be32_to_cpu(req->rcv_isn));
+       pr_debug("%s ep %p tid %u snd_isn %u rcv_isn %u\n", __func__, ep, tid,
+                be32_to_cpu(req->snd_isn), be32_to_cpu(req->rcv_isn));
 
        mutex_lock(&ep->com.mutex);
        dst_confirm(ep->dst);
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_CLOSE;
        event.status = status;
        if (ep->com.cm_id) {
-               PDBG("close complete delivered ep %p cm_id %p tid %u\n",
-                    ep, ep->com.cm_id, ep->hwtid);
+               pr_debug("close complete delivered ep %p cm_id %p tid %u\n",
+                        ep, ep->com.cm_id, ep->hwtid);
                ep->com.cm_id->event_handler(ep->com.cm_id, &event);
                deref_cm_id(&ep->com);
                set_bit(CLOSE_UPCALL, &ep->com.history);
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_DISCONNECT;
        if (ep->com.cm_id) {
-               PDBG("peer close delivered ep %p cm_id %p tid %u\n",
-                    ep, ep->com.cm_id, ep->hwtid);
+               pr_debug("peer close delivered ep %p cm_id %p tid %u\n",
+                        ep, ep->com.cm_id, ep->hwtid);
                ep->com.cm_id->event_handler(ep->com.cm_id, &event);
                set_bit(DISCONN_UPCALL, &ep->com.history);
        }
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_CLOSE;
        event.status = -ECONNRESET;
        if (ep->com.cm_id) {
-               PDBG("abort delivered ep %p cm_id %p tid %u\n", ep,
-                    ep->com.cm_id, ep->hwtid);
+               pr_debug("abort delivered ep %p cm_id %p tid %u\n", ep,
+                        ep->com.cm_id, ep->hwtid);
                ep->com.cm_id->event_handler(ep->com.cm_id, &event);
                deref_cm_id(&ep->com);
                set_bit(ABORT_UPCALL, &ep->com.history);
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u status %d\n", __func__, ep, ep->hwtid, status);
+       pr_debug("%s ep %p tid %u status %d\n",
+                __func__, ep, ep->hwtid, status);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_CONNECT_REPLY;
        event.status = status;
                }
        }
 
-       PDBG("%s ep %p tid %u status %d\n", __func__, ep,
-            ep->hwtid, status);
+       pr_debug("%s ep %p tid %u status %d\n", __func__, ep,
+                ep->hwtid, status);
        set_bit(CONN_RPL_UPCALL, &ep->com.history);
        ep->com.cm_id->event_handler(ep->com.cm_id, &event);
 
        struct iw_cm_event event;
        int ret;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_CONNECT_REQUEST;
        memcpy(&event.local_addr, &ep->com.local_addr,
 {
        struct iw_cm_event event;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        memset(&event, 0, sizeof(event));
        event.event = IW_CM_EVENT_ESTABLISHED;
        event.ird = ep->ord;
        event.ord = ep->ird;
        if (ep->com.cm_id) {
-               PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+               pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
                ep->com.cm_id->event_handler(ep->com.cm_id, &event);
                set_bit(ESTAB_UPCALL, &ep->com.history);
        }
        u32 wrlen = roundup(sizeof(struct cpl_rx_data_ack), 16);
        u32 credit_dack;
 
-       PDBG("%s ep %p tid %u credits %u\n", __func__, ep, ep->hwtid, credits);
+       pr_debug("%s ep %p tid %u credits %u\n",
+                __func__, ep, ep->hwtid, credits);
        skb = get_skb(NULL, wrlen, GFP_KERNEL);
        if (!skb) {
                pr_err("update_rx_credits - cannot alloc skb!\n");
        int err;
        int disconnect = 0;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
 
        /*
         * If we get more than the supported amount of private data
                                MPA_V2_IRD_ORD_MASK;
                        resp_ord = ntohs(mpa_v2_params->ord) &
                                MPA_V2_IRD_ORD_MASK;
-                       PDBG("%s responder ird %u ord %u ep ird %u ord %u\n",
-                            __func__, resp_ird, resp_ord, ep->ird, ep->ord);
+                       pr_debug("%s responder ird %u ord %u ep ird %u ord %u\n",
+                                __func__,
+                                resp_ird, resp_ord, ep->ird, ep->ord);
 
                        /*
                         * This is a double-check. Ideally, below checks are
                if (peer2peer)
                        ep->mpa_attr.p2p_type = p2p_type;
 
-       PDBG("%s - crc_enabled=%d, recv_marker_enabled=%d, "
-            "xmit_marker_enabled=%d, version=%d p2p_type=%d local-p2p_type = "
-            "%d\n", __func__, ep->mpa_attr.crc_enabled,
-            ep->mpa_attr.recv_marker_enabled,
-            ep->mpa_attr.xmit_marker_enabled, ep->mpa_attr.version,
-            ep->mpa_attr.p2p_type, p2p_type);
+       pr_debug("%s - crc_enabled=%d, recv_marker_enabled=%d, xmit_marker_enabled=%d, version=%d p2p_type=%d local-p2p_type = %d\n",
+                __func__, ep->mpa_attr.crc_enabled,
+                ep->mpa_attr.recv_marker_enabled,
+                ep->mpa_attr.xmit_marker_enabled, ep->mpa_attr.version,
+                ep->mpa_attr.p2p_type, p2p_type);
 
        /*
         * If responder's RTR does not match with that of initiator, assign
        struct mpa_v2_conn_params *mpa_v2_params;
        u16 plen;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
 
        /*
         * If we get more than the supported amount of private data
        if (ep->mpa_pkt_len + skb->len > sizeof(ep->mpa_pkt))
                goto err_stop_timer;
 
-       PDBG("%s enter (%s line %u)\n", __func__, __FILE__, __LINE__);
+       pr_debug("%s enter (%s line %u)\n", __func__, __FILE__, __LINE__);
 
        /*
         * Copy the new data into our accumulation buffer.
        if (ep->mpa_pkt_len < sizeof(*mpa))
                return 0;
 
-       PDBG("%s enter (%s line %u)\n", __func__, __FILE__, __LINE__);
+       pr_debug("%s enter (%s line %u)\n", __func__, __FILE__, __LINE__);
        mpa = (struct mpa_message *) ep->mpa_pkt;
 
        /*
                                MPA_V2_IRD_ORD_MASK;
                        ep->ord = min_t(u32, ep->ord,
                                        cur_max_read_depth(ep->com.dev));
-                       PDBG("%s initiator ird %u ord %u\n", __func__, ep->ird,
-                            ep->ord);
+                       pr_debug("%s initiator ird %u ord %u\n",
+                                __func__, ep->ird, ep->ord);
                        if (ntohs(mpa_v2_params->ird) & MPA_V2_PEER2PEER_MODEL)
                                if (peer2peer) {
                                        if (ntohs(mpa_v2_params->ord) &
                if (peer2peer)
                        ep->mpa_attr.p2p_type = p2p_type;
 
-       PDBG("%s - crc_enabled=%d, recv_marker_enabled=%d, "
-            "xmit_marker_enabled=%d, version=%d p2p_type=%d\n", __func__,
-            ep->mpa_attr.crc_enabled, ep->mpa_attr.recv_marker_enabled,
-            ep->mpa_attr.xmit_marker_enabled, ep->mpa_attr.version,
-            ep->mpa_attr.p2p_type);
+       pr_debug("%s - crc_enabled=%d, recv_marker_enabled=%d, xmit_marker_enabled=%d, version=%d p2p_type=%d\n",
+                __func__,
+                ep->mpa_attr.crc_enabled, ep->mpa_attr.recv_marker_enabled,
+                ep->mpa_attr.xmit_marker_enabled, ep->mpa_attr.version,
+                ep->mpa_attr.p2p_type);
 
        __state_set(&ep->com, MPA_REQ_RCVD);
 
        ep = get_ep_from_tid(dev, tid);
        if (!ep)
                return 0;
-       PDBG("%s ep %p tid %u dlen %u\n", __func__, ep, ep->hwtid, dlen);
+       pr_debug("%s ep %p tid %u dlen %u\n", __func__, ep, ep->hwtid, dlen);
        skb_pull(skb, sizeof(*hdr));
        skb_trim(skb, dlen);
        mutex_lock(&ep->com.mutex);
                pr_warn("Abort rpl to freed endpoint\n");
                return 0;
        }
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        mutex_lock(&ep->com.mutex);
        switch (ep->com.state) {
        case ABORTING:
 {
        ep->snd_win = snd_win;
        ep->rcv_win = rcv_win;
-       PDBG("%s snd_win %d rcv_win %d\n", __func__, ep->snd_win, ep->rcv_win);
+       pr_debug("%s snd_win %d rcv_win %d\n",
+                __func__, ep->snd_win, ep->rcv_win);
 }
 
 #define ACT_OPEN_RETRY_COUNT 2
        int iptype;
        __u8 *ra;
 
-       PDBG("%s qp %p cm_id %p\n", __func__, ep->com.qp, ep->com.cm_id);
+       pr_debug("%s qp %p cm_id %p\n", __func__, ep->com.qp, ep->com.cm_id);
        init_timer(&ep->timer);
        c4iw_init_wr_wait(&ep->com.wr_wait);
 
                goto fail4;
        }
 
-       PDBG("%s txq_idx %u tx_chan %u smac_idx %u rss_qid %u l2t_idx %u\n",
-            __func__, ep->txq_idx, ep->tx_chan, ep->smac_idx, ep->rss_qid,
-            ep->l2t->idx);
+       pr_debug("%s txq_idx %u tx_chan %u smac_idx %u rss_qid %u l2t_idx %u\n",
+                __func__, ep->txq_idx, ep->tx_chan, ep->smac_idx, ep->rss_qid,
+                ep->l2t->idx);
 
        state_set(&ep->com, CONNECTING);
        ep->tos = ep->com.cm_id->tos;
        la6 = (struct sockaddr_in6 *)&ep->com.local_addr;
        ra6 = (struct sockaddr_in6 *)&ep->com.remote_addr;
 
-       PDBG("%s ep %p atid %u status %u errno %d\n", __func__, ep, atid,
-            status, status2errno(status));
+       pr_debug("%s ep %p atid %u status %u errno %d\n", __func__, ep, atid,
+                status, status2errno(status));
 
        if (cxgb_is_neg_adv(status)) {
-               PDBG("%s Connection problems for atid %u status %u (%s)\n",
-                    __func__, atid, status, neg_adv_str(status));
+               pr_debug("%s Connection problems for atid %u status %u (%s)\n",
+                        __func__, atid, status, neg_adv_str(status));
                ep->stats.connect_neg_adv++;
                mutex_lock(&dev->rdev.stats.lock);
                dev->rdev.stats.neg_adv++;
        struct c4iw_listen_ep *ep = get_ep_from_stid(dev, stid);
 
        if (!ep) {
-               PDBG("%s stid %d lookup failure!\n", __func__, stid);
+               pr_debug("%s stid %d lookup failure!\n", __func__, stid);
                goto out;
        }
-       PDBG("%s ep %p status %d error %d\n", __func__, ep,
-            rpl->status, status2errno(rpl->status));
+       pr_debug("%s ep %p status %d error %d\n", __func__, ep,
+                rpl->status, status2errno(rpl->status));
        c4iw_wake_up(&ep->com.wr_wait, status2errno(rpl->status));
        c4iw_put_ep(&ep->com);
 out:
        unsigned int stid = GET_TID(rpl);
        struct c4iw_listen_ep *ep = get_ep_from_stid(dev, stid);
 
-       PDBG("%s ep %p\n", __func__, ep);
+       pr_debug("%s ep %p\n", __func__, ep);
        c4iw_wake_up(&ep->com.wr_wait, status2errno(rpl->status));
        c4iw_put_ep(&ep->com);
        return 0;
        int win;
        enum chip_type adapter_type = ep->com.dev->rdev.lldi.adapter_type;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        BUG_ON(skb_cloned(skb));
 
        skb_get(skb);
                if (peer2peer)
                        isn += 4;
                rpl5->iss = cpu_to_be32(isn);
-               PDBG("%s iss %u\n", __func__, be32_to_cpu(rpl5->iss));
+               pr_debug("%s iss %u\n", __func__, be32_to_cpu(rpl5->iss));
        }
 
        rpl->opt0 = cpu_to_be64(opt0);
 
 static void reject_cr(struct c4iw_dev *dev, u32 hwtid, struct sk_buff *skb)
 {
-       PDBG("%s c4iw_dev %p tid %u\n", __func__, dev, hwtid);
+       pr_debug("%s c4iw_dev %p tid %u\n", __func__, dev, hwtid);
        BUG_ON(skb_cloned(skb));
        skb_trim(skb, sizeof(struct cpl_tid_release));
        release_tid(&dev->rdev, hwtid, skb);
 
        parent_ep = (struct c4iw_ep *)get_ep_from_stid(dev, stid);
        if (!parent_ep) {
-               PDBG("%s connect request on invalid stid %d\n", __func__, stid);
+               pr_debug("%s connect request on invalid stid %d\n",
+                        __func__, stid);
                goto reject;
        }
 
        if (state_read(&parent_ep->com) != LISTEN) {
-               PDBG("%s - listening ep not in LISTEN\n", __func__);
+               pr_debug("%s - listening ep not in LISTEN\n", __func__);
                goto reject;
        }
 
 
        /* Find output route */
        if (iptype == 4)  {
-               PDBG("%s parent ep %p hwtid %u laddr %pI4 raddr %pI4 lport %d rport %d peer_mss %d\n"
-                    , __func__, parent_ep, hwtid,
-                    local_ip, peer_ip, ntohs(local_port),
-                    ntohs(peer_port), peer_mss);
+               pr_debug("%s parent ep %p hwtid %u laddr %pI4 raddr %pI4 lport %d rport %d peer_mss %d\n"
+                        , __func__, parent_ep, hwtid,
+                        local_ip, peer_ip, ntohs(local_port),
+                        ntohs(peer_port), peer_mss);
                dst = cxgb_find_route(&dev->rdev.lldi, get_real_dev,
                                      *(__be32 *)local_ip, *(__be32 *)peer_ip,
                                      local_port, peer_port, tos);
        } else {
-               PDBG("%s parent ep %p hwtid %u laddr %pI6 raddr %pI6 lport %d rport %d peer_mss %d\n"
-                    , __func__, parent_ep, hwtid,
-                    local_ip, peer_ip, ntohs(local_port),
-                    ntohs(peer_port), peer_mss);
+               pr_debug("%s parent ep %p hwtid %u laddr %pI6 raddr %pI6 lport %d rport %d peer_mss %d\n"
+                        , __func__, parent_ep, hwtid,
+                        local_ip, peer_ip, ntohs(local_port),
+                        ntohs(peer_port), peer_mss);
                dst = cxgb_find_route6(&dev->rdev.lldi, get_real_dev,
                                local_ip, peer_ip, local_port, peer_port,
                                PASS_OPEN_TOS_G(ntohl(req->tos_stid)),
        child_ep->dst = dst;
        child_ep->hwtid = hwtid;
 
-       PDBG("%s tx_chan %u smac_idx %u rss_qid %u\n", __func__,
-            child_ep->tx_chan, child_ep->smac_idx, child_ep->rss_qid);
+       pr_debug("%s tx_chan %u smac_idx %u rss_qid %u\n", __func__,
+                child_ep->tx_chan, child_ep->smac_idx, child_ep->rss_qid);
 
        init_timer(&child_ep->timer);
        cxgb4_insert_tid(t, child_ep, hwtid);
        int ret;
 
        ep = get_ep_from_tid(dev, tid);
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        ep->snd_seq = be32_to_cpu(req->snd_isn);
        ep->rcv_seq = be32_to_cpu(req->rcv_isn);
 
-       PDBG("%s ep %p hwtid %u tcp_opt 0x%02x\n", __func__, ep, tid,
-            ntohs(req->tcp_opt));
+       pr_debug("%s ep %p hwtid %u tcp_opt 0x%02x\n", __func__, ep, tid,
+                ntohs(req->tcp_opt));
 
        set_emss(ep, ntohs(req->tcp_opt));
 
        if (!ep)
                return 0;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        dst_confirm(ep->dst);
 
        set_bit(PEER_CLOSE, &ep->com.history);
                 * in rdma connection migration (see c4iw_accept_cr()).
                 */
                __state_set(&ep->com, CLOSING);
-               PDBG("waking up ep %p tid %u\n", ep, ep->hwtid);
+               pr_debug("waking up ep %p tid %u\n", ep, ep->hwtid);
                c4iw_wake_up(&ep->com.wr_wait, -ECONNRESET);
                break;
        case MPA_REP_SENT:
                __state_set(&ep->com, CLOSING);
-               PDBG("waking up ep %p tid %u\n", ep, ep->hwtid);
+               pr_debug("waking up ep %p tid %u\n", ep, ep->hwtid);
                c4iw_wake_up(&ep->com.wr_wait, -ECONNRESET);
                break;
        case FPDU_MODE:
                return 0;
 
        if (cxgb_is_neg_adv(req->status)) {
-               PDBG("%s Negative advice on abort- tid %u status %d (%s)\n",
-                    __func__, ep->hwtid, req->status,
-                    neg_adv_str(req->status));
+               pr_debug("%s Negative advice on abort- tid %u status %d (%s)\n",
+                        __func__, ep->hwtid, req->status,
+                        neg_adv_str(req->status));
                ep->stats.abort_neg_adv++;
                mutex_lock(&dev->rdev.stats.lock);
                dev->rdev.stats.neg_adv++;
                mutex_unlock(&dev->rdev.stats.lock);
                goto deref_ep;
        }
-       PDBG("%s ep %p tid %u state %u\n", __func__, ep, ep->hwtid,
-            ep->com.state);
+       pr_debug("%s ep %p tid %u state %u\n", __func__, ep, ep->hwtid,
+                ep->com.state);
        set_bit(PEER_ABORT, &ep->com.history);
 
        /*
                         * do some housekeeping so as to re-initiate the
                         * connection
                         */
-                       PDBG("%s: mpa_rev=%d. Retrying with mpav1\n", __func__,
-                            mpa_rev);
+                       pr_debug("%s: mpa_rev=%d. Retrying with mpav1\n",
+                                __func__, mpa_rev);
                        ep->retry_with_mpa_v1 = 1;
                }
                break;
        case ABORTING:
                break;
        case DEAD:
-               PDBG("%s PEER_ABORT IN DEAD STATE!!!!\n", __func__);
+               pr_debug("%s PEER_ABORT IN DEAD STATE!!!!\n", __func__);
                mutex_unlock(&ep->com.mutex);
                goto deref_ep;
        default:
        if (!ep)
                return 0;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
        BUG_ON(!ep);
 
        /* The cm_id may be null if we failed to connect */
        ep = get_ep_from_tid(dev, tid);
        if (!ep)
                return 0;
-       PDBG("%s ep %p tid %u credits %u\n", __func__, ep, ep->hwtid, credits);
+       pr_debug("%s ep %p tid %u credits %u\n",
+                __func__, ep, ep->hwtid, credits);
        if (credits == 0) {
-               PDBG("%s 0 credit ack ep %p tid %u state %u\n",
-                    __func__, ep, ep->hwtid, state_read(&ep->com));
+               pr_debug("%s 0 credit ack ep %p tid %u state %u\n",
+                        __func__, ep, ep->hwtid, state_read(&ep->com));
                goto out;
        }
 
        dst_confirm(ep->dst);
        if (ep->mpa_skb) {
-               PDBG("%s last streaming msg ack ep %p tid %u state %u "
-                    "initiator %u freeing skb\n", __func__, ep, ep->hwtid,
-                    state_read(&ep->com), ep->mpa_attr.initiator ? 1 : 0);
+               pr_debug("%s last streaming msg ack ep %p tid %u state %u initiator %u freeing skb\n",
+                        __func__, ep, ep->hwtid,
+                        state_read(&ep->com), ep->mpa_attr.initiator ? 1 : 0);
                mutex_lock(&ep->com.mutex);
                kfree_skb(ep->mpa_skb);
                ep->mpa_skb = NULL;
        int abort;
        struct c4iw_ep *ep = to_ep(cm_id);
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
 
        mutex_lock(&ep->com.mutex);
        if (ep->com.state != MPA_REQ_RCVD) {
        struct c4iw_qp *qp = get_qhp(h, conn_param->qpn);
        int abort = 0;
 
-       PDBG("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
+       pr_debug("%s ep %p tid %u\n", __func__, ep, ep->hwtid);
 
        mutex_lock(&ep->com.mutex);
        if (ep->com.state != MPA_REQ_RCVD) {
                        ep->ird = 1;
        }
 
-       PDBG("%s %d ird %d ord %d\n", __func__, __LINE__, ep->ird, ep->ord);
+       pr_debug("%s %d ird %d ord %d\n", __func__, __LINE__, ep->ird, ep->ord);
 
        ep->com.cm_id = cm_id;
        ref_cm_id(&ep->com);
        ep->com.dev = dev;
        ep->com.qp = get_qhp(dev, conn_param->qpn);
        if (!ep->com.qp) {
-               PDBG("%s qpn 0x%x not found!\n", __func__, conn_param->qpn);
+               pr_debug("%s qpn 0x%x not found!\n", __func__, conn_param->qpn);
                err = -EINVAL;
                goto fail2;
        }
        ref_qp(ep);
-       PDBG("%s qpn 0x%x qp %p cm_id %p\n", __func__, conn_param->qpn,
-            ep->com.qp, cm_id);
+       pr_debug("%s qpn 0x%x qp %p cm_id %p\n", __func__, conn_param->qpn,
+                ep->com.qp, cm_id);
 
        /*
         * Allocate an active TID to initiate a TCP connection.
                }
 
                /* find a route */
-               PDBG("%s saddr %pI4 sport 0x%x raddr %pI4 rport 0x%x\n",
-                    __func__, &laddr->sin_addr, ntohs(laddr->sin_port),
-                    ra, ntohs(raddr->sin_port));
+               pr_debug("%s saddr %pI4 sport 0x%x raddr %pI4 rport 0x%x\n",
+                        __func__, &laddr->sin_addr, ntohs(laddr->sin_port),
+                        ra, ntohs(raddr->sin_port));
                ep->dst = cxgb_find_route(&dev->rdev.lldi, get_real_dev,
                                          laddr->sin_addr.s_addr,
                                          raddr->sin_addr.s_addr,
                }
 
                /* find a route */
-               PDBG("%s saddr %pI6 sport 0x%x raddr %pI6 rport 0x%x\n",
-                    __func__, laddr6->sin6_addr.s6_addr,
-                    ntohs(laddr6->sin6_port),
-                    raddr6->sin6_addr.s6_addr, ntohs(raddr6->sin6_port));
+               pr_debug("%s saddr %pI6 sport 0x%x raddr %pI6 rport 0x%x\n",
+                        __func__, laddr6->sin6_addr.s6_addr,
+                        ntohs(laddr6->sin6_port),
+                        raddr6->sin6_addr.s6_addr, ntohs(raddr6->sin6_port));
                ep->dst = cxgb_find_route6(&dev->rdev.lldi, get_real_dev,
                                           laddr6->sin6_addr.s6_addr,
                                           raddr6->sin6_addr.s6_addr,
                goto fail4;
        }
 
-       PDBG("%s txq_idx %u tx_chan %u smac_idx %u rss_qid %u l2t_idx %u\n",
-               __func__, ep->txq_idx, ep->tx_chan, ep->smac_idx, ep->rss_qid,
-               ep->l2t->idx);
+       pr_debug("%s txq_idx %u tx_chan %u smac_idx %u rss_qid %u l2t_idx %u\n",
+                __func__, ep->txq_idx, ep->tx_chan, ep->smac_idx, ep->rss_qid,
+                ep->l2t->idx);
 
        state_set(&ep->com, CONNECTING);
        ep->tos = cm_id->tos;
                goto fail1;
        }
        skb_queue_head_init(&ep->com.ep_skb_list);
-       PDBG("%s ep %p\n", __func__, ep);
+       pr_debug("%s ep %p\n", __func__, ep);
        ep->com.cm_id = cm_id;
        ref_cm_id(&ep->com);
        ep->com.dev = dev;
        int err;
        struct c4iw_listen_ep *ep = to_listen_ep(cm_id);
 
-       PDBG("%s ep %p\n", __func__, ep);
+       pr_debug("%s ep %p\n", __func__, ep);
 
        might_sleep();
        state_set(&ep->com, DEAD);
 
        mutex_lock(&ep->com.mutex);
 
-       PDBG("%s ep %p state %s, abrupt %d\n", __func__, ep,
-            states[ep->com.state], abrupt);
+       pr_debug("%s ep %p state %s, abrupt %d\n", __func__, ep,
+                states[ep->com.state], abrupt);
 
        /*
         * Ref the ep here in case we have fatal errors causing the
        case MORIBUND:
        case ABORTING:
        case DEAD:
-               PDBG("%s ignoring disconnect ep %p state %u\n",
-                    __func__, ep, ep->com.state);
+               pr_debug("%s ignoring disconnect ep %p state %u\n",
+                        __func__, ep, ep->com.state);
                break;
        default:
                BUG();
        rpl_skb = (struct sk_buff *)(unsigned long)req->cookie;
        BUG_ON(!rpl_skb);
        if (req->retval) {
-               PDBG("%s passive open failure %d\n", __func__, req->retval);
+               pr_debug("%s passive open failure %d\n", __func__, req->retval);
                mutex_lock(&dev->rdev.stats.lock);
                dev->rdev.stats.pas_ofld_conn_fails++;
                mutex_unlock(&dev->rdev.stats.lock);
 
        lep = (struct c4iw_ep *)get_ep_from_stid(dev, stid);
        if (!lep) {
-               PDBG("%s connect request on invalid stid %d\n", __func__, stid);
+               pr_debug("%s connect request on invalid stid %d\n",
+                        __func__, stid);
                goto reject;
        }
 
        skb_set_transport_header(skb, (void *)tcph - (void *)rss);
        skb_get(skb);
 
-       PDBG("%s lip 0x%x lport %u pip 0x%x pport %u tos %d\n", __func__,
-            ntohl(iph->daddr), ntohs(tcph->dest), ntohl(iph->saddr),
-            ntohs(tcph->source), iph->tos);
+       pr_debug("%s lip 0x%x lport %u pip 0x%x pport %u tos %d\n", __func__,
+                ntohl(iph->daddr), ntohs(tcph->dest), ntohl(iph->saddr),
+                ntohs(tcph->source), iph->tos);
 
        dst = cxgb_find_route(&dev->rdev.lldi, get_real_dev,
                              iph->daddr, iph->saddr, tcph->dest,
        int abort = 1;
 
        mutex_lock(&ep->com.mutex);
-       PDBG("%s ep %p tid %u state %d\n", __func__, ep, ep->hwtid,
-            ep->com.state);
+       pr_debug("%s ep %p tid %u state %d\n", __func__, ep, ep->hwtid,
+                ep->com.state);
        set_bit(TIMEDOUT, &ep->com.history);
        switch (ep->com.state) {
        case MPA_REQ_SENT:
        struct c4iw_wr_wait *wr_waitp;
        int ret;
 
-       PDBG("%s type %u\n", __func__, rpl->type);
+       pr_debug("%s type %u\n", __func__, rpl->type);
 
        switch (rpl->type) {
        case FW6_TYPE_WR_RPL:
                ret = (int)((be64_to_cpu(rpl->data[0]) >> 8) & 0xff);
                wr_waitp = (struct c4iw_wr_wait *)(__force unsigned long) rpl->data[1];
-               PDBG("%s wr_waitp %p ret %u\n", __func__, wr_waitp, ret);
+               pr_debug("%s wr_waitp %p ret %u\n", __func__, wr_waitp, ret);
                if (wr_waitp)
                        c4iw_wake_up(wr_waitp, ret ? -ret : 0);
                kfree_skb(skb);
                return 0;
        }
        if (cxgb_is_neg_adv(req->status)) {
-               PDBG("%s Negative advice on abort- tid %u status %d (%s)\n",
-                    __func__, ep->hwtid, req->status,
-                    neg_adv_str(req->status));
+               pr_debug("%s Negative advice on abort- tid %u status %d (%s)\n",
+                        __func__, ep->hwtid, req->status,
+                        neg_adv_str(req->status));
                goto out;
        }
-       PDBG("%s ep %p tid %u state %u\n", __func__, ep, ep->hwtid,
-            ep->com.state);
+       pr_debug("%s ep %p tid %u state %u\n", __func__, ep, ep->hwtid,
+                ep->com.state);
 
        c4iw_wake_up(&ep->com.wr_wait, -ECONNRESET);
 out:
 
        ret = c4iw_ofld_send(rdev, skb);
        if (ret)
                goto err4;
-       PDBG("%s wait_event wr_wait %p\n", __func__, &wr_wait);
+       pr_debug("%s wait_event wr_wait %p\n", __func__, &wr_wait);
        ret = c4iw_wait_for_reply(rdev, &wr_wait, 0, 0, __func__);
        if (ret)
                goto err4;
 {
        struct t4_cqe cqe;
 
-       PDBG("%s wq %p cq %p sw_cidx %u sw_pidx %u\n", __func__,
-            wq, cq, cq->sw_cidx, cq->sw_pidx);
+       pr_debug("%s wq %p cq %p sw_cidx %u sw_pidx %u\n", __func__,
+                wq, cq, cq->sw_cidx, cq->sw_pidx);
        memset(&cqe, 0, sizeof(cqe));
        cqe.header = cpu_to_be32(CQE_STATUS_V(T4_ERR_SWFLUSH) |
                                 CQE_OPCODE_V(FW_RI_SEND) |
        int in_use = wq->rq.in_use - count;
 
        BUG_ON(in_use < 0);
-       PDBG("%s wq %p cq %p rq.in_use %u skip count %u\n", __func__,
-            wq, cq, wq->rq.in_use, count);
+       pr_debug("%s wq %p cq %p rq.in_use %u skip count %u\n", __func__,
+                wq, cq, wq->rq.in_use, count);
        while (in_use--) {
                insert_recv_cqe(wq, cq);
                flushed++;
 {
        struct t4_cqe cqe;
 
-       PDBG("%s wq %p cq %p sw_cidx %u sw_pidx %u\n", __func__,
-            wq, cq, cq->sw_cidx, cq->sw_pidx);
+       pr_debug("%s wq %p cq %p sw_cidx %u sw_pidx %u\n", __func__,
+                wq, cq, cq->sw_cidx, cq->sw_pidx);
        memset(&cqe, 0, sizeof(cqe));
        cqe.header = cpu_to_be32(CQE_STATUS_V(T4_ERR_SWFLUSH) |
                                 CQE_OPCODE_V(swcqe->opcode) |
                        /*
                         * Insert this completed cqe into the swcq.
                         */
-                       PDBG("%s moving cqe into swcq sq idx %u cq idx %u\n",
-                                       __func__, cidx, cq->sw_pidx);
+                       pr_debug("%s moving cqe into swcq sq idx %u cq idx %u\n",
+                                __func__, cidx, cq->sw_pidx);
                        swsqe->cqe.header |= htonl(CQE_SWCQE_V(1));
                        cq->sw_queue[cq->sw_pidx] = swsqe->cqe;
                        t4_swcq_produce(cq);
        struct t4_swsqe *swsqe;
        int ret;
 
-       PDBG("%s  cqid 0x%x\n", __func__, chp->cq.cqid);
+       pr_debug("%s  cqid 0x%x\n", __func__, chp->cq.cqid);
        ret = t4_next_hw_cqe(&chp->cq, &hw_cqe);
 
        /*
        u32 ptr;
 
        *count = 0;
-       PDBG("%s count zero %d\n", __func__, *count);
+       pr_debug("%s count zero %d\n", __func__, *count);
        ptr = cq->sw_cidx;
        while (ptr != cq->sw_pidx) {
                cqe = &cq->sw_queue[ptr];
                if (++ptr == cq->size)
                        ptr = 0;
        }
-       PDBG("%s cq %p count %d\n", __func__, cq, *count);
+       pr_debug("%s cq %p count %d\n", __func__, cq, *count);
 }
 
 /*
        if (ret)
                return ret;
 
-       PDBG("%s CQE OVF %u qpid 0x%0x genbit %u type %u status 0x%0x"
-            " opcode 0x%0x len 0x%0x wrid_hi_stag 0x%x wrid_low_msn 0x%x\n",
-            __func__, CQE_OVFBIT(hw_cqe), CQE_QPID(hw_cqe),
-            CQE_GENBIT(hw_cqe), CQE_TYPE(hw_cqe), CQE_STATUS(hw_cqe),
-            CQE_OPCODE(hw_cqe), CQE_LEN(hw_cqe), CQE_WRID_HI(hw_cqe),
-            CQE_WRID_LOW(hw_cqe));
+       pr_debug("%s CQE OVF %u qpid 0x%0x genbit %u type %u status 0x%0x opcode 0x%0x len 0x%0x wrid_hi_stag 0x%x wrid_low_msn 0x%x\n",
+                __func__, CQE_OVFBIT(hw_cqe), CQE_QPID(hw_cqe),
+                CQE_GENBIT(hw_cqe), CQE_TYPE(hw_cqe), CQE_STATUS(hw_cqe),
+                CQE_OPCODE(hw_cqe), CQE_LEN(hw_cqe), CQE_WRID_HI(hw_cqe),
+                CQE_WRID_LOW(hw_cqe));
 
        /*
         * skip cqe's not affiliated with a QP.
        if (!SW_CQE(hw_cqe) && (CQE_WRID_SQ_IDX(hw_cqe) != wq->sq.cidx)) {
                struct t4_swsqe *swsqe;
 
-               PDBG("%s out of order completion going in sw_sq at idx %u\n",
-                    __func__, CQE_WRID_SQ_IDX(hw_cqe));
+               pr_debug("%s out of order completion going in sw_sq at idx %u\n",
+                        __func__, CQE_WRID_SQ_IDX(hw_cqe));
                swsqe = &wq->sq.sw_sq[CQE_WRID_SQ_IDX(hw_cqe)];
                swsqe->cqe = *hw_cqe;
                swsqe->complete = 1;
                BUG_ON(wq->sq.in_use <= 0 && wq->sq.in_use >= wq->sq.size);
 
                wq->sq.cidx = (uint16_t)idx;
-               PDBG("%s completing sq idx %u\n", __func__, wq->sq.cidx);
+               pr_debug("%s completing sq idx %u\n", __func__, wq->sq.cidx);
                *cookie = wq->sq.sw_sq[wq->sq.cidx].wr_id;
                if (c4iw_wr_log)
                        c4iw_log_wr_stats(wq, hw_cqe);
                t4_sq_consume(wq);
        } else {
-               PDBG("%s completing rq idx %u\n", __func__, wq->rq.cidx);
+               pr_debug("%s completing rq idx %u\n", __func__, wq->rq.cidx);
                *cookie = wq->rq.sw_rq[wq->rq.cidx].wr_id;
                BUG_ON(t4_rq_empty(wq));
                if (c4iw_wr_log)
 
 skip_cqe:
        if (SW_CQE(hw_cqe)) {
-               PDBG("%s cq %p cqid 0x%x skip sw cqe cidx %u\n",
-                    __func__, cq, cq->cqid, cq->sw_cidx);
+               pr_debug("%s cq %p cqid 0x%x skip sw cqe cidx %u\n",
+                        __func__, cq, cq->cqid, cq->sw_cidx);
                t4_swcq_consume(cq);
        } else {
-               PDBG("%s cq %p cqid 0x%x skip hw cqe cidx %u\n",
-                    __func__, cq, cq->cqid, cq->cidx);
+               pr_debug("%s cq %p cqid 0x%x skip hw cqe cidx %u\n",
+                        __func__, cq, cq->cqid, cq->cidx);
                t4_hwcq_consume(cq);
        }
        return ret;
        wc->vendor_err = CQE_STATUS(&cqe);
        wc->wc_flags = 0;
 
-       PDBG("%s qpid 0x%x type %d opcode %d status 0x%x len %u wrid hi 0x%x "
-            "lo 0x%x cookie 0x%llx\n", __func__, CQE_QPID(&cqe),
-            CQE_TYPE(&cqe), CQE_OPCODE(&cqe), CQE_STATUS(&cqe), CQE_LEN(&cqe),
-            CQE_WRID_HI(&cqe), CQE_WRID_LOW(&cqe), (unsigned long long)cookie);
+       pr_debug("%s qpid 0x%x type %d opcode %d status 0x%x len %u wrid hi 0x%x lo 0x%x cookie 0x%llx\n",
+                __func__, CQE_QPID(&cqe),
+                CQE_TYPE(&cqe), CQE_OPCODE(&cqe),
+                CQE_STATUS(&cqe), CQE_LEN(&cqe),
+                CQE_WRID_HI(&cqe), CQE_WRID_LOW(&cqe),
+                (unsigned long long)cookie);
 
        if (CQE_TYPE(&cqe) == 0) {
                if (!CQE_STATUS(&cqe))
        struct c4iw_cq *chp;
        struct c4iw_ucontext *ucontext;
 
-       PDBG("%s ib_cq %p\n", __func__, ib_cq);
+       pr_debug("%s ib_cq %p\n", __func__, ib_cq);
        chp = to_c4iw_cq(ib_cq);
 
        remove_handle(chp->rhp, &chp->rhp->cqidr, chp->cq.cqid);
        size_t memsize, hwentries;
        struct c4iw_mm_entry *mm, *mm2;
 
-       PDBG("%s ib_dev %p entries %d\n", __func__, ibdev, entries);
+       pr_debug("%s ib_dev %p entries %d\n", __func__, ibdev, entries);
        if (attr->flags)
                return ERR_PTR(-EINVAL);
 
                mm2->len = PAGE_SIZE;
                insert_mmap(ucontext, mm2);
        }
-       PDBG("%s cqid 0x%0x chp %p size %u memsize %zu, dma_addr 0x%0llx\n",
-            __func__, chp->cq.cqid, chp, chp->cq.size,
-            chp->cq.memsize, (unsigned long long) chp->cq.dma_addr);
+       pr_debug("%s cqid 0x%0x chp %p size %u memsize %zu, dma_addr 0x%0llx\n",
+                __func__, chp->cq.cqid, chp, chp->cq.size,
+                chp->cq.memsize, (unsigned long long)chp->cq.dma_addr);
        return &chp->ibcq;
 err6:
        kfree(mm2);
 
 
        rdev->qpmask = rdev->lldi.udb_density - 1;
        rdev->cqmask = rdev->lldi.ucq_density - 1;
-       PDBG("%s dev %s stag start 0x%0x size 0x%0x num stags %d "
-            "pbl start 0x%0x size 0x%0x rq start 0x%0x size 0x%0x "
-            "qp qid start %u size %u cq qid start %u size %u\n",
-            __func__, pci_name(rdev->lldi.pdev), rdev->lldi.vr->stag.start,
-            rdev->lldi.vr->stag.size, c4iw_num_stags(rdev),
-            rdev->lldi.vr->pbl.start,
-            rdev->lldi.vr->pbl.size, rdev->lldi.vr->rq.start,
-            rdev->lldi.vr->rq.size,
-            rdev->lldi.vr->qp.start,
-            rdev->lldi.vr->qp.size,
-            rdev->lldi.vr->cq.start,
-            rdev->lldi.vr->cq.size);
-       PDBG("udb %pR db_reg %p gts_reg %p "
-            "qpmask 0x%x cqmask 0x%x\n",
-               &rdev->lldi.pdev->resource[2],
-            rdev->lldi.db_reg, rdev->lldi.gts_reg,
-            rdev->qpmask, rdev->cqmask);
+       pr_debug("%s dev %s stag start 0x%0x size 0x%0x num stags %d pbl start 0x%0x size 0x%0x rq start 0x%0x size 0x%0x qp qid start %u size %u cq qid start %u size %u\n",
+                __func__, pci_name(rdev->lldi.pdev), rdev->lldi.vr->stag.start,
+                rdev->lldi.vr->stag.size, c4iw_num_stags(rdev),
+                rdev->lldi.vr->pbl.start,
+                rdev->lldi.vr->pbl.size, rdev->lldi.vr->rq.start,
+                rdev->lldi.vr->rq.size,
+                rdev->lldi.vr->qp.start,
+                rdev->lldi.vr->qp.size,
+                rdev->lldi.vr->cq.start,
+                rdev->lldi.vr->cq.size);
+       pr_debug("udb %pR db_reg %p gts_reg %p qpmask 0x%x cqmask 0x%x\n",
+                &rdev->lldi.pdev->resource[2],
+                rdev->lldi.db_reg, rdev->lldi.gts_reg,
+                rdev->qpmask, rdev->cqmask);
 
        if (c4iw_num_stags(rdev) == 0)
                return -EINVAL;
 
 static void c4iw_remove(struct uld_ctx *ctx)
 {
-       PDBG("%s c4iw_dev %p\n", __func__,  ctx->dev);
+       pr_debug("%s c4iw_dev %p\n", __func__,  ctx->dev);
        c4iw_unregister_device(ctx->dev);
        c4iw_dealloc(ctx);
 }
        devp->rdev.lldi = *infop;
 
        /* init various hw-queue params based on lld info */
-       PDBG("%s: Ing. padding boundary is %d, egrsstatuspagesize = %d\n",
-            __func__, devp->rdev.lldi.sge_ingpadboundary,
-            devp->rdev.lldi.sge_egrstatuspagesize);
+       pr_debug("%s: Ing. padding boundary is %d, egrsstatuspagesize = %d\n",
+                __func__, devp->rdev.lldi.sge_ingpadboundary,
+                devp->rdev.lldi.sge_egrstatuspagesize);
 
        devp->rdev.hw_queue.t4_eq_status_entries =
                devp->rdev.lldi.sge_ingpadboundary > 64 ? 2 : 1;
                }
        }
 
-       PDBG(KERN_INFO MOD "ocq memory: "
-              "hw_start 0x%x size %u mw_pa 0x%lx mw_kva %p\n",
-              devp->rdev.lldi.vr->ocq.start, devp->rdev.lldi.vr->ocq.size,
-              devp->rdev.oc_mw_pa, devp->rdev.oc_mw_kva);
+       pr_debug("ocq memory: hw_start 0x%x size %u mw_pa 0x%lx mw_kva %p\n",
+                devp->rdev.lldi.vr->ocq.start, devp->rdev.lldi.vr->ocq.size,
+                devp->rdev.oc_mw_pa, devp->rdev.oc_mw_kva);
 
        ret = c4iw_rdev_open(&devp->rdev);
        if (ret) {
        }
        ctx->lldi = *infop;
 
-       PDBG("%s found device %s nchan %u nrxq %u ntxq %u nports %u\n",
-            __func__, pci_name(ctx->lldi.pdev),
-            ctx->lldi.nchan, ctx->lldi.nrxq,
-            ctx->lldi.ntxq, ctx->lldi.nports);
+       pr_debug("%s found device %s nchan %u nrxq %u ntxq %u nports %u\n",
+                __func__, pci_name(ctx->lldi.pdev),
+                ctx->lldi.nchan, ctx->lldi.nrxq,
+                ctx->lldi.ntxq, ctx->lldi.nports);
 
        mutex_lock(&dev_mutex);
        list_add_tail(&ctx->entry, &uld_ctx_list);
        mutex_unlock(&dev_mutex);
 
        for (i = 0; i < ctx->lldi.nrxq; i++)
-               PDBG("rxqid[%u] %u\n", i, ctx->lldi.rxq_ids[i]);
+               pr_debug("rxqid[%u] %u\n", i, ctx->lldi.rxq_ids[i]);
 out:
        return ctx;
 }
 {
        struct uld_ctx *ctx = handle;
 
-       PDBG("%s new_state %u\n", __func__, new_state);
+       pr_debug("%s new_state %u\n", __func__, new_state);
        switch (new_state) {
        case CXGB4_STATE_UP:
                pr_info("%s: Up\n", pci_name(ctx->lldi.pdev));
 
                        "%s cxgb4_read_tpte err %d\n", __func__, ret);
                return;
        }
-       PDBG("stag idx 0x%x valid %d key 0x%x state %d pdid %d "
-              "perm 0x%x ps %d len 0x%llx va 0x%llx\n",
-              stag & 0xffffff00,
-              FW_RI_TPTE_VALID_G(ntohl(tpte.valid_to_pdid)),
-              FW_RI_TPTE_STAGKEY_G(ntohl(tpte.valid_to_pdid)),
-              FW_RI_TPTE_STAGSTATE_G(ntohl(tpte.valid_to_pdid)),
-              FW_RI_TPTE_PDID_G(ntohl(tpte.valid_to_pdid)),
-              FW_RI_TPTE_PERM_G(ntohl(tpte.locread_to_qpid)),
-              FW_RI_TPTE_PS_G(ntohl(tpte.locread_to_qpid)),
-              ((u64)ntohl(tpte.len_hi) << 32) | ntohl(tpte.len_lo),
-              ((u64)ntohl(tpte.va_hi) << 32) | ntohl(tpte.va_lo_fbo));
+       pr_debug("stag idx 0x%x valid %d key 0x%x state %d pdid %d perm 0x%x ps %d len 0x%llx va 0x%llx\n",
+                stag & 0xffffff00,
+                FW_RI_TPTE_VALID_G(ntohl(tpte.valid_to_pdid)),
+                FW_RI_TPTE_STAGKEY_G(ntohl(tpte.valid_to_pdid)),
+                FW_RI_TPTE_STAGSTATE_G(ntohl(tpte.valid_to_pdid)),
+                FW_RI_TPTE_PDID_G(ntohl(tpte.valid_to_pdid)),
+                FW_RI_TPTE_PERM_G(ntohl(tpte.locread_to_qpid)),
+                FW_RI_TPTE_PS_G(ntohl(tpte.locread_to_qpid)),
+                ((u64)ntohl(tpte.len_hi) << 32) | ntohl(tpte.len_lo),
+                ((u64)ntohl(tpte.va_hi) << 32) | ntohl(tpte.va_lo_fbo));
 }
 
 static void dump_err_cqe(struct c4iw_dev *dev, struct t4_cqe *err_cqe)
                CQE_STATUS(err_cqe), CQE_TYPE(err_cqe), ntohl(err_cqe->len),
                CQE_WRID_HI(err_cqe), CQE_WRID_LOW(err_cqe));
 
-       PDBG("%016llx %016llx %016llx %016llx\n",
-            be64_to_cpu(p[0]), be64_to_cpu(p[1]), be64_to_cpu(p[2]),
-            be64_to_cpu(p[3]));
+       pr_debug("%016llx %016llx %016llx %016llx\n",
+                be64_to_cpu(p[0]), be64_to_cpu(p[1]), be64_to_cpu(p[2]),
+                be64_to_cpu(p[3]));
 
        /*
         * Ingress WRITE and READ_RESP errors provide
                if (atomic_dec_and_test(&chp->refcnt))
                        wake_up(&chp->wait);
        } else {
-               PDBG("%s unknown cqid 0x%x\n", __func__, qid);
+               pr_debug("%s unknown cqid 0x%x\n", __func__, qid);
                spin_unlock_irqrestore(&dev->lock, flag);
        }
        return 0;
 
 
 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
-extern int c4iw_debug;
-#define PDBG(fmt, args...) \
-do { \
-       if (c4iw_debug) \
-               printk(MOD fmt, ## args); \
-} while (0)
-
 #include "t4.h"
 
 #define PBL_OFF(rdev_p, a) ((a) - (rdev_p)->lldi.vr->pbl.start)
 
        ret = wait_for_completion_timeout(&wr_waitp->completion, C4IW_WR_TO);
        if (!ret) {
-               PDBG("%s - Device %s not responding (disabling device) - tid %u qpid %u\n",
-                    func, pci_name(rdev->lldi.pdev), hwtid, qpid);
+               pr_debug("%s - Device %s not responding (disabling device) - tid %u qpid %u\n",
+                        func, pci_name(rdev->lldi.pdev), hwtid, qpid);
                rdev->flags |= T4_FATAL_ERROR;
                wr_waitp->ret = -EIO;
        }
 out:
        if (wr_waitp->ret)
-               PDBG("%s: FW reply %d tid %u qpid %u\n",
-                    pci_name(rdev->lldi.pdev), wr_waitp->ret, hwtid, qpid);
+               pr_debug("%s: FW reply %d tid %u qpid %u\n",
+                        pci_name(rdev->lldi.pdev), wr_waitp->ret, hwtid, qpid);
        return wr_waitp->ret;
 }
 
                if (mm->key == key && mm->len == len) {
                        list_del_init(&mm->entry);
                        spin_unlock(&ucontext->mmap_lock);
-                       PDBG("%s key 0x%x addr 0x%llx len %d\n", __func__,
-                            key, (unsigned long long) mm->addr, mm->len);
+                       pr_debug("%s key 0x%x addr 0x%llx len %d\n",
+                                __func__, key,
+                                (unsigned long long)mm->addr, mm->len);
                        return mm;
                }
        }
                               struct c4iw_mm_entry *mm)
 {
        spin_lock(&ucontext->mmap_lock);
-       PDBG("%s key 0x%x addr 0x%llx len %d\n", __func__,
-            mm->key, (unsigned long long) mm->addr, mm->len);
+       pr_debug("%s key 0x%x addr 0x%llx len %d\n",
+                __func__, mm->key, (unsigned long long)mm->addr, mm->len);
        list_add_tail(&mm->entry, &ucontext->mmaps);
        spin_unlock(&ucontext->mmap_lock);
 }
 #define MPA_V2_RDMA_READ_RTR            0x4000
 #define MPA_V2_IRD_ORD_MASK             0x3FFF
 
-#define c4iw_put_ep(ep) { \
-       PDBG("put_ep (via %s:%u) ep %p refcnt %d\n", __func__, __LINE__,  \
-            ep, kref_read(&((ep)->kref))); \
-       WARN_ON(kref_read(&((ep)->kref)) < 1); \
-       kref_put(&((ep)->kref), _c4iw_free_ep); \
+#define c4iw_put_ep(ep) {                                              \
+       pr_debug("put_ep (via %s:%u) ep %p refcnt %d\n",                \
+                __func__, __LINE__,                                    \
+                ep, kref_read(&((ep)->kref)));                         \
+       WARN_ON(kref_read(&((ep)->kref)) < 1);                          \
+       kref_put(&((ep)->kref), _c4iw_free_ep);                         \
 }
 
-#define c4iw_get_ep(ep) { \
-       PDBG("get_ep (via %s:%u) ep %p, refcnt %d\n", __func__, __LINE__, \
-            ep, kref_read(&((ep)->kref))); \
-       kref_get(&((ep)->kref));  \
+#define c4iw_get_ep(ep) {                                              \
+       pr_debug("get_ep (via %s:%u) ep %p, refcnt %d\n",               \
+                __func__, __LINE__,                                    \
+                ep, kref_read(&((ep)->kref)));                         \
+       kref_get(&((ep)->kref));                                        \
 }
 void _c4iw_free_ep(struct kref *kref);
 
 
                cmd |= cpu_to_be32(T5_ULP_MEMIO_IMM_F);
 
        addr &= 0x7FFFFFF;
-       PDBG("%s addr 0x%x len %u\n", __func__, addr, len);
+       pr_debug("%s addr 0x%x len %u\n", __func__, addr, len);
        num_wqe = DIV_ROUND_UP(len, C4IW_MAX_INLINE_SIZE);
        c4iw_init_wr_wait(&wr_wait);
        for (i = 0; i < num_wqe; i++) {
                mutex_unlock(&rdev->stats.lock);
                *stag = (stag_idx << 8) | (atomic_inc_return(&key) & 0xff);
        }
-       PDBG("%s stag_state 0x%0x type 0x%0x pdid 0x%0x, stag_idx 0x%x\n",
-            __func__, stag_state, type, pdid, stag_idx);
+       pr_debug("%s stag_state 0x%0x type 0x%0x pdid 0x%0x, stag_idx 0x%x\n",
+                __func__, stag_state, type, pdid, stag_idx);
 
        /* write TPT entry */
        if (reset_tpt_entry)
 {
        int err;
 
-       PDBG("%s *pdb_addr 0x%x, pbl_base 0x%x, pbl_size %d\n",
-            __func__, pbl_addr, rdev->lldi.vr->pbl.start,
-            pbl_size);
+       pr_debug("%s *pdb_addr 0x%x, pbl_base 0x%x, pbl_size %d\n",
+                __func__, pbl_addr, rdev->lldi.vr->pbl.start,
+                pbl_size);
 
        err = write_adapter_mem(rdev, pbl_addr >> 5, pbl_size << 3, pbl, NULL);
        return err;
        mhp->attr.stag = stag;
        mmid = stag >> 8;
        mhp->ibmr.rkey = mhp->ibmr.lkey = stag;
-       PDBG("%s mmid 0x%x mhp %p\n", __func__, mmid, mhp);
+       pr_debug("%s mmid 0x%x mhp %p\n", __func__, mmid, mhp);
        return insert_handle(mhp->rhp, &mhp->rhp->mmidr, mhp, mmid);
 }
 
        int ret;
        u32 stag = T4_STAG_UNSET;
 
-       PDBG("%s ib_pd %p\n", __func__, pd);
+       pr_debug("%s ib_pd %p\n", __func__, pd);
        php = to_c4iw_pd(pd);
        rhp = php->rhp;
 
        struct c4iw_pd *php;
        struct c4iw_mr *mhp;
 
-       PDBG("%s ib_pd %p\n", __func__, pd);
+       pr_debug("%s ib_pd %p\n", __func__, pd);
 
        if (length == ~0ULL)
                return ERR_PTR(-EINVAL);
                ret = -ENOMEM;
                goto dealloc_win;
        }
-       PDBG("%s mmid 0x%x mhp %p stag 0x%x\n", __func__, mmid, mhp, stag);
+       pr_debug("%s mmid 0x%x mhp %p stag 0x%x\n", __func__, mmid, mhp, stag);
        return &(mhp->ibmw);
 
 dealloc_win:
        deallocate_window(&rhp->rdev, mhp->attr.stag, mhp->dereg_skb);
        kfree_skb(mhp->dereg_skb);
        kfree(mhp);
-       PDBG("%s ib_mw %p mmid 0x%x ptr %p\n", __func__, mw, mmid, mhp);
+       pr_debug("%s ib_mw %p mmid 0x%x ptr %p\n", __func__, mw, mmid, mhp);
        return 0;
 }
 
                goto err3;
        }
 
-       PDBG("%s mmid 0x%x mhp %p stag 0x%x\n", __func__, mmid, mhp, stag);
+       pr_debug("%s mmid 0x%x mhp %p stag 0x%x\n", __func__, mmid, mhp, stag);
        return &(mhp->ibmr);
 err3:
        dereg_mem(&rhp->rdev, stag, mhp->attr.pbl_size,
        struct c4iw_mr *mhp;
        u32 mmid;
 
-       PDBG("%s ib_mr %p\n", __func__, ib_mr);
+       pr_debug("%s ib_mr %p\n", __func__, ib_mr);
 
        mhp = to_c4iw_mr(ib_mr);
        rhp = mhp->rhp;
                kfree((void *) (unsigned long) mhp->kva);
        if (mhp->umem)
                ib_umem_release(mhp->umem);
-       PDBG("%s mmid 0x%x ptr %p\n", __func__, mmid, mhp);
+       pr_debug("%s mmid 0x%x ptr %p\n", __func__, mmid, mhp);
        kfree(mhp);
        return 0;
 }
 
        ucontext = container_of(kref, struct c4iw_ucontext, kref);
        rhp = to_c4iw_dev(ucontext->ibucontext.device);
 
-       PDBG("%s ucontext %p\n", __func__, ucontext);
+       pr_debug("%s ucontext %p\n", __func__, ucontext);
        list_for_each_entry_safe(mm, tmp, &ucontext->mmaps, entry)
                kfree(mm);
        c4iw_release_dev_ucontext(&rhp->rdev, &ucontext->uctx);
 {
        struct c4iw_ucontext *ucontext = to_c4iw_ucontext(context);
 
-       PDBG("%s context %p\n", __func__, context);
+       pr_debug("%s context %p\n", __func__, context);
        c4iw_put_ucontext(ucontext);
        return 0;
 }
        int ret = 0;
        struct c4iw_mm_entry *mm = NULL;
 
-       PDBG("%s ibdev %p\n", __func__, ibdev);
+       pr_debug("%s ibdev %p\n", __func__, ibdev);
        context = kzalloc(sizeof(*context), GFP_KERNEL);
        if (!context) {
                ret = -ENOMEM;
        struct c4iw_ucontext *ucontext;
        u64 addr;
 
-       PDBG("%s pgoff 0x%lx key 0x%x len %d\n", __func__, vma->vm_pgoff,
-            key, len);
+       pr_debug("%s pgoff 0x%lx key 0x%x len %d\n", __func__, vma->vm_pgoff,
+                key, len);
 
        if (vma->vm_start & (PAGE_SIZE-1))
                return -EINVAL;
 
        php = to_c4iw_pd(pd);
        rhp = php->rhp;
-       PDBG("%s ibpd %p pdid 0x%x\n", __func__, pd, php->pdid);
+       pr_debug("%s ibpd %p pdid 0x%x\n", __func__, pd, php->pdid);
        c4iw_put_resource(&rhp->rdev.resource.pdid_table, php->pdid);
        mutex_lock(&rhp->rdev.stats.lock);
        rhp->rdev.stats.pd.cur--;
        u32 pdid;
        struct c4iw_dev *rhp;
 
-       PDBG("%s ibdev %p\n", __func__, ibdev);
+       pr_debug("%s ibdev %p\n", __func__, ibdev);
        rhp = (struct c4iw_dev *) ibdev;
        pdid =  c4iw_get_resource(&rhp->rdev.resource.pdid_table);
        if (!pdid)
        if (rhp->rdev.stats.pd.cur > rhp->rdev.stats.pd.max)
                rhp->rdev.stats.pd.max = rhp->rdev.stats.pd.cur;
        mutex_unlock(&rhp->rdev.stats.lock);
-       PDBG("%s pdid 0x%0x ptr 0x%p\n", __func__, pdid, php);
+       pr_debug("%s pdid 0x%0x ptr 0x%p\n", __func__, pdid, php);
        return &php->ibpd;
 }
 
 static int c4iw_query_pkey(struct ib_device *ibdev, u8 port, u16 index,
                           u16 *pkey)
 {
-       PDBG("%s ibdev %p\n", __func__, ibdev);
+       pr_debug("%s ibdev %p\n", __func__, ibdev);
        *pkey = 0;
        return 0;
 }
 {
        struct c4iw_dev *dev;
 
-       PDBG("%s ibdev %p, port %d, index %d, gid %p\n",
-              __func__, ibdev, port, index, gid);
+       pr_debug("%s ibdev %p, port %d, index %d, gid %p\n",
+                __func__, ibdev, port, index, gid);
        dev = to_c4iw_dev(ibdev);
        BUG_ON(port == 0);
        memset(&(gid->raw[0]), 0, sizeof(gid->raw));
 
        struct c4iw_dev *dev;
 
-       PDBG("%s ibdev %p\n", __func__, ibdev);
+       pr_debug("%s ibdev %p\n", __func__, ibdev);
 
        if (uhw->inlen || uhw->outlen)
                return -EINVAL;
        struct net_device *netdev;
        struct in_device *inetdev;
 
-       PDBG("%s ibdev %p\n", __func__, ibdev);
+       pr_debug("%s ibdev %p\n", __func__, ibdev);
 
        dev = to_c4iw_dev(ibdev);
        netdev = dev->rdev.lldi.ports[port-1];
 {
        struct c4iw_dev *c4iw_dev = container_of(dev, struct c4iw_dev,
                                                 ibdev.dev);
-       PDBG("%s dev 0x%p\n", __func__, dev);
+       pr_debug("%s dev 0x%p\n", __func__, dev);
        return sprintf(buf, "%d\n",
                       CHELSIO_CHIP_RELEASE(c4iw_dev->rdev.lldi.adapter_type));
 }
        struct ethtool_drvinfo info;
        struct net_device *lldev = c4iw_dev->rdev.lldi.ports[0];
 
-       PDBG("%s dev 0x%p\n", __func__, dev);
+       pr_debug("%s dev 0x%p\n", __func__, dev);
        lldev->ethtool_ops->get_drvinfo(lldev, &info);
        return sprintf(buf, "%s\n", info.driver);
 }
 {
        struct c4iw_dev *c4iw_dev = container_of(dev, struct c4iw_dev,
                                                 ibdev.dev);
-       PDBG("%s dev 0x%p\n", __func__, dev);
+       pr_debug("%s dev 0x%p\n", __func__, dev);
        return sprintf(buf, "%x.%x\n", c4iw_dev->rdev.lldi.pdev->vendor,
                       c4iw_dev->rdev.lldi.pdev->device);
 }
 {
        struct c4iw_dev *c4iw_dev = container_of(dev, struct c4iw_dev,
                                                 ibdev);
-       PDBG("%s dev 0x%p\n", __func__, dev);
+       pr_debug("%s dev 0x%p\n", __func__, dev);
 
        snprintf(str, str_len, "%u.%u.%u.%u",
                 FW_HDR_FW_VER_MAJOR_G(c4iw_dev->rdev.lldi.fw_vers),
        int ret;
        int i;
 
-       PDBG("%s c4iw_dev %p\n", __func__, dev);
+       pr_debug("%s c4iw_dev %p\n", __func__, dev);
        BUG_ON(!dev->rdev.lldi.ports[0]);
        strlcpy(dev->ibdev.name, "cxgb4_%d", IB_DEVICE_NAME_MAX);
        memset(&dev->ibdev.node_guid, 0, sizeof(dev->ibdev.node_guid));
 {
        int i;
 
-       PDBG("%s c4iw_dev %p\n", __func__, dev);
+       pr_debug("%s c4iw_dev %p\n", __func__, dev);
        for (i = 0; i < ARRAY_SIZE(c4iw_class_attributes); ++i)
                device_remove_file(&dev->ibdev.dev,
                                   c4iw_class_attributes[i]);
 
                ret = -ENOMEM;
                goto free_sq;
        }
-       PDBG("%s sq base va 0x%p pa 0x%llx rq base va 0x%p pa 0x%llx\n",
-               __func__, wq->sq.queue,
-               (unsigned long long)virt_to_phys(wq->sq.queue),
-               wq->rq.queue,
-               (unsigned long long)virt_to_phys(wq->rq.queue));
+       pr_debug("%s sq base va 0x%p pa 0x%llx rq base va 0x%p pa 0x%llx\n",
+                __func__, wq->sq.queue,
+                (unsigned long long)virt_to_phys(wq->sq.queue),
+                wq->rq.queue,
+                (unsigned long long)virt_to_phys(wq->rq.queue));
        memset(wq->rq.queue, 0, wq->rq.memsize);
        dma_unmap_addr_set(&wq->rq, mapping, wq->rq.dma_addr);
 
        if (ret)
                goto free_dma;
 
-       PDBG("%s sqid 0x%x rqid 0x%x kdb 0x%p sq_bar2_addr %p rq_bar2_addr %p\n",
-            __func__, wq->sq.qid, wq->rq.qid, wq->db,
-            wq->sq.bar2_va, wq->rq.bar2_va);
+       pr_debug("%s sqid 0x%x rqid 0x%x kdb 0x%p sq_bar2_addr %p rq_bar2_addr %p\n",
+                __func__, wq->sq.qid, wq->rq.qid, wq->db,
+                wq->sq.bar2_va, wq->rq.bar2_va);
 
        return 0;
 free_dma:
        ucontext = qhp->ucontext;
        rhp = qhp->rhp;
 
-       PDBG("%s qhp %p ucontext %p\n", __func__, qhp, ucontext);
+       pr_debug("%s qhp %p ucontext %p\n", __func__, qhp, ucontext);
        destroy_qp(&rhp->rdev, &qhp->wq,
                   ucontext ? &ucontext->uctx : &rhp->rdev.uctx);
 
        struct c4iw_qp *qhp;
 
        qhp = container_of(kref, struct c4iw_qp, kref);
-       PDBG("%s qhp %p\n", __func__, qhp);
+       pr_debug("%s qhp %p\n", __func__, qhp);
        queue_work(qhp->rhp->rdev.free_workq, &qhp->free_work);
 }
 
 void c4iw_qp_add_ref(struct ib_qp *qp)
 {
-       PDBG("%s ib_qp %p\n", __func__, qp);
+       pr_debug("%s ib_qp %p\n", __func__, qp);
        kref_get(&to_c4iw_qp(qp)->kref);
 }
 
 void c4iw_qp_rem_ref(struct ib_qp *qp)
 {
-       PDBG("%s ib_qp %p\n", __func__, qp);
+       pr_debug("%s ib_qp %p\n", __func__, qp);
        kref_put(&to_c4iw_qp(qp)->kref, queue_qp_free);
 }
 
                        c4iw_invalidate_mr(qhp->rhp, wr->ex.invalidate_rkey);
                        break;
                default:
-                       PDBG("%s post of type=%d TBD!\n", __func__,
-                            wr->opcode);
+                       pr_debug("%s post of type=%d TBD!\n", __func__,
+                                wr->opcode);
                        err = -EINVAL;
                }
                if (err) {
 
                init_wr_hdr(wqe, qhp->wq.sq.pidx, fw_opcode, fw_flags, len16);
 
-               PDBG("%s cookie 0x%llx pidx 0x%x opcode 0x%x read_len %u\n",
-                    __func__, (unsigned long long)wr->wr_id, qhp->wq.sq.pidx,
-                    swsqe->opcode, swsqe->read_len);
+               pr_debug("%s cookie 0x%llx pidx 0x%x opcode 0x%x read_len %u\n",
+                        __func__,
+                        (unsigned long long)wr->wr_id, qhp->wq.sq.pidx,
+                        swsqe->opcode, swsqe->read_len);
                wr = wr->next;
                num_wrs--;
                t4_sq_produce(&qhp->wq, len16);
                wqe->recv.r2[1] = 0;
                wqe->recv.r2[2] = 0;
                wqe->recv.len16 = len16;
-               PDBG("%s cookie 0x%llx pidx %u\n", __func__,
-                    (unsigned long long) wr->wr_id, qhp->wq.rq.pidx);
+               pr_debug("%s cookie 0x%llx pidx %u\n",
+                        __func__,
+                        (unsigned long long)wr->wr_id, qhp->wq.rq.pidx);
                t4_rq_produce(&qhp->wq, len16);
                idx += DIV_ROUND_UP(len16*16, T4_EQ_ENTRY_SIZE);
                wr = wr->next;
        struct sk_buff *skb;
        struct terminate_message *term;
 
-       PDBG("%s qhp %p qid 0x%x tid %u\n", __func__, qhp, qhp->wq.sq.qid,
-            qhp->ep->hwtid);
+       pr_debug("%s qhp %p qid 0x%x tid %u\n", __func__, qhp, qhp->wq.sq.qid,
+                qhp->ep->hwtid);
 
        skb = skb_dequeue(&qhp->ep->com.ep_skb_list);
        if (WARN_ON(!skb))
        int rq_flushed, sq_flushed;
        unsigned long flag;
 
-       PDBG("%s qhp %p rchp %p schp %p\n", __func__, qhp, rchp, schp);
+       pr_debug("%s qhp %p rchp %p schp %p\n", __func__, qhp, rchp, schp);
 
        /* locking hierarchy: cq lock first, then qp lock. */
        spin_lock_irqsave(&rchp->lock, flag);
        int ret;
        struct sk_buff *skb;
 
-       PDBG("%s qhp %p qid 0x%x tid %u\n", __func__, qhp, qhp->wq.sq.qid,
-            ep->hwtid);
+       pr_debug("%s qhp %p qid 0x%x tid %u\n", __func__, qhp, qhp->wq.sq.qid,
+                ep->hwtid);
 
        skb = skb_dequeue(&ep->com.ep_skb_list);
        if (WARN_ON(!skb))
        ret = c4iw_wait_for_reply(&rhp->rdev, &ep->com.wr_wait, qhp->ep->hwtid,
                             qhp->wq.sq.qid, __func__);
 out:
-       PDBG("%s ret %d\n", __func__, ret);
+       pr_debug("%s ret %d\n", __func__, ret);
        return ret;
 }
 
 static void build_rtr_msg(u8 p2p_type, struct fw_ri_init *init)
 {
-       PDBG("%s p2p_type = %d\n", __func__, p2p_type);
+       pr_debug("%s p2p_type = %d\n", __func__, p2p_type);
        memset(&init->u, 0, sizeof init->u);
        switch (p2p_type) {
        case FW_RI_INIT_P2PTYPE_RDMA_WRITE:
        int ret;
        struct sk_buff *skb;
 
-       PDBG("%s qhp %p qid 0x%x tid %u ird %u ord %u\n", __func__, qhp,
-            qhp->wq.sq.qid, qhp->ep->hwtid, qhp->ep->ird, qhp->ep->ord);
+       pr_debug("%s qhp %p qid 0x%x tid %u ird %u ord %u\n", __func__, qhp,
+                qhp->wq.sq.qid, qhp->ep->hwtid, qhp->ep->ird, qhp->ep->ord);
 
        skb = alloc_skb(sizeof *wqe, GFP_KERNEL);
        if (!skb) {
 err1:
        free_ird(rhp, qhp->attr.max_ird);
 out:
-       PDBG("%s ret %d\n", __func__, ret);
+       pr_debug("%s ret %d\n", __func__, ret);
        return ret;
 }
 
        int free = 0;
        struct c4iw_ep *ep = NULL;
 
-       PDBG("%s qhp %p sqid 0x%x rqid 0x%x ep %p state %d -> %d\n", __func__,
-            qhp, qhp->wq.sq.qid, qhp->wq.rq.qid, qhp->ep, qhp->attr.state,
-            (mask & C4IW_QP_ATTR_NEXT_STATE) ? attrs->next_state : -1);
+       pr_debug("%s qhp %p sqid 0x%x rqid 0x%x ep %p state %d -> %d\n",
+                __func__,
+                qhp, qhp->wq.sq.qid, qhp->wq.rq.qid, qhp->ep, qhp->attr.state,
+                (mask & C4IW_QP_ATTR_NEXT_STATE) ? attrs->next_state : -1);
 
        mutex_lock(&qhp->mutex);
 
        }
        goto out;
 err:
-       PDBG("%s disassociating ep %p qpid 0x%x\n", __func__, qhp->ep,
-            qhp->wq.sq.qid);
+       pr_debug("%s disassociating ep %p qpid 0x%x\n", __func__, qhp->ep,
+                qhp->wq.sq.qid);
 
        /* disassociate the LLP connection */
        qhp->attr.llp_stream_handle = NULL;
         */
        if (free)
                c4iw_put_ep(&ep->com);
-       PDBG("%s exit state %d\n", __func__, qhp->attr.state);
+       pr_debug("%s exit state %d\n", __func__, qhp->attr.state);
        return ret;
 }
 
 
        c4iw_qp_rem_ref(ib_qp);
 
-       PDBG("%s ib_qp %p qpid 0x%0x\n", __func__, ib_qp, qhp->wq.sq.qid);
+       pr_debug("%s ib_qp %p qpid 0x%0x\n", __func__, ib_qp, qhp->wq.sq.qid);
        return 0;
 }
 
        struct c4iw_mm_entry *sq_key_mm, *rq_key_mm = NULL, *sq_db_key_mm;
        struct c4iw_mm_entry *rq_db_key_mm = NULL, *ma_sync_key_mm = NULL;
 
-       PDBG("%s ib_pd %p\n", __func__, pd);
+       pr_debug("%s ib_pd %p\n", __func__, pd);
 
        if (attrs->qp_type != IB_QPT_RC)
                return ERR_PTR(-EINVAL);
        qhp->ibqp.qp_num = qhp->wq.sq.qid;
        init_timer(&(qhp->timer));
        INIT_LIST_HEAD(&qhp->db_fc_entry);
-       PDBG("%s sq id %u size %u memsize %zu num_entries %u "
-            "rq id %u size %u memsize %zu num_entries %u\n", __func__,
-            qhp->wq.sq.qid, qhp->wq.sq.size, qhp->wq.sq.memsize,
-            attrs->cap.max_send_wr, qhp->wq.rq.qid, qhp->wq.rq.size,
-            qhp->wq.rq.memsize, attrs->cap.max_recv_wr);
+       pr_debug("%s sq id %u size %u memsize %zu num_entries %u rq id %u size %u memsize %zu num_entries %u\n",
+                __func__,
+                qhp->wq.sq.qid, qhp->wq.sq.size, qhp->wq.sq.memsize,
+                attrs->cap.max_send_wr, qhp->wq.rq.qid, qhp->wq.rq.size,
+                qhp->wq.rq.memsize, attrs->cap.max_recv_wr);
        return &qhp->ibqp;
 err8:
        kfree(ma_sync_key_mm);
        enum c4iw_qp_attr_mask mask = 0;
        struct c4iw_qp_attributes attrs;
 
-       PDBG("%s ib_qp %p\n", __func__, ibqp);
+       pr_debug("%s ib_qp %p\n", __func__, ibqp);
 
        /* iwarp does not support the RTR state */
        if ((attr_mask & IB_QP_STATE) && (attr->qp_state == IB_QPS_RTR))
 
 struct ib_qp *c4iw_get_qp(struct ib_device *dev, int qpn)
 {
-       PDBG("%s ib_dev %p qpn 0x%x\n", __func__, dev, qpn);
+       pr_debug("%s ib_dev %p qpn 0x%x\n", __func__, dev, qpn);
        return (struct ib_qp *)get_qhp(to_c4iw_dev(dev), qpn);
 }
 
 
 
 void c4iw_put_resource(struct c4iw_id_table *id_table, u32 entry)
 {
-       PDBG("%s entry 0x%x\n", __func__, entry);
+       pr_debug("%s entry 0x%x\n", __func__, entry);
        c4iw_id_free(id_table, entry);
 }
 
        }
 out:
        mutex_unlock(&uctx->lock);
-       PDBG("%s qid 0x%x\n", __func__, qid);
+       pr_debug("%s qid 0x%x\n", __func__, qid);
        mutex_lock(&rdev->stats.lock);
        if (rdev->stats.qid.cur > rdev->stats.qid.max)
                rdev->stats.qid.max = rdev->stats.qid.cur;
        entry = kmalloc(sizeof *entry, GFP_KERNEL);
        if (!entry)
                return;
-       PDBG("%s qid 0x%x\n", __func__, qid);
+       pr_debug("%s qid 0x%x\n", __func__, qid);
        entry->qid = qid;
        mutex_lock(&uctx->lock);
        list_add_tail(&entry->entry, &uctx->cqids);
        }
 out:
        mutex_unlock(&uctx->lock);
-       PDBG("%s qid 0x%x\n", __func__, qid);
+       pr_debug("%s qid 0x%x\n", __func__, qid);
        mutex_lock(&rdev->stats.lock);
        if (rdev->stats.qid.cur > rdev->stats.qid.max)
                rdev->stats.qid.max = rdev->stats.qid.cur;
        entry = kmalloc(sizeof *entry, GFP_KERNEL);
        if (!entry)
                return;
-       PDBG("%s qid 0x%x\n", __func__, qid);
+       pr_debug("%s qid 0x%x\n", __func__, qid);
        entry->qid = qid;
        mutex_lock(&uctx->lock);
        list_add_tail(&entry->entry, &uctx->qpids);
 u32 c4iw_pblpool_alloc(struct c4iw_rdev *rdev, int size)
 {
        unsigned long addr = gen_pool_alloc(rdev->pbl_pool, size);
-       PDBG("%s addr 0x%x size %d\n", __func__, (u32)addr, size);
+       pr_debug("%s addr 0x%x size %d\n", __func__, (u32)addr, size);
        mutex_lock(&rdev->stats.lock);
        if (addr) {
                rdev->stats.pbl.cur += roundup(size, 1 << MIN_PBL_SHIFT);
 
 void c4iw_pblpool_free(struct c4iw_rdev *rdev, u32 addr, int size)
 {
-       PDBG("%s addr 0x%x size %d\n", __func__, addr, size);
+       pr_debug("%s addr 0x%x size %d\n", __func__, addr, size);
        mutex_lock(&rdev->stats.lock);
        rdev->stats.pbl.cur -= roundup(size, 1 << MIN_PBL_SHIFT);
        mutex_unlock(&rdev->stats.lock);
        while (pbl_start < pbl_top) {
                pbl_chunk = min(pbl_top - pbl_start + 1, pbl_chunk);
                if (gen_pool_add(rdev->pbl_pool, pbl_start, pbl_chunk, -1)) {
-                       PDBG("%s failed to add PBL chunk (%x/%x)\n",
-                            __func__, pbl_start, pbl_chunk);
+                       pr_debug("%s failed to add PBL chunk (%x/%x)\n",
+                                __func__, pbl_start, pbl_chunk);
                        if (pbl_chunk <= 1024 << MIN_PBL_SHIFT) {
                                pr_warn("Failed to add all PBL chunks (%x/%x)\n",
                                        pbl_start, pbl_top - pbl_start);
                        }
                        pbl_chunk >>= 1;
                } else {
-                       PDBG("%s added PBL chunk (%x/%x)\n",
-                            __func__, pbl_start, pbl_chunk);
+                       pr_debug("%s added PBL chunk (%x/%x)\n",
+                                __func__, pbl_start, pbl_chunk);
                        pbl_start += pbl_chunk;
                }
        }
 u32 c4iw_rqtpool_alloc(struct c4iw_rdev *rdev, int size)
 {
        unsigned long addr = gen_pool_alloc(rdev->rqt_pool, size << 6);
-       PDBG("%s addr 0x%x size %d\n", __func__, (u32)addr, size << 6);
+       pr_debug("%s addr 0x%x size %d\n", __func__, (u32)addr, size << 6);
        if (!addr)
                pr_warn_ratelimited("%s: Out of RQT memory\n",
                                    pci_name(rdev->lldi.pdev));
 
 void c4iw_rqtpool_free(struct c4iw_rdev *rdev, u32 addr, int size)
 {
-       PDBG("%s addr 0x%x size %d\n", __func__, addr, size << 6);
+       pr_debug("%s addr 0x%x size %d\n", __func__, addr, size << 6);
        mutex_lock(&rdev->stats.lock);
        rdev->stats.rqt.cur -= roundup(size << 6, 1 << MIN_RQT_SHIFT);
        mutex_unlock(&rdev->stats.lock);
        while (rqt_start < rqt_top) {
                rqt_chunk = min(rqt_top - rqt_start + 1, rqt_chunk);
                if (gen_pool_add(rdev->rqt_pool, rqt_start, rqt_chunk, -1)) {
-                       PDBG("%s failed to add RQT chunk (%x/%x)\n",
-                            __func__, rqt_start, rqt_chunk);
+                       pr_debug("%s failed to add RQT chunk (%x/%x)\n",
+                                __func__, rqt_start, rqt_chunk);
                        if (rqt_chunk <= 1024 << MIN_RQT_SHIFT) {
                                pr_warn("Failed to add all RQT chunks (%x/%x)\n",
                                        rqt_start, rqt_top - rqt_start);
                        }
                        rqt_chunk >>= 1;
                } else {
-                       PDBG("%s added RQT chunk (%x/%x)\n",
-                            __func__, rqt_start, rqt_chunk);
+                       pr_debug("%s added RQT chunk (%x/%x)\n",
+                                __func__, rqt_start, rqt_chunk);
                        rqt_start += rqt_chunk;
                }
        }
 u32 c4iw_ocqp_pool_alloc(struct c4iw_rdev *rdev, int size)
 {
        unsigned long addr = gen_pool_alloc(rdev->ocqp_pool, size);
-       PDBG("%s addr 0x%x size %d\n", __func__, (u32)addr, size);
+       pr_debug("%s addr 0x%x size %d\n", __func__, (u32)addr, size);
        if (addr) {
                mutex_lock(&rdev->stats.lock);
                rdev->stats.ocqp.cur += roundup(size, 1 << MIN_OCQP_SHIFT);
 
 void c4iw_ocqp_pool_free(struct c4iw_rdev *rdev, u32 addr, int size)
 {
-       PDBG("%s addr 0x%x size %d\n", __func__, addr, size);
+       pr_debug("%s addr 0x%x size %d\n", __func__, addr, size);
        mutex_lock(&rdev->stats.lock);
        rdev->stats.ocqp.cur -= roundup(size, 1 << MIN_OCQP_SHIFT);
        mutex_unlock(&rdev->stats.lock);
        while (start < top) {
                chunk = min(top - start + 1, chunk);
                if (gen_pool_add(rdev->ocqp_pool, start, chunk, -1)) {
-                       PDBG("%s failed to add OCQP chunk (%x/%x)\n",
-                            __func__, start, chunk);
+                       pr_debug("%s failed to add OCQP chunk (%x/%x)\n",
+                                __func__, start, chunk);
                        if (chunk <= 1024 << MIN_OCQP_SHIFT) {
                                pr_warn("Failed to add all OCQP chunks (%x/%x)\n",
                                        start, top - start);
                        }
                        chunk >>= 1;
                } else {
-                       PDBG("%s added OCQP chunk (%x/%x)\n",
-                            __func__, start, chunk);
+                       pr_debug("%s added OCQP chunk (%x/%x)\n",
+                                __func__, start, chunk);
                        start += chunk;
                }
        }
 
        wmb();
        if (wq->sq.bar2_va) {
                if (inc == 1 && wq->sq.bar2_qid == 0 && wqe) {
-                       PDBG("%s: WC wq->sq.pidx = %d\n",
-                            __func__, wq->sq.pidx);
+                       pr_debug("%s: WC wq->sq.pidx = %d\n",
+                                __func__, wq->sq.pidx);
                        pio_copy((u64 __iomem *)
                                 (wq->sq.bar2_va + SGE_UDB_WCDOORBELL),
                                 (u64 *)wqe);
                } else {
-                       PDBG("%s: DB wq->sq.pidx = %d\n",
-                            __func__, wq->sq.pidx);
+                       pr_debug("%s: DB wq->sq.pidx = %d\n",
+                                __func__, wq->sq.pidx);
                        writel(PIDX_T5_V(inc) | QID_V(wq->sq.bar2_qid),
                               wq->sq.bar2_va + SGE_UDB_KDOORBELL);
                }
        wmb();
        if (wq->rq.bar2_va) {
                if (inc == 1 && wq->rq.bar2_qid == 0 && wqe) {
-                       PDBG("%s: WC wq->rq.pidx = %d\n",
-                            __func__, wq->rq.pidx);
+                       pr_debug("%s: WC wq->rq.pidx = %d\n",
+                                __func__, wq->rq.pidx);
                        pio_copy((u64 __iomem *)
                                 (wq->rq.bar2_va + SGE_UDB_WCDOORBELL),
                                 (void *)wqe);
                } else {
-                       PDBG("%s: DB wq->rq.pidx = %d\n",
-                            __func__, wq->rq.pidx);
+                       pr_debug("%s: DB wq->rq.pidx = %d\n",
+                                __func__, wq->rq.pidx);
                        writel(PIDX_T5_V(inc) | QID_V(wq->rq.bar2_qid),
                               wq->rq.bar2_va + SGE_UDB_KDOORBELL);
                }
 {
        cq->sw_in_use++;
        if (cq->sw_in_use == cq->size) {
-               PDBG("%s cxgb4 sw cq overflow cqid %u\n", __func__, cq->cqid);
+               pr_debug("%s cxgb4 sw cq overflow cqid %u\n",
+                        __func__, cq->cqid);
                cq->error = 1;
                BUG_ON(1);
        }
 static inline struct t4_cqe *t4_next_sw_cqe(struct t4_cq *cq)
 {
        if (cq->sw_in_use == cq->size) {
-               PDBG("%s cxgb4 sw cq overflow cqid %u\n", __func__, cq->cqid);
+               pr_debug("%s cxgb4 sw cq overflow cqid %u\n",
+                        __func__, cq->cqid);
                cq->error = 1;
                BUG_ON(1);
                return NULL;