[PATCH v4.7 3/7] net/ncsi: Don't probe on the reserved channel ID (0x1f)

[Joel Stanley]

On Wed, 2016-09-28 at 12:48 +1000, Gavin Shan wrote:
> We needn't send CIS (Clear Initial State) command to the NCSI
> reserved channel (0x1f) in the enumeration. We shouldn't receive
> a valid response from CIS on NCSI channel 0x1f.
> 
> Signed-off-by: Gavin Shan <gwshan at linux.vnet.ibm.com>

Reviewed-by: Joel Stanley <joel at jms.id.au>

> ---
>  net/ncsi/ncsi-manage.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/net/ncsi/ncsi-manage.c b/net/ncsi/ncsi-manage.c
> index b03d1df..8011d51 100644
> --- a/net/ncsi/ncsi-manage.c
> +++ b/net/ncsi/ncsi-manage.c
> @@ -887,12 +887,12 @@ static void ncsi_probe_channel(struct
> ncsi_dev_priv *ndp)
>  		nd->state = ncsi_dev_state_probe_cis;
>  		break;
>  	case ncsi_dev_state_probe_cis:
> -		ndp->pending_req_num = 32;
> +		ndp->pending_req_num = NCSI_RESERVED_CHANNEL;
>  
>  		/* Clear initial state */
>  		nca.type = NCSI_PKT_CMD_CIS;
>  		nca.package = ndp->active_package->id;
> -		for (index = 0; index < 0x20; index++) {
> +		for (index = 0; index < NCSI_RESERVED_CHANNEL;
> index++) {
>  			nca.channel = index;
>  			ret = ncsi_xmit_cmd(&nca);
>  			if (ret)
>