On Fri, Feb 14, 2025 at 10:46:31AM +0800, Jijie Shao wrote:
on 2025/2/14 3:59, Andrew Lunn wrote:Which is why i pointed you toward Gerhard. He found similar
On Thu, Feb 13, 2025 at 11:55:24AM +0800, Jijie Shao wrote:I actually knew about this, but after browsing the source code, I gave up using it.
This patch supports many self test: Mac, SerDes and Phy.Please take a look at the work Gerhard Engleder is doing, and try not
To implement self test, this patch implements a simple packet sending and
receiving function in the driver. By sending a packet in a specific format,
driver considers that the test is successful if the packet is received.
Otherwise, the test fails.
The SerDes hardware is on the BMC side, Therefore, when the SerDes loopback
need enabled, driver notifies the BMC through an event message.
Signed-off-by: Jijie Shao <shaojijie@xxxxxxxxxx>
to reinvent net/core/selftest.c
Andrew
I have an additional requirement: serdes loopback and mac loopback.
However, they are not supported in net/core/selftest.c.
limitations in the code, wanting to add in extra tests, same as you.
Two developers wanting to do that same things, suggests the core
should be extended to support that, not two different copies hidden
away in drivers.
Maybe my initial advice about not exporting the helpers was bad? I
don't know. Please chat with Gerhard and come up with a design that
makes the core usable for both your uses cases, and anybody else
wanting to embed similar self tests in their driver.
Andrew