Re: [PATCH net v4 0/4] net: phy: dp83640: fix shared clock lifetime and probe error cleanup

[luoxuanqiang <xuanqiang.luo@linux.dev>]

在 2026/8/12 00:35, Andrew Lunn 写道:
> On Tue, Aug 11, 2026 at 10:34:17PM +0800, luoxuanqiang wrote:
>> Hi Richard,
>>
>> 在 2026/8/11 21:11, Richard Cochran 写道:
>>> On Fri, Aug 07, 2026 at 03:07:25PM +0800, xuanqiang.luo@linux.dev wrote:
>>>> From: Xuanqiang Luo <xuanqiang.luo@linux.dev>
>>>>
>>>> The DP83640 driver shares one PTP clock between all PHYs on the same MII
>>>> bus.
>>>>
>>>> Its driver-local clock lookup and removal scheme can leak the shared clock
>>>> on probe failure or free it while another probe is acquiring it.
>>> Is this an actual problem that you have seen in the real world?
>>>
>>> Thanks,
>>> Richard
>> No, I have not observed this on real hardware.
> Do you at least have the real hardware, and are testing your patches
> on it?

I should clarify that I have not yet been able to fully test this series 
on real hardware. I looked into putting together a suitable test setup, 
but the most I would be able to get access to is a single DP83640T-EVK. 
The board has only one DP83640 PHY, so it cannot fully exercise the 
changes in patch 4, in particular the case where multiple DP83640 PHYs 
share the same MII bus. So far, the issues identified here and the 
corresponding fixes are based primarily on code inspection, rather than 
on reproducing theproblems and fully validating the fixes on real 
hardware.Sorry I was not clearer earlier about the lack of hardware 
testing. If anyone has access to a setup with multiple DP83640 PHYs on 
the same MII bus, I would appreciate any help testing this series, 
especially the shared-clock lifetime changes in patch 4. Thanks,
Xuanqiang