[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250317185535.7d00444d@jic23-huawei>
Date: Mon, 17 Mar 2025 18:55:35 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Sam Winchenbach <sam.winchenbach@...mepointer.org>
Cc: Krzysztof Kozlowski <krzk@...nel.org>, linux-kernel@...r.kernel.org,
lars@...afoo.de, Michael.Hennerich@...log.com, antoniu.miclaus@...log.com,
robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
bpellegrino@...a.org, Sam Winchenbach <swinchenbach@...a.org>
Subject: Re: [PATCH v7 1/6] dt-bindings: iio: filter: Add lpf/hpf freq
margins
On Mon, 17 Mar 2025 11:07:32 -0400
Sam Winchenbach <sam.winchenbach@...mepointer.org> wrote:
> On Sun, Mar 16, 2025 at 05:38:42PM +0100, Krzysztof Kozlowski wrote:
> > On Sun, Mar 16, 2025 at 09:50:03AM -0400, Sam Winchenbach wrote:
> > > From: Sam Winchenbach <swinchenbach@...a.org>
> > >
> > > Adds two properties to add a margin when automatically finding the
> > > corner frequencies.
> > >
> > > Signed-off-by: Sam Winchenbach <swinchenbach@...a.org>
> > > ---
> > > .../bindings/iio/filter/adi,admv8818.yaml | 20 +++++++++++++++++++
> > > 1 file changed, 20 insertions(+)
> >
> > I don't understand. You got my tag. No changelog here, no cover letter,
> > nothing explains what happened here and why the tag is being removed.
> >
>
> Apologies,
>
> I am still quite new to this workflow, and it deviates significantly
> from my day-to-day work. I mentioned in the previous patch set that I
> would like to update my email address and change:
> "driver core: -> iio: core:"
> I wasn't aware more than that was needed. Sorry for any confusion
> this may have caused.
>
> In the future what is the preferred way to handle a
> situation like this? I wasn't aware of the cover letter feature but
> that looks like a promising option.
Either add stuff below the --- above as that doesn't end up in the
eventual git log, or --cover-letter on your git-format-patch and
put useful things like that in there.
>
> It looks like another option is to add commentary to each patch.
>
> I am less certain about your tag being removed - I don't fully
> understand that. Is there a way to preserve that if changes are made
> after you sign-off?
Once a tag is given it is up to the patch author to add it to the
patches for future versions. That should only be dropped if the
author thinks there are significant enough changes to warrant a fresh
review. If you do drop a tag like that, then the change log
under --- on the particular patch should clearly state why.
>
> Sorry again about the confusion this caused,
No problem, takes a while for everyone to get used to a different
process. There are extensive docs, but it is still easy to miss
things!
Jonathan
> -Sam
>
> > Best regards,
> > Krzysztof
> >
Powered by blists - more mailing lists