Re: [PATCH] scsi_host: fix comment

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/30/2014 01:13 PM, Bart Van Assche wrote:
> On 09/20/14 13:37, Sebastian Herbszt wrote:
>> 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 <herbszt@xxxxxx>
>>
>> diff --git a/include/scsi/scsi_host.h b/include/scsi/scsi_host.h
>> index ba20347..d4e0509 100644
>> --- a/include/scsi/scsi_host.h
>> +++ b/include/scsi/scsi_host.h
>> @@ -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: Bart Van Assche <bvanassche@xxxxxxx>
Reviewed-by: Hannes Reinecke <hare@xxxxxxx>

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare@xxxxxxx			      +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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux