On 9/3/26 22:38, Mateusz Nowicki wrote:
> 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
Hi Mateusz
We got a downstream version which is pending, it will be integrated soon.
When our validation process will be finished, this pinctrl update will be
pushed
on U-Boot mailing list.
Thanks
Patrice
