Hi Andrew,

Thank you for taking the time to review the patch.

-----Original Messages-----
From: "Andrew Lunn" <andrew@lunn.ch>
Send time:Thursday, 22/01/2026 21:27:34
To: 李志 <redacted>
Cc: "Bo Gan" <redacted>, "Russell King (Oracle)" <linux@armlinux.org.uk>, devicetree@vger.kernel.org, andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, netdev@vger.kernel.org, pabeni@redhat.com, mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, ningyu@eswincomputing.com, linmin@eswincomputing.com, pinkesh.vaghela@einfochips.com, weishangjuan@eswincomputing.com
Subject: Re: Re: [PATCH v1 1/2] dt-bindings: ethernet: eswin: add clock sampling control

quoted

We also intend
to upstream complete DTS files for EIC7700 so the bindings can be validated
against real hardware.

I suggest you prioritise this. ARM-SOC is pretty flexible for
accepting work in progress support for new SoCs. All you really need
is for the board to boot to a login prompt on a serial port using an
initramsfs. So the .dtsi and .dts file can be very slim, CPUs, memory
and UARTs.

Once you have that merged you can work on the other drivers, their
bindings, etc.

The eic7700.dtsi and eic7700-hifive-premier-p550.dts are already in
v6.19-rc6. As you said, it only includes CPUs, memory and UARTs right
now. It has been verified to be able to login prompt on a serial port
using an initramfs.
Besides, the reset driver for eic7700 has already been merged in v6.19-rc6.

Because the device drivers such as Ethernet rely on the clock driver,
our plan is that once the clock driver is applied, we will add other
device nodes such as Ethernet and eMMC in the next upstream patch
of the DTS. Is this reasonable?

Regards,
Lin Min