Re: [PATCH] drivers: target: Fixed multiple typos in multiple files

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

 



Hi Muhammad,

> -	 * CID we do connection reinstatement.  Currently we dont not
> +	 * CID we do connection reinstatement.  Currently we don't not

"Currently we don't not" does not parse. Fixing misspelled words in
isolation is not necessarily an improvement. The criteria should be
whether the entire sentence or comment becomes easier to read.

> -		 * status, with the sense key set to ILLEGAL REQUEST,and the additonal
> +		 * status, with the sense key set to ILLEGAL REQUEST,and the additional

There's a space missing after "ILLEGAL REQUEST,".

> - * Called thru fc_lport_iterate().
> + * Called through fc_lport_iterate().

"thru" is perfectly valid and the author's choice.

-- 
Martin K. Petersen	Oracle Linux Engineering



[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