On Thu, 12 Sep 2024, "Murthy, Arun R" <arun.r.murthy@xxxxxxxxx> wrote: >> -----Original Message----- >> From: Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> >> Sent: Thursday, September 12, 2024 2:36 PM >> To: Murthy, Arun R <arun.r.murthy@xxxxxxxxx>; intel-xe@xxxxxxxxxxxxxxxxxxxxx; >> intel-gfx@xxxxxxxxxxxxxxxxxxxxx >> Cc: Murthy, Arun R <arun.r.murthy@xxxxxxxxx>; Srikanth V, NagaVenkata >> <nagavenkata.srikanth.v@xxxxxxxxx> >> Subject: Re: [PATCH 1/3] drm/i915/dp: use fsleep instead of usleep_rage for LT >> >> On Thu, 12 Sep 2024, Arun R Murthy <arun.r.murthy@xxxxxxxxx> wrote: >> > Aux RD Interval value depends on the value read from the dpcd register >> > which is updated from the sink device use flseep thereby we adhere to >> > the Documentation/timers/timers-howto.rst >> > >> > Signed-off-by: Srikanth V NagaVenkata >> > <nagavenkata.srikanth.v@xxxxxxxxx> >> > Signed-off-by: Arun R Murthy <arun.r.murthy@xxxxxxxxx> >> >> Why do all the patches have two Signed-off-by's? >> > Findings are from Srikanth so giving credits to him as well. Contrary to popular belief, Signed-off-by: actually has very little to do with credits and authorship, and everything to do with certifying that the work can be submitted and merged to the kernel [1]. What you're looking for is Co-developed-by: [2] in combination with S-o-b. BR, Jani. [1] https://docs.kernel.org/process/submitting-patches.html#sign-your-work-the-developer-s-certificate-of-origin [2] https://docs.kernel.org/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by -- Jani Nikula, Intel