Re: [PATCH net v4] phy: mscc: Fix timestamping for vsc8584

From: Vladimir Oltean
Date: Tue Aug 19 2025 - 04:28:08 EST


On Mon, Aug 18, 2025 at 10:10:29AM +0200, Horatiu Vultur wrote:
> There was a problem when we received frames and the frames were
> timestamped. The driver is configured to store the nanosecond part of
> the timestmap in the ptp reserved bits and it would take the second part
> by reading the LTC. The problem is that when reading the LTC we are in
> atomic context and to read the second part will go over mdio bus which
> might sleep, so we get an error.
> The fix consists in actually put all the frames in a queue and start the
> aux work and in that work to read the LTC and then calculate the full
> received time.
>
> Fixes: 7d272e63e0979d ("net: phy: mscc: timestamping and PHC support")
> Signed-off-by: Horatiu Vultur <horatiu.vultur@xxxxxxxxxxxxx>

Reviewed-by: Vladimir Oltean <vladimir.oltean@xxxxxxx>

I think the patch is in "Changes Requested" in patchwork because of me,
so I'll try this, see if it helps:

pw-bot: under-review