[PATCH v2] phy: zynqmp: Fix sgmii clk ctrl GTR lane bit shift

Michal Simek michal.simek at amd.com
Thu May 22 11:03:33 CEST 2025



On 5/22/25 10:17, Rutherther wrote:
> Michal Simek <michal.simek at amd.com> writes:
> 
>> On 5/21/25 20:16, Rutherther wrote:
>>> The bitshift in GEM_CLK_CTRL register is five bits, not two. There are
>>> four bits for each GEM, and one bit reserved in between.
>>>
>>> This has caused that using more than one GEM is impossible,
>>> additionally corrupting the GEM0's configuration, leaving GEM0
>>> unusable as well (ie. if GEM0 and GEM1 are used, GEM1 configuration is
>>> going to write to GEM0's registers wrong value, leaving GEM0 unusable)
>>>
>>> Signed-off-by: Frantisek Bohacek <rutherther at ditigal.xyz>
>>
>> You still have the problem that address which you are using for sending is not
>> matching your SOB line.
>> Author:     Rutherther <rutherther at ditigal.xyz>
>> Signed-off-by: Frantisek Bohacek <rutherther at ditigal.xyz>
> 
> Sorry about that, I forgot I configured my smtp client to replace name
> in the From header. Should be fine in v3 I sent few hours ago, I hope.

But your v3 is not in lore.

$ b4 am 20250522060703.4863-1-rutherther at ditigal.xyz
Grabbing thread from 
lore.kernel.org/all/20250522060703.4863-1-rutherther at ditigal.xyz/t.mbox.gz
That message-id is not known.

> 
> Is full name a requirement to contribute to u-boot?

It is generic rule to fully identify yourself in any open source project.

Thanks,
Michal




More information about the U-Boot mailing list