> On 14/08/2024 01:09, Tristram.Ha@xxxxxxxxxxxxx wrote: > >> On 10/08/2024 01:38, Tristram.Ha@xxxxxxxxxxxxx wrote: > >>> From: Tristram Ha <tristram.ha@xxxxxxxxxxxxx> > >>> > >>> The SGMII module of KSZ9477 switch can be setup in 3 ways: 0 for direct > >>> connect, 1 for 1000BaseT SFP, and 2 for 10/100/1000 SFP. > >> > >> Binding should say it, not commit msg. But aren't you duplicating > >> something like phy-connection-type? > > > > The sgmii-mode parameter is just used internally. I am not sure using > > This does not matter. > > > phy-connection-type or phy-mode is appropriate. > > Depends on what this property expressed in terms of hardware. Looks like > you want to say which SGMII mode is being used? The driver can detect whether 10/100/1000Base-T copper SFP is being used. So the main purpose of this device tree parameter is to indicate the SGMII module is directly connected to another one without using any SFP. This is a very rare case. In such case the device tree parameter can be changed to a flag to just indicate SFP is not used.