On 05/02/2012 08:06 AM, Arend van Spriel wrote: > On 04/29/2012 10:13 AM, Arend van Spriel wrote: >> On 04/29/2012 02:50 AM, Hauke Mehrtens wrote: >>> I have not completely runtime tested these patches as I do not have a >>> PCI(e) devices currently supported by brcmsmac. My SoC (BCM4718) and >>> the PCI devices connected to my SoC (BCM43224) are currently not >>> working with brcmsmac, but they are failing much later that without >>> these patches. >> >> I will test this patch series as well on PCI(e) devices. So you do have >> a bcm43224 card, but no laptop to fit it in? Just wondering. I will send >> the results. >> >> Gr. AvS >> > > Hi Hauke, > > I have tested this patch series including the other two series as I > suspect there is a dependancy so: > > [PATCH 0/8] ssb/bcma/bcm47xx: extend boardinfo and sprom > [PATCH 0/4] bcma: add PCI functions from brcmsmac Yes these patch series are mostly providing some stuff needed by brcmsmac. > I only did some basic testing, but wanted to let you know the results so > far: > > BCM43224/OTP/64-bit/2.4G: 63 Mbps > BCM43224/OTP/64-bit/5G: 57 Mbps > BCM43224/SPROM/32-bit/2.4G: 65 Mbps > BCM43224/SPROM/32-bit/5G: 63 Mbps > > These are pretty normal numbers. I will add the test branch to my > nightly regression testing for the next couple of days. > > First impression: Great job! Hi Arend, thanks for testing. Have you tried suspend/resume and hibernate/resume? I think some of the workaround in drivers/bcma/driver_pci.c should be called in the resume path and not only in the initialization path. Hauke -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
