From: "Rafał Miłecki" <zajec5@gmail.com>
To: Hauke Mehrtens <hauke@hauke-m.de>,
"John W. Linville" <linville@tuxdriver.com>
Cc: "Michael Büsch" <m@bues.ch>,
linux-wireless@vger.kernel.org, b43-dev@lists.infradead.org
Subject: [PATCH] b43: HT-PHY: make it BCMA-only
Date: Sat, 2 Mar 2013 10:45:17 +0100 [thread overview]
Message-ID: <CACna6rzJoihR5QtYLhWN+VNdCgc-CpEG30T2ju-Q8ro59f1BsQ@mail.gmail.com> (raw)
In-Reply-To: <5131C5CF.2090306@hauke-m.de>
2013/3/2 Hauke Mehrtens <hauke@hauke-m.de>:
> On 03/02/2013 10:19 AM, Rafa? Mi?ecki wrote:
>> 2013/3/2 Michael B?sch <m@bues.ch>:
>>> On Sat, 2 Mar 2013 02:14:25 +0100
>>> Rafa? Mi?ecki <zajec5@gmail.com> wrote:
>>>
>>>> HT-PHY was found only on BCM4331 which is a BCMA-based chipset. This is
>>>> reallly unlikely we will ever see HT-PHY on SSB thus make the whole code
>>>> BCMA specific. This will allow us to call various BCMA-specific
>>>> functions directly (without extra checks).
>>>>
>>>> Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
>>>> ---
>>>> drivers/net/wireless/b43/Kconfig | 2 +-
>>>> drivers/net/wireless/b43/phy_ht.c | 5 +++++
>>>> 2 files changed, 6 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/drivers/net/wireless/b43/Kconfig b/drivers/net/wireless/b43/Kconfig
>>>> index 7a28d21..09d315d 100644
>>>> --- a/drivers/net/wireless/b43/Kconfig
>>>> +++ b/drivers/net/wireless/b43/Kconfig
>>>> @@ -131,7 +131,7 @@ config B43_PHY_LP
>>>>
>>>> config B43_PHY_HT
>>>> bool "Support for HT-PHY (high throughput) devices (EXPERIMENTAL)"
>>>> - depends on B43 && EXPERIMENTAL
>>>> + depends on B43 && B43_BCMA && EXPERIMENTAL
>>>
>>> Makes sense.
>>> What about removing EXPERIMENTAL? This flag is mostly useless these days.
>>
>> A little later, I've plan for some minor improvement :)
>
> There is already a patch [0] in Linus tree removing EXPERIMENTAL.
>
> Hauke
>
> [0]:
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=afd19ef0b1a98f420ddcfc619182fd589034577f
Glad there was a b43-dev Cc...
John please drop this patch. I'll wait for you merging Linus's tree
and re-submit it rebased.
--
Rafa?
prev parent reply other threads:[~2013-03-02 9:45 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-02 1:14 [PATCH] b43: HT-PHY: make it BCMA-only Rafał Miłecki
2013-03-02 4:09 ` Michael Büsch
2013-03-02 9:19 ` Rafał Miłecki
2013-03-02 9:26 ` Hauke Mehrtens
2013-03-02 9:45 ` Rafał Miłecki [this message]
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=CACna6rzJoihR5QtYLhWN+VNdCgc-CpEG30T2ju-Q8ro59f1BsQ@mail.gmail.com \
--to=zajec5@gmail.com \
--cc=b43-dev@lists.infradead.org \
--cc=hauke@hauke-m.de \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=m@bues.ch \
/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;
as well as URLs for NNTP newsgroup(s).