Re: [PATCH net-next v10 03/11] phy: rockchip: naneng-combphy: add SGMII MAC selection for RK3568

From: Jakub Kicinski

Date: Mon Oct 05 2026 - 17:45:24 EST


On Tue, 6 Oct 2026 05:15:07 +0800 Coia Prant wrote:
> Thanks for the suggestion. I looked at this again, and I think the
> dependency is a bit more involved than it might seem.
>
> The PHY patch (03/11) adds the driver support for
> "rockchip,sgmii-mac-sel", which is documented by the PHY binding (02/11).
> The DTS patch (10/11) then uses this property. If I send 03/11 separately
> to Vinod and keep 10/11 in net-next, dtbs_check will flag the DTS property
> as undocumented until the PHY binding lands in mainline. That would break
> DTS validation for the net-next series.

Wasn't the validation running on top of linux-next to avoid this sort
of problem?

> Also, splitting into separate series means each has to queue and get
> reviewed independently, which I'm worried might not all make the merge
> window in time.
>
> So unless you strongly prefer splitting, I'd rather keep the whole series
> together in net-next. If that works, would it be possible to coordinate an
> Ack from Vinod for the PHY part? Or if you still think it should go
> separately, I can do that too, but I wanted to flag the dependency and
> timing first.

Well, the patch series is already getting stuck because of
cross-subsystem acks. Do what you will, I have 700+ patches
to look at right now :/