All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Henrik Rydberg" <rydberg@euromail.se>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: "John W. Linville" <linville@redhat.com>,
	"Network Development" <netdev@vger.kernel.org>,
	"Rafał Miłecki" <zajec5@gmail.com>
Subject: Re: brcm80211 breakage..
Date: Tue, 31 Jan 2012 13:08:46 +0100	[thread overview]
Message-ID: <20120131120846.GA8799@polaris.bitmath.org> (raw)
In-Reply-To: CA+55aFx2UnVZP1Z44dMHeifB2nD1zJ87kz25zZjUdwv0ozwKPA@mail.gmail.com

Hi Linus,

Thanks for the MBA31 fixes. Sadly, however, with current git
(27ba234), after encountering the known problem

[  882.064120] bcma: Failed to get SPROM: -71

probing simply fails on my machine,

[  882.064256] bcma-pci-bridge: probe of 0000:01:00.0 failed with error -2

resulting in no wireless. Since the bcma SPROM setup allegedly is
unused, simply ignoring the error with this hack

diff --git a/drivers/bcma/main.c b/drivers/bcma/main.c
index febbc0a..cb69fb0 100644
--- a/drivers/bcma/main.c
+++ b/drivers/bcma/main.c
@@ -171,7 +171,6 @@ int bcma_bus_register(struct bcma_bus *bus)
 		pr_err("No SPROM available\n");
 	} else if (err) {
 		pr_err("Failed to get SPROM: %d\n", err);
-		return -ENOENT;
 	}
 
 	/* Register found cores */

makes the device register as

01:00.0 0280: 14e4:4353 (rev 01)
	Subsystem: 106b:00d1
	Kernel driver in use: bcma-pci-bridge

which seems to actually work. Instead of the failed probe, the log shows

[ 1168.048571] bcma: Core 0 found: ChipCommon (manuf 0x4BF, id 0x800, rev 0x22, class 0x0)
[ 1168.048706] bcma: Core 1 found: IEEE 802.11 (manuf 0x4BF, id 0x812, rev 0x17, class 0x0)
[ 1168.048855] bcma: Core 2 found: PCIe (manuf 0x4BF, id 0x820, rev 0x0F, class 0x0)
[ 1168.049051] bcma: Found rev 6 PMU (capabilities 0x108C2606)
[ 1168.120122] bcma: Failed to get SPROM: -71
[ 1168.120340] bcma: Bus registered
[ 1168.125167] brcmsmac bcma0:0: mfg 4bf core 812 rev 23 class 0 irq 23
[ 1168.175670] bcma: Switched to core: 0x812
[ 1168.182061] brcms_module_init: register returned 0

Given that our machines look the same, this is somewhat odd. Could
this be a config issue or did I miss something else?

Henrik

  parent reply	other threads:[~2012-01-31 12:08 UTC|newest]

Thread overview: 89+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-10 20:15 brcm80211 breakage Linus Torvalds
2012-01-11 10:44 ` Arend van Spriel
2012-01-11 15:05   ` Linus Torvalds
2012-01-11 16:04     ` Linus Torvalds
2012-01-11 17:00       ` Larry Finger
2012-01-12  1:06         ` Linus Torvalds
2012-01-12  1:46           ` Larry Finger
2012-01-12  1:58             ` Linus Torvalds
2012-01-12  2:11             ` Linus Torvalds
2012-01-12  4:15               ` Larry Finger
2012-01-12  5:20                 ` Linus Torvalds
2012-01-12  5:30                   ` Linus Torvalds
2012-01-12  7:08                     ` Rafał Miłecki
2012-01-12  7:10                       ` Rafał Miłecki
2012-01-12  7:18                       ` Linus Torvalds
2012-01-12 15:39                       ` Larry Finger
2012-01-12 15:46                         ` Rafał Miłecki
2012-01-12  7:13                     ` Linus Torvalds
2012-01-12  7:18                       ` Rafał Miłecki
2012-01-12  7:22                         ` Linus Torvalds
2012-01-12 10:03                       ` Arend van Spriel
2012-01-12 15:51                         ` Linus Torvalds
2012-01-12 19:00                       ` Linus Torvalds
2012-01-12 19:08                         ` Linus Torvalds
2012-01-12 20:09                           ` Arend van Spriel
2012-01-12 20:27                             ` Linus Torvalds
2012-01-12 20:36                               ` Arend van Spriel
2012-01-12 22:38                               ` Linus Torvalds
2012-01-12 22:42                                 ` Rafał Miłecki
2012-01-12 22:45                                   ` Linus Torvalds
2012-01-12 23:04                                     ` Rafał Miłecki
2012-01-13  0:13                                       ` Linus Torvalds
2012-01-13  2:30                                         ` Linus Torvalds
2012-01-13  5:34                                           ` Linus Torvalds
2012-01-13  6:50                                             ` Rafał Miłecki
2012-01-13  6:57                                               ` Rafał Miłecki
2012-01-13  7:17                                                 ` Linus Torvalds
2012-01-13  7:41                                                   ` [PATCH] bcma: s&r: NULL mapped core Rafał Miłecki
2012-01-13  7:53                                                     ` Linus Torvalds
2012-01-13  7:59                                                     ` [0/5] bcma/brcmsmac suspend/resume cleanups and fixes Linus Torvalds
2012-01-13  8:00                                                       ` [PATCH 1/5] bcma: convert suspend/resume to pm_ops Linus Torvalds
2012-01-13  9:58                                                         ` Arend van Spriel
2012-01-13 10:01                                                           ` Rafał Miłecki
2012-01-13  8:00                                                       ` [PATCH 2/5] bcma: add stub for bcma_bus suspend() Linus Torvalds
2012-01-13  8:01                                                       ` [PATCH 3/5] bcma: connect the bcma bus suspend/resume to the bcma driver suspend/resume Linus Torvalds
2012-01-13  8:01                                                       ` [PATCH 4/5] brcmsmac: remove PCI suspend/resume from bcma driver Linus Torvalds
2012-01-13  9:59                                                         ` Arend van Spriel
2012-01-13  8:02                                                       ` [PATCH 5/5] bcma: invalidate the mapped window over suspend/resume Linus Torvalds
2012-01-13 10:03                                                       ` [0/5] bcma/brcmsmac suspend/resume cleanups and fixes Rafał Miłecki
2012-01-13 16:15                                                         ` Linus Torvalds
2012-01-13 11:06                                                       ` Arend van Spriel
2012-01-13 16:23                                                         ` Linus Torvalds
2012-01-14  3:50                                                         ` Linus Torvalds
2012-01-14 12:22                                                           ` Dominique Martinet
2012-01-14 14:34                                                             ` Rafał Miłecki
2012-01-14 15:07                                                               ` Dominique Martinet
2012-01-14 19:15                                                             ` Linus Torvalds
2012-01-14 23:08                                                               ` Dominique Martinet
2012-01-14 23:28                                                                 ` Linus Torvalds
2012-01-14 17:36                                                           ` Arend van Spriel
2012-01-16 14:00                                                           ` Arend van Spriel
2012-01-16 17:33                                                             ` Arend van Spriel
2012-01-17  1:12                                                               ` Linus Torvalds
2012-01-17 10:37                                                                 ` Arend van Spriel
2012-01-19  9:42                                                                 ` Arend van Spriel
2012-01-19 12:12                                                                 ` Arend van Spriel
2012-01-19 16:15                                                                   ` Linus Torvalds
2012-01-13  7:13                                               ` brcm80211 breakage Linus Torvalds
2012-01-12 19:46                         ` Arend van Spriel
2012-01-13  1:53                         ` Larry Finger
2012-01-31 12:08                         ` Henrik Rydberg [this message]
2012-01-31 17:02                           ` Linus Torvalds
2012-01-31 19:08                             ` Arend van Spriel
2012-01-31 19:25                               ` John W. Linville
2012-01-31 19:35                                 ` Arend van Spriel
2012-01-31 19:58                                 ` Henrik Rydberg
2012-01-31 20:21                                   ` Arend van Spriel
2012-01-31 22:23                                     ` Henrik Rydberg
2012-01-31 19:36                               ` Henrik Rydberg
2012-01-31 19:49                                 ` Arend van Spriel
2012-01-12 13:13                 ` Arend van Spriel
2012-01-12 16:22                   ` Larry Finger
2012-01-12 17:18                   ` Larry Finger
2012-01-12 17:31                     ` Linus Torvalds
2012-01-12 17:44                       ` Larry Finger
2012-01-11 21:01       ` Arend van Spriel
2012-01-12  0:07         ` Linus Torvalds
2012-01-11 17:15   ` Ben Hutchings
2012-01-12  0:10     ` Linus Torvalds

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120131120846.GA8799@polaris.bitmath.org \
    --to=rydberg@euromail.se \
    --cc=linville@redhat.com \
    --cc=netdev@vger.kernel.org \
    --cc=torvalds@linux-foundation.org \
    --cc=zajec5@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.