diff options
author | Gavin Shan <gwshan@linux.vnet.ibm.com> | 2016-10-04 11:25:49 +1100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-10-04 02:11:50 -0400 |
commit | 55e02d0837fb4cf023832252847bfbff453603cc (patch) | |
tree | 6d62c2c125c6774130e8e09f67a9e98d2ea52080 /net/ncsi | |
parent | bc7e0f50aa6958676115bffc1e5e58703579e04b (diff) | |
download | lwn-55e02d0837fb4cf023832252847bfbff453603cc.tar.gz lwn-55e02d0837fb4cf023832252847bfbff453603cc.zip |
net/ncsi: Don't probe on the reserved channel ID (0x1f)
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@linux.vnet.ibm.com>
Reviewed-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ncsi')
-rw-r--r-- | net/ncsi/ncsi-manage.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ncsi/ncsi-manage.c b/net/ncsi/ncsi-manage.c index 97c99bee8b68..8c5e0160d578 100644 --- a/net/ncsi/ncsi-manage.c +++ b/net/ncsi/ncsi-manage.c @@ -911,12 +911,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) |