On Mon, Jul 13, 2020 at 11:40:34AM +0530, sundeep subbaraya wrote:
> > > +static int otx2_ioctl(struct net_device *netdev, struct ifreq *req, int
> > > cmd)
> > > +{
> > > + struct otx2_nic *pfvf = netdev_priv(netdev);
> > > + struct hwtstamp_config *cfg = &pfvf->tstamp;
> > > +
> >
> > Need to test phy_has_hwtstamp() here and pass ioctl to PHY if true.
> >
> For this platform PHY is taken care of by firmware hence it is not
> possible.
This has nothing to do with the FW. The HW design might include a PHY
or MII time stamping device.
> > SKBTX_IN_PROGRESS may be set by the PHY, so you need to test whether
> > time stamping is enabled in your MAC driver as well.
> >
> In our case PHY will not set it and the pfvf/MAC driver sets it.
That might be true today, but the MAC driver should not prevent the
possibility of using an external time stamping device in the future.
Thanks,
Richard
