Bug 212542 - r8169 driver misses device id
Summary: r8169 driver misses device id
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Andy Gospodarek
QA Contact: Brian Brock
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-10-27 11:21 UTC by Ralf Ertzinger
Modified: 2014-06-29 22:57 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-02-02 20:16:50 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
rt8169-add-pciid.patch (441 bytes, patch)
2006-10-27 20:09 UTC, Andy Gospodarek
no flags Details | Diff
rt8169-add-pciid-better.patch (18.21 KB, patch)
2006-10-27 22:01 UTC, Andy Gospodarek
no flags Details | Diff
PHY reset at startup (1.59 KB, patch)
2006-10-27 22:34 UTC, Francois Romieu
no flags Details | Diff

Description Ralf Ertzinger 2006-10-27 11:21:37 UTC
Description of problem:
PCI devices with the ID 10ec:8167 (variants of the Realtek 8169/8110) are
currently not recognized by the r8169 driver. Putting the above id in
/sys/pci/drivers/r8169/new_id makes the driver claim the devices just fine.

Version-Release number of selected component (if applicable):
kernel-2.6.18-1.2798.fc6

How reproducible:
Always

Steps to Reproduce:
1. Boot fedora on board with a Realtek chip having the above ID
2.
3.
  
Actual results:
r8169 is not loaded, device not claimed

Expected results:
r8169 claims the device

Additional info:

Comment 1 Andy Gospodarek 2006-10-27 20:09:36 UTC
Created attachment 139611 [details]
rt8169-add-pciid.patch

Seems like this would be the reasonable patch for this.  It doesn't appear in
the netdev-2.6 tree, so I'll check with Francois.

Comment 2 Francois Romieu 2006-10-27 20:31:58 UTC
The patch has been merged between 2.6.18 and 2.6.19-rc1.

-- 
Ueimor

Comment 3 Andy Gospodarek 2006-10-27 20:37:18 UTC
Ok, thanks.  I didn't see it in netdev-2.6 at first, but now I realize how I
missed it.

Comment 4 Andy Gospodarek 2006-10-27 22:01:30 UTC
Created attachment 139618 [details]
rt8169-add-pciid-better.patch

This patch is probably better since it actually accounts for some PCI region
differences for the newer hardware.

Comment 5 Francois Romieu 2006-10-27 22:34:51 UTC
Created attachment 139620 [details]
PHY reset at startup

If the IDs for the 8168 are added, the patch above may be useful as it fixes an
issue of incorrect link negociation that I have been reported.

--  
Ueimor

Comment 6 Andy Gospodarek 2006-10-27 22:49:27 UTC
Thanks!  I'll include that if we take the larger patch.  

How necessary are the RTL_CFG_X parameters for proper functionality?  It seems
they would be required for everything to work well, but when someone reported
they worked without I began to question their value.

I ask because we would probably prefer the patch that only adds additional PCI
ID's rather than the larger one that has more changes.

Comment 7 Francois Romieu 2006-10-27 23:44:45 UTC
The RTL_CFG_X are needed for the 0x8136 and the 0x8168 which both use a
different PCI BAR and require an extra alignement (8 bytes instead of 2).

Imho the 8168 user base has grown significantly during the last months.

-- 
Ueimor

Comment 8 Andy Gospodarek 2006-10-30 18:58:11 UTC
This is resolved in 2.6.19, so when the fedora kernels move to 2.6.19 you will
get this for free.  

Are you able to workaround this with some trickery in some of your
initialization scripts so the module loads when the box boots?

Comment 9 Ralf Ertzinger 2006-10-30 20:21:17 UTC
I can work around this by piping the id into sysfs, yes.

Comment 10 Andy Gospodarek 2006-10-30 21:01:35 UTC
I would suggest keeping that workaround in place until FC6 moves to 2.6.19.  We
generally don't add patches for new hardware support to Fedora kernels if the
support will appear in an upcoming upstream kernel that will get absorbed into
FC when it is released.  

I will probably try and put together an FC6 test kernel and will post to this BZ
if/when one is ready.

Comment 11 Francois Romieu 2006-10-30 21:38:54 UTC
Well, the MAC address change support in current 2.6.19-rc proved to be fatal for
some users anyway :o/

-- 
Ueimor

Comment 12 Andy Gospodarek 2007-02-02 20:16:50 UTC
This should be resolved with the 2.6.19 kernel.  Please re-open this BZ if you
continue to have problems on a 2.6.19-based fedora kernel.


Note You need to log in before you can comment on or make changes to this bug.