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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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





[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux