* [PATCH] b43: Add new SSB's core id for BCM4328
@ 2010-01-02 16:24 Rafał Miłecki
2010-01-02 16:26 ` Gábor Stefanik
2010-01-02 16:48 ` Larry Finger
0 siblings, 2 replies; 10+ messages in thread
From: Rafał Miłecki @ 2010-01-02 16:24 UTC (permalink / raw)
To: John W. Linville, linux-wireless; +Cc: linux-wireless
[-- Attachment #1: Type: text/plain, Size: 1617 bytes --]
Finally I've possibility to test my Christmas-gift N-PHY :) b43 didn't really "notice" it, so I needed to write this little patch. Now b43 at least look for firmware for my card.
Does this patch look OK?
This time I post from Opera mail so I hope white-spaces will be fine.
From 9b3ea4f29b038f732aba737323e2f89b275ce1ab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
Date: Sat, 2 Jan 2010 16:58:16 +0100
Subject: [PATCH] b43: Add new SSB's core id for BCM4328
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Following line shows my BCM4328 detected on SSB. We didn't include 0x0C rev.
ssb: Core 1 found: IEEE 802.11 (cc 0x812, rev 0x0C, vendor 0x4243)
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
---
drivers/net/wireless/b43/main.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
index 098dda1..3aa45b8 100644
--- a/drivers/net/wireless/b43/main.c
+++ b/drivers/net/wireless/b43/main.c
@@ -110,6 +110,7 @@ static const struct ssb_device_id b43_ssb_tbl[] = {
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 9),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 10),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 11),
+ SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 12),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 13),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 15),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 16),
--
1.6.4.2
[-- Attachment #2: 0001-b43-Add-new-SSB-s-core-id-for-BCM4328.patch --]
[-- Type: application/octet-stream, Size: 1257 bytes --]
From 9b3ea4f29b038f732aba737323e2f89b275ce1ab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
Date: Sat, 2 Jan 2010 16:58:16 +0100
Subject: [PATCH] b43: Add new SSB's core id for BCM4328
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Following line shows my BCM4328 detected on SSB. We didn't include 0x0C rev.
ssb: Core 1 found: IEEE 802.11 (cc 0x812, rev 0x0C, vendor 0x4243)
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
---
drivers/net/wireless/b43/main.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
index 098dda1..3aa45b8 100644
--- a/drivers/net/wireless/b43/main.c
+++ b/drivers/net/wireless/b43/main.c
@@ -110,6 +110,7 @@ static const struct ssb_device_id b43_ssb_tbl[] = {
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 9),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 10),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 11),
+ SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 12),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 13),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 15),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 16),
--
1.6.4.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] b43: Add new SSB's core id for BCM4328
2010-01-02 16:24 [PATCH] b43: Add new SSB's core id for BCM4328 Rafał Miłecki
@ 2010-01-02 16:26 ` Gábor Stefanik
2010-01-02 16:40 ` Rafał Miłecki
2010-01-02 17:46 ` Michael Buesch
2010-01-02 16:48 ` Larry Finger
1 sibling, 2 replies; 10+ messages in thread
From: Gábor Stefanik @ 2010-01-02 16:26 UTC (permalink / raw)
To: Rafał Miłecki; +Cc: John W. Linville, linux-wireless
2010/1/2 Rafał Miłecki <zajec5@gmail.com>:
> Finally I've possibility to test my Christmas-gift N-PHY :) b43 didn't
> really "notice" it, so I needed to write this little patch. Now b43 at least
> look for firmware for my card.
>
> Does this patch look OK?
>
> This time I post from Opera mail so I hope white-spaces will be fine.
>
> From 9b3ea4f29b038f732aba737323e2f89b275ce1ab Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
> Date: Sat, 2 Jan 2010 16:58:16 +0100
> Subject: [PATCH] b43: Add new SSB's core id for BCM4328
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> Following line shows my BCM4328 detected on SSB. We didn't include 0x0C rev.
> ssb: Core 1 found: IEEE 802.11 (cc 0x812, rev 0x0C, vendor 0x4243)
>
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
> ---
> drivers/net/wireless/b43/main.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/b43/main.c
> b/drivers/net/wireless/b43/main.c
> index 098dda1..3aa45b8 100644
> --- a/drivers/net/wireless/b43/main.c
> +++ b/drivers/net/wireless/b43/main.c
> @@ -110,6 +110,7 @@ static const struct ssb_device_id b43_ssb_tbl[] = {
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 9),
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 10),
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 11),
> + SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 12),
Rev.14 is also missing (which AFAIK is used by some LP cards).
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 13),
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 15),
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 16),
> --
> 1.6.4.2
--
Vista: [V]iruses, [I]ntruders, [S]pyware, [T]rojans and [A]dware. :-)
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] b43: Add new SSB's core id for BCM4328
2010-01-02 16:26 ` Gábor Stefanik
@ 2010-01-02 16:40 ` Rafał Miłecki
2010-01-02 16:43 ` Gábor Stefanik
2010-01-02 17:46 ` Michael Buesch
1 sibling, 1 reply; 10+ messages in thread
From: Rafał Miłecki @ 2010-01-02 16:40 UTC (permalink / raw)
To: Gábor Stefanik; +Cc: John W. Linville, linux-wireless
W dniu 2 stycznia 2010 17:26 użytkownik Gábor Stefanik
<netrolller.3d@gmail.com> napisał:
> 2010/1/2 Rafał Miłecki <zajec5@gmail.com>:
>> Finally I've possibility to test my Christmas-gift N-PHY :) b43 didn't
>> really "notice" it, so I needed to write this little patch. Now b43 at least
>> look for firmware for my card.
>>
>> Does this patch look OK?
>>
>> This time I post from Opera mail so I hope white-spaces will be fine.
>>
>> From 9b3ea4f29b038f732aba737323e2f89b275ce1ab Mon Sep 17 00:00:00 2001
>> From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
>> Date: Sat, 2 Jan 2010 16:58:16 +0100
>> Subject: [PATCH] b43: Add new SSB's core id for BCM4328
>> MIME-Version: 1.0
>> Content-Type: text/plain; charset=UTF-8
>> Content-Transfer-Encoding: 8bit
>>
>> Following line shows my BCM4328 detected on SSB. We didn't include 0x0C rev.
>> ssb: Core 1 found: IEEE 802.11 (cc 0x812, rev 0x0C, vendor 0x4243)
>>
>> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
>> ---
>> drivers/net/wireless/b43/main.c | 1 +
>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/net/wireless/b43/main.c
>> b/drivers/net/wireless/b43/main.c
>> index 098dda1..3aa45b8 100644
>> --- a/drivers/net/wireless/b43/main.c
>> +++ b/drivers/net/wireless/b43/main.c
>> @@ -110,6 +110,7 @@ static const struct ssb_device_id b43_ssb_tbl[] = {
>> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 9),
>> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 10),
>> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 11),
>> + SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 12),
>
> Rev.14 is also missing (which AFAIK is used by some LP cards).
Thanks, I'll post patch for that as well. Is adding this 0x0C (==13)
fine in this form? Or maybe we should use ifdef CONFIG_B43_NPHY for
some reason?
--
Rafał
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] b43: Add new SSB's core id for BCM4328
2010-01-02 16:40 ` Rafał Miłecki
@ 2010-01-02 16:43 ` Gábor Stefanik
2010-01-02 16:48 ` Rafał Miłecki
0 siblings, 1 reply; 10+ messages in thread
From: Gábor Stefanik @ 2010-01-02 16:43 UTC (permalink / raw)
To: Rafał Miłecki; +Cc: John W. Linville, linux-wireless
2010/1/2 Rafał Miłecki <zajec5@gmail.com>:
> W dniu 2 stycznia 2010 17:26 użytkownik Gábor Stefanik
> <netrolller.3d@gmail.com> napisał:
>> 2010/1/2 Rafał Miłecki <zajec5@gmail.com>:
>>> Finally I've possibility to test my Christmas-gift N-PHY :) b43 didn't
>>> really "notice" it, so I needed to write this little patch. Now b43 at least
>>> look for firmware for my card.
>>>
>>> Does this patch look OK?
>>>
>>> This time I post from Opera mail so I hope white-spaces will be fine.
>>>
>>> From 9b3ea4f29b038f732aba737323e2f89b275ce1ab Mon Sep 17 00:00:00 2001
>>> From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
>>> Date: Sat, 2 Jan 2010 16:58:16 +0100
>>> Subject: [PATCH] b43: Add new SSB's core id for BCM4328
>>> MIME-Version: 1.0
>>> Content-Type: text/plain; charset=UTF-8
>>> Content-Transfer-Encoding: 8bit
>>>
>>> Following line shows my BCM4328 detected on SSB. We didn't include 0x0C rev.
>>> ssb: Core 1 found: IEEE 802.11 (cc 0x812, rev 0x0C, vendor 0x4243)
>>>
>>> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
>>> ---
>>> drivers/net/wireless/b43/main.c | 1 +
>>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/drivers/net/wireless/b43/main.c
>>> b/drivers/net/wireless/b43/main.c
>>> index 098dda1..3aa45b8 100644
>>> --- a/drivers/net/wireless/b43/main.c
>>> +++ b/drivers/net/wireless/b43/main.c
>>> @@ -110,6 +110,7 @@ static const struct ssb_device_id b43_ssb_tbl[] = {
>>> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 9),
>>> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 10),
>>> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 11),
>>> + SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 12),
>>
>> Rev.14 is also missing (which AFAIK is used by some LP cards).
>
> Thanks, I'll post patch for that as well. Is adding this 0x0C (==13)
> fine in this form? Or maybe we should use ifdef CONFIG_B43_NPHY for
> some reason?
I would say it's OK this way (BTW 0x0C == 12); we also didn't put
rev.15 under CONFIG_B43_PHY_LP.
(BTW I'm planning to rename CONFIG_B43_NPHY to CONFIG_B43_PHY_N in
preparation for further PHY types, as well as the ability to only
build certain PHYs as needed if EMBEDDED is set - a kernel for an
LP-PHY SoC might save some space by not building G-PHY code.)
>
> --
> Rafał
>
--
Vista: [V]iruses, [I]ntruders, [S]pyware, [T]rojans and [A]dware. :-)
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] b43: Add new SSB's core id for BCM4328
2010-01-02 16:24 [PATCH] b43: Add new SSB's core id for BCM4328 Rafał Miłecki
2010-01-02 16:26 ` Gábor Stefanik
@ 2010-01-02 16:48 ` Larry Finger
2010-01-02 16:55 ` Rafał Miłecki
2010-01-02 17:22 ` Rafał Miłecki
1 sibling, 2 replies; 10+ messages in thread
From: Larry Finger @ 2010-01-02 16:48 UTC (permalink / raw)
To: Rafał Miłecki; +Cc: John W. Linville, linux-wireless
On 01/02/2010 10:24 AM, Rafał Miłecki wrote:
> Finally I've possibility to test my Christmas-gift N-PHY :) b43 didn't
> really "notice" it, so I needed to write this little patch. Now b43 at
> least look for firmware for my card.
>
> Does this patch look OK?
>
> This time I post from Opera mail so I hope white-spaces will be fine.
>
> From 9b3ea4f29b038f732aba737323e2f89b275ce1ab Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
> Date: Sat, 2 Jan 2010 16:58:16 +0100
> Subject: [PATCH] b43: Add new SSB's core id for BCM4328
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> Following line shows my BCM4328 detected on SSB. We didn't include 0x0C
> rev.
> ssb: Core 1 found: IEEE 802.11 (cc 0x812, rev 0x0C, vendor 0x4243)
>
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
> ---
> drivers/net/wireless/b43/main.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/b43/main.c
> b/drivers/net/wireless/b43/main.c
> index 098dda1..3aa45b8 100644
> --- a/drivers/net/wireless/b43/main.c
> +++ b/drivers/net/wireless/b43/main.c
> @@ -110,6 +110,7 @@ static const struct ssb_device_id b43_ssb_tbl[] = {
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 9),
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 10),
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 11),
> + SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 12),
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 13),
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 15),
> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 16),
Your tabs have been converted to spaces.
Most of my patches are sent using the mail command; however, I can send them
with Thunderbird using the following steps:
(1) Prepare the mail message with the patch attached.
(2) Save the message and close the open copy.
(3) Go to the "Draft" folder and click on the entry.
(4) The attachment will be listed at the bottom. Highlight that test and copy
into the clipboard.
(5) Click the "Edit" button, paste the attachment text into the message, clear
the attachment entry, and send.
I'm not sure how far you have gotten with the code, but I have been working with
a new driver with version 5.10.56.46. There are a lot of changes from the
4.174... that was previously reverse-engineered.
To answer your question in a later E-mail. You do not need to conditionalize the
core revision entry. If the N PHY code is not turned on, the driver will get an
unsupported PHY message and fail later in an acceptable error.
Larry
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] b43: Add new SSB's core id for BCM4328
2010-01-02 16:43 ` Gábor Stefanik
@ 2010-01-02 16:48 ` Rafał Miłecki
0 siblings, 0 replies; 10+ messages in thread
From: Rafał Miłecki @ 2010-01-02 16:48 UTC (permalink / raw)
To: Gábor Stefanik; +Cc: John W. Linville, linux-wireless
W dniu 2 stycznia 2010 17:43 użytkownik Gábor Stefanik
<netrolller.3d@gmail.com> napisał:
> 2010/1/2 Rafał Miłecki <zajec5@gmail.com>:
>> W dniu 2 stycznia 2010 17:26 użytkownik Gábor Stefanik
>> <netrolller.3d@gmail.com> napisał:
>>> 2010/1/2 Rafał Miłecki <zajec5@gmail.com>:
>>>> Finally I've possibility to test my Christmas-gift N-PHY :) b43 didn't
>>>> really "notice" it, so I needed to write this little patch. Now b43 at least
>>>> look for firmware for my card.
>>>>
>>>> Does this patch look OK?
>>>>
>>>> This time I post from Opera mail so I hope white-spaces will be fine.
>>>>
>>>> From 9b3ea4f29b038f732aba737323e2f89b275ce1ab Mon Sep 17 00:00:00 2001
>>>> From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
>>>> Date: Sat, 2 Jan 2010 16:58:16 +0100
>>>> Subject: [PATCH] b43: Add new SSB's core id for BCM4328
>>>> MIME-Version: 1.0
>>>> Content-Type: text/plain; charset=UTF-8
>>>> Content-Transfer-Encoding: 8bit
>>>>
>>>> Following line shows my BCM4328 detected on SSB. We didn't include 0x0C rev.
>>>> ssb: Core 1 found: IEEE 802.11 (cc 0x812, rev 0x0C, vendor 0x4243)
>>>>
>>>> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
>>>> ---
>>>> drivers/net/wireless/b43/main.c | 1 +
>>>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/drivers/net/wireless/b43/main.c
>>>> b/drivers/net/wireless/b43/main.c
>>>> index 098dda1..3aa45b8 100644
>>>> --- a/drivers/net/wireless/b43/main.c
>>>> +++ b/drivers/net/wireless/b43/main.c
>>>> @@ -110,6 +110,7 @@ static const struct ssb_device_id b43_ssb_tbl[] = {
>>>> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 9),
>>>> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 10),
>>>> SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 11),
>>>> + SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 12),
>>>
>>> Rev.14 is also missing (which AFAIK is used by some LP cards).
>>
>> Thanks, I'll post patch for that as well. Is adding this 0x0C (==13)
>> fine in this form? Or maybe we should use ifdef CONFIG_B43_NPHY for
>> some reason?
>
> I would say it's OK this way (BTW 0x0C == 12); we also didn't put
> rev.15 under CONFIG_B43_PHY_LP.
OK, thanks. And of course I've meant 0x0C == 12 :) After applying this
patch b43 won't try to init N-PHY device until CONFIG_B43_NPHY is set.
So I believe this patch is fine, thanks for sharing opinion :)
> (BTW I'm planning to rename CONFIG_B43_NPHY to CONFIG_B43_PHY_N in
> preparation for further PHY types, as well as the ability to only
> build certain PHYs as needed if EMBEDDED is set - a kernel for an
> LP-PHY SoC might save some space by not building G-PHY code.)
Heh, I was planning to do that :) OK, I'll focus on other stuff,
waiting for your patch :)
--
Rafał
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] b43: Add new SSB's core id for BCM4328
2010-01-02 16:48 ` Larry Finger
@ 2010-01-02 16:55 ` Rafał Miłecki
2010-01-02 17:40 ` Larry Finger
2010-01-02 17:22 ` Rafał Miłecki
1 sibling, 1 reply; 10+ messages in thread
From: Rafał Miłecki @ 2010-01-02 16:55 UTC (permalink / raw)
To: Larry Finger; +Cc: John W. Linville, linux-wireless
W dniu 2 stycznia 2010 17:48 użytkownik Larry Finger
<Larry.Finger@lwfinger.net> napisał:
> Your tabs have been converted to spaces.
>
> Most of my patches are sent using the mail command; however, I can send them
> with Thunderbird using the following steps:
Oh great :| I guess I'll just try mail command.
> I'm not sure how far you have gotten with the code, but I have been working with
> a new driver with version 5.10.56.46. There are a lot of changes from the
> 4.174... that was previously reverse-engineered.
I don't have anything more than posted patches.
What do you suggest about writing next (real) parts? Is that worth
effort basing code on current RE docs? Maybe some parts only, that
didn't change? Or should I just wait for update (is anyone working on
it)?
> To answer your question in a later E-mail. You do not need to conditionalize the
> core revision entry. If the N PHY code is not turned on, the driver will get an
> unsupported PHY message and fail later in an acceptable error.
One more confirmation :) Thanks.
--
Rafał
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] b43: Add new SSB's core id for BCM4328
2010-01-02 16:48 ` Larry Finger
2010-01-02 16:55 ` Rafał Miłecki
@ 2010-01-02 17:22 ` Rafał Miłecki
1 sibling, 0 replies; 10+ messages in thread
From: Rafał Miłecki @ 2010-01-02 17:22 UTC (permalink / raw)
To: Larry Finger; +Cc: John W. Linville, linux-wireless
White-spaces fixed.
From 9b3ea4f29b038f732aba737323e2f89b275ce1ab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
Date: Sat, 2 Jan 2010 16:58:16 +0100
Subject: [PATCH] b43: Add new SSB's core id for BCM4328
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Following line shows my BCM4328 detected on SSB. We didn't include 0x0C rev.
ssb: Core 1 found: IEEE 802.11 (cc 0x812, rev 0x0C, vendor 0x4243)
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
---
drivers/net/wireless/b43/main.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
index 098dda1..3aa45b8 100644
--- a/drivers/net/wireless/b43/main.c
+++ b/drivers/net/wireless/b43/main.c
@@ -110,6 +110,7 @@ static const struct ssb_device_id b43_ssb_tbl[] = {
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 9),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 10),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 11),
+ SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 12),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 13),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 15),
SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 16),
--
1.6.4.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] b43: Add new SSB's core id for BCM4328
2010-01-02 16:55 ` Rafał Miłecki
@ 2010-01-02 17:40 ` Larry Finger
0 siblings, 0 replies; 10+ messages in thread
From: Larry Finger @ 2010-01-02 17:40 UTC (permalink / raw)
To: Rafał Miłecki; +Cc: John W. Linville, linux-wireless
On 01/02/2010 10:55 AM, Rafał Miłecki wrote:
> W dniu 2 stycznia 2010 17:48 użytkownik Larry Finger
> <Larry.Finger@lwfinger.net> napisał:
>> Your tabs have been converted to spaces.
>>
>> Most of my patches are sent using the mail command; however, I can send them
>> with Thunderbird using the following steps:
>
> Oh great :| I guess I'll just try mail command.
>
>
>> I'm not sure how far you have gotten with the code, but I have been working with
>> a new driver with version 5.10.56.46. There are a lot of changes from the
>> 4.174... that was previously reverse-engineered.
>
> I don't have anything more than posted patches.
>
> What do you suggest about writing next (real) parts? Is that worth
> effort basing code on current RE docs? Maybe some parts only, that
> didn't change? Or should I just wait for update (is anyone working on
> it)?
I have finished the initialization routine found in
http://bcm-v4.sipsolutions.net/802.11/PHY/Init/N. I still have 6 or 8 of the
routines called to prepare and some of the others to check, but you could at
least code the main routine with dummy stubs for the called routines..
You should check with Gabor regarding PHY and radio register names.
Larry
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] b43: Add new SSB's core id for BCM4328
2010-01-02 16:26 ` Gábor Stefanik
2010-01-02 16:40 ` Rafał Miłecki
@ 2010-01-02 17:46 ` Michael Buesch
1 sibling, 0 replies; 10+ messages in thread
From: Michael Buesch @ 2010-01-02 17:46 UTC (permalink / raw)
To: Gábor Stefanik
Cc: Rafał Miłecki, John W. Linville, linux-wireless
On Saturday 02 January 2010 17:26:55 Gábor Stefanik wrote:
> > @@ -110,6 +110,7 @@ static const struct ssb_device_id b43_ssb_tbl[] = {
> > SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 9),
> > SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 10),
> > SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 11),
> > + SSB_DEVICE(SSB_VENDOR_BROADCOM, SSB_DEV_80211, 12),
>
> Rev.14 is also missing (which AFAIK is used by some LP cards).
"AFAIK" doesn't count.
We only add IDs for cards that do actually exist in reality.
So do you have such a card or a report by someone owning one?
--
Greetings, Michael.
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2010-01-02 17:46 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-02 16:24 [PATCH] b43: Add new SSB's core id for BCM4328 Rafał Miłecki
2010-01-02 16:26 ` Gábor Stefanik
2010-01-02 16:40 ` Rafał Miłecki
2010-01-02 16:43 ` Gábor Stefanik
2010-01-02 16:48 ` Rafał Miłecki
2010-01-02 17:46 ` Michael Buesch
2010-01-02 16:48 ` Larry Finger
2010-01-02 16:55 ` Rafał Miłecki
2010-01-02 17:40 ` Larry Finger
2010-01-02 17:22 ` Rafał Miłecki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).