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

Gavin Shan gwshan at linux.vnet.ibm.com
Fri Oct 21 11:47:40 AEDT 2016


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 3d1d775..b728614 100644
--- a/net/ncsi/ncsi-manage.c
+++ b/net/ncsi/ncsi-manage.c
@@ -616,12 +616,12 @@ static void ncsi_dev_probe(struct ncsi_dev_priv *ndp)
 		nd->nd_state = ncsi_dev_state_probe_cis;
 		break;
 	case ncsi_dev_state_probe_cis:
-		atomic_set(&ndp->ndp_pending_reqs, 0x20);
+		atomic_set(&ndp->ndp_pending_reqs, NCSI_RESERVED_CHANNEL);
 
 		/* Clear initial state */
 		nca.nca_type = NCSI_PKT_CMD_CIS;
 		nca.nca_package = ndp->ndp_active_package->np_id;
-		for (index = 0; index < 0x20; index++) {
+		for (index = 0; index < NCSI_RESERVED_CHANNEL; index++) {
 			nca.nca_channel = index;
 			ret = ncsi_xmit_cmd(&nca);
 			if (ret)
-- 
2.1.0



More information about the openbmc mailing list