Hi, On 23. 09. 24 15:14, Peng Fan (OSS) wrote: > From: Ye Li <[email protected]> > > Because iMX93 has a internal 1/2 divider before clock input to > network controller, so have to set twice frequency rate > > Fixes: 09de565f76b ("net: fec_mxc: support i.MX93") > Signed-off-by: Ye Li <[email protected]> > Signed-off-by: Peng Fan <[email protected]> > --- > drivers/net/fec_mxc.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c > index 0a0d92bc2cd..e5d7f0f3e1e 100644 > --- a/drivers/net/fec_mxc.c > +++ b/drivers/net/fec_mxc.c > @@ -1215,6 +1215,9 @@ static int fecmxc_set_ref_clk(struct clk *clk_ref, > phy_interface_t interface) > else > return -EINVAL; > > + if (is_imx93()) > + freq = freq << 1;
Humm, but how did it work so far? On phyboard-segin-imx93, this change breaks fec ethernet. I had to revert this already in NXP downstream u-boot. BR, Primoz > + > ret = clk_set_rate(clk_ref, freq); > if (ret < 0) > return ret; -- Primoz Fiser | phone: +386-41-390-545 <tel:+386-41-390-545> | ---------------------------------------------------------| Norik systems d.o.o. | https://www.norik.com <https://www.norik.com> | Your embedded software partner | email: [email protected] <mailto:[email protected]> | Slovenia, EU | phone: +386-41-540-545 <tel:+386-41-540-545> |

