RE: [PATCH net-next 1/4] dt-bindings: net: dsa: microchip: add SGMII port support to KSZ9477 switch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 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.





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux