All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Franky Lin" <frankyl@broadcom.com>
To: "Jason Cooper" <jason@lakedaemon.net>
Cc: rvossen@broadcom.com, arend@broadcom.com, linux-wireless@vger.kernel.org
Subject: Re: [RFC PATCH] net: brcmfmac: add sdio chip id 0x4319
Date: Mon, 17 Jun 2013 10:38:23 -0700	[thread overview]
Message-ID: <51BF498F.1010007@broadcom.com> (raw)
In-Reply-To: <20130617161903.GN31667@titan.lakedaemon.net>

On 06/17/2013 09:19 AM, Jason Cooper wrote:
> Added Franky...
>
> On Wed, Jun 12, 2013 at 06:21:23PM -0400, Jason Cooper wrote:
>> All,
>>
>> I have a Seagate Wireless Plus I am trying to put a vanilla kernel on
>> (currently v3.10-rc5 for various omap DT bits).
>>
>> I have the board booting and running a debian rootfs on the HD.  I just
>> got mmc to come up and I've discovered that the wireless card is vendor
>> 0x02d0, device 0x4319.
>>
>> My hope, once I got to this point, was that I would be able to use
>> the mainline, open source driver.  Unfortunately, it looks like brcmfmac
>> lost support for the 0x4319 while it was in staging.
>>
>> The commit in question is:
>>
>>    4dad253 staging: brcm80211: remove code for unsupported chip
>>
>> Is adding it back in a bridge too far?
> It looks like only two things are needed, adding the chip id and then
> setting the addresses.  Here's a preliminary patch to do just that.
> Note, the addresses I have used are a straight copy from the 4329.  It
> causes this:
>
> [   12.346466] mmcblk mmc0:0001: no of_node; not parsing pinctrl DT
> [   12.353149] brcmfmac_sdio mmc0:0001:1: no of_node; not parsing pinctrl DT
> [   12.360717] brcmfmac_sdio mmc0:0001:2: no of_node; not parsing pinctrl DT
> [   12.368804] brcmfmac: brcmf_sdio_chip_drivestrengthinit: No SDIO Drive strength init done for chip 4319 rev 1 pmurev 7
> [   12.380462] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte F0:@0x00408=03, Err: -22
> [   12.391906] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte F0:@0x00408=03, Err: -22
> [   12.403289] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte F0:@0x00408=03, Err: -22
> [   12.412261] brcmfmac: brcmf_sdio_regrw_helper: failed with -22
> [   12.418457] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte F0:@0x00408=01, Err: -22
> [   12.429718] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte F0:@0x00408=01, Err: -22
> [   12.441101] brcmfmac: brcmf_sdioh_request_byte: Failed to write byte F0:@0x00408=01, Err: -22
> [   12.450073] brcmfmac: brcmf_sdio_regrw_helper: failed with -22
>
> I've taken a quick look at aosp, the provided GPL broadcom code, and I
> haven't seen anything for the 4319 wrt to addresses.  How hard would it
> be to get those addresses from you guys?

Hi Jason,

The major obstacle of adding 4319 support is the obsolete firmware. 
Since it's an EOL chip we are not planning to add the support to brcmfmac.

Franky


  reply	other threads:[~2013-06-17 17:38 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-12 22:21 brcmfmac sdio device id 0x4319 ? Jason Cooper
2013-06-17 16:19 ` [RFC PATCH] net: brcmfmac: add sdio chip id 0x4319 Jason Cooper
2013-06-17 17:38   ` Franky Lin [this message]
2013-06-17 17:45     ` Rafał Miłecki
2013-06-17 17:51       ` Jason Cooper
2013-06-17 17:45     ` Jason Cooper

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=51BF498F.1010007@broadcom.com \
    --to=frankyl@broadcom.com \
    --cc=arend@broadcom.com \
    --cc=jason@lakedaemon.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=rvossen@broadcom.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.