lkml.org 
[lkml]   [2017]   [Oct]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH net-next RFC 5/9] net: dsa: forward hardware timestamping ioctls to switch driver
    On Thu, Sep 28, 2017 at 10:25:34AM -0700, Florian Fainelli wrote:
    > This echoes back to Andrew's comments in patch 2, but we may have to
    > prefer PHY timestamping over MAC timestamping if both are available?
    > Richard, is that usually how the preference should be made?

    No, if the MAC supports time stamping, then it will take precedence,
    because the MAC driver doesn't know that the PHY also supports this.
    In the case where a board design includes the PHYTER (the one and only
    PHY PHC) and a MAC PHC, the user must de-select the MAC support in the
    Kconfig in order to use the PHYTER.

    So in general, we don't support PHC/timestamping simultaneously in the
    MAC and PHY. It would be a lot of work to support this, and the user
    timestamping API would have to be extended yet again, and so I think
    it is not worth the effort.

    Getting back to this patch, it should fall back to PHY timestamping
    when the switch device doesn't support timestamping:

    case SIOCGHWTSTAMP:
    if (ds->ops->port_hwtstamp_get)
    return ds->ops->port_hwtstamp_get(ds, port, ifr);
    else
    return phy_mii_ioctl(dev->phydev, ifr, cmd);

    That way, if someone combines a PHYTER with a non-PTP capable switch,
    it will just work.

    Thanks,
    Richard

    \
     
     \ /
      Last update: 2017-10-08 15:13    [W:3.237 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site