On 05/03/2012 12:34 AM, Hauke Mehrtens wrote:

On 05/02/2012 08:06 AM, Arend van Spriel wrote:

quoted

On 04/29/2012 10:13 AM, Arend van Spriel wrote:

quoted

On 04/29/2012 02:50 AM, Hauke Mehrtens wrote:

quoted

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.

quoted

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