Re: [PATCH] infiniband: hw: amso1100: Fixed compile warning.

From: Bart Van Assche
Date: Mon Jan 24 2011 - 14:43:14 EST


On Sun, Jan 23, 2011 at 5:30 PM, Ralf Thielow
<ralf.thielow@xxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxx>
> ---
>  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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/