mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: mvtodevnull@gmail.com
To: "Michael Buesch" <mb@bu3sch.de>
Cc: "Larry Finger" <larry.finger@lwfinger.net>,
	"Rafael J. Wysocki" <rjw@sisk.pl>,
	"Daniel Walker" <dwalker@mvista.com>,
	"Ray Lee" <ray-lk@madrabbit.org>,
	"Simon Holm Thøgersen" <odie@cs.aau.dk>,
	matthias.kaehlcke@gmail.com, linux-kernel@vger.kernel.org,
	linux@bohmer.net, "Ingo Molnar" <mingo@elte.hu>,
	kjwinchester@gmail.com, jonathan@jonmasters.org,
	akpm@linux-foundation.org, bcm43xx-dev@lists.berlios.de
Subject: Re: [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem semaphore to mutex
Date: Mon, 17 Dec 2007 18:27:44 -0500	[thread overview]
Message-ID: <520b60a90712171527h4563542ane34796b32e99635d@mail.gmail.com> (raw)
In-Reply-To: <200712180018.57272.mb@bu3sch.de>

On Dec 17, 2007 6:18 PM, Michael Buesch <mb@bu3sch.de> wrote:
> On Tuesday 18 December 2007 00:12:30 mvtodevnull@gmail.com wrote:
> >
> > Well, I'm not sure what you mean by "requires" b43, but I did say that
> > the device uses the b43 driver.
>
> Requires means requires.
>

Ok, noted.

> > Sorry, I should have been more specific. I figured since the device
> > could use bcm43xx, it could also use b43legacy, so I copied the
> > entries in b43_ssb_tbl[] to b43legacy_ssb_tbl[] and rebuilt the
> > b43legacy driver. I removed the b43 and ssb modules, and inserted the
> > b43legacy module. Afterwards, I noticed b43 had still been autoloaded,
> > so I removed it (I checked dmesg and only b43legacy had initialized
> > anyway) , and proceeded to use the b43legacy driver with the version 3
> > firmware. And like I said, it works exactly like the b43 driver with
> > the version 4 firmware.
>
> I'm not sure what you are trying to show with this hack.
> It's been said several times in this thread that the firmware has
> nothing to do with the device radio.
> So it won't affect the transmit rate or something like that.
>
> Note that the difference between b43 and b43legacy is NOT that the
> driver is "legacy". It is called legacy because it does _only_ support
> legacy _devices_. So if you hack it to drive a new card it will only
> work by luck (luck as in there might be some code left over which
> is able to initialize the device somehow.). My point being, we removed
> code for new devices from b43legacy and are probably going to remove
> even more stuff in the future. So there is simply no point in testing
> any new device with b43legacy.
>

At the start of my discussion with Larry, I asked if the firmware
could cause such a difference. Larry said no, and I believed him, but
I'm very lost as to what could be causing such a difference between
bcm43xx and b43. I figured it was at least worth testing, I mean, it
wasn't like I was recommending others do the same.

I'm actually still using the b43legacy driver right now, and it really
does work exactly the same as b43. If you're interested, here's the
relevant section of dmesg from when I loaded the b43legacy driver:

ssb: SPROM revision 2 detected.
ssb: Sonics Silicon Backplane found on PCI device 0000:05:00.0
b43legacy-phy1: Broadcom 4311 WLAN found
b43legacy-phy1 debug: Found PHY: Analog 4, Type 2, Revision 8
b43legacy-phy1 debug: Found Radio: Manuf 0x17F, Version 0x2050, Revision 2
b43legacy-phy1 debug: Radio initialized
phy1: Selected rate control algorithm 'simple'
b43legacy-phy1 debug: Loading firmware version 0x127, patch level 14
(2005-04-18 02:36:27)
Registered led device: b43legacy-phy1:tx
Registered led device: b43legacy-phy1:rx
b43legacy-phy1 debug: Chip initialized
b43legacy-phy1 debug: 32-bit DMA initialized
b43legacy-phy1 debug: Wireless interface started
b43legacy-phy1 debug: Adding Interface type 2
wlan0: Initial auth_alg=0
wlan0: authenticate with AP 00:09:5b:70:2d:68
wlan0: RX authentication from 00:09:5b:70:2d:68 (alg=0 transaction=2 status=0)
wlan0: authenticated
wlan0: associate with AP 00:09:5b:70:2d:68
wlan0: RX AssocResp from 00:09:5b:70:2d:68 (capab=0x5 status=0 aid=86)
wlan0: associated
b43legacy-phy1 debug: Removing Interface type 2
b43legacy-phy1 debug: Wireless interface stopped
b43legacy-phy1 debug: DMA-32 0x0200 (RX) max used slots: 1/64
b43legacy-phy1 debug: DMA-32 0x02A0 (TX) max used slots: 0/128
b43legacy-phy1 debug: DMA-32 0x0280 (TX) max used slots: 0/128
b43legacy-phy1 debug: DMA-32 0x0260 (TX) max used slots: 0/128
b43legacy-phy1 debug: DMA-32 0x0240 (TX) max used slots: 0/128
b43legacy-phy1 debug: DMA-32 0x0220 (TX) max used slots: 2/128
b43legacy-phy1 debug: DMA-32 0x0200 (TX) max used slots: 0/128
b43legacy-phy1 debug: Radio initialized
b43legacy-phy1 debug: Radio initialized
b43legacy-phy1 debug: Loading firmware version 0x127, patch level 14
(2005-04-18 02:36:27)
Registered led device: b43legacy-phy1:tx
Registered led device: b43legacy-phy1:rx
b43legacy-phy1 debug: Chip initialized
b43legacy-phy1 debug: 32-bit DMA initialized
b43legacy-phy1 debug: Wireless interface started
b43legacy-phy1 debug: Adding Interface type 2
wlan0: Initial auth_alg=0
wlan0: authenticate with AP 00:09:5b:70:2d:68
wlan0: RX authentication from 00:09:5b:70:2d:68 (alg=0 transaction=2 status=0)
wlan0: authenticated
wlan0: associate with AP 00:09:5b:70:2d:68
wlan0: RX AssocResp from 00:09:5b:70:2d:68 (capab=0x5 status=0 aid=86)
wlan0: associated
wlan0: disassociate(reason=3)
wlan0: Initial auth_alg=0
wlan0: authenticate with AP 00:09:5b:70:2d:68
wlan0: RX authentication from 00:09:5b:70:2d:68 (alg=0 transaction=2 status=0)
wlan0: authenticated
wlan0: associate with AP 00:09:5b:70:2d:68
wlan0: RX ReassocResp from 00:09:5b:70:2d:68 (capab=0x5 status=0 aid=87)
wlan0: associated

  reply	other threads:[~2007-12-17 23:28 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20071213003023.117964080@mvista.com>
2007-12-12  8:00 ` [PATCH 1/3] ps3: vuart: fix error path locking Daniel Walker
2007-12-13  2:00   ` Geoff Levand
2007-12-19  1:10     ` Andrew Morton
2007-12-19  1:54       ` Daniel Walker
2007-12-19  3:04       ` Geoff Levand
2007-12-20 19:32         ` Daniel Walker
2007-12-20 20:06           ` Andrew Morton
2007-12-20 20:13             ` Daniel Walker
2007-12-12  8:00 ` [PATCH 2/3] driver base: memory: semaphore to mutex Daniel Walker
2007-12-12  8:00 ` [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem " Daniel Walker
2007-12-13  0:48   ` Michael Buesch
2007-12-13  1:17     ` Ray Lee
2007-12-13 13:45       ` Michael Buesch
2007-12-14  0:05         ` Ray Lee
2007-12-14  0:43           ` Michael Buesch
2007-12-14  0:55             ` Harvey Harrison
2007-12-14 10:45               ` Michael Buesch
2007-12-14  1:12             ` Ray Lee
2007-12-14 10:49               ` Michael Buesch
2007-12-14 11:15                 ` Ingo Molnar
2007-12-14 11:39                   ` Michael Buesch
2007-12-14 12:16                     ` Ingo Molnar
2007-12-14 12:31                       ` Michael Buesch
2007-12-14 12:53                         ` Ingo Molnar
2007-12-14 13:08                           ` Michael Buesch
2007-12-14 16:06                             ` Ray Lee
2007-12-14 16:49                               ` Michael Buesch
2007-12-14 18:01                                 ` Ray Lee
2007-12-14 18:11                                   ` Ingo Molnar
2007-12-14 18:47                                     ` Ray Lee
2007-12-14 18:22                                   ` Michael Buesch
2007-12-14 18:45                                     ` Ray Lee
2007-12-14 19:05                                       ` Michael Buesch
2007-12-14 19:25                                         ` Ray Lee
2007-12-14 19:38                                           ` Michael Buesch
2007-12-14 19:55                                             ` Ray Lee
2007-12-14 20:13                                               ` Michael Buesch
2007-12-14 21:39                                                 ` Ray Lee
2007-12-14 12:59                         ` Simon Holm Thøgersen
2007-12-14 13:04                           ` Michael Buesch
2007-12-15  0:51                             ` Rafael J. Wysocki
2007-12-15  0:58                               ` Larry Finger
2007-12-15  1:59                                 ` mvtodevnull
2007-12-15  2:27                                   ` Larry Finger
2007-12-15  4:37                                     ` mvtodevnull
2007-12-15  4:39                                       ` mvtodevnull
2007-12-15  7:18                                         ` Larry Finger
2007-12-15 12:38                                           ` mvtodevnull
2007-12-17  2:17                                             ` mvtodevnull
2007-12-17  6:52                                               ` Larry Finger
2007-12-17  7:17                                                 ` mvtodevnull
2007-12-17  9:49                                                   ` Michael Buesch
2007-12-17 10:15                                                     ` mvtodevnull
2007-12-17 10:35                                                       ` mvtodevnull
2007-12-17 22:04                                                         ` mvtodevnull
2007-12-17 22:45                                                           ` Michael Buesch
2007-12-17 23:12                                                             ` mvtodevnull
2007-12-17 23:18                                                               ` Michael Buesch
2007-12-17 23:27                                                                 ` mvtodevnull [this message]
2007-12-18  1:16                                                   ` Larry Finger
2007-12-18  2:34                                                     ` mvtodevnull
2007-12-19  2:12                                                       ` mvtodevnull
2007-12-19  8:11                                                         ` Larry Finger
     [not found]                                                           ` <200712191046.54147.mb@bu3sch.de>
2007-12-19 13:48                                                             ` Ingo Molnar
2007-12-15 10:57                               ` Michael Buesch
2007-12-15 23:18                                 ` Rafael J. Wysocki
2007-12-15 23:27                                   ` Michael Buesch
2007-12-16  2:30                                     ` Larry Finger
2007-12-16 11:13                                       ` Michael Buesch
2007-12-16 13:56                                     ` Johannes Berg
2007-12-16 14:35                                       ` Rafael J. Wysocki
2007-12-16 14:18                                         ` Johannes Berg
2007-12-14  2:09             ` Larry Finger
2007-12-14 10:56             ` Ingo Molnar
2007-12-14 14:14               ` John W. Linville
2007-12-14 14:40                 ` stefano.brivio
2007-12-14 16:27                   ` Ray Lee
2007-12-14 16:45                     ` Ray Lee
2007-12-14 16:59                       ` Michael Buesch
2007-12-14 17:22                         ` Ray Lee
2007-12-14 17:59                         ` Ingo Molnar
2007-12-14 18:38                           ` Michael Buesch
2007-12-15  1:25                             ` Rafael J. Wysocki
2007-12-15 21:43                               ` John W. Linville
2007-12-15 23:26                                 ` Rafael J. Wysocki
2007-12-16  3:20                                   ` Stefan Lippers-Hollmann
2007-12-16  9:22                                 ` Ingo Molnar
2007-12-16 11:23                                   ` Michael Buesch
2007-12-13  1:20     ` Daniel Walker
2007-12-13 10:13       ` Ingo Molnar
2007-12-13 11:11         ` Michael Buesch
2007-12-13 14:23           ` John W. Linville

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=520b60a90712171527h4563542ane34796b32e99635d@mail.gmail.com \
    --to=mvtodevnull@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=bcm43xx-dev@lists.berlios.de \
    --cc=dwalker@mvista.com \
    --cc=jonathan@jonmasters.org \
    --cc=kjwinchester@gmail.com \
    --cc=larry.finger@lwfinger.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@bohmer.net \
    --cc=matthias.kaehlcke@gmail.com \
    --cc=mb@bu3sch.de \
    --cc=mingo@elte.hu \
    --cc=odie@cs.aau.dk \
    --cc=ray-lk@madrabbit.org \
    --cc=rjw@sisk.pl \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®