Thread (35 messages) 35 messages, 7 authors, 2020-07-15

Re: [PATCH v2 08/16] spi: davinci: Remove uninitialized_var() usage

From: Joe Perches <joe@perches.com>
Date: 2020-07-02 16:23:15
Also in: linux-block, linux-clk, linux-doc, linux-ide, linux-mm, linux-spi, linux-wireless, lkml

On Thu, 2020-07-02 at 08:42 -0700, Kees Cook wrote:
On Thu, Jul 02, 2020 at 04:23:35PM +0100, Mark Brown wrote:
quoted
On Thu, Jul 02, 2020 at 08:21:40AM -0700, Kees Cook wrote:
quoted
On Wed, Jul 01, 2020 at 09:39:20PM +0100, Mark Brown wrote:
quoted
Please copy maintainers on patches :(
Hi! Sorry about that; the CC list was giant, so I had opted for using
subsystem mailing lists where possible.
If you're going to err in a direction there I'd err in the direction of
CCing the people not the list - I only saw this since I was looking for
something else, I don't normally see stuff in the mailing list folder.
Yeah, I've gotten conflicting feedback on treewide changes:
- please CC me on only the one patch, I don't want to see everything else
- please CC me on the whole series, I want the full context for the change

I opted toward "CC me on this series", but then I get stuck when the CC
is giant. I think I may switch back to individual CCs for specific
patches, and point people to lore if they want greater context. (lore
didn't exist before...)
IMO:

For a patch series that spans multiple subsystems,
each patch should always CC any specific subsystem
maintainers..

A good trick would be to use the cover letter
message-id: and have each individual patch in the
series reference the cover letter id below the
--- line so any reviewer doesn't have to find the
in-reply-to: message id and then reference the
lore link.

Something like:

---

For complete series see: https://lore.kernel.org/r/<cover_letter_message_id>

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help