On 22 June 2015 at 10:10, Hauke Mehrtens <ha...@hauke-m.de> wrote: > On 06/22/2015 02:51 AM, Ian Kent wrote: >> On Sun, 2015-06-21 at 19:19 +0200, Rafał Miłecki wrote: >>> On 10 March 2015 at 04:30, Ian Kent <ra...@themaw.net> wrote: >>>> The Netgear R8000 has a PEX8603 connected to the BCM53012 and if >>>> it isn't configured during the bus scan the PCI layer goes crazy >>>> trying to configure phantom devices. >>> >>> This is kind of magic for me. >>> Hauke: are you able to review this? >> >> Yeah, it's a bridge so it needs upstream and downstream bus ids set but >> the main issue is that it doesn't respond with appropriate return values >> for non-existent devices. >> >> I modelled the code on the corresponding source from the Broadcom SDK >> file that this source was originally developed from (at least it >> appeared to match). >> >> I confirmed that PEX860X devices don't return proper return code from >> another PEX driver, from the WRT1900 source I think, its been a while >> now. >> >> Ian > > Hi, > > I completely missed these patches, thanks for pointing me to them again. > I will try to review them in the net days. I already saw this code in > the vendor driver but did not add it because my devices did not use it. > We will change to a different PCIe driver with the next kernel in > OpenWrt, because Broadcom added a kernel for this PCIe core into the > mainline kernel. We should get it into this new driver. Then also some > people from Broadcom can review this.
This would be nice to patch the old driver in CC branch. To get R8000 supported by CC. -- Rafał _______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel