On Thu, 2012-02-09 at 11:15 -0800, Chad Dupuis wrote: > @@ -882,7 +892,7 @@ qla2x00_sns_gpn_id(scsi_qla_host_t *vha, sw_info_t *list) > uint16_t i; > struct sns_cmd_pkt *sns_cmd; > > - for (i = 0; i < MAX_FIBRE_DEVICES; i++) { > + for (i = 0; i < ha->max_fibre_devices; i++) { > /* Issue GPN_ID */ > /* Prepare SNS command request. */ > sns_cmd = qla2x00_prep_sns_cmd(vha, GPN_ID_CMD, You missed adding - int rval; + int rval = QLA_SUCCESS; Here, which causes a compile warning: Since you did the above in every other case, I've just fixed it up. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html