From: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
To: ally heev <allyheev@gmail.com>
Cc: Krzysztof Kozlowski <krzk@kernel.org>,
Miri Korenblit <miriam.rachel.korenblit@intel.com>,
linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH RESEND iwlwifi-next] wifi: iwlwifi: mld: remove unused variable in d3.c
Date: Mon, 24 Nov 2025 11:28:28 -0500 [thread overview]
Message-ID: <20251124-encouraging-snobbish-nuthatch-230bcb@lemur> (raw)
In-Reply-To: <5d6e492ee9194b1125340a66fbd878279286e433.camel@gmail.com>
On Mon, Nov 24, 2025 at 09:28:39PM +0530, ally heev wrote:
> > > > Best regards,
> > > > -----BEGIN PGP SIGNATURE-----
> > > >
> > > > iHUEABYKAB0WIQQBFRpOLrIakF7DYvaWPaLUP9d7HAUCaRLyowAKCRCWPaLUP9d7
> > > > HLG9AQDZmggT93tMLv32pq5myG3e83t12KWON9efOBg//lH4vgD+Ppw3KoTpdYtJ
> > > > 20VDaWuqUVj9KLqjx9hblfEiB7s9gQw=
> > > > =A6F+
> > > > -----END PGP SIGNATURE-----
> > >
> > > You should switch to patatt instead, not signing emails. We don't use
> > > that, I think at all, unlike patatt which is being actively checked by
> > > toolkit (b4).
> >
> > It's even more curious. The patch was sent with b4, but I'm pretty sure we
> > wouldn't have put the PGP signature in like that, so I'm curious how that even
> > happened.
> >
> > Ally, can you shed some light on that?
> >
> > -K
>
> Yes, I use b4 for my patches. I work on other open source projects
> which use Github. So, my global git config commit autosign is set to
> `true`, hence you have seen PGP signatures.
Interesting! I've not seen this cause PGP signatures to be added to "git show"
output before. I've tried to replicate it locally, but even if I have a signed
git commit, I can't make the "gpgsig" header to show up in the output of "git
show --format=email" (which is what b4 uses). Do you have some local git
configuration changes that make "gpgsig" fields show up in the output of "git
show"?
-K
next prev parent reply other threads:[~2025-11-24 16:28 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-21 14:27 [PATCH RESEND iwlwifi-next] wifi: iwlwifi: mld: remove unused variable in d3.c Ally Heev
2025-11-22 14:18 ` Krzysztof Kozlowski
2025-11-24 15:38 ` Konstantin Ryabitsev
2025-11-24 15:58 ` ally heev
2025-11-24 16:28 ` Konstantin Ryabitsev [this message]
2025-11-25 7:20 ` ally heev
2025-11-25 13:54 ` Konstantin Ryabitsev
2025-11-25 16:00 ` ally heev
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20251124-encouraging-snobbish-nuthatch-230bcb@lemur \
--to=konstantin@linuxfoundation.org \
--cc=allyheev@gmail.com \
--cc=krzk@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=miriam.rachel.korenblit@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox