Re: [PATCH 6.6 resubmit 0/3] Fix PPS channel routing

Hi,

On 2024. 12. 13. 12:41, Greg Kroah-Hartman wrote:

This series is really totally confusing. Here's what it looks like in my mbox:

1   C Dec 13 Csókás, Bence   (0.8K) [PATCH 6.6 resubmit 0/3] Fix PPS channel routing
2   C Dec 13 Csókás, Bence   (1.9K) ├─>[PATCH 6.11 v4 2/3] net: fec: refactor PPS channel configuration
3   C Dec 13 Csókás, Bence   (1.8K) ├─>[PATCH 6.11 v4 3/3] net: fec: make PPS channel configurable
4   C Dec 13 Csókás, Bence   (1.4K) ├─>[PATCH 6.11 v4 1/3] dt-bindings: net: fec: add pps channel property
5   C Dec 13 Csókás, Bence   (1.9K) ├─>[PATCH 6.6 resubmit 2/3] net: fec: refactor PPS channel configuration
6   C Dec 13 Csókás, Bence   (1.8K) ├─>[PATCH 6.6 resubmit 3/3] net: fec: make PPS channel configurable
7   C Dec 13 Csókás, Bence   (0.9K) ├─>[PATCH 6.11 v4 0/3] Fix PPS channel routing
8   C Dec 13 Csókás, Bence   (1.4K) └─>[PATCH 6.6 resubmit 1/3] dt-bindings: net: fec: add pps channel property

I see some 6.11 patches (which make no sense as 6.11 is long end-of-life)

Ah, sorry, it seems those were left in my maildir from a previous format-patch as I invoked send-email ./* ...

and a "resubmit?" for 6.6, but no explaination as to _why_ this is being resubmitted here, or in the patches themselves.

I submitted it to 6.6 once here, but it got rejected because it wasn't in 6.11.y and 6.12.y: Link: https://lore.kernel.org/netdev/2024120204-footer-riverbed-0daa@gregkh/

Since then, it got into 6.12.y, and - as you said - 6.11 got EOL, before it could ever get this patch. So I thought to resubmit it for 6.6, as that's the version that is of interest to us.

Two different branches in the same series is also really really hard for any type of tooling to tease apart, making this a manual effort on our side if we want to deal with them.

What would you do if you got a series that looked like this and had to handle it? Would you do what I'm doing now and ask, "What in the world is going on?" :)

Please be kind to those on the other side of your emails, make it blindingly obvious, as to what they need to do with them, AND make it simple for them to handle the patches.

Series is now deleted from my review queue, sorry.

greg k-h

Sorry for the confusion. So, should I submit the series yet again, without "resubmit" (and, obviously, without the left-over 6.11 patches)?

Bence