Hi Patrice,
On 2/24/26 13:57, Patrice CHOTARD wrote:
> Good catch, you are right, i wasn't aware of bindings update on kernel's side
> uypstream vs downstrem.
> I will drop this patch and resubmit an update later.
I hit this on STM32MP257F-DK with mainline U-Boot: with networking
enabled in stm32mp25_defconfig, every RGMII frame sent by the board is
received with a CRC error by the link partner, at 100 and 1000 Mb/s.
Setting the RET and DE bits in GPIOx_ADVCFGR on the RGMII data pins,
which is what this patch does, fixes it. The kernel side is already
covered: "st,io-sync" landed in stm32mp25-pinctrl.dtsi for
eth1_rgmii_pins_a in v6.19 and for eth1_rgmii_pins_b in stm32-next.
Unless you have an updated version pending, I would like to resubmit
this patch as v2 parsing the upstream binding ("st,io-sync" and
"skew-delay-{input,output}-ps"), keeping Fabien as author, tested on
STM32MP257F-DK.
Regards,
Mateusz
