Modify

Opened 9 years ago

Closed 7 years ago

Last modified 4 years ago

#5193 closed defect (fixed)

phylib on ar71xx

Reported by: patrick.horn@… Owned by: juhosg
Priority: normal Milestone: Barrier Breaker 14.07
Component: kernel Version: Trunk
Keywords: ar71xx, ag71xx, ip175c, rb450 Cc:

Description

The ar71xx platform has two issues that prevent it from loading the new ip175c driver.

The first is that the ag71xx network driver does not attach to any PHYs at all, if the ethernet port is connected to more than one PHY. This just involves changing the logic so it always connects to the first PHY it finds, even if it finds more than one.

The second problem is that the ag71xx network driver is loaded before phylib, which means that the network driver will only attach to "Generic PHY" instead of the more specific driver. This just requires a change to the ordering in drivers/net/Makefile.

This patch has been successfully tested on the RB450 (ip175c) and RB493AH (ip178c).

Attachments (3)

ag71xx_phy.patch (1.8 KB) - added by patrick.horn@… 9 years ago.
Changes to ag71xx_phy.c and drivers/net/Makefile
ar71xx-enable-ip175c.patch (801 bytes) - added by patrick.horn@… 9 years ago.
Enable "ip175c" PHY driver instead of "icplus" driver on ar71xx
patch-5193-19047 (1.6 KB) - added by mtadeu@… 8 years ago.
Same of both above, but update to trunk r19047

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by patrick.horn@…

Changes to ag71xx_phy.c and drivers/net/Makefile

Changed 9 years ago by patrick.horn@…

Enable "ip175c" PHY driver instead of "icplus" driver on ar71xx

comment:1 Changed 8 years ago by javier

The fix proposed in this ticket has wider repercussions than what the description implies. Without it the LAN ports on the RouterStation are unusable. I would like to vote for the inclusion of this patch in the development branch.

comment:2 Changed 8 years ago by mtadeu

This patch is essential to RB450.
I update ar71xx-enable-ip175c.patch to trunk r18782, but this is easy to replace by "make kernel_menuconfig".
But, without ag71xx_phy.patch, the 4 ports attached to eth1 do not work.

Changed 8 years ago by mtadeu@…

Same of both above, but update to trunk r19047

comment:3 Changed 8 years ago by thepeople

  • Owner changed from developers to juhosg
  • Status changed from new to assigned

comment:4 Changed 7 years ago by nbd

  • Resolution set to fixed
  • Status changed from assigned to closed

fixed in r26670, r26671

comment:5 Changed 4 years ago by jow

  • Milestone changed from Attitude Adjustment 12.09 to Barrier Breaker 14.07

Milestone Attitude Adjustment 12.09 deleted

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.