On Tue, Jun 04, 2024 at 11:29:43AM +0000, si.yanteng@linux.dev wrote:

2024年5月30日 15:22, "Russell King (Oracle)" [off-list ref] 写到:

Hi, Russell, Serge,

quoted

On Thu, May 30, 2024 at 10:25:01AM +0800, Huacai Chen wrote:

quoted

Hi, Yanteng,

 

 The title should be "Fix ....." rather than "Fixed .....", and it is

I would avoid the ambiguous "Fix" which for stable folk imply that this

is a bug fix - but it isn't. It's adding support for requiring 1G

speeds to always be negotiated.

Oh, I get it now. Thanks!

quoted

I would like this patch to be held off until more thought can be put

into how to handle this without having a hack in the driver (stmmac

has too many hacks and we're going to have to start saying no to

these.)

Yeah, you have a point there, but I would also like to hear Serge's opinion.

I would be really glad not to have so many hacks in the STMMAC driver.
It would have been awesome if we are able to find a solution without
introducing one more quirk in the common driver code.

I started digging into this sometime ago, but failed to come up with
any decent hypothesis about the problem nature. One of the glimpse
idea was that the loongson_gnet_fix_speed() method code might be somehow
connected with the problem. But I didn't have much time to go further
than that.

Anyway I guess we'll need to have at least two more patchset
re-spins to settle all the found issues. Until then we can keep
discussing the problem Yanteng experience on his device. Russell, do
you have any suggestion of what info Yanteng should provide to better
understand the problem and get closer to it' cause?

-Serge(y)

Thanks,
Yanteng