On 05/03/2012 12:34 AM, Hauke Mehrtens wrote: > 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 > Not yet. I also want to cover the bcm4313 pcie card. I will add the suspend resume testing to my list. Gr. AvS -- 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
