Re: [PATCH 2/2] scsi: qla2xxx: change some variables to hexadecimal string via %*phN format

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

 



[ - bunch of QLogic people from Cc list, I don't know why you put them
there, but I think my comments are not to bother people personally,
so, I left few and SCSI ML]

First of all, didn't you notice the
http://www.spinics.net/lists/linux-scsi/msg66798.html ?
I think there is subject is wrong a bit and this patch either should
be combined with that one, or goes after.
I'm sorry for this little mess, Oleksandr is newbie in the community,
I'm kinda a mentor to him.

Also, comment below.

On Wed, Jul 10, 2013 at 2:13 PM, Saurav Kashyap
<saurav.kashyap@xxxxxxxxxx> wrote:
> HI Oleksandr,
> Thanks for the patch. In the below patch couple of lines exceed 80 characters.

I think is quite okay to combine string literals to long lines since
checkpatch.pl recommends to do so.
What is your opinion?

--
With Best Regards,
Andy Shevchenko
--
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