On Sun, Dec 15, 2024 at 06:15:41PM +0100, Csókás Bence wrote: > 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)? What would you want to see if you were on my end? (hint, a resend of the correct things to apply, along with an explaination of what they are and what version they are and what has changed between versions...) thanks, greg k-h
