Re: [PATCH 8/9] net: mdio: ipq4019: add qca8084 configurations

[Jie Luo]

On 11/16/2023 1:03 AM, Robert Marko wrote:
> On Wed, Nov 15, 2023 at 6:01 PM Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> wrote:
> > On 11/15/23 17:20, Andrew Lunn wrote:
> > > On Wed, Nov 15, 2023 at 11:25:14AM +0800, Luo Jie wrote:
> > > > The PHY & PCS clocks need to be enabled and the reset
> > > > sequence needs to be completed to make qca8084 PHY
> > > > probeable by MDIO bus.
> > >
> > > Is all this guaranteed to be the same between different boards?
> > No, this looks like a total subsystem overreach, these should be
> > taken care of from within clk framework and consumed with the clk
> > APIs.
> >
> > Konrad
>
> There are patches for QCA8084 clocks:
> https://patchwork.kernel.org/project/linux-arm-msm/cover/20231104034858.9159-1-quic_luoj@xxxxxxxxxxx/
>
> I guess all of the clocking should be done there, it isn't really a MDIO issue.
>
> Regards,
> Robert

Thanks Robert for the link of this patch.

Yes, the clock driver of qca8084 is probed as the MDIO device, the
configuration sequence here to lighten the qca8084 PHY need to
be completed before the clock APIs available to call.