Re: [PATCH 0/1] iio: accel: kionix-kx022a: Timestamp minor fix

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

 



On Sat, 18 Feb 2023 14:32:46 +0100
Mehdi Djait <mehdi.djait.k@xxxxxxxxx> wrote:

> Hi Matti,
> 
> On Fri, Feb 17, 2023 at 10:04:30PM +0200, Matti Vaittinen wrote:
> > Thanks again Mehdi. It's great to have this fixed :)
> > 
> > On 2/17/23 21:19, Mehdi Djait wrote:  
> > > As discussed in [1] here is a patch to get the timestamp in the
> > > trigger handler from the driver's private data
> > > 
> > > [1] https://lore.kernel.org/linux-iio/Y+%2FLpcc0Wa2VGsGO@carbian/T/#mec923c14d1324ffabfa88dde225eeb227005b48f
> > >   
> > 
> > By the way, I don't think a cover-letter is required for single patches. You
> > can usually put the relevant information in the commit message.
> > Additionally, if there is some questions, suggestions or other relevant bits
> > you wish to say - but don't want to include in commit logs - you can usually
> > place them in the patch mail directly after the first
> > "---"
> > 
> > Eg:
> > 
> > <commit message>
> > <Signed-off-by and other tags>
> > 
> > ---
> > 
> > Here we can add a note / request if required.
> >   
> 
> Yes, that's why I sent a cover-letter, I wanted to include relevant bits
> but not in the commit message (a reference to the discussion). 
> 
> I should have sent the patch --in-reply-to the thread with the discussion 
> anyway. 

Don't do that.  Use a Link tag to the discussion instead.
Otherwise the patch series can get buried and lost.


> 
> Thank you for the help :) 
> 
> --
> Kind Regards
> Mehdi Djait




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux