Re: [PATCH 2/2] sd: Inline sd_probe_part2()

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

 



On Wed, 2019-03-20 at 21:14 +-0100, Greg Kroah-Hartman wrote:
+AD4 On Wed, Mar 20, 2019 at 01:09:20PM -0700, Bart Van Assche wrote:
+AD4 +AD4 This patch does not change any functionality.
+AD4 
+AD4 Then why do it?
+AD4 
+AD4 You need to say +AF8-why+AF8 a patch is needed in order for people to want to
+AD4 take it...

Hi Greg,

You are right of course. The reason I would like to inline that function is
because it was essential in the past to have that code as a separate function
but after patch 1/2 inlining that function makes sd+AF8-probe() easier to read.
Martin, do you want me to resubmit this patch series with an updated commit
message or do you rather prefer to update the commit message yourself?

Thanks,

Bart.



[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