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.