Re: [PATCH net] net: explicitly check in of_phy_is_fixed_link() for managed = "in-band-status"

From: Vladimir Oltean
Date: Tue Aug 12 2025 - 11:09:45 EST


On Tue, Aug 12, 2025 at 12:33:36PM +0100, Russell King (Oracle) wrote:
> As an additional point, I'm not sure what has broken that justifies
> this change for the net tree. You mention at one point in the commit
> description about wanting to use "c73" as a string for "managed",
> which suggests new development, and thus shouldn't this be targetting
> net-next?
>
> Note that at present, all dts files in the kernel either omit the
> managed property, or have it present with value "in-band-status".
>
> Thus, I think the commit makes sense for net-next.

Correct, thanks for challenging this, nothing seems to be broken if
stable kernels are left alone in their thinking that managed = "c73"
would be a fixed link. It was out of caution that old and new kernels
would disagree on this when faced with the same device tree, but it does
not seem to matter. I will re-target the patch to net-next.