Re: [PATCH net-next 4/8] net: dsa: lan9645x: add basic dsa driver for LAN9645X

[Jens Emil Schulz Ostergaard]

On Fri, 2026-03-06 at 17:33 +0100, Andrew Lunn wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On Fri, Mar 06, 2026 at 04:03:36PM +0100, Jens Emil Schulz Ostergaard wrote:
> > On Thu, 2026-03-05 at 18:37 +0100, Andrew Lunn wrote:
> > > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> > > 
> > > > > However, this is a multi-giga switch, with high speed SERDES ports. My
> > > > > guess is, a SERDES port is used to connect to the host, not RGMII. So
> > > > > i would not implement this, until somebody reports having a board with
> > > > > RGMII connecting to the host. KISS.
> > > > 
> > > > We intend to do exactly that with this switch, just because the host
> > > > doesn't support SGMII, just RMII/RGMII.
> > > 
> > > :-)
> > > 
> > > Then i suggest using {rx|tx}-internal-delay-ps as Vladimir suggested.
> > > 
> > >      Andrew
> > 
> > OK, I will use {rx|tx}-internal-delay-ps instead. I can add that the
> > current EVB board for lan9645x also uses RGMI mac2mac to the host, so there
> > are already users who would appreciate this. This is the LAN96459 EDS2
> > Daughter Card with part number EV14H52A.
> 
> O.K. Do you plan to add a DT file for this? Since it is a daughter
> card, is it a DT overlay? I see there is an I2C EEPROM, used for board
> identification. So that tells you which overlay to load?
> 
>         Andrew

Yes, we would like to add a DT, and it would be an as an overlay. I believe
the EEPROM was intended for the firmware in unmanaged mode. We do not use
it for managed mode at the moment.

Thanks,
Emil