lkml.org 
[lkml]   [2024]   [Feb]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/2] dt-bindings: net: Add TI DP83640
    > > > +  ti,phy-control-frames:
    > > > + $ref: /schemas/types.yaml#/definitions/uint32
    > > > + enum: [0, 1]
    > > > + description: |
    > > > + If present, enables or disables the PHY control frames.
    > > > + PHY Control Frames support can also be strapped. If the strap pin is not
    > > > + set correctly or not set at all then this can be used to configure it.
    > > > + - 0 = PHY Control Frames disabled
    > > > + - 1 = PHY Control Frames enabled
    > > > + - unset = Configured by straps
    > > What is a control frame?
    > I'm not an expert on this but it seems that if the PHY's Serial Management
    > interface is not available, it is possible to build PCF (PHY Control Frame)
    > packets that will be passed to PHY through the MAC Transmit Data interface.
    > The
    > PHY is then able to intercept and interpret these packets. Enabling it
    > increases
    > the MII Transmit packet latency.
    > You'll find details in §5.4.6 of datasheet
    > [https://www.ti.com/lit/gpn/dp83640]

    Do you actually need this feature?

    [Looks at data sheet]

    Ah, so it allows you to access PHY registers by sending it commands in
    Ethernet frames. That should in theory be faster than MDIO. However,
    my experience with Ethernet switches which offer similar capabilities,
    it is often not faster, because of interrupt coalescing.

    Anyway, the serial management interface is the MDIO bus. You know this
    is available, because that is how the PHY driver it talking to the
    PHY! Also, i've not seen any code which implements sending commands to
    the PHY using Ethernet frames.

    So why not just hard code it in the driver to disable this feature?

    Andrew

    \
     
     \ /
      Last update: 2024-05-27 15:07    [W:2.734 / U:0.000 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site