Re: [PATCH] net: phylink: Separating two unrelated definitions for improving code readability

[Russell King (Oracle)]

On Thu, Nov 21, 2024 at 02:15:48PM +0200, Vladimir Oltean wrote:
> On Thu, Nov 21, 2024 at 12:11:02PM +0000, Russell King (Oracle) wrote:
> > On Thu, Nov 21, 2024 at 01:52:30PM +0200, Vladimir Oltean wrote:
> > > I don't understand what's to defend about this, really.
> > 
> > It's not something I want to entertain right now. I have enough on my
> > plate without having patches like this to deal with. Maybe next year
> > I'll look at it, but not right now.
> 
> I can definitely understand the lack of time to deal with trivial
> matters, but I mean, it isn't as if ./scripts/get_maintainer.pl
> drivers/net/phy/phylink.c lists a single person...

Trivial patches have an impact beyond just reviewing the patch. They
can cause conflicts, causing work that's in progress to need extra
re-work.

I have the problems of in-band that I've been trying to address since
April. I have phylink EEE support that I've also been trying to move
forward. However, with everything that has happened this year (first,
a high priority work item, followed by holiday, followed by my eye
operations) I've only _just_ been able to get back to looking at these
issues... meanwhile I see that I'm now being asked for stuff about
stacked PHYs which is also going to impact phylink. Oh, and to top it
off, I've discovered that mainline is broken on my test platform
(IRQ crap) which I'm currently trying to debug what has been broken.
Meaning I'm not working on any phylink stuff right now because of
other people's breakage.

It's just been bit of crap after another after another.

Give me a sodding break.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!