Modify ↓

Opened 2 years ago

Closed 2 years ago

#21393 closed defect (fixed)

Regression: pcie_iproc_bcma detects only 1 device on Netgear R8000

Reported by: rmilecki Owned by: developers
Priority: normal Milestone:
Component: packages Version: Trunk
Keywords: Cc:

Description

Netgear R8000 is BCM4709A0 based router with 3 x BCM43602:
​https://wikidevi.com/wiki/Netgear_R8000

At some point, after some trunk change, pcie_iproc_bcma stopped detecting 2nd and 3rd PCIe device.

Attachments (4)

r47387-good.txt​ (73.9 KB) - added by rmilecki 2 years ago.
r47461-good.txt​ (82.8 KB) - added by rmilecki 2 years ago.
r47817-bad.txt​ (18.0 KB) - added by rmilecki 2 years ago.
r47884-bad.txt​ (61.6 KB) - added by rmilecki 2 years ago.

Download all attachments as: .zip

Change History (8)

Changed 2 years ago by rmilecki

Changed 2 years ago by rmilecki

Changed 2 years ago by rmilecki

Changed 2 years ago by rmilecki

comment:1 Changed 2 years ago by rmilecki

It seems regression was introduced somewhere between r47461 and r47817. I listed commits from that range touching target/linux/bcm53xx/patches-4.1/:

  • r47800 kernel: add bcm47xxpart patches sent for 4.5 merge window
  • r47688 bcm53xx: update PCIe driver
  • r47687 bcm53xx: update SMP patches
  • r47686 bcm53xx: activate workarounds for erratas
  • r47685 bcm53xx: move patch 132-ARM-BCM5301X-Add-missing-Netgear-R8000-LEDs.patch
  • r47467 kernel: take bcm47xx_nvram.h from kernel and not backports

The most obvious candidate is r47688 bcm53xx: update PCIe driver

comment:2 Changed 2 years ago by arokh

I can confirm that r47688 was the culprit. r47954 works when it's reversed.

comment:3 Changed 2 years ago by rmilecki

r47688 backported many iProc patches, thanks to adityaxavier help (testing) I was able to track down the one that caused this regression: PCI: iproc: Add PAXC interface support.

I reverted this change for now (in r48382) and it was confirmed that all 3 devices are detected again.

comment:4 Changed 2 years ago by rmilecki

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

Workaround replaced with a proper fix in r48423.

Thanks for your help adityaxavier!

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.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.