Hi Nikita,
> -----Original Message-----
> From: Simek, Michal <michal.si...@amd.com>
> Sent: Thursday, August 11, 2022 1:16 PM
> To: Nikita Shubin <nikita.shu...@maquefel.me>; Agarwal, Swati
> <swati.agar...@amd.com>; Pandey, Radhey Shyam
> <radhey.shyam.pan...@amd.com>
> Cc: li...@yadro.com; Nikita Shubin <n.shu...@yadro.com>; Joe
> Hershberger <joe.hershber...@ni.com>; Ramon Fried
> <rfried....@gmail.com>; u-boot@lists.denx.de
> Subject: Re: [RFC PATCH 1/1] net: xilinx: handle internal PHY/PCS
>
> Hi Nikita,
>
> On 8/2/22 12:53, Nikita Shubin wrote:
> > From: Nikita Shubin <n.shu...@yadro.com>
> >
> > In SGMII/1000BaseX Xilinx AXI Ethernet may also have an Internal PHY
> > (PCS) in addition to external PHY, in that case we should also set at
> > least BMCR_ANENABLE.
> >
> > PCS are not visible until axinet bringup, so init should be done
> > after, controller is brought up, then we should poll BMSR_ANEGCOMPLETE
> > prior to polling the external PHY.
> >
> > The PCS handling relies on "pcs-handle" device tree handle which
> > serves the similar purpose in Linux device tree.
> >
> > Signed-off-by: Nikita Shubin <n.shu...@yadro.com>
> > ---
<snip>
> > @@ -540,6 +565,20 @@ static int axiemac_start(struct udevice *dev)
> > return -1;
> > }
> >
> > + if (pcsdev) {
> > + /* It looks like we need a bit of delay for core to come up
> > + * may be we could poll MgtRdy or PhyRstCmplt bit
> > + * in 0x00000010, but 1 msec is no a big deal.
> > + */
> > + udelay(1000);
> > + ret = phywrite(priv, pcsdev->addr, MII_BMCR,
> > + BMCR_ANENABLE | BMCR_FULLDPLX |
> BMCR_SPEED1000);
Thanks for the patch.
Could you please confirm that BMCR_ISOLATE is also being handled in
the autonegotiation path? For ex., the equivalent in linux can be found
here:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/phy/phylink.c#n3050
Could you also please handle the SGMII/1000BaseX standard selection
in the PCS PMA IP? For reference, please see:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/ethernet/xilinx/xilinx_axienet_main.c#n1619
Regards,
Harini
