On 03.06.2020 20:57, Eivind Uggedal wrote:
> Hi,
> 
> I saw you added the initial RTL8125 support in r8169.
> 
> I have a recent x570 motherboard (MSI x570 Tomahawk) with onboard RTL8125. 
> With r8169 from 5.4 and 5.7 I get:
> 
>> unknown chip XID 641
> 
> from the r8169 driver since XID==641 does not match any of the mask/val in 
> rtl8169_get_mac_version().
> 
> I did not see any relevant unmerged changes in the netdev-next tree.
> 
> I have a hard time following these "magic" masks/values and could not 
> correlate them to anything in the Realtek oot r8125 driver. I have some C 
> experience, but zero driver/low level experience.
> 
Thanks for the report. XID 641 is CFG_METHOD_5 = RTL8125B in the r8125 vendor 
driver.
This new chip version isn't supported yet by the r8169 driver. Typically I wait 
for
reports like yours because:
- Not every chip version in the vendor driver makes it to the market.
  Adding support for it would be wasted effort.
- I don't have hardware for testing. I need users like you for it ..

> I could try to just fallback to RTL_GIGIA_MAC_VER_60 or RTL_GIGIA_MAC_VER_61 
> locally, but ideally I'd like a proper patch that can be mainlined.
> 
This may or may not work, partially also depending on the link partner. 
Typically each
chip version needs it own quirks, therefore they are treated differently by the 
vendor driver.

> At least one other poor soul has the same issue: 
> https://forums.unraid.net/topic/92998-dhcp-networking-issue-v-683/
> 
> Seeing as this board had a world wide release just days ago maybe this is a 
> new revision of the RTL8125 chipset?
> 
> Appreciate any pointers and help to debug it further!
> 
There is not really something to debug. I have to port all chip quirks from
the vendor driver and ask Realtek for a firmware file.
I'll put it on my roadmap, maybe it makes it into 5.9 or 5.10. Until then
you would have to use vendor driver r8125.

> Regards,
> Eivind Uggedal

Heiner

Reply via email to