[ofa-general] ***SPAM*** Re: [PATCH] : Define debugging variables only when CONFIG_INFINIBAND_NES_DEBUG is enabled

[Manish Katiyar]

On Tue, Jan 27, 2009 at 11:58 PM, Manish Katiyar <mkatiyar at gmail.com> wrote:
> Below patch removes following compilation warnings :
> drivers/infiniband/hw/nes/nes_cm.c:781: warning: unused variable 'tmp_addr'
> drivers/infiniband/hw/nes/nes_cm.c:820: warning: unused variable 'tmp_addr'
>

Hi,

Any feedback on this ?

Thanks -
manish

>
> Signed-off-by: Manish Katiyar <mkatiyar at gmail.com>
> ---
>  drivers/infiniband/hw/nes/nes_cm.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/infiniband/hw/nes/nes_cm.c
> b/drivers/infiniband/hw/nes/nes_cm.c
> index a01b448..2b34859 100644
> --- a/drivers/infiniband/hw/nes/nes_cm.c
> +++ b/drivers/infiniband/hw/nes/nes_cm.c
> @@ -778,7 +778,9 @@ static struct nes_cm_node *find_node(struct
> nes_cm_core *cm_core,
>        unsigned long flags;
>        struct list_head *hte;
>        struct nes_cm_node *cm_node;
> +#ifdef CONFIG_INFINIBAND_NES_DEBUG
>        __be32 tmp_addr = cpu_to_be32(loc_addr);
> +#endif
>
>        /* get a handle on the hte */
>        hte = &cm_core->connected_nodes;
> @@ -817,7 +819,9 @@ static struct nes_cm_listener
> *find_listener(struct nes_cm_core *cm_core,
>  {
>        unsigned long flags;
>        struct nes_cm_listener *listen_node;
> +#ifdef CONFIG_INFINIBAND_NES_DEBUG
>        __be32 tmp_addr = cpu_to_be32(dst_addr);
> +#endif
>
>        /* walk list and find cm_node associated with this session ID */
>        spin_lock_irqsave(&cm_core->listen_list_lock, flags);
> --
> 1.5.4.3
>
>
> Thanks -
> Manish
>