Re: [PATCH 08/11] qla2xxx: Add framework for Async fabric discovery.

2017-01-13 Thread Madhani, Himanshu 


On 1/11/17, 12:08 PM, "Bart Van Assche"  wrote:

>On 12/23/2016 08:23 PM, Himanshu Madhani wrote:
>> +sess->logout_completed = 0;
>> +be_sid[0] = sess->d_id.b.domain;
>> +be_sid[1] = sess->d_id.b.area;
>> +be_sid[2] = sess->d_id.b.al_pa;
>
>Hello Himanshu and Quinn,
>
>When building the qlt_create_sess() code with W=1 the compiler warns
>that values are assigned to the elements of the be_sid[] array but that
>these values are not used. Please review the code.

Thanks for review. will review code and update patch.
 
>
>Thanks,
>
>Bart.
N�r��yb�X��ǧv�^�)޺{.n�+{���"�{ay�ʇڙ�,j��f���h���z��w���
���j:+v���w�j�mzZ+�ݢj"��!�i

Re: [PATCH 08/11] qla2xxx: Add framework for Async fabric discovery.

2017-01-11 Thread Bart Van Assche 
On 12/23/2016 08:23 PM, Himanshu Madhani wrote:
> + sess->logout_completed = 0;
> + be_sid[0] = sess->d_id.b.domain;
> + be_sid[1] = sess->d_id.b.area;
> + be_sid[2] = sess->d_id.b.al_pa;

Hello Himanshu and Quinn,

When building the qlt_create_sess() code with W=1 the compiler warns
that values are assigned to the elements of the be_sid[] array but that
these values are not used. Please review the code.

Thanks,

Bart.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html