diff options
| author | Gavin Shan <gwshan@linux.vnet.ibm.com> | 2016-10-03 20:25:49 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-10-04 02:11:50 -0400 |
| commit | 55e02d0837fb4cf023832252847bfbff453603cc (patch) | |
| tree | 6d62c2c125c6774130e8e09f67a9e98d2ea52080 /net/ncsi | |
| parent | bc7e0f50aa6958676115bffc1e5e58703579e04b (diff) | |
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) | |||
| 911 | nd->state = ncsi_dev_state_probe_cis; | 911 | nd->state = ncsi_dev_state_probe_cis; |
| 912 | break; | 912 | break; |
| 913 | case ncsi_dev_state_probe_cis: | 913 | case ncsi_dev_state_probe_cis: |
| 914 | ndp->pending_req_num = 32; | 914 | ndp->pending_req_num = NCSI_RESERVED_CHANNEL; |
| 915 | 915 | ||
| 916 | /* Clear initial state */ | 916 | /* Clear initial state */ |
| 917 | nca.type = NCSI_PKT_CMD_CIS; | 917 | nca.type = NCSI_PKT_CMD_CIS; |
| 918 | nca.package = ndp->active_package->id; | 918 | nca.package = ndp->active_package->id; |
| 919 | for (index = 0; index < 0x20; index++) { | 919 | for (index = 0; index < NCSI_RESERVED_CHANNEL; index++) { |
| 920 | nca.channel = index; | 920 | nca.channel = index; |
| 921 | ret = ncsi_xmit_cmd(&nca); | 921 | ret = ncsi_xmit_cmd(&nca); |
| 922 | if (ret) | 922 | if (ret) |
