Messages in this thread | | | From | Bart Van Assche <> | Date | Mon, 24 Jan 2011 20:42:36 +0100 | Subject | Re: [PATCH] infiniband: hw: amso1100: Fixed compile warning. |
| |
On Sun, Jan 23, 2011 at 5:30 PM, Ralf Thielow <ralf.thielow@googlemail.com> wrote: > > Fixed compile warning by use "unsigned long" instead of "u64" > on assign NULL to "c2_vq_req->reply_msg". > > Signed-off-by: Ralf Thielow <ralf.thielow@googlemail.com> > --- > drivers/infiniband/hw/amso1100/c2_vq.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/infiniband/hw/amso1100/c2_vq.c b/drivers/infiniband/hw/amso1100/c2_vq.c > index 9ce7819..807b730 100644 > --- a/drivers/infiniband/hw/amso1100/c2_vq.c > +++ b/drivers/infiniband/hw/amso1100/c2_vq.c > @@ -107,7 +107,7 @@ struct c2_vq_req *vq_req_alloc(struct c2_dev *c2dev) > r = kmalloc(sizeof(struct c2_vq_req), GFP_KERNEL); > if (r) { > init_waitqueue_head(&r->wait_object); > - r->reply_msg = (u64) NULL; > + r->reply_msg = (unsigned long) NULL; > r->event = 0; > r->cm_id = NULL; > r->qp = NULL; > @@ -123,7 +123,7 @@ struct c2_vq_req *vq_req_alloc(struct c2_dev *c2dev) > */ > void vq_req_free(struct c2_dev *c2dev, struct c2_vq_req *r) > { > - r->reply_msg = (u64) NULL; > + r->reply_msg = (unsigned long) NULL; > if (atomic_dec_and_test(&r->refcnt)) { > kfree(r); > } > @@ -151,7 +151,7 @@ void vq_req_get(struct c2_dev *c2dev, struct c2_vq_req *r) > void vq_req_put(struct c2_dev *c2dev, struct c2_vq_req *r) > { > if (atomic_dec_and_test(&r->refcnt)) { > - if (r->reply_msg != (u64) NULL) > + if (r->reply_msg != (unsigned long) NULL) > vq_repbuf_free(c2dev, > (void *) (unsigned long) r->reply_msg); > kfree(r);
Hello Ralf,
Have you considered using "0" instead of "(u64) NULL" or "(unsigned long) NULL" ?
Bart. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |