Skip to contentSkip to conversation

scsi_host: fix comment

PATCH Applied Sebastian Herbszt 4 participants 4 messages Sep 20, 2014 – Sep 30, 2014 SCSI

lore
Share & tools
lore

Sebastian Herbszt

author
#0 lore raw

PATCH 1/1 scsi_host: fix comment

Commit 1abf635 (scsi: use 64-bit value for 'max_luns') changed the order of Scsi_Host members. Update the comment to reflect this.

Signed-off-by Sebastian Herbszt <[email protected]>
include/scsi/scsi_host.h +1 -1
index ba20347..d4e0509 100644
@@ -603,13 +603,13 @@ struct Scsi_Host {
 	unsigned long last_reset;
 
 
 	/*
 	 * These three parameters can be used to allow for wide scsi,
 	 * and for host adapters that support multiple busses
-	 * The first two should be set to 1 more than the actual max id
+	 * The last two should be set to 1 more than the actual max id
 	 * or lun (e.g. 8 for SCSI parallel systems).
 	 */
 	unsigned int max_channel;
 	unsigned int max_id;
 	u64 max_lun;
Reviewed-by on v1 1/1
#1 lore raw
Quoted 13 lines · Sebastian Herbszt

Commit 1abf635 (scsi: use 64-bit value for 'max_luns') changed the order of Scsi_Host members. Update the comment to reflect this.

Signed-off-by Sebastian Herbszt <[email protected]>
include/scsi/scsi_host.h +0 -0
index ba20347..d4e0509 100644
@@ -603,13 +603,13 @@ struct Scsi_Host {
  	unsigned long last_reset;

include/scsi/scsi_host.h struct Scsi_Host { · line 614

Quoted 10 lines
include/scsi/scsi_host.h +1 -1
  	/*
  	 * These three parameters can be used to allow for wide scsi,
  	 * and for host adapters that support multiple busses
-	 * The first two should be set to 1 more than the actual max id
+	 * The last two should be set to 1 more than the actual max id
  	 * or lun (e.g. 8 for SCSI parallel systems).
  	 */
  	unsigned int max_channel;
  	unsigned int max_id;
  	u64 max_lun;
Reviewed-by Bart Van Assche <[email protected]>
Reviewed-by
#2 lore raw
Quoted 15 lines · Bart Van Assche
Quoted 14 lines · Sebastian Herbszt

Commit 1abf635 (scsi: use 64-bit value for 'max_luns') changed the order
of Scsi_Host members. Update the comment to reflect this.

Signed-off-by Sebastian Herbszt <[email protected]>
include/scsi/scsi_host.h +0 -0
index ba20347..d4e0509 100644
@@ -603,13 +603,13 @@ struct Scsi_Host {
      unsigned long last_reset;
Quoted 12 lines

include/scsi/scsi_host.h struct Scsi_Host { · line 614

Quoted 10 lines
include/scsi/scsi_host.h +1 -1
      /*
       * These three parameters can be used to allow for wide scsi,
       * and for host adapters that support multiple busses
-     * The first two should be set to 1 more than the actual max id
+     * The last two should be set to 1 more than the actual max id
       * or lun (e.g. 8 for SCSI parallel systems).
       */
      unsigned int max_channel;
      unsigned int max_id;
      u64 max_lun;
Reviewed-by Bart Van Assche <[email protected]>
Reviewed-by Hannes Reinecke <[email protected]>

Cheers,

Hannes

Signature · 8 lines
Dr. Hannes Reinecke		      zSeries & Storage
[email protected]			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html