RE: [PATCH net-next] net: phy: micrel: Fix latency issues in LAN8814 PHY

[Divya.Koppera]

Hi Andrew,

> -----Original Message-----
> From: Andrew Lunn <andrew@xxxxxxx>
> Sent: Tuesday, July 5, 2022 8:45 PM
> To: Divya Koppera - I30481 <Divya.Koppera@xxxxxxxxxxxxx>
> Cc: hkallweit1@xxxxxxxxx; linux@xxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx;
> edumazet@xxxxxxxxxx; kuba@xxxxxxxxxx; pabeni@xxxxxxxxxx;
> netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; UNGLinuxDriver
> <UNGLinuxDriver@xxxxxxxxxxxxx>; Madhuri Sripada - I34878
> <Madhuri.Sripada@xxxxxxxxxxxxx>
> Subject: Re: [PATCH net-next] net: phy: micrel: Fix latency issues in LAN8814
> PHY
> 
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the
> content is safe
> 
> On Tue, Jul 05, 2022 at 04:35:54PM +0530, Divya Koppera wrote:
> > Latency adjustments done for 1-step and 2-step from default latency
> > register values to fix negative and high mean path delays.
> 
> You forgot to Cc: the PTP maintainer.
> 
> Doesn't this change break user space tools which have already measured the
> latency and is doing a correction?

This corrections can be done using user space tools. But I tried to fix it in phy rather than giving chance to application.

This will work on only certain platforms. So I'm dropping this change to get into kernel.

Thanks
Divya

> 
>     Andrew